在 VS2013 的 C# 代码中查找异常隐藏/吞咽答案

【问题标题】：Find exception hiding/swallowing in C# code in VS2013在 VS2013 的 C# 代码中查找异常隐藏/吞咽
【发布时间】：2014-07-15 16:12:40
【问题描述】：

在 VS2013 的 C# 解决方案(ASP.NET WebForms)n 中是否有某种内置函数/扩展/工具的方法来查找所有异常隐藏/异常吞咽。

谢谢

编辑：

我有现有的解决方案，其中一些程序员使用隐藏/吞咽异常（空捕获，只捕获一些无用的代码）。我正在寻找某种方法来找到代码中的所有这些地方，分析它们，然后修复它们。

【问题讨论】：

您正在寻找一个全局错误处理程序来捕获 ASP.NET 网站中任何未处理的异常？如果是这样，it's been asked many times before.
我有现有的解决方案，其中一些程序员使用隐藏/吞咽异常（空捕获，只捕获一些无用的代码）。我正在寻找某种方法来在代码中找到所有这些地方，分析它们，然后修复它们。
搜索？ ctrl-f"catch("
数以千计的结果 - 其中 99% 是正确的
FxCop 会检查这些以及 resharper。事实上，任何名副其实的代码分析工具都可以。

标签： c# visual-studio exception code-analysis static-code-analysis

【解决方案1】：

您可以使用 Roslyn 编写一些代码来轻松处理这个问题。

我实际上为朋友写了一些代码来做这件事。这是我第一次尝试使用 Roslyn SDK，所以我的代码可能一团糟，但它绝对是功能性的。

    static void Main(string[] args)
    {
        var result = Microsoft.CodeAnalysis.CSharp.CSharpSyntaxTree.ParseFile(@"..\..\Test.cs");

        var root = result.GetRoot();

        var exceptionNodes = FindCatchNodes(root);

        foreach (var node in exceptionNodes)
        {
            var line = node.GetLocation().GetLineSpan().StartLinePosition.Line + 1;
            if (IsTotallyEmptyCatch(node))
            {
                Console.WriteLine("Totally empty catch: line {0}", line);
            }
            if (JustRethrows(node))
            {
                Console.WriteLine("Pointless rethrow: line {0}", line);
            }
        }
    }


    static List<SyntaxNodeOrToken> FindCatchNodes(SyntaxNodeOrToken node)
    {
        var exceptions = new List<SyntaxNodeOrToken>();
        var isCatchBlock = node.IsKind(SyntaxKind.CatchClause);
        if (isCatchBlock)
        {
            exceptions.Add(node);
        }

        foreach (var result in node.ChildNodesAndTokens().Select(FindCatchNodes).Where(result => result != null))
        {
            exceptions.AddRange(result);
        }
        return exceptions;

    }

    static bool IsTotallyEmptyCatch(SyntaxNodeOrToken catchBlock)
    {
        var block = catchBlock.ChildNodesAndTokens().First(t => t.CSharpKind() == SyntaxKind.Block);
        var children = block.ChildNodesAndTokens();
        return (children.Count == 2 && children.Any(c => c.CSharpKind() == SyntaxKind.OpenBraceToken) &&
                children.Any(c => c.CSharpKind() == SyntaxKind.CloseBraceToken));
    }

    static bool JustRethrows(SyntaxNodeOrToken catchBlock)
    {
        var block = catchBlock.ChildNodesAndTokens().First(t => t.CSharpKind() == SyntaxKind.Block);
        var children = block.ChildNodesAndTokens();
        return (children.Count == 3 && children.Any(c => c.CSharpKind() == SyntaxKind.OpenBraceToken) &&
                children.Any(c => c.CSharpKind() == SyntaxKind.CloseBraceToken) && children.Any(c=>c.CSharpKind() == SyntaxKind.ThrowStatement));
    }

鉴于此测试文件：

using System;
namespace RoslynTest
{
    public class Test
    {
        public void Foo()
        {
            try
            {
                var x = 0;
            }
            catch
            {

            }
        }

        public void Bar()
        {
            try
            {
                var x = 0;
            }
            catch (Exception ex)
            {
                throw;
            }
        }


        public void Baz()
        {
            try
            {
                var x = 0;
            }
            catch (Exception ex)
            {
                throw ex;
            }
        }
    }
}

输出是：

完全空的捕获：..\Test.cs：第 12 行

毫无意义的重新抛出：....\Test.cs：第 24 行

毫无意义的重新抛出：....\Test.cs：第 37 行

【讨论】：

【解决方案2】：

我不知道内置方法。但是您可以编写自己的工具来查找这些地方。只需在解决方案中正则表达式所有文件并计算捕获和抛出。每个文件的数量应该相同:)

【讨论】：