Часто, при пинге сервера программа подвисает на 1-2 секунды и выдаёт значение пинга 0

@OpenGoodwin · Регистрация: 15.01.2014

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

Создал программу для проверки пинга. Часто, при пинге сервера программа подвисает на 1-2 секунды и выдаёт значение пинга 0. Есть ли какое-то исключение, которое можно заточить в try catch, что бы программа не зависала?

@OwenGlendower · 22.04.2015, 14:27

OpenGoodwin, "зависание" программы говорит что ты делаешь ping в главном потоке приложения. Выполняй этот код в отдельном потоке и зависаний не будет. Ну и надо отметить что никакой try/catch здесь не поможет.

Что касается нулевого пинга, то нужно смотреть как ты его реализовал.

@OpenGoodwin · 22.04.2015, 14:30 **[ТС]**

OwenGlendower, даю отрезок кода.

C#

try
                    {
                        System.Net.NetworkInformation.Ping ping = new System.Net.NetworkInformation.Ping();
                        System.Net.NetworkInformation.PingReply pingReply = ping.Send("login.worldofwarships.ru");
                        if (pingReply.RoundtripTime <= 70)
                        {
                            label32.ForeColor = System.Drawing.Color.Green;
                        }
                        else if (pingReply.RoundtripTime >= 70 && pingReply.RoundtripTime <= 120)
                        {
                            label32.ForeColor = System.Drawing.Color.Orange;
                        }
                        else if (pingReply.RoundtripTime >= 120)
                        {
                            label32.ForeColor = System.Drawing.Color.Red;
                        }
                        label32.Text = pingReply.RoundtripTime.ToString();
                    }
                    catch (PingException)
                    {
                        label32.ForeColor = System.Drawing.Color.Red;
                        label32.Text = "Ошибка";
                    }

@OwenGlendower · 22.04.2015, 14:40

OpenGoodwin, ты отправляешь ping только один раз. Похоже что иногда он не проходит и поэтому RoundtripTime равен 0. Тут возможны два решения - простое и правильное

Простое - добавить проверку свойства Status. Если оно равно IPStatus.Success, то тогда можно выводить RoundtripTime, иначе ошибка. Правильное решение это отправлять ping несколько раз подряд чтобы накопить статистику. Тебе уже приводили пример для отправки нескольких ping-ов. Модифицируй его чтобы еще накапливалось среднее время ответа и ориентируйся на него.

@dok221 · 22.04.2015, 14:52

Кто мешает сделать вот так:

C#

try
            {
                System.Net.NetworkInformation.Ping ping = new System.Net.NetworkInformation.Ping();
                System.Net.NetworkInformation.PingReply pingReply = ping.Send("login.worldofwarships.ru");
                if (pingReply.RoundtripTime <= 70 && pingReply.RoundtripTime >= 1)
                {
                    label32.ForeColor = System.Drawing.Color.Green;
                }
                else if (pingReply.RoundtripTime >= 70 && pingReply.RoundtripTime <= 120)
                {
                    label32.ForeColor = System.Drawing.Color.Orange;
                }
                else if (pingReply.RoundtripTime >= 120)
                {
                    label32.ForeColor = System.Drawing.Color.Red;
                }
                else if (pingReply.RoundtripTime == 0)
                {
                    label32.Text = "Сервер умер :)";
                }
                if (pingReply.RoundtripTime != 0)
                {
                    label32.Text = pingReply.RoundtripTime.ToString();
                }
             }
            catch (PingException)
            {
                label32.ForeColor = System.Drawing.Color.Red;
                label32.Text = "Ошибка";
            }

Добавлено через 8 минут
Вывести это в отдельный метод pingtoserver, вызвать его в новом потоке:

C#

1 2	Thread pingThread = new Thread(pingtoserver); pingThread.Start();

@OpenGoodwin · 22.04.2015, 15:30 **[ТС]**

OwenGlendower, спасибо

Буду разбираться с тем кодом.

Добавлено через 35 минут
dok221, Вы забыли указать, что нужно так же реализовывать систему Invoke. Ведь поток не может получить доступ к label из другого потока. Ну да ладно, я с этим помаялся. Сделал. Теперь другая проблема. При инвоке не выводится пинг:

C#

1	if (label33.InvokeRequired) label33.Invoke(new Action(() => pingReply.RoundtripTime.ToString()));

