抽象还是不抽象

Question

预先感谢您阅读本文。我不完全理解如何/何时使用摘要，所以我试图思考我从事的每个项目，看看有一天它是否会全部点击微笑 | :)

另外，可访问级别（私有、受保护、内部）与关键字 static、abstract 和 override 的混合往往会让我有点困惑。我如何定义这个方法/属性/类......

这对我来说并不是一个大谜团，但有些项目让我在处理这些主题时进行编码。

话虽如此，

我有一个读取 XML 文档并输出文本和图像文件的应用程序。我还将所有信息存储在数据库中。我让它工作得很好。

XML 具有包含必填字段的标准实现，并且被多个组织用来向我的应用程序提交数据。所有组织都应该（至少）使用 XML 实施指南中概述的所需节点/元素。

因此，我希望有一个默认的数据对象类型，以便能够为所需元素派生特定组织的数据类型。 （如果要使用该对象，则必须实现这些字段）。

如果组织。只要使用默认的要求，我就可以使用默认的对象。如果他们使用附加（可选）字段，我将必须创建一个继承默认类型的新类型。

我的第一个想法是使用具有受保护属性的抽象类来满足我的最低要求：

public abstract partial class AbstractDataObject
{
   protected string DataObjectName;
   protected DateTime? DataObjectDate;
   etc...
}

然后，如果组织只使用节点的必需元素而没有可选元素，我可以使用“默认”对象。

internal partial class DefaultDataObject : AbstractDataObject
{
   public new string DataObjectName { get; set; }
   public new DateTime? DataObjectDate { get; set; }
   etc...
}

但是，如果组织使用所需节点的可选字段，我可以使用派生的组织数据对象。

internal sealed partial class OranizationDataObject : AbstractDataObject
{
   public new string DataObjectName { get; set; }
   public new DateTime? DataObjectDate { get; set; }
   etc...

   //Optional fields used by this organization
   public string DataObjectCode { get; set; }
   etc...

}

我需要抽象类吗？在我看来，我可以只拥有一个 DefaultDataObject （类似的东西）：

internal partial class DefaultDataObject
{
   public virtual string DataObjectName { get; set; }
   public virtual DateTime? DataObjectDate { get; set; }
   etc...
}

然后：

internal sealed partial class OranizationDataObject : DefaultDataObject
{
   public override string DataObjectName { get; set; }
   public override DateTime? DataObjectDate { get; set; }
   etc...

   //Optional fields used by this organization
   public string DataObjectCode { get; set; }
   etc...

}

我只是真的想了解如何定义这些对象，以便我可以在每个组织中重用它们。两种方法似乎都有效，但我希望了解如何正确定义它们。

将 XML 放入上述对象中：

