c# - 抽象，还是不抽象

Question

提前感谢您阅读本文。我不完全理解如何/何时使用摘要，所以我试图在我从事的每个项目中考虑它，看看它是否会在某一天全部点击 Smile | :)

此外，可访问性级别(私有(private)、 protected 、内部)与关键字 static、abstract 和 override 的组合往往让我有点困惑。我如何定义这个方法/属性/类....

这对我来说并不是什么大谜团，但有些项目让我在处理这些主题时绕着圈子编码。

话虽如此，

我有一个应用程序可以读取 XML 文档并输出文本和图像文件。我还将所有信息存储在数据库中。我让它运行良好。

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

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

如果组织。只要使用默认要求，我就可以使用默认对象。如果他们使用额外的(可选)字段，我将不得不创建一个继承默认类型的新类型。

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

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....

或

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 类，则它不需要是抽象的。如果您没有任何需要由派生类以不同方式实现的功能，您可以简单地使其成为一个普通基类，它将包含公共(public)属性。

2. 接下来，如果您要转到 hide them，那么在基类(在您的情况下是抽象的)中声明属性是没有意义的。 (使用 new 关键字)。 DefaultDataObject 的第一个代码片段不必要地创建了一堆具有相同名称的新属性。完全删除它们 - 它们在基类中已经定义。

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

   换句话说，这很简单:

   // this doesn't need to be abstract.
   // just put all the common stuff inside.
   public class BaseDO
   {
       // as svick pointed out, these should also be properties.
       // you should *never* expose public fields in your classes.
   
       public string Name { get; set; }
       public DateTime? Date { get; set; }
   }
   
   // don't use the new keyword to hide stuff.
   // in most cases, you won't need that's behavior
   public class DerivedDO : BaseDO
   {
       // no need to repeat those properties from above,
       // only add **different ones**
       public string Code { get; set; }
   }
   

3. 作为旁注，但恕我直言很重要，您应该简化命名(并使其成为 more clearer 您的代码所做的)。例如，无需在每个属性名称中重复“DataObject”。但由于您的代码可能只是一个简化版本，所以没关系。

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