Формы Windows начинают зависать и зависать, как я могу сделать этот код эффективным?

Question

Формы Windows начинают зависать и зависать, как я могу сделать этот код эффективным?

1

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

private bool Ping(string url, Label label)
    {
        try
        {
            HttpWebRequest request = (HttpWebRequest)HttpWebRequest.Create(url);
            request.Timeout = 300;
            request.AllowAutoRedirect = false;
            request.Method = "HEAD";

            using (var response = request.GetResponse())
            {
                label.Text = "Online";
                label.ForeColor = Color.LimeGreen;
                return true;
            }
        }
        catch
        {
            label.Text = "Offline";
            label.ForeColor = Color.DarkRed;
            return false;
        }
    }

Я предполагаю, что причина, по которой приложение продолжает висит, - это из-за таймаута, когда там, по меньшей мере, на 15 сайтах Pinged потребуется некоторое время, есть ли способ, которым я могу изменить это, чтобы он не влиял на производительность приложений. Я бы хотел, чтобы Ping сайтов в таймер, так что каждые 10-30 секунд приложение снова засылает сайты.

Любые идеи относительно того, что мне нужно изменить, чтобы достичь этого?

Sandro Perez 27 авг. 2014, в 22:52

Источник

7

Вы должны делать это асинхронно, а не синхронно.
Servy 27 авг. 2014, в 20:05

Теги:

c#

optimization

performance

1 ответ

Ещё вопросы

Вы должны делать это асинхронно, а не синхронно.

empi · Accepted Answer · 2014-08-27T18-22-00.000Z

Вы наблюдаете, как приложение висит, потому что вы блокируете поток пользовательского интерфейса. Лучший способ избежать этого - использовать асинхронную модель программирования. Если вы используете С# 5 и.net 4.5, проще всего использовать ключевые слова async/await.

Вы должны изменить подпись метода на:

private async Task<bool> Ping(string url, Label label)

и изменить код на:

using (var response = await request.GetResponseAsync())

Извините, если кажется, что я спрашиваю немного, не могли бы вы привести пример?
Это выдает ошибку, ансик должен быть пустым
В моем коде произошла ошибка - тип возвращаемого значения должен быть Task <bool>, а не bool. Я исправил это.
Асинхронные методы @SandroPerez должны быть либо Task , Task<T> либо void . void должен использоваться ТОЛЬКО для обработчиков событий.
Как бы я сейчас использовал этот метод? Ping ("site.com/", lblLabel);
Как вы в настоящее время используете это? Это зависит от контекста. Если вы также можете пометить вызывающий метод как асинхронный, вы можете просто написать await Ping (...);
У меня есть каждый Ping внутри Timer_Tick, я использую его как таковой, жду Ping ("sit", lblLabel);
@SandroPerez: есть проблемы с этим? Просто отметьте Timer_Tick как асинхронный.
Да, я немного поиграл с этим и решил, что это проблема, спасибо Эмпи :)