StyleCop规则SA1600要求每个类型的成员都有自己的文档头。我认为这很合理,我喜欢这个规则。但是,假设我们有以下层次:StyleCop SA1600规则和接口实现
/// <summary>
/// Documentation for interface ISomeModule.
/// </summary>
interface ISomeModule
{
/// <summary>
/// Documentation for DoA.
/// </summary>
void DoA();
/// <summary>
/// Documentation for DoB.
/// </summary>
void DoB();
}
/// <summary>
/// Documentation for StandardModule.
/// </summary>
class StandardModule : ISomeModule
{
private readonly SomeCoolType _value;
/// <summary>
/// Documentation for constructor.
/// </summary>
public StandardModule(SomeCoolType value)
{
_value = value;
}
// SA1600 violation here!
public void DoA()
{
// realisation of DoA().
}
// SA1600 violation here!
public void DoB()
{
// realisation of DoB().
}
/// <summary>
/// Documentation for MyOwnDoC.
/// </summary>
public void MyOwnDoC()
{
// realisation of MyOwnDoC().
}
}
在这里,我完全公开的接口成员的DoA()和DOB(),我们知道这些方法究竟从接口文档做的。 VS Intellisence也知道它,即使在类StandardModule中,我们也可以通过将鼠标悬停在这些方法上来查看方法的描述。所以没有必要将文档从接口复制到派生类。但StyleCop要求做到这一点。为什么?有人知道吗?
如果我们尝试解决这个问题,我们可以去4点不同的方式:从接口
1.复制文件。 这里的问题是,如果我们复制文档,我们将遇到更新所有派生类中文档的问题,如果界面行为发生变化。
2.禁止消息SuppressMessageAttribute。 那么,假设我们说“好的,我可以使用SuppressMessageAttribute”来抑制这种违反我不同意的做法。对于规则SA1600,我在SuppressMessageAttribute前面添加了StandardModule类。但是现在StyleCop停止检查类StandardModule中的文档标题。我不想要它,因为我们有构造函数和其他一些方法。
3鸿沟类划分成多个区域, 我们可以把类StandardModule成2个区域,只有在实现了接口ISomeModule部分使用消息抑制。我认为所有部分都应该放在一个文件中。我最喜欢这种方法(在#4之后),但现在我们必须处理一个类的多个部分。
4.修改规则SA1600。是否有可能自己实现规则SA1600,以便它考虑到类成员是否记录在基类或接口中? (这里我不问我们是否可以为StyleCop编写我们自己的规则,我知道我们可以,但我的意思是StyleCop引擎是否可以检查某些成员是否来自接口或基类)。
解决SA1600接口实现问题的最佳方法是什么?
对于一个写得很好的质量问题+1。 – 2012-03-24 18:47:38