专家:一个重构的故事(1/5)

前一段时间,文斯,一位经验丰富的PHP开发人员,让我看一看他的第一个symfony项目,产品商店。ob娱乐下载

他的客户很高兴为网站运行良好。文斯也满足他准时交付网站。但他没有对他的代码质量非常有信心。

他寄给我的代码有一些关于它的质量的反馈。而不是通过电子邮件回复,我决定邀请他< a href ="http://www.sensiolabs.com/">Sensio赞助实验室工作现场重构会话。

他确实是很常见的错误,我决定告诉你这个重构会话的故事,希望你会学到尽可能多的文斯。

今天,我们将介绍一些网站的功能,我们将有一个快速查看代码。我们将开始下一部分的重构过程。

这个项目了ob娱乐下载symfony 1.1。

的模式

数据库模式是由两个表,一个产品表,类别表:

#配置/模式。yml推动:产品:标题:{类型:varchar(255),要求:真实}形象:varchar(255)描述:用longvarchar价格:{类型:小数,要求:真}is_new:布尔is_in_stock:布尔created_at:时间戳category_id添加:~类别:名称:varchar (255)

主页

网站的首页是由所有可用的产品列表(is_in_stock设置为true)。

所有的功能我们将重构的管理产品模块。

这是指数行动管理主页:

/ /应用程序/前端/模块/产品/动作/ actions.class.php公共函数executeIndex(){美元标准=新标准();美元标准- >添加(ProductPeer::IS_IN_STOCK,真正的);美元标准- >addAscendingOrderByColumn(ProductPeer::价格);美元标准- >setLimit(10);这个美元- >产品= ProductPeer::doSelectJoinCategory(美元标准);这个美元- >getResponse()- >setTitle(“所有产品”);这个美元- >getResponse()- >addStylesheet(“homepage.css”);返回sfView::成功;}

和相关的indexSuccess.php模板:

我们产品<标题> < / h1 >< ? phpforeach(美元的产品作为美元的产品):? >< div > < h2 >< ? php回声美元的产品- >getTitle()? >< ? php如果(美元的产品- >getIsNew()):? > < span =风格“margin-left: 10 px;颜色:# e55”>新! < / span > < ? phpendif;? >< / h2 > < div =风格“margin-bottom: 10 px”> < / em > < em >类别:< ? php回声美元的产品- >getCategory()- >getName()? >- < em > < / em >价格:$ < ? php回声美元的产品- >getPrice()? >- - - - - -< ? php如果(in_array(美元的产品- >getId(),中的(sf_user美元- >getAttribute(“最爱”,数组())))):? >< a href =“< ?php echo url_for(“产品/ removeFromFavorites ? id =”。$产品- > getId ()) ? > "> < img src =“/图片/ favorite.png”/ > < / >< ? php其他的:? ><小> < ? php回声link_to(“添加到我的最爱”,“产品/ addToFavorites id = ?”。美元的产品- >getId())? > < /小>< ? phpendif;? >< / div > = < div > < div风格“浮动:左”> < img宽度=“100 px”src =“/图片/产品/ < ?php echo $产品- > getImage () ? > "/ > < / div > < p >< ? php回声美元的产品- >getDescription()? >< ? php如果(sf_user美元- >isAuthenticated()):? >< p风格=“text-align:正确的”> < a href =“< ?php echo url_for(“产品/编辑? id =”。$产品- > getId ()) ? > ">编辑这个产品< / > < / p >< ? phpendif;? >< / p > < br风格=“明确:“/ > < / div > < div =风格“text-align:正确的”>< ? php回声link_to(image_tag(“/图片/ add_to_cart.png”),“产品/买? id =”。美元的产品- >getId())? >< / div > <人力资源/ > < / div >< ? phpendforeach;? >

的最爱

您可能已经看到主页截图,用户可以添加一个产品给他的最爱。功能非常简单:你点击“添加到我的最爱”链接将产品添加到您的最爱,和你点击星星图像删除它从你的最爱。

最喜欢的是管理的executeAddToFavorites ()和executeRemoveFromFavorites ()方法从产品模块:

公共函数executeAddToFavorites(){美元的产品= ProductPeer::retrieveByPk(这个美元- >getRequestParameter(“id”));这个美元- >forward404Unless(美元的产品);美元的最爱=这个美元- >getUser()- >getAttribute(“最爱”);美元的最爱(美元的产品- >getId()]=真正的;这个美元- >getUser()- >setAttribute(“最爱”,美元的最爱);这个美元- >重定向(“产品/指数”);}公共函数executeRemoveFromFavorites(){美元的产品= ProductPeer::retrieveByPk(这个美元- >getRequestParameter(“id”));这个美元- >forward404Unless(美元的产品);美元的最爱=这个美元- >getUser()- >getAttribute(“最爱”);设置(美元的最爱(美元的产品- >getId()]);这个美元- >getUser()- >setAttribute(“最爱”,美元的最爱);这个美元- >重定向(“产品/指数”);}

产品编辑

当登录,管理员还可以编辑产品信息。

这一页是管理的编辑行动:

公共函数executeEdit(){这个美元- >产品= ProductPeer::retrieveByPk(这个美元- >getRequestParameter(“id”));如果(is_null(这个美元- >产品)){这个美元- >forward404();}这个美元- >形式=新ProductForm(这个美元- >产品);如果(sfRequest::帖子= =这个美元- >getRequest()- >getMethod()){这个美元- >形式- >绑定(这个美元- >getRequestParameter(“产品”),这个美元- >getRequest()- >getfile(“产品”));如果(这个美元- >形式- >isValid()){/ /获取当前的产品美元的产品=这个美元- >形式- >getObject();美元图片=这个美元- >形式- >getValue(“图像”);currentImage美元=美元的产品- >getImage();如果(美元图片){/ /删除旧的形象如果(file_exists(美元的文件= sfConfig::得到(“sf_web_dir”)。' /图片/产品/。currentImage美元)){拆开(美元的文件);}}/ /更新产品美元的产品=这个美元- >形式- >updateObject();如果(美元图片){/ /保存新形象美元的文件名=sha1(美元图片- >getOriginalName())。美元图片- >getExtension(美元图片- >getOriginalExtension());美元图片- >保存(sfConfig::得到(“sf_web_dir”)。' /图片/产品/。美元的文件名);美元的产品- >setImage(美元的文件名);}其他的{美元的产品- >setImage(currentImage美元);}/ /保存产品美元的产品- >保存();这个美元- >getUser()- >setFlash(“通知”,sprintf(的产品“% s”已经成功更新。,美元的产品- >getTitle()));这个美元- >重定向(“产品/指数”);}}}

和editSuccess.php模板:

<标题>编辑“< ?php echo $产品- > getTitle () ? > "<小># < ?php echo $产品- > getId() ? > < /小> < / h1 >< p > < ? php回声link_to(“返回主页”,“@homepage”)? action = > < / p > <形式“< ?php echo url_for(“产品/编辑? id =”。$产品- > getId ()) ? > "方法=“职位”enctype =“多部分/格式”单元格间距= > <表“0”>< ? php回声美元的形式? >< tr > < td colspan =“2”风格=“text-align:正确的”> < input type =“提交”值=“保存”/ > < / td > < / tr > < /表> > < /形式

今天就到这儿了。我们将开始下一部分的重构过程。同时,花点时间去阅读代码,并开始思考你会重构,如何以及为什么。