|
锁定老帖子 主题：java 代码审核的重要性
精华帖 (0) :: 良好帖 (0) :: 新手帖 (0) :: 隐藏帖 (0)
|
|
|---|---|
| 作者 | 正文 |
|
发表时间：2014-11-12
越发的注意到代码审核的重要性 越是大的，时间跨越很久，经手人数多的项目，代码审核的重要性越明显。 今天领导给我一个小任务，让我查一下几个sql语句，结果我查了半天啊，原因就是我得跟踪查询条件，而这些个条件真是一个乱啊，各种sql语句拼接，还有赋值，没写三条代码呢，就是一个execute(sql),每个rs数据又放在list中，list又放在每个map中，map又根据key找value，找不到又有新的逻辑，而且我这个小程序猿都发现很多的小功能可以封装起来，当然我也知道，我们在实际做任务的时候，哪能顾得上那么多的东东啊，想的就是快点完成任务。这时就体现出代码审核的重要性了。对于特别容易看出应该封装的代码由于是自己写的，思想可能不好跳出来，一时发现不了，可是别人就会一眼发现这个问题。所以同事之间代码审核我觉得是既简单又高效的行为，既提高代码的可读性、维护性又不占用多少时间，换来的却是以后能缩减一半以上时间的维护，对项目长远的意义更是重大。 可是，实际公司中却几乎没有这个审核环节，挺悲哀的··· ··· 声明：ITeye文章版权属于作者，受法律保护。没有作者书面许可不得转载。
推荐链接
|
| 返回顶楼 | |
|
发表时间：2014-11-14
一般来说，在实际开发的时候，都是按照功能点开发，这个功能点做好了，基本没人再去理会，包括开发者本人，除非有了bug需要。很多可以复用的功能，因为没有比较好的设计，基本都是在需要重用的时候，直接重写了。
我觉得代码的审核或者代码的评审，完全可以自己完成。 |
| 返回顶楼 | |
|
发表时间：2014-11-14
那我只能说你真的很猛，能够在不同的思维中自由的转化，记得看过一篇文章，英国一个小男孩，自己同时扮演十个角色来相互对话，这就是牛人啊，这思维，真酸爽
|
| 返回顶楼 | |
|
发表时间：2014-11-17
楼主你说的对。
项目紧什么的都是借口，写出那种代码的，就是能力问题。 那么行业里这种情况确实占大多数，所以一定不是偶然，而且你只能去理解体谅这种情况。 比如一个实习生，刚毕业到一家公司，团队开发氛围就是上面那种，项目充斥大量垃圾代码。而他是一张白纸，他浸染了这个风格，三年后成为团队核心，依然带领新的实习生写这样的代码。于是，不管起什么新项目，不管领导决定重构N年前的老系统，殊途同归，只要氛围没变，结果还是一样。 在这个环境中，所有人都觉得代码审查时浪费时间，所有人都觉得老子能写出没有bug的代码你还要怎么样，所有人每天都在不断吐槽更老的代码是多么的差同时自己也依然这么写。软件工程是一个很潜移默化的事物，当你没达到一个层次时，你是不会知道它有多么的先进。 而大部分达不到这个层次，所以很多项目结果还不错，内部都很糟。 看到一句话很有意思：世界上最痛苦的事情不是看翔一样的代码，而是为了保持统一，自己要模仿翔的样子去写代码。 |
| 返回顶楼 | |
|
发表时间：2014-11-17
大环境就是这个样子，现在我还是个小程，希望以后我能做到我今天说的并一直保持下去
|
| 返回顶楼 | |
|
发表时间：2014-11-17
代码评审的作用毋庸置疑是重要的，尤其是在项目启动阶段，对所有人的代码尤其是新手的代码要做重点代码审查，当然在做审查之前要做好一个基本的代码规范表以供参考
|
| 返回顶楼 | |
|
发表时间：2014-11-19
代码审查确实很重要 记得做项目的时候没有做入口参数检查 结果前台传过来的参数不对 导致把数据库的记录全部删除了 真是作死的节奏
|
| 返回顶楼 | |
|
发表时间：2014-11-19
thly1234 写道 代码审查确实很重要 记得做项目的时候没有做入口参数检查 结果前台传过来的参数不对 导致把数据库的记录全部删除了 真是作死的节奏
这么屌。。被别人恶意注入的？ |
| 返回顶楼 | |
|
发表时间：2015-01-07
white_crucifix 写道 thly1234 写道 代码审查确实很重要 记得做项目的时候没有做入口参数检查 结果前台传过来的参数不对 导致把数据库的记录全部删除了 真是作死的节奏
这么屌。。被别人恶意注入的？ 不是 前台页面传值的时候丢了 后台又没有校验 |
| 返回顶楼 | |
