返回一个新对象与修改作为参数传入的对象

Question

我在代码审查期间遇到了以下代码片段。

我的直觉告诉我，这不符合正确的 OOP。

我认为 LoadObject 方法应该返回一个新的 SomeObject 对象，而不是修改传递给它的对象。尽管我无法真正找到为什么这样做更好的正确解释。

我的解决方案更好吗？如果是的话为什么？具体来说，给定的代码示例中违反了哪些 OOP 原则或标准（如果有）？

   public void someMethod()
    {
        ...
        var someObject = new SomeObject();
        LoadSomeObject(reader,someObject);
    }

    private void LoadSomeObject(SqlDataReader reader, SomeObject someObject)
    {
       someObject.Id = reader.GetGuid(0);
    }

Answer 1

代码的编写方式没有任何问题，因为您只是修改了 someObject 的属性。

然而，在 LoadSomeObject 中创建 someObject 并返回它也是正确的。

在这一点上，两种选择都是正确的。

Answer 2

我不是 OO 大师，所以对这些持保留态度。

1. 对象应尽可能自行管理/定义其行为。如果您曾经了解过松散耦合，那么这背后的原理是显而易见的。将 LoadSomeObject 的细节转移到 SomeObject 的实现中可能是更好的设计决策，但是对于这样一个通用示例来说，这很难讨论。

2. 可变状态在任何命令式代码（包括面向对象代码）中都是完美的，它是这些范例的核心“功能”。 OTOH，不可变状态具有不可描述的优势（我想我们对这个主题有几个问题，否则就问任何 FP 拥护者），并且拥有一些不可变对象并不是特别非 OO。

编辑：您也可以将 reader 传递给 SomeObject 的构造函数。

Answer 3

不存在通用的面向对象概念，这与类似的代码相冲突。
但稍后您会发现，如果您不遵循一些设计原则，那么很难收集和理解操作 SomeObject 实例的方法。
对于初学者来说，最简单的方法可能是分离两种主要类型的过程：

• 功能性 - 旨在创建新实例而不是改变其他对象。
• Methodic - 旨在更改其主机实例的状态。

这里的好主意是，如果您想分离 SomeObject 操作逻辑，可以使用

 public void loadFromReader( SqlDataReader reader )

方法和方法

创建 SomeObjectBuilder 类型

公共 SomeObject getValue()

属性

Answer 4

无论哪种方式都是完全可以接受的，但是在决定哪种方式适合您的特定情况时，需要考虑您希望如何处理返回的对象。不变性，即在每个操作中创建一个新实例是 .NET 中字符串的处理方式。

复制方法显然需要返回一个副本。作为作用于单例对象的方法，例如由全局引用保存的方法，不适合返回新对象，因为更改可能会丢失。

Answer 5

如果 LoadSomeObject 要返回新的 SomeObject，您可能需要更改方法名称以避免混淆。也许到NewAndLoadSomeObject？

Answer 6

我确实同意你的直觉，在大多数情况下感觉有点不对劲。我同意返回一些东西更好，因为这样可以更清楚地表明某些东西正在被修改。看到正在使用的返回值可以立即清楚发生了什么。

不过，我认为这感觉还是好一点（无论如何在大多数情况下）：

public void someMethod()
{
    ...
    var someObject = new SomeObject();
    someObject.Load(reader);
}

然后显然在 SomeObject 类中，您会发现

public void Load(SqlDataReader reader)
{
   this.Id = reader.GetGuid(0);
}

这个想法是支持实例方法而不是静态方法。当您可以让对象对其自己的数据进行操作时，为什么还要创建一个对象并传递其数据呢？