Завершение потоков, после закрытия формы

@Good1101 · Регистрация: 28.12.2011

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

После закрытия формы в процессах продолжает висеть процесс.
Я так понимаю вот этот поток не закрывается.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
        //принятие данных от сервера
        void Receiver()
        {
            Thread th = new Thread(delegate()
            {
                while (client_running)
                {
                    try
                    {
                        byte[] bytes = new byte[1024];
                        // Принимает данные от сервера
                        client.Receive(bytes);
                        if (bytes.Length != 0)
                        {
                            string data = Encoding.UTF8.GetString(bytes);
                            textBox1.Text += data;
                        }
                    }
                    catch { }
                }
            });
            th.Start();
            threads.Add(th);
        }

Честно искал как отследить закрытие приложения чтобы попытаться убить все потоки в списке, подскажите как завершить поток при закрытии формы на крестик.

@Петррр · 30.08.2012, 20:03

Смотрите события FormClosed, FormClosing

@INF1NUM · 31.08.2012, 11:48

была у меня такая "шляпа". так и не разобрался в чём дело, просто повесил на закрытие формы

C#
1
System.Environment.Exit(1);

помогло

Добавлено через 1 минуту
кстати пустой catch не комильфо.

@KristopherRobin · 31.08.2012, 12:56

если есть доступная во всех методах ссылка на тред:

C#
1
2
3
4
5
6
private void MainForm_FormClosed(object sender, FormClosedEventArgs e)
        {
            if (thrdStart != null)  //thrdStart - глобальная ссылка на тред
                if (thrdStart.IsAlive)
                    thrdStart.Abort();
        }

@Good1101 · 01.09.2012, 00:34 **[ТС]**

всем спасибо, вроде бы ничего не делал но проблема сама пропала...

@Bigsmoke · 01.02.2014, 21:49

INF1NUM, спасибо, действительно помогает

@Grishaco · 02.02.2014, 11:15

Good1101, при работе с Thread не забывайте выставлять свойство IsBackground в true и таких проблем не будет.

@kolorotur · 02.02.2014, 16:02

Неужели только у меня возникает чувство дискомфорта от завершения работы программы методом "а гори оно всё синим пламенем"?
Это я про принудительное завершение приложения, аборт потока или предоставление потока рантайму на уничтожение по своему усмотрению.

Под рукой же имеется все что нужно для нормального завершения работы: при закрытии приложения выставляем флаг client_running на false, закрываем сокет client и вызываем Join на потоке th. Если сокет заблочен из-за чтения, то он бросает исключение при вызове Receive (исключение ловим и пишем в лог, что сокет закрыт), потом переходит на следующую итерацию, в которой не проходит условие client_running, после чего работа потока благополучно завершается.
Все счастливы, дзен достигнут.

И нет необходимости стрелять себе в голову, когда пора идти спать.

@Grishaco · 02.02.2014, 16:10

Сообщение от kolorotur

Неужели только у меня возникает чувство дискомфорта от завершения работы программы методом "а гори оно всё синим пламенем"?

Я не думаю что это принципиально, если нужно выйти из приложения. Мы же не беспокоимся если процесс закрыт через диспетчер задач. Я про IsBackground.

@kolorotur · 02.02.2014, 16:16

Сообщение от Grishaco

Я не думаю что это принципиально, если нужно выйти из приложения.

В данном конкретном примере — не принципиально, да.
Однако, лучше "с детства" приучаться к опрятности

Сообщение от Grishaco

Мы же не беспокоимся если процесс закрыт через диспетчер задач.

Оно, конечно, понятно, что при выдергивании шнура из розетки никакой finally не поможет — на то оно и экстренное/непредвиденное завершение.
Но при штатном завершении работу все-таки нужно заканчивать нормально.

@Grishaco · 02.02.2014, 16:25

Сообщение от kolorotur

Оно, конечно, понятно, что при выдергивании шнура из розетки никакой finally не поможет — на то оно и экстренное/непредвиденное завершение.
Но при штатном завершении работу все-таки нужно заканчивать нормально.

Не соглашусь, если с проектом работает несколько разработчиков и потоков запускается и завершается очень много, то пусть ставят IsBackground = true, а потом корректно завершают свои потоки. Во всяком случае это не приведет к тому что разработчик не учел некоторые моменты и процесс повис в памяти.

Добавлено через 5 минут
При этом что такое штатное завершение? А если у меня куча сборок, которые используются в моем проекте и там используются потоки, мне что каждый раз передавать флаг завершения через всю иерархию классов?

@kolorotur · 02.02.2014, 17:15

Сообщение от Grishaco

потоков запускается и завершается очень много

Количество потоков — не повод валить их все без разбора

Сообщение от Grishaco

пусть ставят IsBackground = true, а потом корректно завершают свои потоки. Во всяком случае это не приведет к тому что разработчик не учел некоторые моменты и процесс повис в памяти.