public DefaultDataObject ExtractXmlData(XContainer root)
    {
        var myObject = (from t in root.
        Elements("ElementA").Elements("ElementB")
              select new DefaultDataObject()
              {
        DataObjectName = (String)t.Element("ChildElement1"),
        DataObjectDate = 
                      Program.TryParseDateTime((String)
                      t.Elements("ChildElement2")
                      .ElementAtOrDefault(0)
        ),
        etc....

OR

public OranizationDataObject ExtractXmlData(XContainer root)
    {
        var myObject = (from t in root.
        Elements("ElementA").Elements("ElementB")
            select new OranizationDataObject()
              {
    DataObjectName = (String)t.Element("ChildElement1"),
    DataObjectDate = Program.TryParseDateTime(
             (String)t.Elements("ChildElement2")
             .ElementAtOrDefault(0)),
    DataObjectCode = (String)t.Element("ChildElement3"),

等...

再次感谢您的阅读。别忘了给服务员小费……

乔

Answer 1

1. 首先，如果您的基类是普通的 DTO 类，则它不需要是抽象的。如果您没有任何需要由派生类以不同方式实现的功能，则只需将其设置为一个普通基类，它将包含通用属性。

2. 接下来，如果您要 隐藏它们（使用new关键字）。您的第一个 DefaultDataObject 代码片段不必要地创建了一堆具有相同名称的新属性。完全删除它们 - 它们已经在基类中定义。

   [编辑]我最初没有注意到这一点，@svick 警告我，你的基类实际上包含字段而不是属性，这让我想知道为什么您根本需要添加 new 关键字。我快速浏览了你的代码并将它们视为属性。在任何情况下，您都不应该公开公共字段 - 至少通过添加 { get; 将它们更改为自动实现的属性。放; } 块。

   换句话说，这很简单：

   // 这不需要是抽象的。
   // 把所有常用的东西都放进去。
   公共类BaseDO
   {
       // 正如 svick 指出的，这些也应该是属性。
       // 你不应该*永远*在你的类中公开公共字段。
   
       公共字符串名称{获取;放; }
       公共日期时间？日期{获取;放; }
   }
   
   // 不要使用 new 关键字来隐藏内容。
   // 在大多数情况下，你不需要这样的行为
   公共类 DerivedDO ：BaseDO
   {
       // 无需重复上面的这些属性，
       // 只添加**不同的**
       公共字符串代码{获取;放; }
   }
   

3. 作为旁注，但无论如何重要的恕我直言，您应该简化命名（并使其更清楚您的代码的作用）。例如，无需在每个属性名称中重复“DataObject”。但由于您的代码可能只是一个简化版本，所以这并不重要。

4. 最后，您听说过 XmlSerializer？您不需要手动遍历 XML 元素。调用 XmlSerializer 来序列化和反序列化数据就足够了。

Answer 2

我需要知道的一切都是我从芝麻街学到的
仔细检查您的类设计，以确保您已识别出所有相同和不同的内容。可以这么说，与您的班级一起玩电脑，看看他们如何做相同、不同或相同但方式不同的事情。

什么是相同、不同、相同但不同可能会随着您玩电脑而改变。

一般性地思考 OO 类的两大支柱。多态与继承
正如你所做的那样。就 C# 实现本身而言，没有那么多。

事物如何聚集成相同与不同将有助于推动实施
这都是相对的。

• 更多相同的默认行为？也许是具体的基类而不是抽象的。
• 更多相同的事情，但不同？也许是一个抽象类而不是具体的基类。
• 执行x的默认方式？也许是一个虚拟方法。
• 每个人都做同样的事情，但没有两个人做同样的事？也许是一个代表。

实施建议

• 将方法和字段设置为默认受保护。 Private 不会被继承。设计发生变化，保持灵活性。如果某些东西必须是私有的，那没问题。

• 虚拟意味着您可以更改子类中的实现。这并不意味着您必须这样做。

• 人们似乎没有充分利用委托。它们非常适合多态方法。

• 公共字段没有任何问题。公共字段和公共自动实现属性之间的实际区别是什么？没有什么。它们都直接返回（或设置）基础值。那么，为财产而烦恼又有什么意义呢？如果您想公开揭示与“自然”状态不同的潜在价值。例如，返回特定格式的数字。当然，同一个字段可以有不同的属性。

• 属性可以有get，但没有set。或者反之亦然。 get 和 set 也可以有不同的访问级别。通常，您会将其视为公共 get 和受保护（或私有）集。

Answer 3

这取决于派生类型想要做什么。如果他们打算使用默认实现并仅以某种方式对其进行扩展，那么将默认类作为非抽象基类就可以了。

另一方面，如果他们最有可能重新实现功能，您应该有一个抽象基类（或接口）和一个单独的默认类。

如果您由于某种原因不知道它是哪一个，您可以通过使用抽象基类并保留默认类未密封的方式让继承者进行选择。

另外，看看你的代码，你似乎误解了各种关键字的作用。大多数时候，您不想像这样使用new。它的作用是定义另一个具有相同名称的成员，与原始成员无关。此外，如果您不想更改某些内容，就没有理由覆盖它。因此，如果您希望派生类不必重新实现这些属性，则根本不必将它们设为虚拟的。

Answer 4

抽象类已经可以实现可以继承的东西

public abstract class DataObjectBase
{
    public string DataObjectName { get; set; }
    public DateTime? DataObjectDate { get; set; }
}

具体类可以添加新的属性和方法

public class DerivedDataObject : DataObjectBase 
{
    public int NewProperty { get; set; }
}

属性DataObjectName和DataObjectDate在新类中已经可用，因为它们是自动从基类继承。

但是，如果抽象类定义了抽象成员，则必须在派生类中实现它。

假设基类定义了

public abstract void SomeMethod(string name);

派生类必须这样做，

public override void SomeMethod(string name)
{
    ...
}

如果您的基类没有抽象成员，则它不需要是抽象的，可以直接发挥默认数据对象的作用。

---

这里不需要关键字“partial”。仅当您想将一个类拆分为多个文件中的多个部分时，它才有用。

这里关键字new是错误的。它用于隐藏继承的成员。这意味着继承的成员将隐藏在新声明的“后面”。你需要的是覆盖。这不会隐藏成员，而是在派生类中提供同一成员的替代实现。