return Task.FromResult(default(object));の意味を教えてください

Question

こちら（http://oki2a24.com/2015/11/29/auth-based-on-identity-asp-net-mvc5/）でTaskの戻り値を「return Task.FromResult(default(object));」としていますが、これは何が返るのでしょうか？
抜粋すると
/// <summary>
/// ユーザーを削除します。
/// </summary>
/// <param name="user">ユーザーオブジェクト</param>
/// <returns>IdentityResult オブジェクト</returns>
public Task DeleteAsync(User user)
{
User deletedTargetUser = db.Users.Find(user.Id);
db.Users.Remove(deletedTargetUser);
db.SaveChanges();
return Task.FromResult(default(object));
}
で、コメントにはIdentityResult オブジェクトが返ると書いてありますが、コードからはそのように読み取れません。

Answer 1

こんにちは。
まず結論から、「IdentityResultオブジェクトが返る」は完全に嘘コメントですね！

メソッドTask.FromResultは、Completedな状態のTaskを作り出すファクトリメソッドです。
メソッドDeleteAsyncは、戻り値がTask<T>ではなくTaskとなっているので、結果を持たない非同期処理を表しています。
なので、単純に「完了した」「失敗した」などを表現できるTaskを結果として返却してしまえばいいわけです。
よって、Task.FromResultを使用して、無意味な値default(object)(== null)を入れた「最初から完了しているTask」を作成して返しているのです。

実際、このコードはかなり悪い例です。
DeleteAsyncという名前を持っていますが、やっていることは、

• 「同期的に」ユーザーを検索 db.Users.Find(user.Id);
• 「同期的に」ユーザーを削除 db.Users.Remove(deletedTargetUser);
• 「同期的に」DBに反映 db.SaveChanges();
• 「完了状態で中身の無いTask」を作成し、それを結果として返却 Task.FromResult(default(object));

と、最初から最後まで完全に同期処理になっており、Taskを返す意味が全くないコードになっています。メソッド名も嘘というのはちょっと笑えないですね。
普通に非同期をやるなら、メソッド内の処理を丸ごとTask.Runに包んで、そのTaskを返すのが一般的でしょう。

Comments

[rias]

すっきりしました。
ありがとうございました。