在项目中寻找代码的坏命名

澳门新葡亰网站注册 29

常做的和常想的事情

 

澳门新葡亰网站注册 1

介绍

晦涩的if条件

 

1)对于||的处理

  这段时间一直做项目,所以相对忙碌些,今天终于有时间回过头来好好看一下自己写的代码,看哪里有问题,哪里有“坏味道”。

澳门新葡亰网站注册 2

 

                             
  澳门新葡亰网站注册 3上面的代码要比下面的代码理解起来花更多的时间

  慢慢过,慢慢回忆代码。开始捕捉坏味道。

澳门新葡亰网站注册 4

 

  style=”FONT-SIZE: 15px”>尽管减少代码行数是一个好目标,但把理解代码所需的时间最小化是一个更好的目标。

澳门新葡亰网站注册,常做的和常想的事情

澳门新葡亰网站注册 5

 

 

澳门新葡亰网站注册 6

返回值

 

澳门新葡亰网站注册 7

晦涩的if条件

                                
  澳门新葡亰网站注册 8上面的”$ret”是“我想不出名字“的托词。与其使用这样空洞的名字,不如挑一个能描述这个实体的值或者目的的名字。

 

澳门新葡亰网站注册 9

1)对于||的处理

                                   
 澳门新葡亰网站注册 10$alias声明了这个变量是用来承载别名的——标明了这个变量的目的。并且可能帮我们找到缺陷

澳门新葡亰网站注册 11

澳门新葡亰网站注册 12

                               
  澳门新葡亰网站注册 13上面的代码要比下面的代码理解起来花更多的时间

style=”FONT-SIZE: 16px”>好的名字应当描述变量的目的或者它所承载的值。

澳门新葡亰网站注册 14

澳门新葡亰网站注册 15

 

临时变量

  style=”font-size: 15px;”>尽管减少代码行数是一个好目标,但把理解代码所需的时间最小化是一个更好的目标。

澳门新葡亰网站注册 16

澳门新葡亰网站注册 17

                               
 澳门新葡亰网站注册 18 这里的$i是“我的临时变量”,专门用来让统计数据自动增加,避免统计点的重复。但是对这个$i来讲最重要的并不是临时变量。用charset_index表示“我的统计数据的下标”,更具“描述性”。

 

澳门新葡亰网站注册 19

 

澳门新葡亰网站注册 20

返回值

循环迭代器

 

我在我的js代码中也发现了坏味道。

 

澳门新葡亰网站注册 21

澳门新葡亰网站注册 22

                               
 澳门新葡亰网站注册 23i变量名字很空泛,所以不要这么做。

                                
  澳门新葡亰网站注册 24上面的”$ret”是“我想不出名字“的托词。与其使用这样空洞的名字,不如挑一个能描述这个实体的值或者目的的名字。

澳门新葡亰网站注册 25

澳门新葡亰网站注册 26

像i,j,iter和it等名字常用做索引和循环迭代器。尽管名字很空泛,但是大家都知道它们的意思是“我是一个迭代器“。——实际上你用这些名字来表示其他含义,那会很混乱。所以不要这么做。

                                   
 澳门新葡亰网站注册 27$alias声明了这个变量是用来承载别名的——标明了这个变量的目的。并且可能帮我们找到缺陷

style=”FONT-SIZE: 16px”>如果你非要用i,j,it这样空泛的名字,那么你要有个好的理由说服自己。

澳门新葡亰网站注册 28

澳门新葡亰网站注册 29

style=”font-size: 16px;”>好的名字应当描述变量的目的或者它所承载的值。

总结

澳门新葡亰网站注册 30

  我们在编码的过程中,多花几秒钟想出一个好名字,你会发现我们的“命名能力“很快提升上去。