0

我创建了一个实现接口的抽象类。这个抽象类将是几个需要填充该接口属性的具体类的基础。

CLR 合规性警告会在前两个示例中弹出。我了解它们所代表的含义,这里有几个问题涵盖了它们。

为了使字段不同,我可以添加一个尾随下划线。它被编译器接受。这是一个正确的风格选择。我认为它不是很突出,可能是代码味道。但我可能只是不习惯。

或者我创建一个定义属性字段的抽象祖先是错误的?这个想法当然是为了节省重复工作并帮助强制执行标准实现,但我可以看到,当它开始为这些“隐藏”字段分配值时,它可能在后代中具有自己的味道。

namespace MyLittleCompany.Widgety
{
  public abstract class MlcWidgetInformation : IMlcWidgetInformation
  {
    //Compiler complains of Non-CLR Compliance (case difference only)
    protected int sides;  //Number of sides for this widget

    //Compiler complains of Non-CLR Compliance (non-private name with underscore
    // is not compliant)
    protected int _hooks; //Number of hooks on this widget

    //Compiler is happy with a trailing underscore
    protected int feathers_; //Number of feathers on this widget

    // Interface items
    public Sides { get { return sides; } } 
    public Hooks { get { return _hooks; } }
    public Feathers { get { return feathers_; } }
  }
}
=====================================
namespace MyLittleCompany.Widgety
{
  public class SmallWidgetInformation : MlcWidgetInformation 
  {
    public SmallWidgetInformation()
    {
      // Is this a smell? As in "What are these things?"
      sides = 6;
      _hooks = 3;
      feathers_ = 1;
     }
  }
}        
4

2 回答 2

2

为了避免重复定义三个字段而必须创建一个抽象基类不是代码异味,而是:

  1. 确实感觉将 DRY 发挥到了极致,
  2. 并且(假设您在其他地方使用继承),您阻止了从其他类继承的机会。

但是,如果您愿意/能够使用 VS2015 和 C# 6,那么帮助就在眼前。新的只读自动属性允许您这样做,无需重复就无需基类:

public interface IMlcWidgetInformation
{
    int Sides { get; }
    int Hooks { get; }
    int Feathers { get; }
}

public class SmallWidgetInformation : IMlcWidgetInformation
{
    public int Sides { get; } = 6;
    public int Hooks { get; } = 3;
    public int Feathers { get; } = 1;
}

在 C# 6 被更广泛地采用之前,您只能在继承和重复自己之间做出选择。

于 2015-07-09T20:22:43.210 回答
2

在抽象类中创建受保护字段是绝对可以接受的。

命名约定仅供参考,它取决于您使用的样式工具。使用您和/或您的团队喜欢的风格并从中定制您的工具。最重要的是项目本身是一致的。

我个人以前从未见过使用尾随下划线,但我可以看到它的好处。可能是显示受保护字段的一种非常聪明的方式。如果我遇到一个使用它的团队,我肯定会赞成这个约定。

于 2015-07-09T16:49:26.700 回答