Вывод результатов PING в Label.Text

@DemoDiplom · Регистрация: 11.05.2016

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

Решил начать изучение C# с простенькой проги для себя по выводу индикатора результов команды Ping. Почитал форум, приспособил код для себя немного. Работает, но форма зависает. НАсоклько я понял это из-за того что программа выполняет функцию ping_server() в главном потоке. Пытаюсь вынести в отдельный поток, но тогда выражение не срабатывате и результат не выводится.

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
25
26
  private void button1_Click(object sender, EventArgs e)
        {
            
           Task.Run(() => ping_server());
            Task.Run(() => ping_server()).Wait();
            //ping_server();
            //Thread newThread = new Thread(ping_server);
            //newThread(ping_server()).Start();
        }
        private void ping_server()
        {
            string path = AppDomain.CurrentDomain.BaseDirectory + "Serverconfig.ini";
            INIManager ipserv = new INIManager(path);
            string ipadress = ipserv.GetPrivateString("serv", "ip2");
 
            var Ping = new Ping();
             System.Windows.Forms.Timer timer = new System.Windows.Forms.Timer() { Interval = 250 };
   
 
            timer.Tick += (sender, e) => label2.Text = Ping.Send(ipadress).RoundtripTime < 1
                                             ? "good"
                                              : "no good";
            timer.Start();
            Thread.Sleep(100);
            
        }

Подскажите пжлст, что я делаю не так? Может где какую строку кода пропустил?

@Элд Хасп · 29.11.2019, 21:50

DemoDiplom, обновление UI элементов должно происходить в основном потоке.
Для этого лучше использовать асинхронные методы.

Экземпляр таймера нужно объявить как член (поле) типа.

Метод для пинга должен возвращать значение пинга и сам он не должен создавать потоки.

После асинхронного вызова метода, управление вернётся в основной поток и уже в нем надо присвоить значение UI элементу.

@DemoDiplom · 29.11.2019, 22:16 **[ТС]**

Все отлично, но я пока настолько глубоко не понимаю C#. А можно как то на примере показать пжлст?) Читаю литературу но пока мало что понимаю, пытаюсь вот на примерах и практики что то понять)

@Элд Хасп · 30.11.2019, 10:25

DemoDiplom, с WinForms давно не работал, сейчас я в поездке и без компа до НГ.
Попробую частично накидать в коде для прояснения, но полный рабочий код не смогу сделать.

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
25
26
27
28
29
30
31
32
33
34
35
  private readonly Timer timer = new System.Windows.Forms.Timer(); 
  private void button1_Click(object sender, EventArgs e)
        {
            ping_server();
           //Task.Run(() => ping_server());
            //Task.Run(() => ping_server()).Wait();
            //ping_server();
            //Thread newThread = new Thread(ping_server);
            //newThread(ping_server()).Start();
        }
        private string ipadress;
        private Ping ping;
        private void ping_server()
        {
            string path = AppDomain.CurrentDomain.BaseDirectory + "Serverconfig.ini";
            INIManager ipserv = new INIManager(path);
            ipadress = ipserv.GetPrivateString("serv", "ip2");
 
            ping = new Ping();
             timer.Interval = 250;
   
 
            timer.Tick += TimerTickAsync;
            timer.Start();
           // Thread.Sleep(100);
            
        }
 
        private async void TimerTickAsync(object sender, EventArgs e)
        {
             int pingValue = await Task.Run(()=>ping.Send(ipadress).RoundtripTime);
             label2.Text = pingValue < 1
                                             ? "good"
                                              : "no good";
         }

@DemoDiplom · 30.11.2019, 10:48 **[ТС]**

Попробовал, но получились ошибки, в коде вынес в комменты, где ругается компилятор:

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
25
26
27
28
29
30
31
32
33
34
35
36
private void button1_Click(object sender, EventArgs e)
        {
            //Task.Run(() => ping_server());
            ping_server();
 
 
            //Thread newThread = new Thread(ping_server);
            //newThread.Start();
 
        }
        private string ipadress;
        private Ping ping;
        public void ping_server()
 
        {
            string path = AppDomain.CurrentDomain.BaseDirectory + "Serverconfig.ini";
            INIManager ipserv = new INIManager(path);
            string ipadress = ipserv.GetPrivateString("serv", "ip2");
            ping = new Ping();
            System.Windows.Forms.Timer timer = new System.Windows.Forms.Timer();
            timer.Interval = 250;
            timer.Tick += TimerTickAsync; // тут ругался на timer что он не существует в данном контексте поэтому объявил его выше
            timer.Start();
            
 
 
        }
 
        private async void TimerTickAsync(object sender, EventArgs e) {
 
            int pingValue = await Task.Run(() => ping.Send(ipadress).RoundtripTime); //в этой строке ругается что "Неудается явно преобразовать тип "Long" в "Int". Существует явное преобразование(возможно, пропущено приведение типов) "
 
            label2.Text = pingValue < 1
                                           ? "Good"
                                           : "No good";
        }

