ASP.NET Core MVC(C# .NET 5)で構築したWebアプリケーションにおいてユーザー入力チェックを行いたいと考えています。
単純な入力チェックはModelの検証機能を用いたValidationAttributeを用いることで非常に簡単に行うことができます。しかしDatabaseアクセスを伴うようなValidationAttributeを作った場合、ASP.NET Core MVCには非同期Validationの仕組みがありません。どのような対応方法を取るのが適切でしょうか。
以下3案は思いついた案ですが、もっと適切な方法があるのではないかと考えています。
Validationの場合はDatabaseアクセスも同期処理として実装する
AnyAsync()
,ToListAsync()
などを用いず、Any()
,ToList()
を用いる非同期処理を待つ
protected override ValidationResult IsValid(object value, ValidationContext validationContext)
{
var model = validationContext.ObjectInstance as SomeModel;
var domainService = validationContext.GetService<IMyDomainService>();
var task = domainService.SomeCheckAsync(model);
Task.WaitAll(task);
var result = task.Result;
if (!result)
{
return new ValidationResult(FormatErrorMessage(ErrorMessageString));
}
return ValidationResult.Success;
}
- Taskで実行
protected override ValidationResult IsValid(object value, ValidationContext validationContext)
{
var model = validationContext.ObjectInstance as SomeModel;
var domainService = validationContext.GetService<IMyDomainService>();
var result = Task.Run(() => {
return domainService.SomeCheckAsync(model);
}).GetAwaiter().GetResult();
if (!result)
{
return new ValidationResult(FormatErrorMessage(ErrorMessageString));
}
return ValidationResult.Success;
}
いまいち煮え切らないのは下記のように考えたからです。
- ASP.NET Coreは非同期処理を使うことでブロッキングが発生せずに適切なパフォーマンスで動作するという基本設計に反していると思いました。Micorosoft技術者のBlogで同期と非同期を混在させるべきではないとの記載もあったかと思います。
- おそらくデッドロックが発生するのではないか(お試しでは発生していない)。
- ASP.NETの通常の処理でTask.Run()を使うのはご法度とどこかで読んだ気がするのですが…探せませんでした。
個人的に、ModelのAnnotationとして設定することができるData Annotation Model Validatorは、ビジネスロジックと分離した検証機構として非常に優れた設計だと考えていますので、ビジネスロジックとして入力チェックを行う案はできる限り避けたいと考えています(そうするのが適切なら、そうするしかないですが)。