召唤专家:重构故事(第5/5部分)

在最后一部分中，我们将一些代码移到了ProductForm．今天，我们将通过将代码移动到视图以及使用symfony为常见情况提供的一些很好的快捷方式来进一步增强操作。ob娱乐下载

在第3部分中，我们重构了指数动作，现在看起来像这样:

/ /应用程序/前端/模块/产品/ actions.class.php公共函数executeIndex（）｛这个美元->产品= ProductPeer::getAvailableProducts（）；这个美元->getResponse（）->setTitle（“所有产品”）；这个美元->getResponse（）->addStylesheet（“homepage.css”）；返回sfView::成功；｝

在syob娱乐下载mfony中，默认视图是sfView:成功，所以我们可以从动作中省略它:

/ /应用程序/前端/模块/产品/ actions.class.php公共函数executeIndex（）｛这个美元->产品= ProductPeer::getAvailableProducts（）；这个美元->getResponse（）->setTitle（“所有产品”）；这个美元->getResponse（）->addStylesheet（“homepage.css”）；｝

在此操作中，我们设置页面标题并添加特定的样式表。即使这段代码按预期工作，也不是正确的方式。这段代码实际上属于视图。让我们把这段代码移到indexSuccess.php模板，这样我们的动作看起来就像这样:

/ /应用程序/前端/模块/产品/ actions.class.php公共函数executeIndex（）｛这个美元->产品= ProductPeer::getAvailableProducts（）；｝

要在模板中添加样式表，可以使用use_stylesheet ()助手:

<！——apps/frontend/modules/product/templates/indexSuccess.php——><？phpuse_stylesheet（“homepage.css”）；我们产品< h1 > < / h1 ><？phpforeach（美元的产品作为美元的产品）：? ><！——……-->

要将标题从动作移到模板，就有点麻烦了。我们需要在布局中创建一个插槽，我们将在模板中填充:

<！——apps/frontend/modules/templates/layout.php——>< html>  




      <？phpuse_stylesheet（“homepage.css”）；<？php槽（“标题”，“所有产品”）? >我们产品< h1 > < / h1 ><？phpforeach（美元的产品作为美元的产品）：? ><！——……-->

插槽是在布局中添加一些动态部件的好方法，可以被每个模板覆盖。

让我们继续编辑行动。下面是当前的代码:

公共函数executeEdit（）｛这个美元->产品= ProductPeer::retrieveByPk（这个美元->getRequestParameter（“id”））；如果（is_null（这个美元->产品））｛这个美元->forward404（）；｝这个美元->形式＝新ProductForm（这个美元->产品）；如果（sfRequest::帖子= =这个美元->getRequest（）->getMethod（））｛这个美元->形式->绑定（这个美元->getRequestParameter（“产品”），这个美元->getRequest（）->getfile（“产品”））；如果（这个美元->形式->isValid（））｛美元的产品＝这个美元->形式->保存（）；这个美元->getUser（）->setFlash（“通知”，sprintf（“产品“%s”已成功更新。”，美元的产品->getTitle（）））；这个美元->重定向（“产品/指数”）；｝｝｝

由于在满足条件时转发到404页面是很常见的，symfony有一些很好的快捷方法:ob娱乐下载forward404If ()而且forward404Unless ()：

公共函数executeEdit（）｛这个美元->forward404Unless（这个美元->产品= ProductPeer::retrieveByPk（这个美元->getRequestParameter（“id”）））；/ /……｝

从symfoob娱乐下载ny 1.1开始，symfony将请求对象作为第一个参数传递给执行*方法。所以，不用getRequestParameter ()代理方法，我们可以用getParameter ()方法。我们也可以用newisMethod ()方法来简化帖子检查。

这是决赛executeEdit ()方法:

公共函数executeEdit（美元的请求）｛这个美元->forward404Unless（这个美元->产品= ProductPeer::retrieveByPk（美元的请求->getParameter（“id”）））；这个美元->形式＝新ProductForm（这个美元->产品）；如果（美元的请求->isMethod（“职位”））｛这个美元->形式->绑定（美元的请求->getParameter（“产品”），美元的请求->getfile（“产品”））；如果（这个美元->形式->isValid（））｛美元的产品＝这个美元->形式->保存（）；这个美元->getUser（）->setFlash（“通知”，sprintf（“产品“%s”已成功更新。”，美元的产品->getTitle（）））；这个美元->重定向（“产品/指数”）；｝｝｝

的executeEdit ()代码现在非常普遍，您可以使用一个很好的快捷方式进行表单处理。的bindAndSave ()方法的快捷方式bind ()方法，然后save ()如果表单有效，则返回对象:

公共函数executeEdit（美元的请求）｛这个美元->forward404Unless（这个美元->产品= ProductPeer::retrieveByPk（美元的请求->getParameter（“id”）））；这个美元->形式＝新ProductForm（这个美元->产品）；如果（美元的请求->isMethod（“职位”）& &这个美元->形式->bindAndSave（美元的请求->getParameter（“产品”），美元的请求->getfile（“产品”）））｛这个美元->getUser（）->setFlash（“通知”，sprintf（“产品“%s”已成功更新。”，美元的产品->getTitle（）））；这个美元->重定向（“产品/指数”）；｝｝

控制器的目的现在表达得非常好。的最终代码产品模块:

/ /应用程序/前端/模块/产品/ actions.class.php公共函数executeIndex（）｛这个美元->产品= ProductPeer::getAvailableProducts（）；｝公共函数executeEdit（美元的请求）｛这个美元->forward404Unless（这个美元->产品= ProductPeer::retrieveByPk（美元的请求->getParameter（“id”）））；这个美元->形式＝新ProductForm（这个美元->产品）；如果（美元的请求->isMethod（“职位”）& &这个美元->形式->bindAndSave（美元的请求->getParameter（“产品”），美元的请求->getfile（“产品”）））｛这个美元->getUser（）->setFlash（“通知”，sprintf（“产品“%s”已成功更新。”，美元的产品->getTitle（）））；这个美元->重定向（“产品/指数”）；｝｝公共函数executeAddToFavorites（美元的请求）｛这个美元->forward404Unless（美元的产品= ProductPeer::retrieveByPk（美元的请求->getParameter（“id”）））；这个美元->getUser（）->addProductToFavorites（美元的产品）；这个美元->重定向（“产品/指数”）；｝公共函数executeRemoveFromFavorites（美元的请求）｛这个美元->forward404Unless（美元的产品= ProductPeer::retrieveByPk（美元的请求->getParameter（“id”）））；这个美元->getUser（）->removeProductFromFavorites（美元的产品）；这个美元->重定向（“产品/指数”）；｝

新productActions班级现在31代码行而不是96．我们已经把代码削减了三分之二以上!

下面是我们项目的LOC表:

因此，在重构之后，我们的代码减少了一些，特别是关注点更好地分离了。我们还创建了可重用的对象和方法:

• ProductForm
• ProductPeer: getAvailableProducts ()
• myUser: addProductToFavorites ()
• myUser: removeProductFromFavorites ()
• myUser: hasInFavorites ()

可重用对象也意味着它们是自包含的。所以，现在你甚至可以把你的模型类和表单打包成一个插件。

作为一个额外的好处，为我们的代码编写单元测试现在也更容易了。

每个重构过程的主要目标总是相同的:

• 使代码更易于维护
• 允许代码轻松地演进
• 通过重用现有代码，减少编写新特性所需的代码量

即使Vince在这次重构之后很累，但他还是很高兴，并立即开始重构网站的其他部分。我希望你也学到了新的技巧，你将能够使用你的下一个symfony项目。ob娱乐下载