@Элд Хасп · 30.11.2019, 19:26

DemoDiplom, таймер должен быть полем класса только для чтения.
Может вы пропустили первую строку из моего кода?
Уберите объявление таймера из метода - обязательно!

Добавлено через 3 минуты
DemoDiplom, тип свойства RoundtripTime не смотрел.
Если оно long, то поменяйте тип переменной.
Это ни на что не влияет.

C#
31
          long pingValue = await Task.Run(() => ping.Send(ipadress).RoundtripTime); //в этой строке ругается что "Неудается явно преобразовать тип "Long" в "Int". Существует явное преобразование(возможно, пропущено приведение типов) "

@DemoDiplom · 01.12.2019, 08:59 **[ТС]**

Сообщение от Элд Хасп

DemoDiplom, таймер должен быть полем класса только для чтения.
Может вы пропустили первую строку из моего кода?
Уберите объявление таймера из метода - обязательно!
Добавлено через 3 минуты
DemoDiplom, тип свойства RoundtripTime не смотрел.
Если оно long, то поменяйте тип переменной.
Это ни на что не влияет.
C#Выделить код
31
long pingValue = await Task.Run(() => ping.Send(ipadress).RoundtripTime); //в этой строке ругается что "Неудается явно преобразовать тип "Long" в "Int". Существует явное преобразование(возможно, пропущено приведение типов) "

ДА я пропустил первую строку. Испарвил все по вашим рекомендациям. КОд заработал, но программа ругается.
System.InvalidOperationException: "Уже выполняется асинхронный вызов. Он должен быть завершен или отменен, прежде чем можно будет вызвать этот метод."

C#
1
     long pingValue = await Task.Run(() => ping.Send("10.10.60.3").RoundtripTime); //ругается в этйо строке

@Элд Хасп · 01.12.2019, 13:10

DemoDiplom, значит таймер работаем быстрое чем успевает метод вернут значение.
Измените метод таймера

C#
29
30
31
32
33
34
35
36
37
38
        private async void TimerTickAsync(object sender, EventArgs e) 
        {
            timer. Stop ();
            long pingValue = await Task.Run(() => ping.Send(ipadress).RoundtripTime); //в этой строке ругается что "Неудается явно преобразовать тип "Long" в "Int". Существует явное преобразование(возможно, пропущено приведение типов) "
 
            label2.Text = pingValue < 1
                                           ? "Good"
                                           : "No good";
             timer. Start ();
        }

@DemoDiplom · 02.12.2019, 10:52 **[ТС]**

C#
1
2
3
4
5
6
7
8
9
10
        private async void TimerTickAsync(object sender, EventArgs e) 
        {
            timer. Stop ();
            long pingValue = await Task.Run(() => ping.Send(ipadress).RoundtripTime); //в этой строке ругается что "Неудается явно преобразовать тип "Long" в "Int". Существует явное преобразование(возможно, пропущено приведение типов) "
 
            label2.Text = pingValue < 1
                                           ? "Good"
                                           : "No good";
             timer. Start ();
        }

Сделал так. Все ранв овыдает ту же ошибку System.InvalidOperationException: "Уже выполняется асинхронный вызов. Он должен быть завершен или отменен, прежде чем можно будет вызвать этот метод."

Добавлено через 16 секунд
Сайт глючит сильно. Вобщем код работает. Ошибку выдает при повторном нажатии на button1. НАверно надо в исключениях обрабатывать данную ситуацию?!)
ЗА помощь огромное спасибо, буду дальше ковыряться.

@Элд Хасп · 02.12.2019, 11:58

