Модераторы: LSD, AntonSaburov

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> getRuntime().exec и потоки 
:(
    Опции темы
LSD
Дата 13.2.2008, 14:44 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Leprechaun Software Developer
****


Профиль
Группа: Модератор
Сообщений: 15718
Регистрация: 24.3.2004
Где: Dublin

Репутация: 210
Всего: 538



Проблема состоит в том, что CmdTest.exec() не ждет пока будет выполнена команда. Команда просто передаётся cmd.exe и все. Сколько она будет выполняться - неизвестно. С выводом в System.out таких проблем нет, потому что по мере выполнения команды данные перекачиваются в System.out, а тут ты сам должен определить когда команда выполнилась, или просто подождать.

Код

import java.io.*;

public class CmdTest
{
  private OutputStream outputStream;
  private InputStream inputStream;
  private InputStream errorStream;
  private ByteArrayOutputStream output = new ByteArrayOutputStream();
  private ByteArrayOutputStream errors = new ByteArrayOutputStream();
  private Process proc;

  public CmdTest() throws IOException
  {
    proc = new ProcessBuilder("cmd.exe").start();
    this.inputStream = proc.getInputStream();
    this.outputStream = proc.getOutputStream();
    this.errorStream = proc.getErrorStream();

    new CmdTest.Spooler(inputStream, output).start();
    new CmdTest.Spooler(errorStream, errors).start();
  }

  public void exec(String... commands)
  {
    BufferedWriter writer = new BufferedWriter(new OutputStreamWriter(this.outputStream));
    for(String command : commands)
    {
      try
      {
        writer.write(command);
        writer.newLine();
        writer.flush();
      }
      catch(IOException e)
      {
        e.printStackTrace();
      }
    }
  }

  public String getOutput(String charset) throws UnsupportedEncodingException
  {
    return new String(output.toByteArray(), charset);
  }

  public void clearOutput()
  {
    output.reset();
  }

  public String getErrors(String charset) throws UnsupportedEncodingException
  {
    return new String(errors.toByteArray(), charset);
  }

  public void clearErrors()
  {
    errors.reset();
  }

  public int exit()
  {
    try
    {
      BufferedWriter writer = new BufferedWriter(new OutputStreamWriter(this.outputStream));
      writer.write("exit");
      writer.newLine();
      writer.flush();
      return proc.waitFor();
    }
    catch(Exception e)
    {
      e.printStackTrace();
    }
    return -1;
  }

  public static void main(String[] args) throws Exception
  {
    CmdTest test = new CmdTest();

    test.exec("C:", "CD C:\\Windows", "dir");
    Thread.sleep(1000);
    System.out.println("output = " + test.getOutput("Cp866"));
    test.clearOutput();
    System.out.println("errors = " + test.getErrors("Cp866"));
    test.clearErrors();
    System.out.println("\n=======================================\n");

    test.exec("ipconfig");
    Thread.sleep(1000);
    System.out.println("output = " + test.getOutput("Cp1251"));
    test.clearOutput();
    System.out.println("errors = " + test.getErrors("Cp1251"));
    test.clearErrors();
    System.out.println("\n=======================================\n");

    test.exec("find /?");
    Thread.sleep(1000);
    System.out.println("output = " + test.getOutput("Cp866"));
    test.clearOutput();
    System.out.println("errors = " + test.getErrors("Cp866"));
    test.clearErrors();
    System.out.println("\n=======================================\n");

    test.exit();
    Thread.sleep(1000);
    System.out.println("output = " + test.getOutput("Cp866"));
    test.clearOutput();
    System.out.println("errors = " + test.getErrors("Cp866"));
    test.clearErrors();
    System.out.println("\n=======================================\n");
  }

  private static class Spooler extends Thread
  {
    private InputStream in;
    private OutputStream out;

    public Spooler(InputStream in, OutputStream out)
    {
      super("SpoolerThread");
      this.in = in;
      this.out = out;
    }

    @Override
    public void run()
    {
      byte[] buffer = new byte[1024];
      int read;
      try
      {
        while((read = in.read(buffer)) != -1)
          out.write(buffer, 0, read);
      }
      catch(IOException e)
      {
        e.printStackTrace();
      }
    }
  }
}



--------------------
Disclaimer: this post contains explicit depictions of personal opinion. So, if it sounds sarcastic, don't take it seriously. If it sounds dangerous, do not try this at home or at all. And if it offends you, just don't read it.
PM MAIL WWW   Вверх
Akeem
  Дата 13.2.2008, 15:15 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 381
Регистрация: 14.7.2006
Где: Киев/Украина

Репутация: нет
Всего: 3



LSD, спасибо!
Все работает. Премного благодарен. 

Только вопрос: обезательно указывать кодировку? спрашиваю потому що если указать явно то выдает ошибку что неопознает кодировки. (мне не нужны лишние параметры)

П.С. Код переписал под свою программу. 




--------------------
Мир огромен и мы только муравьи в нём! (С) formicha

PM MAIL WWW ICQ   Вверх
LSD
Дата 13.2.2008, 15:22 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Leprechaun Software Developer
****


Профиль
Группа: Модератор
Сообщений: 15718
Регистрация: 24.3.2004
Где: Dublin

Репутация: 210
Всего: 538



Цитата(Akeem @  13.2.2008,  15:15 Найти цитируемый пост)
Только вопрос: обезательно указывать кодировку?

У меня без указания кодировки не все работает корректно, вместо русских буквы кракозябры. Можешь переписать так:
Код

  public String getOutput(Charset charset)
  {
    return new String(output.toByteArray(), charset);
  }

тогда никаких Exception-ов выкидываться не будет.

Добавлено через 37 секунд
К тому же часть команд выводит данные в 866 кодировке, часть в 1251.


--------------------
Disclaimer: this post contains explicit depictions of personal opinion. So, if it sounds sarcastic, don't take it seriously. If it sounds dangerous, do not try this at home or at all. And if it offends you, just don't read it.
PM MAIL WWW   Вверх
Akeem
Дата 13.2.2008, 15:26 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 381
Регистрация: 14.7.2006
Где: Киев/Украина

Репутация: нет
Всего: 3



Ну в принципе у меня проблем с кодировкой не должно быть. Все результаты и команды на англ. 

П.С. Я вообще убрал объявление кодировки с функции.
Код

public String getOutput()
  {
    return new String(output.toByteArray());
  }



--------------------
Мир огромен и мы только муравьи в нём! (С) formicha

PM MAIL WWW ICQ   Вверх
Resilient
Дата 27.11.2008, 18:21 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок



Профиль
Группа: Участник
Сообщений: 13
Регистрация: 26.11.2008

Репутация: нет
Всего: нет



Приветствую... вобщем-то тоже чайник в Яве, надеюсь, что только пока что smile
А нету ли способа гарантированно дождаться выполнения команды? и узнать, что вернула та програмка.
PM MAIL   Вверх
LSD
Дата 29.11.2008, 15:37 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Leprechaun Software Developer
****


Профиль
Группа: Модератор
Сообщений: 15718
Регистрация: 24.3.2004
Где: Dublin

Репутация: 210
Всего: 538



Цитата(Resilient @  27.11.2008,  18:21 Найти цитируемый пост)
А нету ли способа гарантированно дождаться выполнения команды? и узнать, что вернула та програмка.

А нафига?
Если тебе нужно выполнить некую программу, то вызывай непосредственно её, а не через cmd.exe и таких проблем не будет. А если тебе нужно выполнить команду именно cmd.exe, то в любом случае ты будешь парсить вывод этой команды и по нему уже можно будет определить, что команда завершена.


--------------------
Disclaimer: this post contains explicit depictions of personal opinion. So, if it sounds sarcastic, don't take it seriously. If it sounds dangerous, do not try this at home or at all. And if it offends you, just don't read it.
PM MAIL WWW   Вверх
Resilient
Дата 1.12.2008, 16:52 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок



Профиль
Группа: Участник
Сообщений: 13
Регистрация: 26.11.2008

Репутация: нет
Всего: нет



В данном случае неизвестно других методов вызова, кроме как из командной строки :(
Нужно же дождаться окончания выполнения и в зависимости от кода возврата в cmd продолжить.
PM MAIL   Вверх
LSD
Дата 1.12.2008, 18:23 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Leprechaun Software Developer
****


Профиль
Группа: Модератор
Сообщений: 15718
Регистрация: 24.3.2004
Где: Dublin

Репутация: 210
Всего: 538



Цитата(Resilient @  1.12.2008,  16:52 Найти цитируемый пост)
В данном случае неизвестно других методов вызова, кроме как из командной строки :(

Вызова чего? Ты программу пытаешься вызвать или внутреннюю команду cmd.exe?


--------------------
Disclaimer: this post contains explicit depictions of personal opinion. So, if it sounds sarcastic, don't take it seriously. If it sounds dangerous, do not try this at home or at all. And if it offends you, just don't read it.
PM MAIL WWW   Вверх
Resilient
Дата 2.12.2008, 15:11 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок



Профиль
Группа: Участник
Сообщений: 13
Регистрация: 26.11.2008

Репутация: нет
Всего: нет



программу. И другого вызова в данном случае не получится.
НО у меня оно дожидается окончания работы. smile
PM MAIL   Вверх
LSD
Дата 2.12.2008, 17:33 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Leprechaun Software Developer
****


Профиль
Группа: Модератор
Сообщений: 15718
Регистрация: 24.3.2004
Где: Dublin

Репутация: 210
Всего: 538



Цитата(Resilient @  2.12.2008,  15:11 Найти цитируемый пост)
программу.

Ну так и запускай программу, а не cmd.exe smile 


--------------------
Disclaimer: this post contains explicit depictions of personal opinion. So, if it sounds sarcastic, don't take it seriously. If it sounds dangerous, do not try this at home or at all. And if it offends you, just don't read it.
PM MAIL WWW   Вверх
Ответ в темуСоздание новой темы Создание опроса
Правила форума "Java"
LSD   AntonSaburov
powerOn   tux
javastic
  • Прежде, чем задать вопрос, прочтите это!
  • Книги по Java собираются здесь.
  • Документация и ресурсы по Java находятся здесь.
  • Используйте теги [code=java][/code] для подсветки кода. Используйтe чекбокс "транслит", если у Вас нет русских шрифтов.
  • Помечайте свой вопрос как решённый, если на него получен ответ. Ссылка "Пометить как решённый" находится над первым постом.
  • Действия модераторов можно обсудить здесь.
  • FAQ раздела лежит здесь.

Если Вам помогли, и атмосфера форума Вам понравилась, то заходите к нам чаще! С уважением, LSD, AntonSaburov, powerOn, tux, javastic.

 
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | Java: Общие вопросы | Следующая тема »


 




[ Время генерации скрипта: 0.0991 ]   [ Использовано запросов: 22 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.