c# - 抽象还是不抽象

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

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

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

照这样说，

我有一个读取 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....

或者

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"),

等等....

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

乔

最佳答案

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

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

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

换句话说，这很简单:

// 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; }
}

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

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

关于c# - 抽象还是不抽象，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/8865663/