DemoDiplom, нельзя дважды подряд вызывать ping_server().
В нём есть присоединение метода timer.Tick += TimerTickAsync;.
И получается в одном Tick этот метод вызывается дважды - ловите исключение.
Для WinForms оптимальным будет добавить на форму таймер и там сразу указать метод для Tick.
А из кода убрать поле таймера и присоединение метода.

Также надо добавить функционал на кнопку.
Если таймер не запущен - кнопка его запускает.
А если запущен - кнопка его останавливает.

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

@DemoDiplom · 02.12.2019, 22:54 **[ТС]**

Так и сделал) спасибо за помощь и разьяснения)

Новые блоги и статьи Все статьи Все блоги /
Установка Qt Creator для C и C++: ставим среду, CMake и MinGW без фреймворка Qt 8Observer8 05.04.2026 Среду разработки Qt Creator можно установить без фреймворка Qt. Есть отдельный репозиторий для этой среды: https:/ / github. com/ qt-creator/ qt-creator, где можно скачать установщик, на вкладке Releases:. . .	AkelPad-скрипты, структуры, и немного лирики.. testuser2 05.04.2026 Такая программа, как AkelPad существует уже давно, и также давно существуют скрипты под нее. Тем не менее, прога живет, периодически что-то не спеша дополняется, улучшается. Что меня в первую очередь. . .	Отображение реквизитов в документе по условию и контроль их заполнения Maks 04.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеСпецтехники", разработанного в конфигурации КА2. Данный документ берёт данные из другого нетипового документа. . .	Фото всей Земли с борта корабля Orion миссии Artemis II kumehtar 04.04.2026 Это первое подобное фото сделанное человеком за 50 лет. Снимок называют новым вариантом легендарной фотографии «The Blue Marble» 1972 года, сделанной с борта корабля «Аполлон-17». Новое фото. . .
Вывод диалогового окна перед закрытием, если документ не проведён Maks 04.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: реализовать программный контроль на предмет проведения документа. . .	Программный контроль заполнения реквизитов табличной части документа Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: 1. Реализовать контроль заполнения реквизита. . .	wmic не является внутренней или внешней командой Maks 02.04.2026 Решение: DISM / Online / Add-Capability / CapabilityName:WMIC~~~~ Отсюда: https:/ / winitpro. ru/ index. php/ 2025/ 02/ 14/ komanda-wmic-ne-naydena/	Программная установка даты и запрет ее изменения Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: при создании документов установить период списания автоматически. . .

@Элд Хасп Модератор 16141 / 11264 / 2888 Регистрация: 21.04.2018 Сообщений: 33,120 Записей в блоге: 2
	29.11.2019, 21:50
	DemoDiplom, обновление UI элементов должно происходить в основном потоке. Для этого лучше использовать асинхронные методы. Экземпляр таймера нужно объявить как член (поле) типа. Метод для пинга должен возвращать значение пинга и сам он не должен создавать потоки. После асинхронного вызова метода, управление вернётся в основной поток и уже в нем надо присвоить значение UI элементу. 1

@DemoDiplom 0 / 0 / 0 Регистрация: 11.05.2016 Сообщений: 9
	29.11.2019, 22:16 [ТС]
	Все отлично, но я пока настолько глубоко не понимаю C#. А можно как то на примере показать пжлст?) Читаю литературу но пока мало что понимаю, пытаюсь вот на примерах и практики что то понять) 0

@Элд Хасп Модератор 16141 / 11264 / 2888 Регистрация: 21.04.2018 Сообщений: 33,120 Записей в блоге: 2
	02.12.2019, 11:58
	DemoDiplom, нельзя дважды подряд вызывать ping_server(). В нём есть присоединение метода `timer.Tick += TimerTickAsync;`. И получается в одном Tick этот метод вызывается дважды - ловите исключение. Для WinForms оптимальным будет добавить на форму таймер и там сразу указать метод для Tick. А из кода убрать поле таймера и присоединение метода. Также надо добавить функционал на кнопку. Если таймер не запущен - кнопка его запускает. А если запущен - кнопка его останавливает. Получится, что перед повторным запуском сначала таймер будет остановлен. 1

@DemoDiplom 0 / 0 / 0 Регистрация: 11.05.2016 Сообщений: 9
	02.12.2019, 22:54 [ТС]
	Так и сделал) спасибо за помощь и разьяснения) 0

Вывод результатов PING в Label.Text

Решение