Question

我意识到这个问题所寻求的查询不足以找到IDisposable实现的每一个小问题，但每个早期警告都很重要，所以我会采取我能得到的。

我想知道是否有人为NDepend提出了一个CQL查询，它将列出所有未实现IDisposable的类，但是有一个或多个字段。一个类可能通过一个bug（即有人忘了检查IDisposable实现的字段类型）或者通过代码进化（或者某个字段中使用的类在某个地方获得IDisposable）来结束此查询的结果列表。以后的日期没有更新所有用法。）

查找未实现IDisposable的所有类的简单查询是：

SELECT TYPES WHERE !Implement "System.IDisposable"

但是，这当然不会检查类是否为上述规则实现IDisposable。

有没有人有这样的疑问？我仍然在掌握CQL，所以这部分让我不知所措。

Answer 1

Lasse，感谢CQLinq（LINQ上的代码规则）功能，现在可以实现匹配应该实现IDisposable的类型。实际上现在提供了两个相关的默认规则，您可以轻松编写自己的相关规则：

// <Name>Types with disposable instance fields must be disposable</Name>
warnif count > 0

let iDisposable = ThirdParty.Types.WithFullName("System.IDisposable").FirstOrDefault() 
where iDisposable != null // iDisposable can be null if the code base doesn't use at all System.IDisposable

from t in Application.Types where 
   !t.Implement(iDisposable) && 
   !t.IsGeneratedByCompiler 

let instanceFieldsDisposable = 
    t.InstanceFields.Where(f => f.FieldType != null &&
                                f.FieldType.Implement(iDisposable))

where instanceFieldsDisposable.Count() > 0
select new { t, instanceFieldsDisposable }

// <Name>Disposable types with unmanaged resources should declare finalizer</Name>
// warnif count > 0
let iDisposable = ThirdParty.Types.WithFullName("System.IDisposable").SingleOrDefault()
where iDisposable != null // iDisposable can be null if the code base deosn't use at all System.IDisposable

let disposableTypes = Application.Types.ThatImplement(iDisposable)
let unmanagedResourcesFields = disposableTypes.ChildFields().Where(f => 
   !f.IsStatic && 
    f.FieldType != null && 
    f.FieldType.FullName.EqualsAny("System.IntPtr","System.UIntPtr","System.Runtime.InteropServices.HandleRef")).ToHashSet()
let disposableTypesWithUnmanagedResource = unmanagedResourcesFields.ParentTypes()

from t in disposableTypesWithUnmanagedResource
where !t.HasFinalizer
let unmanagedResourcesTypeFields = unmanagedResourcesFields.Intersect(t.InstanceFields)
select new { t, unmanagedResourcesTypeFields }

NDepend CQL查询缺少IDisposable实现

1 个答案: