在方法中传递数据对象或数据对象的一部分？

Question

public class DataObject{
    public int Value1;
    public int Value2;
    public int Value3;
    public int Value4;
    public int Value5;

    public DataObject(){}
}

public class SomeClass{
    public SomeClass(){}
    public void MultiplyFirstThreeValues(int Value1, int Value2, int Value3){
        return Value1*Value2*Value3;
    }
    public void MultiplyFirstThreeValues(DataObject d){
        return d.Value1*d.Value2*d.Value3;
    }
}

将所需的每个变量发送到方法或整个数据对象（在并非使用数据对象中的每个变量的情况下），哪种做法更好？这可能是一个“艺术大于科学”的问题，但如果你不能给出一个硬性规定，那么我有兴趣听听做一个而不是另一个的动机是什么（在这种情况下，如果不该方法中使用了数据对象的所有值）。

谢谢，我期待了解更多相关信息。

Answer 1

我会选择选项C将逻辑保留在对象本身内：

  public class DataObject
  {
    private int Value1;
    private int Value2;
    private int Value3;
    private int Value4;
    private int Value5;

    public DataObject(){}

    public int MultiplyFirstThreeValues()
    {
        return Value1*Value2*Value3;
    }
 }

编辑：
为了回应评论 - 我将添加一句话：

程序代码获取信息然后
做出决定。面向对象的代码
告诉对象做事。

在我看来，这只是一种更好的设计方法，因为它可以让您隐藏实际获取结果/执行实现的方式。您当前的设置方式更多地反映了访问者模式，它可能很强大，但也由于类的接口现在拥有所有这些公共成员，因此增加了更多的复杂性。如果您的计算跨越多个 DataObject 和/或其他类型，这实际上可能有意义，但如果我能摆脱它，我肯定会使用封装。

Answer 2

经验法则：如果您有超过 7 个参数（尽管有些人会说超过 3 个），请考虑使用 引入参数对象重构。

一般来说，如果参数数量较少，请传递对象的部分内容，特别是如果您不希望参数数量及其类型发生变化。

如果上述情况并非如此（您期望参数的数量/类型发生变化和/或您有很多参数），请使用该对象。

---

注意：

有很多参数是一种代码味道，可能表明您的方法/类做得太多，应该分解。

Answer 3

具有许多输入参数的方法签名通常被认为表明缺少一个类。有多少是多少？很难说，但三个对我来说并不特别多。

思考过程中需要考虑的另一件事是这些值是否总是以相同的方式来自同一个对象。如果该方法可以用于其他输入组合，也许保留多参数输入会更好。当然，如果参数确实很多，你可以为参数集提供一个专门设计的类型，然后使用它。

Answer 4

就个人而言，我倾向于仅传递对象，尽管不使用所有属性，因为这会导致更小的方法签名，我觉得在读取代码时比许多参数更容易在心理上解析