7

我试图让 NDepend 使用标准“方法太大”查询的修改版本来识别长方法。

我不想报告开发人员几乎无法控制的长方法,所以我使用DebuggerNonUserCode属性和InitializeComponent().

不幸的是,我仍然得到一些误报,因为生成类型中的方法也被报告了。问题是虽然类型本身具有DebuggerNonUserCode属性,但方法没有,因此尽管它们是生成的,但它们仍包含在输出中。

我正在寻找类型和方法之间的连接之类的东西:给我所有没有DebuggerNonUserCode属性的类型并在这些类型上运行查询,但我不知道如何在 CQL 中表达这一点。

对于某些程序集,我可以简单地过滤全名,但不幸的是,我们的一些程序集混合了开发人员制作和生成的类型。不幸的IsGeneratedByCompiler是,在这种情况下也不能使用。

我的查询

WARN IF Count > 0 IN SELECT METHODS WHERE 
   NbLinesOfCode > 30 AND
   !HasAttribute "System.Diagnostics.DebuggerNonUserCodeAttribute" AND
   !NameIs "InitializeComponent()"
   ORDER BY NbLinesOfCode DESC
4

2 回答 2

2

Brian,感谢LINQ 查询上的代码规则(CQLinq),您要求的代码规则的源代码是:

warnif count > 0
from m in Application.Methods where
  m.NbLinesOfCode > 30 &&
 !m.ParentType.HasAttribute( "System.Diagnostics.DebuggerNonUserCodeAttribute") &&
  m.Name != "InitializeComponent()"
orderby m.NbLinesOfCode descending
select new { m, m.NbLinesOfCode }

CQLinq 语法还提供了一种定义什么是Just-My-Code的方法。此处描述了此功能。基本上,您需要通过以 notmycode 为前缀的查询来定义集合JustMyCode。然后你要求的规则可以很容易地重写:

warnif count > 0
from m in JustMyCode.Methods where
   m.NbLinesOfCode > 30
orderby m.NbLinesOfCode descending    
select new { m, m.NbLinesOfCode }

一次性定义的JustMyCode集可以在任何代码规则上重复使用。另外,您可以查看来自 JustMyCode 的notmycode默认查询丢弃生成和设计器方法

于 2010-03-05T10:23:06.787 回答
0

我非常喜欢 NDepend,但它仍然是最大的单一缺点,即命名空间/类型/方法信息无法加入到单个查询中。该功能将使 CQL 成为真正强大的东西。

除此之外,检查“ IsGeneratedByCompiler ”和“ IsInFrameworkAssembly ”可能会有所帮助。您还可以从查询中删除某些命名空间(OUT OF NAMESPACES "..."

于 2010-03-04T14:11:49.183 回答