04天:重构

以前在symfonyob娱乐下载

在第三天,所有的MVC架构层被显示和修改问题正确地显示在主页的列表。应用程序越来越好,但仍然缺乏内容。

第四天的目标是显示的列表一个问题的答案,给一个URL问题详细信息页面,添加一个自定义类,有些大块代码转移到一个更好的地方。这将有助于你了解模板的概念,模型、路由策略、和重构。你可能认为这是为时过早重写代码,只有几天大,但是我们将会看到你的感觉在本教程的结束。

阅读本教程,您应该熟悉的概念在symfony MVC实现ob娱乐下载。它还将帮助如果你有了一个主意敏捷开发是多少。

显示一个问题的答案

首先,让我们继续改编自生成的模板问题CRUD期间第二天

的问题/显示行动致力于显示的细节问题,通过一个提供id。来测试它,就叫(你将不得不改变2真正的id这个问题在你的表):

http://askeet/frontend_dev.php/question/show/id/2

问题的细节

你可能已经看到了显示如果你玩过与应用程序页面。这就是我们要添加一个问题的答案。

快速浏览一下

首先,让我们看一看显示行动,位于使用/应用程序/前端/模块/问题/动作/ actions.class.php文件:

公共函数executeShow(){这个美元- >问题= QuestionPeer::retrieveByPk(这个美元- >getRequestParameter(“id”));这个美元- >forward404Unless(这个美元- >问题);}

如果您熟悉推动,你在这里认识到一个简单的请求问题表。它的目的是让独特的有价值的记录id请求的参数作为一个主键。在上面的例子在URL中,id参数的值为1,所以- > retrieveByPk ()的方法QuestionPeer类将返回的对象类问题与1作为一个主键。如果你不熟悉推动,回来后你所读的内容一些文档欧宝官网下载app在他们的网站上。

这个请求被传递到的结果showSuccess.php模板通过美元问题变量。

的- > getRequestParameter (id)的方法sfAction对象被…请求参数称为id,无论是在GET或POST传递模式。例如,如果您需要:

http://askeet/frontend_dev.php/question/show/id/1/myparam/myvalue

…然后显示行动将能够检索括号通过请求$ this - > getRequestParameter (“myparam”)。

修改showSuccess.php模板

生成的showSuccess.php模板不是我们需要的,所以我们将完全重写它。打开前端/模块/问题/模板/ showSuccess.php文件和替换的内容:

< ? phpuse_helper(“日期”)? >< div类=“interested_block”> < div类=“interested_mark”id =“mark_ < ?php echo $问题- > getId () ? > ">< ? php回声数(美元问题- >getInterests())? >< / div > < / div > < h2 > < ? php回声美元问题- >getTitle()? > < / h2 > < div类=“question_body”>< ? php回声美元问题- >getBody()? >< / div > < div id =“答案”>< ? phpforeach(美元问题- >getAnswers()作为美元的答案):? >< div类=“答案”>发布的< ? php回声美元的答案- >getUser()- >getFirstName()。' '。美元的答案- >getUser()- >getLastName()? >在< ? php回声format_date(美元的答案- >getCreatedAt(),“p”)? >< div >< ? php回声美元的答案- >getBody()? >< / div > < / div >< ? phpendforeach;? >< / div >

你认识这里的interested_block已经添加到divlistSuccess.php昨天模板。它只是显示了对于一个给定的问题感兴趣的用户数量。之后,标记看起来也很像的一个列表,只是没有link_to在标题上。它只是一个重写的初始代码只显示必要的信息的问题。

新部分是答案div。它显示所有问题的答案(使用简单$问题- > getAnswers ()推动方法),对每个人来说,显示总相关性,作者的名字,创建日期除了身体。

的format_date ()是另一个例子的模板助手最初的声明是必需的。你可以找到更多关于这个辅助的语法和其他帮手国际化助手章symfonyob娱乐下载的书(这些帮手加快显示日期的乏味的任务在一个漂亮的格式)。

添加一些新的测试数据

是时候添加一些数据回答和相关性表的末尾数据/夹具/ test_data.yml(随意添加你自己的):

答:a1_q1: question_id: q1 user_id:弗朗索瓦身体:|你可以试着读她的诗。小鸡喜欢这一类的东西。a2_q1: question_id: q1 user_id:法比身体:|不要带她去一家甜甜圈店。永远。女孩不喜欢被吃用手指——尽管很高兴。a3_q2: question_id: q2 user_id:法比身体:|问题答案是:给她买一个步骤,所以她能得到一些锻炼和感激她将失去的重量。a4_q3: question_id:第三季度user_id:法比身体:|与symfony建立它,人们会喜欢的。ob娱乐下载

重新加载您的数据:

php批/ load_data.php美元

导航到行动显示第一个问题检查修改是否成功:

http://askeet/frontend_dev.php/question/show/id/XX

问题是现在显示在一个空想家,紧随其后的是答案。这是更好的,不是吗?

修改模型,第一部分

基本上可以肯定的是,作者的全名需要其他地方的应用程序。你也可以考虑的全名是一个属性用户对象。这意味着应该有方法的用户模型允许检索的全名,而不是重建一个动作。让我们写它。打开使用/ lib /模型/ User.php并添加以下方法:

公共函数__toString(){返回这个美元- >getFirstName()。' '。这个美元- >getLastName();}

为什么这个方法命名__toString ()而不是getFullName ()或类似的东西?因为__toString ()方法是默认的方法使用PHP5对象表示为字符串。这意味着您可以替换

发布的< ? php回声美元的答案- >getUser()- >getFirstName()。' '。美元的答案- >getUser()- >getLastName()? >

线的使用/应用程序/前端/模块/问题/模板/ showSuccess.php由简单的模板

发布的< ? php回声美元的答案- >getUser()? >

实现相同的结果。整洁的,不是吗?

不要重复自己

敏捷开发的良好原则之一就是避免重复代码。它说“不要重复自己”(D.R.Y.)。这是因为重复代码是两倍的时间来审查,修改,比一块独特的封装测试和验证代码。这也使得应用程序维护复杂得多。如果你注意到今天的最后部分教程中,你可能会注意到一些之间的重复代码listSuccess.php昨天和模板写showSuccess.php模板:

< div类=“interested_block”> < div类=“interested_mark”id =“mark_ < ?php echo $问题- > getId () ? > ">< ? php回声数(美元问题- >getInterests())? >< / div > < / div >

所以我们的第一次会议重构将从两个模板中删除这段代码放到一个吗片段或可重用的代码块。创建一个_interested_user.php文件使用/应用程序/前端/模块/问题/模板/目录下面的代码:

< div类=“interested_mark”id =“mark_ < ?php echo $问题- > getId () ? > ">< ? php回声数(美元问题- >getInterests())? >< / div >

然后,替换原来的代码模板(listSuccess.php和showSuccess.php):

< div类=“interested_block”>< ? phpinclude_partial(“interested_user”,数组(“问题”= >美元问题))? >< / div >

一个片段没有本地访问任何当前的对象。片段使用美元问题中定义的变量,所以它必须是include_partial调用。额外的_前面的片段文件名称可以很容易区分从实际模板片段模板/目录。如果你想了解更多关于片段,阅读看一章symfonyob娱乐下载的书。

修改模型,第二部分

的$问题- > getInterests ()的呼唤新的片段并请求数据库并返回一个对象数组的类感兴趣。这是一个沉重的请求数量的利害关系人,并可能加载数据库太多。记住,这叫也做的listSuccess.php模板,但这一次在一个循环中,为每个列表的问题。这将是一个好主意来优化它。

一个很好的解决方法是添加一个列到问题表被称为interested_users本专栏,并更新每次创建一个感兴趣的问题。

添加一个字段用户对象模型

没有恐惧和修改使用/ config / schema.xml通过添加的数据模型ask_question表:

<列的名字=“interested_users”类型=“整数”默认的=“0”/ >

然后重建模型:

syob娱乐下载mfony propel-build-model美元

没错,我们已经重建模型,而不用担心现有的扩展!这是因为扩展用户类的使用/ lib /模型/ User.php继承自生成的推动使用/ lib /模型/ om / BaseUser.php类。这就是为什么你不应该编辑的代码使用/ lib /模型/ om /目录:每次覆盖建造模式被称为。ob娱乐下载Symfony有助于缓解正常生命周期模型更改的任何web项目的早期阶段。

您还需要更新实际的数据库。为了避免写一些SQL语句,你应该重建你的SQL架构和重新加载您的测试数据:

syob娱乐下载mfony propel-build-sql mysql - u对于美元- p / sql / lib.model.schema askeet <数据。sql批php / load_data.php美元

mysql - u对于美元- p使用- e " alter table ask_question添加interested_users int违约‘0’”

修改save ()的方法感兴趣对象

更新这个新字段的值每次用户宣布完成对一个问题,即每次记录添加到感兴趣表。可以实现与MySQL的触发,但这将依赖于一个数据库解决方案,你不能切换到另一个数据库。

最好的解决方案是修改模型通过重写save ()的方法感兴趣类。每次调用此方法类的一个对象感兴趣被创建。所以打开使用/ lib /模型/ Interest.php文件和写在以下方法:

公共函数保存(反对美元=零){美元受潮湿腐烂=父::保存(反对美元);/ /更新interested_users问题表美元问题=这个美元- >getQuestion();interested_users美元=美元问题- >getInterestedUsers();美元问题- >setInterestedUsers(interested_users美元+1);美元问题- >保存(反对美元);返回美元受潮湿腐烂;}

新save ()方法的问题与当前利益,并增加其interested_users字段。然后,它通常的save (),但因为$ this - >保存();最终在一个无限循环,它使用类方法父:保存()代替。

安全事务的更新请求

会发生什么如果数据库的更新之间失败问题对象的一个感兴趣对象?你会得到损坏数据。这是同样的问题在银行转账是指第一个请求时减少的一个账户,和第二个请求增加另一个帐户。

如果两个请求是高度相关的,你应该确保他们的执行事务。事务是两个请求会成功的保险,或没有。如果错了事务的一个请求,所有以前成功请求取消,和数据库返回到之前的状态,这是交易。

我们的save ()方法是一个很好的机会来说明在symfony的实现交易。ob娱乐下载替换代码:

公共函数保存(反对美元=零){反对美元=推动:getConnection();试一试{反对美元- >开始();美元受潮湿腐烂=父::保存(反对美元);/ /更新interested_users问题表美元问题=这个美元- >getQuestion();interested_users美元=美元问题- >getInterestedUsers();美元问题- >setInterestedUsers(interested_users美元+1);美元问题- >保存(反对美元);反对美元- >提交();返回美元受潮湿腐烂;}抓(异常$ e){反对美元- >回滚();扔$ e;}}

首先,该方法通过克里奥尔语打开一个直接连接到数据库。之间的- >开始()和- > commit ()声明,事务确保所有将没有完成。如果失败,将会抛出一个异常,数据库将执行回滚到先前的状态。

更改模板

现在- > getInterestedUsers ()的方法问题对象正常工作,是时候来简化_interested_user.php片段替换:

< ? php回声数(美元问题- >getInterests())? >

通过

< ? php回声美元问题- >getInterestedUsers()? >

的请求数量和执行时间,这应该是更好的。您可以验证它与数据库请求的数量显示在web调试工具栏,在数据库图标。注意,你也可以得到详细的SQL查询当前页面通过点击数据库图标本身:

测试修改的有效性

我们将检查没有被请求显示再次行动,但在此之前,再次运行数据导入一批,我们昨天写道:

cd /home/sfprojects/askeet/batch php load_data.php美元

当创建的记录感兴趣表,sfPropelData对象将使用覆盖save ()方法正确,应该更新相关用户记录。这是一个好方法来测试模型的修改,即使没有CRUD接口的感兴趣对象了。

检查请求主页和第一个问题的细节:

http://askeet/frontend_dev.php/http://askeet/frontend_dev.php/question/show/id/XX

感兴趣的用户的数量并没有改变。这是一个成功的移动!

相同的答案

只是做什么count($问题- > getInterests ())也可以做的count($回答- > getRelevancys ())。唯一的区别是,一个答案可以有积极的和消极的用户的投票,而一个问题只能投票“有趣”。现在你了解如何修改模型,我们可以很快。这里有更改,就像一个提醒。你不需要手动复制为明天的教程如果使用使用SVN储存库。

• 添加下面的列回答表中schema.xml

  <列的名字=“relevancy_up”类型=“整数”默认的=“0”/ ><列的名字=“relevancy_down”类型=“整数”默认的=“0”/ >

• 重建模型,并相应地更新数据库

  syob娱乐下载mfony propel-build-model symfony propel-build-sql mysql - u对于美元美元- p / sql / lib.model.schema.sql askeet <数据

• 覆盖- >保存()的方法相关性类的lib /模型/ Relevancy.php

  公共函数保存(反对美元=零){反对美元=推动:getConnection();试一试{反对美元- >开始();美元受潮湿腐烂=父::保存();/ /更新相关性回答表美元的答案=这个美元- >getAnswer();如果(这个美元- >getScore()= =1){美元的答案- >setRelevancyUp(美元的答案- >getRelevancyUp()+1);}其他的{美元的答案- >setRelevancyDown(美元的答案- >getRelevancyDown()+1);}美元的答案- >保存(反对美元);反对美元- >提交();返回美元受潮湿腐烂;}抓(异常$ e){反对美元- >回滚();扔$ e;}}

• 添加以下两个方法回答类的模型:

  公共函数getRelevancyUpPercent(){总美元=这个美元- >getRelevancyUp()+这个美元- >getRelevancyDown();返回总美元吗?sprintf(“% .0f”,这个美元- >getRelevancyUp()*One hundred./总美元):0;}公共函数getRelevancyDownPercent(){总美元=这个美元- >getRelevancyUp()+这个美元- >getRelevancyDown();返回总美元吗?sprintf(“% .0f”,这个美元- >getRelevancyDown()*One hundred./总美元):0;}

• 改变部分有关的答案问题/模板/ showSuccess.php由:

  < div id =“答案”>< ? phpforeach(美元问题- >getAnswers()作为美元的答案):? >< div类=“答案”>< ? php回声美元的答案- >getRelevancyUpPercent()? > %< ? php回声美元的答案- >getRelevancyDownPercent()? >%下发布的< ? php回声美元的答案- >getUser()- >getFirstName()。' '。美元的答案- >getUser()- >getLastName()? >在< ? php回声format_date(美元的答案- >getCreatedAt(),“p”)? >< div >< ? php回声美元的答案- >getBody()? >< / div > < / div >< ? phpendforeach;? >< / div >

• 添加一些测试数据的装置

  相关性:rel1: answer_id: a1_q1 user_id:法比得分:1 rel2: answer_id: a1_q1 user_id:弗朗索瓦分数:1

• 启动人口批

• 检查问题/显示页面

路由

年初以来,本教程中,我们叫做URL

http://askeet/frontend_dev.php/question/show/id/XX

symfony的默认路由规则理解这个要求实际上如果你有要求ob娱乐下载

http://askeet/frontend_dev.php?module=question&action=show&id=XX

但有一个路由系统开辟了很多其他的可能性。我们可以使用URL的标题问题,能够要求相同的页面:

http://askeet/frontend_dev.php/question/what-shall-i-do-tonight-with-my-girlfriend

这将优化的搜索引擎索引网站的页面,并使url更具可读性。

创建另一个版本的标题

首先,我们需要一个转换版本的标题——剥夺了标题——作为一个URL。有一个以上的方法替代,我们将选择存储这个标题的新列问题表。在schema.xml添加以下行,问题表:

<列的名字=“stripped_title”类型=“varchar”大小=“255”/ ><独特的名字=“unique_stripped_title”><惟一的列的名字=“stripped_title”/ >< /独特的>

…和rebuild the model and update the database:

syob娱乐下载mfony propel-build-model symfony propel-build-sql mysql - u对于美元美元- p / sql / lib.model.schema.sql askeet <数据

我们将很快覆盖setTitle ()的方法问题对象,同时设置了标题。

自定义类

但是在那之前,我们将创建一个自定义类来改变一个标题为剥夺了标题,特别是因为这个函数并不关注问题对象(我们也可能会使用它回答对象)。

