项目心得－提高代码质量的体会（3）

 

1.1        代码审查

从科技局项目开始，代码审查已经被提上日程，可是现在想起来，当时的做法真是有些太初级。我觉得是我的关注面错了：代码审查的过程是完成代码重构的过程！如果有些不好的代码我自己都没有看出来它需要进行重构，这个时候代码审查会议就是一个很好的剔除烂代码的过程，大家一起讨论如何对已有代码进行修改，怎么样能更好的提高代码的重用率和可读性，而不仅仅的只是关心代码中哪里少了一个空格，哪里有多了一个空格等等。

下面的代码是科技局中出现的（在一个jsp中摘抄,举例说明，不看也罢）：

上面的代码中，有边框的变量很明显是重复声明，因为从脚本开始到结束，各次查询和上次查询的结果没有任何关系，在下半部分的jsp中也表示所有的BaseList只是充当临时的查询对象，所有的XXX_Sql也只是一个临时存储Sql语句的一个变量。所以象这样的代码必须在代码审查中得到很好的重构，避免重复代码的出现！提高代码编写的效率。

        我觉得重构阶段可以有两种方法：1 依赖重构方案，对不满足方案的代码进行重构

2 找出一些常见的需要重构的代码，并列举重构方案

个人觉得第二个方法比较可行，首先因为重构的方案有非常多种，一本书几百页都在讲重构的方法，而且随着程序员编码经验的增加，每个人对重构方案的看法也不同，不能盲目的去套这些东西，不能比较技术或方法的好坏，只能说这个技术（方法）用在某个地方比其他技术（方法）更合适。第二种方法将给大家提供一些常用的（特别是我们公司经常会出现的一些情况）重构方案，并在编码过程中推行，这样大家可以一点一点的提高自己，提高编码的效率。

楼主，你们公司项目中还把SQL语句放在JSP里？？

个人体会：

评审的确是发现问题，提高个人技能，改善团队效率，有效沟通的好办法。

blogbin

代码评审是保证软件质量的重要手段 也是检查软件设计是否落实的手段
代码审查可以是新手快速进步，找出程序中的一些低级bug，使代码结构统一
还有一点是可以让大家有一个相互学习的机会
但是目前能做到代码评审的公司能有几个呢？
我知道华为有 其他的公司没见着

公司招人有问题，很多就是经过培训就进来，甚至没有经验的毕业生都要～～～

郁闷的很。。。最近也想走呢～～

没有经验不要紧

关键是要有规范

经验是可以积累的，良好的个人素质更有利于成长。

blogbin

积累了一定的经验，可能会由于福利待遇等问题就会离开。。。
这样公司的辛勤栽培付之一炬～～～
对于刚起步的公司来说，这是很危险的事情～～

重构是改进代码质量的一个很好的手段。但是重构是有风险的，如果没有充分的单元测试作保证，那么产生的问题会让你停止重构。

如果是学习重构技术，提高程序员水平，这是个不错的方法。
如果是应用重构技术，正如楼上说所-单元测试
如果是为了代码检查，重点别放在重构上，重点放在团队成员规范检查和水平提高上。