2

Есть форма с button и Label. При нажатии на кнопку выполняется метод button1_Click:

    private async void button1_Click(object sender, EventArgs e)
    {
        // Список IP адресов.
        List<string> ListIP = new List<string>();

        // Метод getListPC(timeout) возвращает список ip адресов.
        ListIP = getListPC(50);

        // WMI-Класс, который позволяет получать информацию о ПК по IP:
        // Имя, процессор, материнская плата, список установленных программ и др.
        WMI current = new WMI();
        foreach (string el in ListIP)
        {
            label3.Text = el;
            // CheckАccess(IP) - метод класса WMI, позволяет проверить доступность ПК.
            await current.CheckАccess(el);
        }
    }

Код класса WMI

class WMI
{
    Logger logger = LogManager.GetCurrentClassLogger();

    private ConnectionOptions Connection { get; set; } = new ConnectionOptions();

    public Task<bool> CheckАccess(string ComputerName)
    {
        try
        {

            ManagementObjectSearcher searcher =
                new ManagementObjectSearcher("\\\\" + ComputerName + "\\root\\CIMV2", "SELECT * FROM Win32_ComputerSystem");
            ManagementScope scope =
                new ManagementScope("\\\\" + ComputerName + "\\root\\CIMV2", Connection);
            scope.Connect();
            searcher.Scope = scope;
            searcher.Get();
            return Task.FromResult(true);
        }
        catch (Exception e)
        {
            //MessageBox.Show(e.Message);
            logger.Trace(e.StackTrace + " " + e.Message);
            return Task.FromResult(false);
        }
    }
}

Во время подключения к ПК scope.Connect() в экземпляре класса WMI происходит блокировка UI. Почему это происходит? Разве перебор цикла foreach не должен быть продолжен без ожидания завершения метода CheckАccess?

3
  • 2
    scope.Connect(); ... return Task.FromResult(true); - никакой асинхронности (читай ухода в другой поток кроме UI потока), а просто костыль для впихивания в TPL
    – vitidev
    3 апр 2017 в 13:48
  • 1
    Ближайший костыль - обернуть вызов CheckАccess или внутри самого CheckАccess (по необходимости) в Task.Run(). По нормальному - переписать доступ к ManagementScope в асихнронный вид (если он это поддерживает).
    – vitidev
    3 апр 2017 в 13:54
  • Перебор цикла ждёт завершения каждого элемента. Вы об этом явно говорите, когда пишите await. Хотя блокирует вас не это =)
    – Monk
    3 апр 2017 в 14:19

2 ответа 2

6

Тот факт, что метод CheckАccess возвращает Task и вы рядом с его вызовом написали await, не делает его асинхронным. Почитайте краткое изложение того, что делает async/await, посмотрите видео.

Проблема в том, что у вас весь метод CheckАccess выполняется синхронно. Варианта решения проблемы, как уже подсказали, два:

  1. Переписать вызов WMI в асинхронный вид, если это возможно.
  2. Обернуть вызов CheckАccess или вызов WMI самого CheckАccess в Task.Run().
4

Лучшим вариантом будет использование Task.Run, поскольку вы тогда явно укажете кодом, что хотите выполнять данный метод на потоке из ThreadPool, и что контекст UI вам в нем не интересен, и что это обертка над синхронным методом. Это если у классов ManagementObjectSearcher и ManagementScope нет асинхронных методов.

await Task.Run(()=>current.CheckАccess(el));
6
  • Совет про ConfigureAwait и Task.Delay -- плохой. Там в цикле идет обращение к UI, а значит контекст должен сохраняться. Уберите первую часть ответа, не путайте людей.
    – andreycha
    3 апр 2017 в 14:02
  • @andreycha Не совсем согласен с тем, что контекст сохранится - ConfigureAwait(false) по идее заставит запланировать задачу в пуле. Ну а сам хак грязный.
    – vitidev
    3 апр 2017 в 14:09
  • 1
    Убрал хак из ответа:) 3 апр 2017 в 14:20
  • @vitidev я не говорю, что контекст сохранится. Я говорю, что он должен сохраняться, потому что после асинхронной операции идет обращение к UI (которое должно быть в UI-контексте).
    – andreycha
    3 апр 2017 в 14:24
  • 1
    Я проглядел обращение к Label, тогда да, тот хак работать не будет. 3 апр 2017 в 14:26

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.