NDepend CQL查询缺少IDisposable实现

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 }