锁定老帖子 主题:碰到一个钉子户,请大家给点意见
精华帖 (0) :: 良好帖 (0) :: 隐藏帖 (0)
|
|
---|---|
作者 | 正文 |
发表时间:2008-05-20
抛出异常的爱 写道 bulargy 写道 miracle9i 写道 斗胆问一句,俺、俺没发现这段有啥不正常的啊
额的个神呐~~~ resourceTypeList=resourcesManager.findResourceType(); this.setResourceTypeList(resourceTypeList); 得到了值后,又把自己set给自己了~~~~你不觉得奇怪么~~~~汗一个先~~ 这个不算什么。。。。 这种错。。。很常见。。。 不用太在意。。。。这种不算是错误。。。junit能过。 抛兄很务实啊,不过测试能过不等于易于维护,所以还是被看做是坏代码。 |
|
返回顶楼 | |
发表时间:2008-05-20
|
|
返回顶楼 | |
发表时间:2008-05-20
你确定你没问题,而且你的其他小弟同你一条心,那么想办法把他踢掉。
不值得白养一张嘴。 |
|
返回顶楼 | |
发表时间:2008-05-20
freej 写道 抛兄很务实啊,不过测试能过不等于易于维护,所以还是被看做是坏代码。 应该只属于多复制了无用代码。我觉得这种代码也不是什么大问题,因为这里重复赋值也不会出现问题,就是会让后来维护的人员迷糊。 不知道pmd,findbugs这些检测工具能不能检测出这种无用的重复代码。 PS:这个,我觉得完全靠人管理,没法解决这个兄弟的问题,他写的代码毕竟是可运行的。 但是呢,如果追求完美的话,不如指定一些checkstyle规则,然后用pmd,findbugs加强代码质量检测,写出违反规则的代码考虑是记录一下,还是发回去重写啊。 不过这样的话,你眼前清净了,手下的同志们可就要诉苦了。 |
|
返回顶楼 | |
发表时间:2008-05-20
xyz20003 写道 freej 写道 抛兄很务实啊,不过测试能过不等于易于维护,所以还是被看做是坏代码。 应该只属于多复制了无用代码。我觉得这种代码也不是什么大问题,因为这里重复赋值也不会出现问题,就是会让后来维护的人员迷糊。 不知道pmd,findbugs这些检测工具能不能检测出这种无用的重复代码。 PS:这个,我觉得完全靠人管理,没法解决这个兄弟的问题,他写的代码毕竟是可运行的。 但是呢,如果追求完美的话,不如指定一些checkstyle规则,然后用pmd,findbugs加强代码质量检测,写出违反规则的代码考虑是记录一下,还是发回去重写啊。 不过这样的话,你眼前清净了,手下的同志们可就要诉苦了。 如果都养成了好习惯的话,那写出干净的代码是很自然的事,不会觉得苦。我觉得好的习惯是培养的重点,这样有利于整个开发维护过程的顺利进行。 pmd和findbugs我都没用过,有空真的学学习了。 |
|
返回顶楼 | |
发表时间:2008-05-20
抛出异常的爱 写道 bulargy 写道 miracle9i 写道 斗胆问一句,俺、俺没发现这段有啥不正常的啊
额的个神呐~~~ resourceTypeList=resourcesManager.findResourceType(); this.setResourceTypeList(resourceTypeList); 得到了值后,又把自己set给自己了~~~~你不觉得奇怪么~~~~汗一个先~~ 这个不算什么。。。。 这种错。。。很常见。。。 不用太在意。。。。这种不算是错误。。。junit能过。 额的神啊,我又发现了无数让我又想笑又想哭的代码了~~~~ 例如: 自己拼hql 然后人云亦云的抄别人用占位符传参数的代码 修改公用类,不通知其他人 一个东西有树和列表2种都可以操作,为了做树了就把用列表操作的给删掉了,导致错误 让他模糊查询,他用"="号 有多个查询条件,他对所有的组合进行if else 还有那命名。。。。XxxId的意思竟然能够是Xxx不能为空 我彻底的崩溃了,他老人家现在回学校答辩去了,一个人写的代码,我们3帮他改bug,前天统计出43个。。。。今天又n个 他要是回来了,还是好好回炉几个月,谁的面子我都不卖了~~~ |
|
返回顶楼 | |
发表时间:2008-05-20
|
|
返回顶楼 | |
发表时间:2008-05-20
pikachu 写道 你确定你没问题,而且你的其他小弟同你一条心,那么想办法把他踢掉。
不值得白养一张嘴。 我也想,可我没权利~~~ |
|
返回顶楼 | |
发表时间:2008-05-20
xyz20003 写道 freej 写道 抛兄很务实啊,不过测试能过不等于易于维护,所以还是被看做是坏代码。 应该只属于多复制了无用代码。我觉得这种代码也不是什么大问题,因为这里重复赋值也不会出现问题,就是会让后来维护的人员迷糊。 不知道pmd,findbugs这些检测工具能不能检测出这种无用的重复代码。 PS:这个,我觉得完全靠人管理,没法解决这个兄弟的问题,他写的代码毕竟是可运行的。 但是呢,如果追求完美的话,不如指定一些checkstyle规则,然后用pmd,findbugs加强代码质量检测,写出违反规则的代码考虑是记录一下,还是发回去重写啊。 不过这样的话,你眼前清净了,手下的同志们可就要诉苦了。 其实也没追求完美,就是能写出“正确点”“正常点”“规范点”的代码 团队代码规范文档java的和js的都订好了,之前都让他们看过。 我估计他也是看过就忘干净了,让他去读读《think in java》补补基础,读读《重构》改善改善代码质量,至今未读。。。我都放弃了。。。。 |
|
返回顶楼 | |
发表时间:2008-05-20
freej 写道 xyz20003 写道 freej 写道 抛兄很务实啊,不过测试能过不等于易于维护,所以还是被看做是坏代码。 应该只属于多复制了无用代码。我觉得这种代码也不是什么大问题,因为这里重复赋值也不会出现问题,就是会让后来维护的人员迷糊。 不知道pmd,findbugs这些检测工具能不能检测出这种无用的重复代码。 PS:这个,我觉得完全靠人管理,没法解决这个兄弟的问题,他写的代码毕竟是可运行的。 但是呢,如果追求完美的话,不如指定一些checkstyle规则,然后用pmd,findbugs加强代码质量检测,写出违反规则的代码考虑是记录一下,还是发回去重写啊。 不过这样的话,你眼前清净了,手下的同志们可就要诉苦了。 如果都养成了好习惯的话,那写出干净的代码是很自然的事,不会觉得苦。我觉得好的习惯是培养的重点,这样有利于整个开发维护过程的顺利进行。 pmd和findbugs我都没用过,有空真的学学习了。 对呀,养成好一点的代码习惯,最好是团队内大家的代码习惯有一定的默契,那样才能提高工作效率和质量,也便于维护。 |
|
返回顶楼 | |