@OwenGlendower · 22.04.2015, 15:35

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

if (label33.InvokeRequired) label33.Invoke(new Action(() => pingReply.RoundtripTime.ToString()));

Это равносильно коду:

C#

1	pingReply.RoundtripTime.ToString();

То есть результат ToString() уходит в никуда. Надо так:

C#

1	label33.Invoke(new Action(() => { label33.Text = pingReply.RoundtripTime.ToString(); } ));

@OpenGoodwin · 22.04.2015, 15:39 **[ТС]**

OwenGlendower, точно, спасибо.

@OpenGoodwin · 22.04.2015, 15:54 **[ТС]**

А что это за ошибка?

@OwenGlendower · 22.04.2015, 16:27

OpenGoodwin, для работы Invoke() требуется чтобы элемент управления был создан и для окна работал цикл обработки сообщений. Ты очевидно вызываешь метод ru3 слишком "рано" - скорее всего из конструктора. Советую перенести его вызов в обработчик события Load формы.

P.S. Судя по тому что я вижу на скриншоте у тебя несколько функций ruN которые делают по сути одно и тоже. Советую переписать код так чтобы повторений было как можно меньше потому что длинные простыни кода состоящие из copy-paste тяжело поддерживать. Как пример посмотри на еще одно старое обсуждение.

@OpenGoodwin · 22.04.2015, 16:34 **[ТС]**

OwenGlendower, я создал несколько методов(а точнее - 9). И вызывал поочерёдно их в потоках в обработчике нажатия на кнопку.

C#

Thread PingRU1 = new Thread(ru1);
                    Thread PingRU2 = new Thread(ru2);
                    Thread PingRU3 = new Thread(ru3);
                    Thread PingRU4 = new Thread(ru4);
                    Thread PingRU5 = new Thread(ru5);
                    Thread PingRU6 = new Thread(ru6);
                    Thread PingRU7 = new Thread(ru7);
                    Thread PingRU8 = new Thread(ru8);
                    Thread PingRU9 = new Thread(ru9);
                    //ru1
                    PingRU1.Start();
 
                    //ru2
                    PingRU2.Start();
 
                    //ru3
                    PingRU3.Start();
 
                    //ru4
                    PingRU4.Start();
 
                    //ru5
                    PingRU5.Start();
 
                    //ru6
                    PingRU6.Start();
 
                    //ru7
                    PingRU7.Start();
 
                    //ru8
                    PingRU8.Start();
 
                    //ru9
                    PingRU9.Start();

Сделал это специально для тех случаев, когда один сервер не работает. Получается так, что все сервера уже отправили запрос, получили ответ и готовы выдать значение на экран, но им приходится ждать timeout неработающего сервера.

@OwenGlendower · 22.04.2015, 17:03

OpenGoodwin, это не оправдание для дублирования кода. Методу Thread.Start можно передать параметр который будет передан методу потока. С его помощью можно написать универсальный метод для всех ситуаций. Примерно так:

Поток

C#

static void DoPing(object o)
{
    PingData data = (PingData)o;
    try
    {
        Ping ping = new Ping();
        PingReply pingReply = ping.Send(data.Address);
        if (pingReply.RoundtripTime <= 70 && pingReply.RoundtripTime >= 1)
        {
            data.Label.ForeColor = System.Drawing.Color.Green;
        }
        else if (pingReply.RoundtripTime >= 70 && pingReply.RoundtripTime <= 120)
        {
            data.Label.ForeColor = System.Drawing.Color.Orange;
        }
        else if (pingReply.RoundtripTime >= 120)
        {
            data.Label.ForeColor = System.Drawing.Color.Red;
        }
        else if (pingReply.RoundtripTime == 0)
        {
            data.Label.Text = "Сервер умер :)";
        }
        if (pingReply.RoundtripTime != 0)
        {
            data.Label.Text = pingReply.RoundtripTime.ToString();
        }
    }
    catch (PingException)
    {
        data.Label.ForeColor = System.Drawing.Color.Red;
        data.Label.Text = "Ошибка";
    }    
}

Вызов

C#

new Thread(DoPing).Start(new PingData { Address = "server1", Label = label1 } );
new Thread(DoPing).Start(new PingData { Address = "server2", Label = label2 } );
new Thread(DoPing).Start(new PingData { Address = "server3", Label = label3 } );
new Thread(DoPing).Start(new PingData { Address = "server4", Label = label4 } );
new Thread(DoPing).Start(new PingData { Address = "server5", Label = label5 } );

