在将ASP.NET应用程序迁移到异步/等待模型时,我偶然发现了一个相当危险的场景.
I've just stumbled across a rather dangerous scenario while migrating an ASP.NET application to the async/await model.
这种情况是,我使方法异步:async Task DoWhateverAsync(),将接口中的声明更改为Task DoWhateverAsync(),并希望编译器通过该警告.好吧,祝你好运.通过接口将对象注入的任何地方,都不会发生警告. :-(
The situation is that I made a method async: async Task DoWhateverAsync(), changed the declaration in the interface to Task DoWhateverAsync() and hoped that the compiler would tell me where the code now is wrong, via that Warning. Well, tough luck. Wherever that object gets injected via the interface, the warning doesn't happen. :-(
这很危险.有什么方法可以自动检查返回任务的未等待的方法吗?我不介意太多警告,但我不想错过任何一个警告.
This is dangerous. Is there any way to check automatically for non-awaited methods that return tasks? I don't mind a few warnings too many, but I wouldn't want to miss one.
这是一个例子:
using System.Threading.Tasks; namespace AsyncAwaitGames { // In my real case, that method just returns Task. public interface ICallee { Task<int> DoSomethingAsync(); } public class Callee: ICallee { public async Task<int> DoSomethingAsync() => await Task.FromResult(0); } public class Caller { public void DoCall() { ICallee xxx = new Callee(); // In my real case, the method just returns Task, // so there is no type mismatch when assigning a result // either. xxx.DoSomethingAsync(); // This is where I had hoped for a warning. } } }推荐答案
最后,我们使用roslyn查找忽略了Task或Task<>的返回值的所有实例:
In the end, we used roslyn to find all instances where a return value of Task or Task<> was ignored:
if (methodSymbol.ReturnType.Equals(syntaxNodeAnalysisContext.SemanticModel.Compilation.GetTypeByMetadataName(typeof(Task).FullName))) { // For all such symbols, produce a diagnostic. var diagnostic = Diagnostic.Create(Rule, node.GetLocation(), methodSymbol.ToDisplayString()); syntaxNodeAnalysisContext.ReportDiagnostic(diagnostic); } if (((INamedTypeSymbol) methodSymbol.ReturnType).IsGenericType && ((INamedTypeSymbol) methodSymbol.ReturnType).BaseType.Equals(syntaxNodeAnalysisContext.SemanticModel.Compilation.GetTypeByMetadataName(typeof(Task).FullName))) { // For all such symbols, produce a diagnostic. var diagnostic = Diagnostic.Create(Rule, node.GetLocation(), methodSymbol.ToDisplayString()); syntaxNodeAnalysisContext.ReportDiagnostic(diagnostic); }更多推荐
查找未等待的异步方法调用
发布评论