改善既有代码的设计》之四 简化条件表达式(2)

状况:你手上有个表达式,它根据对象型别的不同而选择不同的行为,那么将这个条件式的每个分支放进一个subclass内的覆写函数中,然后将原始函数声明为抽象函数。

 

此代码的坏味道:

  1、它太长,当视频有新类型的时候,它会变得更长。

  2、它明显做了不止一件事。

  3、它违反了单一权责原则,因为它有好几个修改它的理由。

  4、它违反了开放闭合原则,因为每当添加新类型时,必须修改它。不过最麻烦的可能是到处皆有类似结构(_get类型名Rank())的函数。

 

 Introduce Assertion

 

状况:某一段代码需要对程序状态(state)做出某种假设,那么以断言(assertion)明确表现这种假设。

 

 

运行结果:

运行结果:

 

采点:

  1、常常会有这样的代码,只有当某个条件为真时,该段代码才能正常运行。——实际上程序最后成品往往将assertion统统删除。

  2、这样的假设通常并没有在代码中明确表现出来,你必须阅读整个算法才能看出。——有时候程序员会以注释写出这样的假设,而assetion是一种更好的技术。

  3、assertion是一个条件式,应该总是为真。如果失败,表示程序员犯了错误

  4、assertion可以作为交流与调试的辅助。——交流:可以帮助程序员阅读理解代码所做的假设。调试:帮助程序员找到bug,可以在距离最近的地方抓住bug。

  5、assertion并不改变程序的任何行为。

  6、assertion价值:帮助程序员理解代码正确运行的必要条件。

  7、建议最好把assertion的条件式使用Extract Method,为了将若干地方的重复码提炼到同一个函数中,也许只是为了更清楚说明条件式的用途。

 

 总结

 

       这一章我比较喜欢“Replace Nested Conditional with Guard Clauses “这个方式,我在平时的代码中也经常这样用,还有人给这种方式取名叫”卫从句“。

      还有一个就是我经常在php开发中用的调试是var_dump()或print_r(),我也第一次发现php中还有assert这种方式,不错!

 

        在学习和实践的过程中,我也学到了很多不错的方式。但是我觉得在团队开发中,有的时候还是”大局为重“,按照团队的习惯方式去编码,或者你可以跟团队沟通,得到大家的认可之后,在使用这里面的方法,这样大家彼此调试和阅读对方代码的时候比较方便。

您可能感兴趣的文章:

内容版权声明:除非注明,否则皆为本站原创文章。

转载注明出处:http://www.heiqu.com/e9add83248766e79fd64096efde87327.html