Ждать завершения потока (без зависания формы)

@6eH · Регистрация: 05.05.2014

Author24 — интернет-сервис помощи студентам

Здравствуйте товарищи!

Столкнулся с такой вот проблемой.
Пишу программу, которая запускает через cmd.exe некие команды. Форма зависает на прослушивании ответа от cmd (Оно и понятно, там цикл прослушивания и вывода результата) и отвисает после выполнения.
Для решения этой проблемы я использовал поток, но теперь программа не дожидается выполнения функции из потока, а просто запускает его и продолжает работать дальше.

Подскажите пожалуйста как сделать так, чтобы запускался отдельный поток, но программа ждала (без зависания формы) его окончания (ну или какого-либо события).

Структура работы:
Как сейчас:

Пользователь выбрал команду (например.ping ya.ru) -> Запускается поток с функцией cmd.exe + эта команда -> Программа НЕ ЖДЕТ завершения потока и продолжает выполнять код.

Как необходимо:

Пользователь выбрал команду (например.ping ya.ru) -> Запускается поток с функцией cmd.exe + эта команда -> Программа ЖДЕТ завершения потока и продолжает выполнять код.

Вот код функции командной строки :

C#

        public void RunCmd(object command)
        {
            Thread newThread = new Thread(delegate() { DoWork(command); });
            newThread.Start();
            // Тут нужно ждать ответа от функции DoWork
        }
 
        public void DoWork(object command)
        {
            Process proc = new Process()
            {
                StartInfo = new ProcessStartInfo("cmd.exe", "/c" + command)
                {
                    StandardOutputEncoding = Encoding.GetEncoding(866),
                    RedirectStandardOutput = true,
                    UseShellExecute = false,
                    CreateNoWindow = true,
                    WindowStyle = ProcessWindowStyle.Hidden
                }
            };
            proc.Start();
 
            if (!proc.StartInfo.RedirectStandardOutput)
                return;
 
            string line;
            StreamReader sr = proc.StandardOutput;
            while (!sr.EndOfStream)
            {
                line = sr.ReadLine();
                richTextBox1.Invoke(new MethodInvoker(() =>
                {
                    richTextBox1.Text += line + "\n";
                }), null);
            }
        }

PS: newThread.Join() тоже вешает форму.

@sk007 · 01.06.2014, 23:06

Вот так?

C#

Process proc = new Process()
            {
                StartInfo = new ProcessStartInfo("cmd.exe", "/c" + command)
                {
                    StandardOutputEncoding = Encoding.GetEncoding(866),
                    RedirectStandardOutput = true,
                    UseShellExecute = false,
                    CreateNoWindow = true,
                    WindowStyle = ProcessWindowStyle.Hidden
                }
            };
            proc.Start();
            proc.WaitForExit();

Добавлено через 5 минут
или может лучше повесить обработчик?

C#

proc.OutputDataReceived += new DataReceivedEventHandler(proc_OutputDataReceived);
        //....................................
 
        private void proc_OutputDataReceived(object sender, DataReceivedEventArgs e)
        {
            richTextBox1.Invoke(new Action(() =>
            {
                richTextBox1.Text += e.Data + "\n";
            }), null);
        }

@6eH · 03.06.2014, 10:27 **[ТС]**

sk007, Ваш способ к сожалению не работает.
А не работает, как я понимаю, потому, что вся функция уже идет в отдельном потоке и основному коду по барабану, выполнилась она уже или нет. То есть эти проверки и обработчики не могут "заморозить" основной код из своего потока. Поправьте если ошибаюсь.
Вопрос актуален.

Добавлено через 2 часа 32 минуты
В общем решилась проблема проще простого.
Может и немного по индуски, но работает как нужно.

Приведу решение для последующих поколений

C#

// bool checkThread = false;        // Внести в глобальные переменные, это наш сигнал завершения выполнения потока
 
public void RunCmd(object command)
        {
            Thread newThread = new Thread(delegate() { DoWork(command); });
            newThread.Start();
            while (checkThread == false)        // Цикл не дающий выполняться коду дальше пока
            {                                            //  сигнал checkThread не будет true
                Application.DoEvents();
            }
            checkThread = false;              // Возвращаем сигнал на исходную
     // ....................                         //  Продолжаем выполнять код
        }
 
public void DoWork(object command)
        {
            Process proc = new Process()
            {
                StartInfo = new ProcessStartInfo("cmd.exe", "/c" + command)
                {
                    StandardOutputEncoding = Encoding.GetEncoding(866),
                    RedirectStandardOutput = true,
                    UseShellExecute = false,
                    CreateNoWindow = true,
                    WindowStyle = ProcessWindowStyle.Hidden
                }
            };
            proc.Start();
 
            if (!proc.StartInfo.RedirectStandardOutput)
                return;
 
            string line;
            StreamReader sr = proc.StandardOutput;
            while (!sr.EndOfStream)
            {
                line = sr.ReadLine();
                richTextBox1.Invoke(new MethodInvoker(() =>
                {
                    richTextBox1.Text += line + "\n";
                }), null);
            }
            checkThread = true;     // Сигнал (функция выполнена)
        }

@Spawn · 03.06.2014, 11:34

6eH, забудьте про Application.DoEvents(), про "ручной" запуск потоков тоже, ThreadPool есть, он разберётся что создать.
что касается ожидания, Вы неправильно подходите к решению задачи, можно и просто RichTextBox для чтения сделать, коль уж нужно дождаться.