@OpenGoodwin · 22.04.2015, 17:11 **[ТС]**

OwenGlendower, хм, интересно реализовано. Только вот теперь другая проблема. Студия не распознаёт объект PingData. Жалуется на то, что не подключена директива, хотя она включена:

C#

1	Error 6 The type or namespace name 'PingData' could not be found (are you missing a using directive or an assembly reference?)

C#

1	using System.Net.NetworkInformation;

@OwenGlendower · 22.04.2015, 17:13

OpenGoodwin, ой

Забыл привести его код:

C#

class PingData
{
    public string Address;
    public Label Label;
}

@OpenGoodwin · 22.04.2015, 17:20 **[ТС]**

OwenGlendower, спасибо ОГРОМНОЕ за то, что помогаете. Добавил инвоки, всё заработало

@OwenGlendower Администратор 17696 / 13823 / 5313 Регистрация: 17.03.2014 Сообщений: 28,340 Записей в блоге: 1
	22.04.2015, 16:27	10
	OpenGoodwin, для работы Invoke() требуется чтобы элемент управления был создан и для окна работал цикл обработки сообщений. Ты очевидно вызываешь метод ru3 слишком "рано" - скорее всего из конструктора. Советую перенести его вызов в обработчик события Load формы. P.S. Судя по тому что я вижу на скриншоте у тебя несколько функций ruN которые делают по сути одно и тоже. Советую переписать код так чтобы повторений было как можно меньше потому что длинные простыни кода состоящие из copy-paste тяжело поддерживать. Как пример посмотри на еще одно старое обсуждение. 0

Часто, при пинге сервера программа подвисает на 1-2 секунды и выдаёт значение пинга 0

Решение

Решение

@OpenGoodwin 7 / 7 / 3 Регистрация: 15.01.2014 Сообщений: 459
		1
	Часто, при пинге сервера программа подвисает на 1-2 секунды и выдаёт значение пинга 0 22.04.2015, 14:08. Показов 2248. Ответов 14 Метки icmp (Все метки) Создал программу для проверки пинга. Часто, при пинге сервера программа подвисает на 1-2 секунды и выдаёт значение пинга 0. Есть ли какое-то исключение, которое можно заточить в try catch, что бы программа не зависала? 0

@OwenGlendower Администратор 17696 / 13823 / 5313 Регистрация: 17.03.2014 Сообщений: 28,340 Записей в блоге: 1
	22.04.2015, 14:27	2
	OpenGoodwin, "зависание" программы говорит что ты делаешь ping в главном потоке приложения. Выполняй этот код в отдельном потоке и зависаний не будет. Ну и надо отметить что никакой try/catch здесь не поможет. Что касается нулевого пинга, то нужно смотреть как ты его реализовал. 0

@OwenGlendower Администратор 17696 / 13823 / 5313 Регистрация: 17.03.2014 Сообщений: 28,340 Записей в блоге: 1
	22.04.2015, 14:40	4
	Сообщение было отмечено OpenGoodwin как решение Решение OpenGoodwin, ты отправляешь ping только один раз. Похоже что иногда он не проходит и поэтому RoundtripTime равен 0. Тут возможны два решения - простое и правильное Простое - добавить проверку свойства Status. Если оно равно IPStatus.Success, то тогда можно выводить RoundtripTime, иначе ошибка. Правильное решение это отправлять ping несколько раз подряд чтобы накопить статистику. Тебе уже приводили пример для отправки нескольких ping-ов. Модифицируй его чтобы еще накапливалось среднее время ответа и ориентируйся на него. 1

@OpenGoodwin 7 / 7 / 3 Регистрация: 15.01.2014 Сообщений: 459
	22.04.2015, 15:39 [ТС]	8
	OwenGlendower, точно, спасибо. 0

@OpenGoodwin 7 / 7 / 3 Регистрация: 15.01.2014 Сообщений: 459
	22.04.2015, 15:54 [ТС]	9
	А что это за ошибка? Миниатюры 0

@OpenGoodwin 7 / 7 / 3 Регистрация: 15.01.2014 Сообщений: 459
	22.04.2015, 17:20 [ТС]	15
	OwenGlendower, спасибо ОГРОМНОЕ за то, что помогаете. Добавил инвоки, всё заработало 0