设为首页 收藏本站
查看: 981|回复: 0

[经验分享] 一个python程序员的ruby三日游(五) ——重构(一)

[复制链接]

尚未签到

发表于 2017-5-8 13:20:56 | 显示全部楼层 |阅读模式
  写在这最后的最后,这个算是在最后面学到的东西,关于这五篇的来源,之所以是三天内学习的,是因为ThoughtWords的Homework。这部分内容在可以在十一月这篇博文了解到,按照顺序来说也差不多是那样子的,按照敏捷开发的思想一步步过来,不过总的来说Ruby的元编程很美妙,Ruby很美妙。而这也是在电话面试的时候,问到的几本书而构成的Homework的顺序。总的来说,可以学到很多东西真的很不错,虽然没有多少机会可以进去。
  那么,接下来就用之中重构的代码一部分,一部分的说,总得来说重构很有趣。如果让我对我所做的重构进行一个定义的话,那就是下面两个内容:


  • 使程序更容易读懂
  • 删除重复代码


基本部分




Repalce Subclass with Fields(以字段代替子类)


DSC0000.jpg



原来的结构就是类似于如上所示,请原谅我暂时还不能用原文中的代码来表述,不过总体上是差不多的,原先的结构理论上还是不错的,问题是唯一的区别只在“返回数量数据”上。

原先是考虑到后期,当不同的书有不同的属性,或者需要不同的操作,后来,在系统另一部分优化的时候发现,这样做有利于系统后期扩展,也就是我们可以加一本新的书进去,而不是添加一个新的类,添加新的类,只会使系统越来越复杂。(转载保留Phodal's
BlogPhodal'szenthink)


DSC0001.jpg



原先考虑过用适配器模式来解决这个问题,问题是这样子仿佛只会使问题复杂化。





Decompose Conditional(分解条件表达式)


这个东西就用得比较多,只限于原先的条件式比较复杂,先贴出重构后的代码。

def is_weekend(sum)
sum>5 && sum<=7
end
def is_weekday(sum)
sum<=5 && sum >0
end


也就是一个用于条件判断的表达式,只是不知道在这里的系统开销会比原来多多少,问题是这样可以使系统更容易维护,结合代码

if is_weekend(sum)
day.weekend+=1
elsif is_weekday(sum)
day.weekday+=1


不过,让我觉得有点多此一举的感觉,只是可读性比原来强多了,对于复杂的系统来说还是相当有利的。

if sum>5 && sum<=7
day.weekend+=1
elsif sum<=5 && sum>0
day.weekday+=1


不过,这也适合于在重构一书中提到的另外一个原则。


Replace Magic Numer with Symbolic Constant(以字面常量取代魔法数)


放在原来的代码里,有时候我们不一定能理解0、5、7是干嘛的,但是要是放在

is_weekend()
is_weekday()


函数里面,就变得简单明了了。





Extrace Method(提炼函数)


这个就在里面用得比较多,不好举例,也就是把一个函数分出来,换句话说,也就是让函数有着真正的意义 。

原先,用的是比较价格函数

compare()


问题是在我们比较的时候,我们还需要取得不同的几个值,而不同的几个值还需要比较取得最小值,于是函数就独立了几个出来了,如

get_costsmin()
list_costsmin()
select_costsmin()





换句放说,就是把原来的大函数分解为小函数。

其他可参见《重构 改善既有代码的设计》





总结


一开始整理那些代码的时候,让人觉得不是很愉快,而且很容易无从下手,几个简单的方法,


  • 挑大函数
  • 复杂的条件语句
  • 不容易看懂的代码

总的来说,重构很好玩~~~

运维网声明 1、欢迎大家加入本站运维交流群:群②:261659950 群⑤:202807635 群⑦870801961 群⑧679858003
2、本站所有主题由该帖子作者发表,该帖子作者与运维网享有帖子相关版权
3、所有作品的著作权均归原作者享有,请您和我们一样尊重他人的著作权等合法权益。如果您对作品感到满意,请购买正版
4、禁止制作、复制、发布和传播具有反动、淫秽、色情、暴力、凶杀等内容的信息,一经发现立即删除。若您因此触犯法律,一切后果自负,我们对此不承担任何责任
5、所有资源均系网友上传或者通过网络收集,我们仅提供一个展示、介绍、观摩学习的平台,我们不对其内容的准确性、可靠性、正当性、安全性、合法性等负责,亦不承担任何法律责任
6、所有作品仅供您个人学习、研究或欣赏,不得用于商业或者其他用途,否则,一切后果均由您自己承担,我们对此不承担任何法律责任
7、如涉及侵犯版权等问题,请您及时通知我们,我们将立即采取措施予以解决
8、联系人Email:admin@iyunv.com 网址:www.yunweiku.com

所有资源均系网友上传或者通过网络收集,我们仅提供一个展示、介绍、观摩学习的平台,我们不对其承担任何法律责任,如涉及侵犯版权等问题,请您及时通知我们,我们将立即处理,联系人Email:kefu@iyunv.com,QQ:1061981298 本贴地址:https://www.yunweiku.com/thread-374753-1-1.html 上篇帖子: 一种分析代金券使用分布情况的方法python实现版(下) 下篇帖子: 一个python程序员的ruby三日游(四)——观察者模式
您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

扫码加入运维网微信交流群X

扫码加入运维网微信交流群

扫描二维码加入运维网微信交流群,最新一手资源尽在官方微信交流群!快快加入我们吧...

扫描微信二维码查看详情

客服E-mail:kefu@iyunv.com 客服QQ:1061981298


QQ群⑦:运维网交流群⑦ QQ群⑧:运维网交流群⑧ k8s群:运维网kubernetes交流群


提醒:禁止发布任何违反国家法律、法规的言论与图片等内容;本站内容均来自个人观点与网络等信息,非本站认同之观点.


本站大部分资源是网友从网上搜集分享而来,其版权均归原作者及其网站所有,我们尊重他人的合法权益,如有内容侵犯您的合法权益,请及时与我们联系进行核实删除!



合作伙伴: 青云cloud

快速回复 返回顶部 返回列表