Task.Run Serial Communications

Question

Мне действительно понравилось использовать MVVM и async/await, поэтому, естественно, я взломал проект, чтобы создать новую разработку (MVVM), которая обменивается данными с устройствами по последовательному интерфейсу, и пользовательский интерфейс будет реагировать. Мои классы-драйверы - это синхронные вызовы, которые взаимодействуют с устройством по последовательному каналу и могут занимать время (хотя, как правило, нет) в зависимости от того, сколько данных возвращается, если я получаю соответствующий возврат или если драйвер должен продолжать ждать еще несколько секунд чтобы узнать, подходит ли соответствующая доходность и т. д.

В любом случае, что я закончил делать, это обматывание сообщений в ожидании Task.Run (() => {}); обертка. Таким образом, с точки зрения моего пользовательского интерфейса, когда пользователь нажимает кнопку, волшебным образом ее бросают в пул потоков, и мой пользовательский интерфейс остается отзывчивым при обработке задачи.

Является ли приведенная ниже реализация неправильной? Поскольку я общаюсь с Serial, это не моя работа, технически связанная с I/O работа вместо работы с привязкой к ЦП (Task.Run предположительно используется только для работы с ЦП, из того, что я прочитал)? Ниже метод вызывается в моей модели представления, выполнив:

var returnVal = await _objName.InitAsync(); 

Тогда вызов Task.Run ....

public async Task<bool> InitAsync() 
    { 
     bool returnVal = false; 

     await Task.Run(() => 
     { 
      System.Threading.Thread.Sleep(COMMAND_DELAY_MILLISECONDS); 
      returnVal = _obj.InitiateProvision(); 
     }); 

     return returnVal; 
    } 

Возвращаемое значение BOOL просто говорится, если команда была успешной или нет. Код работает очень хорошо, поскольку я тестировал его уже около месяца без каких-либо проблем, но я не уверен, что это лучше ... Есть ли «лучший» способ сделать это, хотя он все еще может поддерживать асинхронные операции? Любое руководство ценится!

Answer 1

Если вы используете Task.Run, просто чтобы у вас была задержка до того, как вы сделаете свою команду, просто используйте delay и не свяжите threadpool.

public async Task<bool> InitAsync() 
{ 
    bool returnVal = false; 

    await Task.Delay(COMMAND_DELAY_MILLISECONDS).ConfigureAwait(false); 
    returnVal = _obj.InitiateProvision(); 

    return returnVal; 
} 

Вы сказали, что было 3 команды, вы можете просто сделать все три команды подряд, если она подходит для вашего дизайна, вы можете иметь более одного await в функции.

public async Task<bool> InitAsync() 
{ 
    bool returnVal = false; 

    await Task.Delay(COMMAND_DELAY_MILLISECONDS).ConfigureAwait(false); 
    returnVal = _obj.InitiateProvision(); 

    if(returnVal == true) 
    { 
     await Task.Delay(COMMAND_DELAY_MILLISECONDS).ConfigureAwait(false); 
     returnVal = _obj.CommandTwo(); 
    } 

    if(returnVal == true) 
    { 
     await Task.Delay(COMMAND_DELAY_MILLISECONDS).ConfigureAwait(false); 
     returnVal = _obj.CommandThree(); 
    } 

    return returnVal; 
} 

.ConfigureAwait(false) делает это так, функция не должна возвращать в поток пользовательского интерфейса, когда он продолжает свою работу после await и может сделать программу быстрее из-за этого не необходимости ждать своей очереди в потоке пользовательского интерфейса ,