Установка IsBackground = true очень быстро приведет к ситуации "поставил и забыл", и чем больше команда и объем работы на отдельного ее члена, тем больше будет этих ситуаций.
А вот если процесс повис в памяти, то это очень хорошее напоминание, что в коде косяк, и это напоминание позволит обнаружить и пофиксить этот косяк еще на стадии тестирования, а не приводить потом к undefined behavior на клиентской системе.

Сообщение от Grishaco

что такое штатное завершение?

Это риторический вопрос?
Спрашиваю без издевки, просто чтобы зря не расписывать ответ на вопрос, на который ответа не ожидается.

Сообщение от Grishaco

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

Каждый класс сам следит за своими ресурсами — это ж азбука.
Задача разработчика, использующего стороннюю сборку, дать сигнал ее компонентам о завершении работы, например используя паттерн Disposable.
Задача разработчка, предоставляющего стороннюю сборку, предоставить интерфейс для корректного завершения ее компонентов, если таковой интерфейс необходим.

Вы отсигналили, а класс той сборки уже сам своим потокам выставил нужные флаги. И так по цепочке.
Используется много разных классов и в итоге завершение работы занимает некоторое время? Ну что ж, система значит сложная — бывает. Покажите пользователю плашку (можете с прогрессом) о том, что работа приложения завершается — подождите немножко, пожалуйста.
Виндуз вон, при каждом завершении это окно показывает и ничего, практически никто не жалуется.

Исключение — необходимость внедрить хак, когда разработчик какой-либо сборки не предоставил интерфейса для завершения работы ее классов и нет возможности связаться с разработчиком или заменить сборку на хорошую, годную.

Но чтобы не возникало необходимости в хаках, в том числе и кому-то из-за вас ("вы" в данном контексте — не конкретно Вы, а некий гипотетический разраб), я выше и написал про приучение себя к опрятности "с детства".
Погадил — будь добр, прибери за собой.

@Grishaco · 02.02.2014, 17:34

Сообщение от kolorotur

Установка IsBackground = true очень быстро приведет к ситуации "поставил и забыл", и чем больше команда и объем работы на отдельного ее члена, тем больше будет этих ситуаций.

И чем грозит ситуация поставил и забыл? Обычно пользователь прежде чем закрыть программу пытается отменить операцию, тем самым завершается поток. Просто подстраховались на случай не предвиденной ошибки, которая может помешать завершить поток.

Сообщение от kolorotur

А вот если процесс повис в памяти, то это очень хорошее напоминание, что в коде косяк, и это напоминание позволит обнаружить и пофиксить этот косяк еще на стадии тестирования, а не приводить потом к undefined behavior на клиентской системе.

Особенно прикольно, когда это воспроизводится на компе у пользователя за 4000 км и без удаленного доступа. Тестеры вроде все делают как пользователь, а поймать не могут.

Сообщение от kolorotur

Каждый класс сам следит за своими ресурсами — это ж азбука.
Задача разработчика, использующего стороннюю сборку, дать сигнал ее компонентам о завершении работы, например используя паттерн Disposable.

Т.е. вы хотите сказать, что если я использую поток в своем классе, то я обязан реализовывать IDisposable?
А если вызывается статический метод и там создается поток?

Я считаю, что если разработчик использует поток, то он должен либо предоставить пользователю отменить операции идущую в потоке, любо выставить флаг завершения потока вместе с процессом. А не то получится, я же реализовал IDisposable, что же ты не использовал using.

@Rameron · 02.02.2014, 18:54

Сообщение от kolorotur

Но при штатном завершении работу все-таки нужно заканчивать нормально.

И всё же лень - двигатель быдлокода и от этого никуда не денешься.

@ksk · 02.02.2014, 22:01

kolorotur, к сожалению, у большинства разработчиков отсутствуют инклудья, дающие способность понять необходимость писать аккуратно. Наша профессия вырождается, как выродилась в автослесарей профессия специалистов по двигателям. Культура производства в сфере ПО уже не востребована, как минимум вопрошающими здесь. Всё чаще прихожу к такому мнению.

@kolorotur · 03.02.2014, 16:18

Сообщение от Grishaco

И чем грозит ситуация поставил и забыл?

Всяко бывает: от "ничем" до "запоротым файлом" чем по-хуже. Зависит от того, что происходит в потоке.

Сообщение от Grishaco

Обычно пользователь прежде чем закрыть программу пытается отменить операцию, тем самым завершается поток.

Так это если операция все-таки подразумевает ее отмену.
Я ведь и говорю именно о разнице в подходе "отменить перед выходом" и "да пошло оно всё...".

Сообщение от Grishaco

Просто подстраховались на случай не предвиденной ошибки, которая может помешать завершить поток.

