That depends a bit on when you want the error to be raised - i.e. eagerly, or as part of the awaitable. As with iterator blocks, if you want eager error checks, you need two methods, for example:
public Task<int> SomeMethod(..args..) {
if(..args fail..) throw new InvalidOperationException(...);
return SomeMethodImpl(...args...);
}
private async Task<int> SomeMethodImpl(...args...)
{
... await etc ...
}
This will then perform any argument checking as part of the initial call, not the awaitable. If you want the exception to be part of the awaitable, you can just throw it:
public async Task<int> SomeMethod(..args..) {
if(..args fail..) throw new InvalidOperationException(...);
... await etc ...
}
However, in your example, the fact that you are return
ing a Task
suggests that this is not actually an async
method, but is an async (but not async
) method. You can't just do:
return new Task(() => { throw new ArgumentNullException("argument"); });
because that Task
will never have been started - and never will be. I suspect you would need to do something like:
try {
throw new InvalidArgumentException(...); // need to throw to get stacktrace
} catch(Exception ex) {
var source = new TaskCompletionSource<int>();
source.SetException(ex);
return source.Task;
}
which is... a bit of a mouthful and could probably be encapsulated a bit better. This will return a Task
that indicates it is in the Faulted
state.