项目心得－提高代码质量的体会（3）?上面的代码中，有边框的变量很明显是重复声明，因为从脚本开始到结束，各次
项目心得－提高代码质量的体会（3）
?
上面的代码中，有边框的变量很明显是重复声明，因为从脚本开始到结束，各次查询和上次查询的结果没有任何关系，在下半部分的jsp中也表示所有的BaseList只是充当临时的查询对象，所有的XXX_Sql也只是一个临时存储Sql语句的一个变量。所以象这样的代码必须在代码审查中得到很好的重构，避免重复代码的出现！提高代码编写的效率。<o:p></o:p>
??????? 我觉得重构阶段可以有两种方法：1 依赖重构方案，对不满足方案的代码进行重构<o:p></o:p>
2 找出一些常见的需要重构的代码，并列举重构方案<o:p></o:p>
个人觉得第二个方法比较可行，首先因为重构的方案有非常多种，一本书几百页都在讲重构的方法，而且随着程序员编码经验的增加，每个人对重构方案的看法也不同，不能盲目的去套这些东西，不能比较技术或方法的好坏，只能说这个技术（方法）用在某个地方比其他技术（方法）更合适。第二种方法将给大家提供一些常用的（特别是我们公司经常会出现的一些情况）重构方案，并在编码过程中推行，这样大家可以一点一点的提高自己，提高编码的效率。<o:p></o:p>
1 楼 blogbin 2007-05-23 个人体会：
评审的确是发现问题，提高个人技能，改善团队效率，有效沟通的好办法。
blogbin 2 楼 xly_971223 2007-05-23 代码评审是保证软件质量的重要手段 也是检查软件设计是否落实的手段
代码审查可以是新手快速进步，找出程序中的一些低级bug，使代码结构统一
还有一点是可以让大家有一个相互学习的机会
但是目前能做到代码评审的公司能有几个呢？
我知道华为有 其他的公司没见着 3 楼 nicemike 2007-05-23 vampire423 写道楼主，你们公司项目中还把SQL语句放在JSP里？？
公司招人有问题，很多就是经过培训就进来，甚至没有经验的毕业生都要～～～
郁闷的很。。。最近也想走呢～～ 4 楼 jlusdy 2007-05-23 没有经验不要紧
关键是要有规范
5 楼 blogbin 2007-05-23 经验是可以积累的，良好的个人素质更有利于成长。
blogbin 6 楼 nicemike 2007-05-24 blogbin 写道经验是可以积累的，良好的个人素质更有利于成长。
blogbin
积累了一定的经验，可能会由于福利待遇等问题就会离开。。。
这样公司的辛勤栽培付之一炬～～～
对于刚起步的公司来说，这是很危险的事情～～
7 楼 zyl 2007-05-24 重构是改进代码质量的一个很好的手段。但是重构是有风险的，如果没有充分的单元测试作保证，那么产生的问题会让你停止重构。 8 楼 mario 2007-05-24 如果是学习重构技术，提高程序员水平，这是个不错的方法。
如果是应用重构技术，正如楼上说所-单元测试
如果是为了代码检查，重点别放在重构上，重点放在团队成员规范检查和水平提高上。
9 楼 xly_971223 2007-05-25 zyl 写道重构是改进代码质量的一个很好的手段。但是重构是有风险的，如果没有充分的单元测试作保证，那么产生的问题会让你停止重构。
没有单元测试 还谈什么重构， 重构和单元测试是一个整体 10 楼 zlkn2005 2007-05-25 怎么看不到代码啊？
如果正像LS的几位说的那样，我觉得需要学的地方会有很多。 11 楼 xj4150 2007-05-25 nicemike 写道vampire423 写道楼主，你们公司项目中还把SQL语句放在JSP里？？
公司招人有问题，很多就是经过培训就进来，甚至没有经验的毕业生都要～～～
郁闷的很。。。最近也想走呢～～
jsp中不能写sql语句可以作为一项规定啊。这么简单的检查也不做吗？
另外，确实看不到代码。 12 楼 klyuan 2007-05-25 zyl 写道重构是改进代码质量的一个很好的手段。但是重构是有风险的，如果没有充分的单元测试作保证，那么产生的问题会让你停止重构。
不做单元测试那做什么？重什么构？