Вы меня, возможно, не так поняли. Я не говорю, что IsBackground — абсолютное зло.
Я о том, что меня коробит, когда программист даже не пытается нормально отменить параллельную операцию перед выходом.

Сообщение от Grishaco

Т.е. вы хотите сказать, что если я использую поток в своем классе, то я обязан реализовывать IDisposable?

Почему обязан? У меня же там слово "например" стоит.
Можно использовать любой удобный механизм, но механизм присутствовать должен. Банально для управления лайфтаймом объекта. А то объект заспавнил поток, сам при этом сдох, потому что больше не нужен, а поток продолжает что-то делать.

Сообщение от Grishaco

А если вызывается статический метод и там создается поток?

То создается другой статический метод для завершения работы этого потока.

Сообщение от Grishaco

Я считаю, что если разработчик использует поток, то он должен либо предоставить пользователю отменить операции идущую в потоке, либо выставить флаг завершения потока вместе с процессом.

Да кто ж спорит.
Вот у автора в коде предоставлена прекрасная возможность при выходе красиво отменить операцию, идущую в потоке. Эту возможность я описал в первом своем сообщении.
Я ж ворчу о том, что этой возможностью никто не воспользовался, почему-то все предложения были из серии либо убей, либо забей — само, мол, отвалится.

Сообщение от Grishaco

А не то получится, я же реализовал IDisposable, что же ты не использовал using.

Ну дык поэтому и встраивают защиту от идиота и закрывают ресурсы в деструкторе, если Dispose не был вызван.

@Good1101 · 12.02.2014, 22:11 **[ТС]**

Всем спасибо конечно, но прошло уже полтора года с того момента как меня интересовал данный вопрос)

@kolorotur · 13.02.2014, 11:33

Good1101, это ж проблема из серии вечных, она никогда не стареет

@DataPlanner · 13.02.2014, 11:41

Good1101, неправильно написан код работы лисенера

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .	SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .

@Петррр 6721 / 3570 / 900 Регистрация: 28.10.2010 Сообщений: 5,937
	30.08.2012, 20:03
	Смотрите события FormClosed, FormClosing 0

@Good1101 63 / 62 / 11 Регистрация: 28.12.2011 Сообщений: 280
	01.09.2012, 00:34 [ТС]
	всем спасибо, вроде бы ничего не делал но проблема сама пропала... 0

@Bigsmoke 0 / 0 / 0 Регистрация: 30.03.2017 Сообщений: 3
	01.02.2014, 21:49
	INF1NUM, спасибо, действительно помогает 0

@Grishaco 438 / 362 / 100 Регистрация: 29.06.2010 Сообщений: 981 Записей в блоге: 1
	02.02.2014, 11:15
	Good1101, при работе с Thread не забывайте выставлять свойство IsBackground в true и таких проблем не будет. 0

@kolorotur 17823 / 12973 / 3382 Регистрация: 17.09.2011 Сообщений: 21,261
	02.02.2014, 16:02
	Неужели только у меня возникает чувство дискомфорта от завершения работы программы методом "а гори оно всё синим пламенем"? Это я про принудительное завершение приложения, аборт потока или предоставление потока рантайму на уничтожение по своему усмотрению. Под рукой же имеется все что нужно для нормального завершения работы: при закрытии приложения выставляем флаг client_running на false, закрываем сокет client и вызываем Join на потоке th. Если сокет заблочен из-за чтения, то он бросает исключение при вызове Receive (исключение ловим и пишем в лог, что сокет закрыт), потом переходит на следующую итерацию, в которой не проходит условие client_running, после чего работа потока благополучно завершается. Все счастливы, дзен достигнут. И нет необходимости стрелять себе в голову, когда пора идти спать. 1

@ksk 624 / 495 / 43 Регистрация: 05.07.2010 Сообщений: 1,589
	02.02.2014, 22:01
	kolorotur, к сожалению, у большинства разработчиков отсутствуют инклудья, дающие способность понять необходимость писать аккуратно. Наша профессия вырождается, как выродилась в автослесарей профессия специалистов по двигателям. Культура производства в сфере ПО уже не востребована, как минимум вопрошающими здесь. Всё чаще прихожу к такому мнению. 0

@Good1101 63 / 62 / 11 Регистрация: 28.12.2011 Сообщений: 280
	12.02.2014, 22:11 [ТС]
	Всем спасибо конечно, но прошло уже полтора года с того момента как меня интересовал данный вопрос) 0

@kolorotur 17823 / 12973 / 3382 Регистрация: 17.09.2011 Сообщений: 21,261
	13.02.2014, 11:33
	Good1101, это ж проблема из серии вечных, она никогда не стареет 0

@DataPlanner 158 / 187 / 48 Регистрация: 25.11.2013 Сообщений: 978
	13.02.2014, 11:41
	Good1101, неправильно написан код работы лисенера 0

Завершение потоков, после закрытия формы

Решение