Не по теме:

Сообщение от 6eH

В общем решилась проблема проще простого.

Скорее убилась... причём с разбега

@6eH · 04.06.2014, 00:43 **[ТС]**

Был бы благодарен за наглядный пример.
Обязательно почитаю про ThreadPool.
И что значит : "... RichTextBox для чтения сделать ..."

@Spawn · 04.06.2014, 09:22

Создайте формочку Form1, киньте туда следующее:

C#

        // Элементы управления
        // кнопка запуска
        Button btnStart;
        // текстовое поле для команды
        TextBox tbCommand;
        // RichTextBox с результатами
        RichTextBox rtbResult;
        // Панелька для поля и кнопки
        Panel pnlCommads;
 
        public Form1()
        {
            InitializeComponent();
 
            // инициализируем панельку
            pnlCommads = new Panel()
            {
                Height = 22,
                // будет прибита к нижней части формы
                Dock = DockStyle.Bottom
            };
            // инициализируем кнопку
            btnStart = new Button()
            {
                Width = 60,
                // будет прибита к правой части панели
                Dock = DockStyle.Right,
                Text = "Start",
            };
            // событие запуска команды (нажатие на кнопку)
            btnStart.Click += btnStart_Click;
            // инициализируем поле для команды
            tbCommand = new TextBox()
            {
                // заполнит всё свободное пространство
                Dock = DockStyle.Fill,
                // команда по-умолчанию
                Text = "ping 127.0.0.1"
            };
            // событие изменения текста в поле
            tbCommand.TextChanged += tbCommand_TextChanged;
            // инициализируем RichTextBox
            rtbResult = new RichTextBox()
            {
                // займёт всё свободное пространство
                Dock = DockStyle.Fill,
                // только для чтения
                ReadOnly = true,
                // чтобы нельзя было переключиться Tab'ом
                TabStop = false
            };
 
            // текстовое поле и кнопку кладём на панель
            pnlCommads.Controls.AddRange(new Control[] { btnStart, tbCommand });
            // RichTextBox и панель кладём на форму
            // порядок добавления важен, так как используем свойство Dock у элементов
            this.Controls.AddRange(new Control[] { rtbResult, pnlCommads });
            // изменяем заголовок формы
            this.Text = "Программа запущена";
        }
 
        /// <summary>
        /// Событие запуска команды
        /// </summary>
        /// <param name="sender"></param>
        /// <param name="e"></param>
        void btnStart_Click(object sender, EventArgs e)
        {
            // изменяем заголовок формы
            this.Text = "Выполняем команду";
            // делаем панель недоступной, чтобы не нажали ещё разок
            pnlCommads.Enabled = false;
            // запуускаем выполнение команды в отдельном потоке
            ThreadPool.QueueUserWorkItem(new WaitCallback(DoWork), tbCommand.Text);
        }
 
        /// <summary>
        /// Выполнение команды
        /// </summary>
        /// <param name="command">Команда</param>
        private void DoWork(object command)
        {
            Process proc = new Process()
            {
                StartInfo = new ProcessStartInfo("cmd.exe", String.Format("/c \"{0}\"", command as string))
                {
                    StandardOutputEncoding = Encoding.GetEncoding(866),
                    RedirectStandardOutput = true,
                    UseShellExecute = false,
                    CreateNoWindow = true,
                    WindowStyle = ProcessWindowStyle.Hidden
                }
            };
            proc.Start();
 
            if (proc.StartInfo.RedirectStandardOutput)
            {
                string line;
                StreamReader sr = proc.StandardOutput;
                // Если процесс ещё не завершился
                while (!proc.HasExited)
                {
                    // Если есть что показать
                    while (!sr.EndOfStream)
                    {
                        line = sr.ReadLine();
                        rtbResult.Invoke(new MethodInvoker(() =>
                        {
                            rtbResult.AppendText(line + "\n");
                        }), null);
                    }
                    // даём передышку потоку, пока в консоли появятся данные
                    Thread.Sleep(10);
                }
            }
            // Сигнализируем о завершении выполнения команды
            this.Invoke(new MethodInvoker(CommandFinished));
        }
 
        /// <summary>
        /// Завершение выполнения команды
        /// </summary>
        private void CommandFinished()
        {
            // делаем панельку вновь активной
            pnlCommads.Enabled = true;
            // изменяем заголовок
            this.Text = "Команда выполнена";
        }
 
        void tbCommand_TextChanged(object sender, EventArgs e)
        {
            // если текст в поле отсутствует, делаем кнопку неактивной
            btnStart.Enabled = tbCommand.TextLength > 0;
        }

@Spawn 995 / 893 / 354 Регистрация: 24.03.2014 Сообщений: 2,381 Записей в блоге: 2
	03.06.2014, 11:34	4
	6eH, забудьте про Application.DoEvents(), про "ручной" запуск потоков тоже, ThreadPool есть, он разберётся что создать. что касается ожидания, Вы неправильно подходите к решению задачи, можно и просто RichTextBox для чтения сделать, коль уж нужно дождаться. Не по теме: Сообщение от 6eH В общем решилась проблема проще простого. Скорее убилась... причём с разбега 1

@6eH 0 / 0 / 0 Регистрация: 05.05.2014 Сообщений: 10
	04.06.2014, 00:43 [ТС]	5
	Был бы благодарен за наглядный пример. Обязательно почитаю про ThreadPool. И что значит : "... RichTextBox для чтения сделать ..." 0

	04.06.2014, 09:22