11

我正在使用一个服务层，直到现在我使用了一个 ServiceObject（它实现了 ArrayAccess、Iterator、Countable），但我想知道这是否是个好主意。

你会怎么做：

ArticleService::createArticle($articleData, $userId);

或者

ArticleService::createArticle(ServiceObject $data);

在哪里$data：

array(
  'title' => 'Lorem ipsum',
  'body'  => 'Dolor sid amet',
  'userId' => 55,
);

ServiceObject 的好处是为每个方法提供一个公共签名，但是有时它看起来效率不高，并且没有被广泛使用，它失去了它的兴趣。

任何反馈？

4

6 回答 6

5

将其更改为：

ArticleService::createArticle($title, $body, $user_id);

这使得创建“文章”所需的内容非常清楚。

$articleData像您这样的“选项”数组$data不可能明智地进行智能感知，我建议不要这样做。

把你的想法当作一个通用ServiceObject的垃圾，这实际上是一个非常糟糕的主意。你的动机是崇高的，但这是错误的解决方案。

如果您需要更有说服力，请随意戳。

于 2011-06-11T23:20:17.733 回答
2

我认为这里最好的方法是有类似的东西

$article = new Article;
$article->title = "Lorem ipsum"
$article->body = "Dolor sit amet"
$article->creator = $userID
ArticleService::createArticle($article)

尽管“createArticle”可能不再是该函数的最佳名称，因为 Article 对象已经创建。ArticleService::publishArticle($article)虽然我不知道你在做什么，但你可能会有类似的东西。

您想要的是将数据的构造 ( Article) 与数据的使用() 分开ArticleService。

我真正想说的是，不要将 createArticle 的参数设置为通用数组或“ServiceObject”，这两者都是参数列表的无用抽象，使其成为真正代表相关实体的类。

于 2011-06-17T18:51:34.483 回答
0

我认为更好的方法是注入一个对象。但是为什么有 ServiceObject 呢？您有一个方法 createArticle，那么将文章对象传递给它是合乎逻辑的，不是吗？这样，组织验证过程会更简单，您只需在注释中标记要验证的字段即可。

这也是您使用的方法的问题。如果您在服务层中使用数据映射器模式，那么您显然应该将对象传递给它。

使用一个对象，您可以在对象之间拥有透明和清晰的通信界面。对于数组，您传输的数据类型、字段类型等并不清楚。

于 2011-06-17T22:44:58.603 回答
0

第二个更好，你永远不会遇到人们以错误的方式调用你的方法的问题！它也更容易阅读和查看所需内容。对于作为开发人员的我来说，很明显我必须传递一个类型的对象，ServiceObject然后我开始搜索该类的文档或示例或接口。但在第一个示例中，我必须阅读 的代码createArticle以了解如何处理这两个参数。

第二种方法也将帮助您随着时间的推移保持 API/签名的清洁。当内部需要一些额外的数据时，createArticle您只需将它们传递给ServiceObject您，您无需更改签名！

第一个例子的更好的签名是：

ArticleService::createArticle(array $articleData, $userId);

在这种情况下$articleData必须是一个数组。这可以防止错误。在 PHP 函数参数中，只能使用数组和类名进行强制转换。

于 2011-06-11T00:51:34.250 回答
0

没有正确的方法，但我会ArticleService::createArticle($articleData, $userId);在这种特定情况下使用。

我假设文章有一个必需的非空属性 userId （您从上下文中获得）和一个可选的（不是很重要的）内容数据，它可能是空的。

这样文章和用户对象之间的联系就很明显了。可读性略有提高。否则，您需要完全熟悉所需的数据和 createArticle 方法。

作为证明：当我看到ArticleService::createArticle($articleData, $userId)我立即了解该方法正在做什么以及对输入数据的期望时，第二个让我感到困惑。

此外，如果缺少 userId，此时您将收到一个错误，而不是在您将其插入数据库时​​，您可能会收到一个更难以跟踪的 SQL 错误。

另一方面，在 ZF 中，使用数组作为参数是很常见的，因此您的代码可能看起来不同。

但是，这主要取决于您的喜好。

于 2011-06-14T16:12:47.217 回答
0

我投票支持该对象，因为复杂系统中的代码完成是非常好的事情，而且还有一些自动验证，因为当您知道对象类型时，您可以以某种方式验证它。但是如果你在 createSomething 中没有复杂的逻辑，那么数组也可以适合。

于 2011-05-15T22:00:22.013 回答