创建一个新的myTools.class.php下的文件使用/ lib /目录:

< ? php类myTools{公共静态函数stripText(美元的文本){美元的文本=函数(美元的文本);/ /删除了所有非单词字符美元的文本=preg_replace(' /\ W/ ',' ',美元的文本);/ /替换所有空白部分破折号美元的文本=preg_replace(' /\+ / ',“- - -”,美元的文本);/ /修剪破折号美元的文本=preg_replace(' /\ -美元/ ',”,美元的文本);美元的文本=preg_replace(' / ^\ -/ ',”,美元的文本);返回美元的文本;}}

现在打开使用/ lib /模型/ Question.php类文件并添加:

公共函数setTitle(v美元){父::setTitle(v美元);这个美元- >setStrippedTitle(myTools::stripText(v美元));}

请注意,myTools自定义类不需要宣称:symfony自动装载在需要的时候,它位于提供ob娱乐下载lib /目录中。

我们现在可以重新加载数据:

syob娱乐下载mfony cc php批/ load_data.php美元

如果你想了解更多关于自定义类和定制的帮手,阅读扩展章symfonyob娱乐下载的书。

改变链接显示行动

在listSuccess.php模板,更改线

< h2 > < ? php回声link_to(美元问题- >getTitle(),“问题/显示? id =”。美元问题- >getId())? > < / h2 >

通过

< h2 > < ? php回声link_to(美元问题- >getTitle(),的问题/显示? stripped_title = '。美元问题- >getStrippedTitle())? > < / h2 >

现在打开actions.class.php的问题模块,和改变显示行动:

公共函数executeShow(){$ c=新标准();$ c- >添加(QuestionPeer::STRIPPED_TITLE,这个美元- >getRequestParameter(“stripped_title”));这个美元- >问题= QuestionPeer::doSelectOne($ c);这个美元- >forward404Unless(这个美元- >问题);}

再次显示的问题列表,访问每个人通过点击标题:

http://askeet/frontend_dev.php/

url正确显示了标题的问题:

http://askeet/frontend_dev.php/question/show/stripped-title/what-shall-i-do-tonight-with-my-girlfriend

改变路由规则

但这并不是我们希望如何被显示。现在,是时候编辑路由规则。打开routing.yml配置文件(位于使用/应用程序/前端/ config /目录),并添加以下规则的文件:

问题:url: / /问题:stripped_title参数:{模块:问题,行动:}

在url行,这个词问题是一个自定义的文本将出现在最终的URL,而stripped_title是一个参数(之前:)。他们形成一个模式symfony路由ob娱乐下载系统将适用于的链接问题/显示行动电话,因为所有的链接在我们的模板使用link_to ()帮手。

它是最后的测试时间:显示主页,点击标题的第一个问题。第一个问题不仅显示(证明没有什么坏了),但现在你的浏览器的地址栏显示:

http://askeet/frontend_dev.php/question/what-shall-i-do-tonight-with-my-girlfriend

如果你想了解更多关于路由功能,读取路由策略一章symfonyob娱乐下载的书。

明天见

今天,网站本身没有得到许多新特性。然而,你看到更多模板编码,您知道如何修改模型,整个代码已经在很多地方重构。

这发生在一个symfony项目的生活:可以重用的代码重构一个片段或一个自定义类,代码出ob娱乐下载现在一个动作或者一个模板,实际上属于模型移动到模型。即使这传播大量的小文件中的代码传播许多目录,更容易维护和演化。此外,一个symfony项目的文件结构,很容易找到一段代码实际上是根据其性ob娱乐下载质(辅助、模型、模板、行动、自定义类,等等)。

重构工作今天将加快发展即将到来的日子。我们会定期做一些重构的这个项目,因为我们开发——做一个功能的工作方式而不用担心即将到来的功能,需要一个好的结构代码如果我们不想最后搞得一团糟。

明天是什么?我们将开始编写一个形式,看看如何获取信息。我们还将主页的列表的问题分割成页面。与此同时,今天感觉免费下载的代码从SVN库(标记release_day_4):

http://svn.askeet.com/tags/release_day_4/

和寄给我们使用的任何问题使用邮件列表或者是专门的论坛。