如何确定传入一个对象是否比几个参数（int、string等）更好[关闭]

Question

我有一个名为“User”的类/对象，它有大约十几个属性（例如：UserGUID、UserName 等）。它有一个构造函数、静态方法、一些其他的辅助/支持方法等等。

该网站有数百个函数/方法，其中 2+ 参数来自用户对象。例如：

public string HelloWorld(Guid userGUID, Guid accountGUID, bool somethingElse)
{
    //Do something
}

我真的想传入 User 对象本身以使调用更清晰，而不必在每次需要来自 User 对象的新值时不断添加参数。像这样：

public string HelloWorld(User user)
{
    //Do something
    Guid userGUID = user.UserGUID;
}

所以我的问题是，在什么时候传入对象好/坏与传入几个参数？它取决于物体的大小吗？我如何确定什么是“太大”与“好”？是参数的数量吗？多少参数太多了？

Answer 1

您应该考虑该方法应该做什么。方法为什么存在？

方法的语义将决定它的参数。因此，例如，如果 HelloWord 应该打印出一些内容，例如 userId 和 something else，那么签名应该包含 userId 和 something else 作为参数。

另一方面，如果HelloWord 应该打印出一些关于User 的信息，那么方法签名应该将对象 User 作为参数。

这完全取决于方法语义。

Answer 2

在 Clean Code 中，Robert Martin 说首选 0 个参数，1 或 2 个参数是可以接受的，3 个太多了。

在我看来，只要你在同一个过程中，我认为传递对象比传递参数更可取。您不希望发送（或接收）超出另一个进程（例如 Web 服务）所需的内容。

我强烈推荐 Clean Code，这是一本很好的读物，并且对结构有很多话要说。

Answer 3

这里有一个非常重要的区别，这不是意见。

我有一个名为“User”的类/对象，它有十几个属性

鉴于上述情况，如果您当时允许(User user) 而不是只允许(Guid userGUID, Guid accountGUID, bool somethingElse)，那么您刚刚引入了一个安全漏洞。

通过发布User 类的额外名称，客户端将能够发送比他们应该访问的更多的数据。例如，如果您使整个类可用（并且它具有外部关系），则客户端可以以这种方式更改外部导航属性键。客户端也可以根据存储在该类中的信息更改时间戳，甚至逻辑分隔。

如果您允许整个类都被接受，那么防止这种类型的违规行为很容易，您只需要手动检查每个属性以确保它没有被错误发送，或者只选择它的子集来筛选它发送的信息。不管怎样，这都是个坏主意。

虽然使用具有与所示 3 相同属性的 User 类可能没有区别，但如果不选中，允许具有比 3 更大的集合的 User 类的模型绑定可能会出现问题。