Асинхронный метод MVVM Light Call для свойства изменен?
Итак, я немного покопался и нигде не смог найти реального окончательного ответа на этот вопрос.
Я пишу приложение, используя MVVM Light и WPF. У меня есть служба, внедренная в мою ViewModel, которая проверяет действительность определенного установленного свойства. Сервис совершает веб-вызов, и он асинхронный. Вызову не нужно останавливать выполнение приложения, и он предназначен исключительно для обеспечения визуальной обратной связи с пользователем относительно правильности введенного значения.
Таким образом, мне удалось кое-что взломать, чтобы заставить его работать, но это выглядит несколько хакерским.
Как правильно выполнить асинхронный метод при изменении свойства, не прибегая к чему-то вроде async void?
Вот что у меня есть на данный момент.
public string OmmaLicenseNumber
{
get => _ommaLicenseNumber;
set
{
Set(() => OmmaLicenseNumber, ref _ommaLicenseNumber, value);
Patient.OmmaLicenseNumber = value;
var _ = CheckLicenseValid();
}
}
private async Task CheckLicenseValid()
{
var valid = await _licenseValidationService.IsValidAsync(OmmaLicenseNumber);
// We don't want the UI piece showing up prematurely. Need 2 properties for this;
LicenseValid = valid;
LicenseInvalid = !valid;
}
Если я просто попытаюсь вызвать .Result для асинхронного метода, это приведет к тупиковой ситуации, для устранения которой потребуется перезапуск приложения. И хотя то, что у меня есть, работает, я не особо фанат. Каковы мои другие варианты?
Ответы 3
Проблема здесь не в том, как выполнить async задачу незаметно, а в том, что вы делаете с исключениями. Я говорю это, потому что они могут появиться, когда задача будет решена.
В идеале просто покажите следующему читателю, что он получит. Так как вы против использования async void.
Опция 1
// running an async method unobserved
Task.Run(async () =>
{
try
{
await DoSomethingAsync();
}
catch (Exception e)
{
// Deal with unobserved exceptions
// or there will be dragons
}
});
Note : This is technically offloading (it will lose the context BEWARE) and the async lamda makes it an async void anyway, however you will have to deal with the exceptions in any case though
Вариант 2 и более противоречиво:
public async void DoSomethingFireAndForget()
{
try
{
await DoSomethingAsync();
}
catch (Exception e)
{
// Deal with unobserved exceptions
// or the will be dragons
}
}
Вариант 3 лучшее из обоих миров:
Note : Use whatever plumbing you need to observe the exceptions, ie Action etc
public static class TaskUtils
{
#pragma warning disable RECS0165 // Asynchronous methods should return a Task instead of void
public static async void FireAndForgetSafeAsync(this Task task, Action<Exception> onErrors = null)
#pragma warning restore RECS0165 // Asynchronous methods should return a Task instead of void
{
try
{
await task;
}
catch (Exception ex)
{
onErrors?.Invoke(ex);
}
}
}
Обработчики событий позволяют использовать async void
Ссылка Async/Await — лучшие практики асинхронного программирования
public string OmmaLicenseNumber {
get => _ommaLicenseNumber;
set {
Set(() => OmmaLicenseNumber, ref _ommaLicenseNumber, value);
Patient.OmmaLicenseNumber = value;
//Assuming event already subscribed
//i.e. OmmaLicenseNumberChanged += OmmaLicenseNumberChanged;
OmmaLicenseNumberChanged(this,
new LicenseNumberEventArgs { LicenseNumber = value }); //Raise event
}
}
private event EventHandler<LicenseNumberEventArgs> OmmaLicenseNumberChanged = delegate { };
private async void OnOmmaLicenseNumberChanged(object sender, LicenseNumberEventArgs args) {
await CheckLicenseValid(args.LicenseNumber); //<-- await async method call
}
private async Task CheckLicenseValid(string licenseNumber) {
var valid = await _licenseValidationService.IsValidAsync(licenseNumber);
// We don't want the UI piece showing up prematurely. Need 2 properties for this;
LicenseValid = valid;
LicenseInvalid = !valid;
}
//...
public class LicenseNumberEventArgs : EventArgs {
public string LicenseNumber { get; set; }
}
Считаю ли я, что это немного жестко?
да. Это просто пример, чтобы показать, что это выполнимо.
Можно ли преобразовать это в более простой вызов вспомогательного/служебного метода?
да. Может выглядеть как ожидаемый шаблон обратного вызова с использованием выражений для получения значения для проверки.
Взгляните на класс NotifyTask, написанный Стивеном Клири.
Это хороший способ обработки асинхронных операций в конструкторах и свойствах.
Вы можете реорганизовать свой код так:
private NotifyTask _OmmaLicenseNumberChangedNotifyTask
= NotifyTask.Create(Task.CompletedTask);
public NotifyTask OmmaLicenseNumberChangedNotifyTask
{
get => this._OmmaLicenseNumberChangedNotifyTask;
set
{
if (value != null)
{
this._OmmaLicenseNumberChangedNotifyTask = value;
OnPropertyChanged("OmmaLicenseNumberChangedNotifyTask");
}
}
}
public string OmmaLicenseNumber
{
get => _ommaLicenseNumber;
set
{
Set(() => OmmaLicenseNumber, ref _ommaLicenseNumber, value);
Patient.OmmaLicenseNumber = value;
OmmaLicenseNumberChangedNotifyTask = NotifyTask.Create(CheckLicenseValid());
}
}
Затем вы можете забыть о Task или привязать элементы своего пользовательского интерфейса к свойствам NotifyTask, например, IsCompleted, чтобы сделать что-то видимым, только если Task завершен.