论坛首页 招聘求职论坛

碰到一个钉子户,请大家给点意见

浏览 43429 次
精华帖 (0) :: 良好帖 (0) :: 隐藏帖 (0)
作者 正文
   发表时间:2008-05-20  
抛出异常的爱 写道
bulargy 写道
miracle9i 写道
斗胆问一句,俺、俺没发现这段有啥不正常的啊

额的个神呐~~~
resourceTypeList=resourcesManager.findResourceType();  
this.setResourceTypeList(resourceTypeList);  

得到了值后,又把自己set给自己了~~~~你不觉得奇怪么~~~~汗一个先~~

这个不算什么。。。。
这种错。。。很常见。。。
不用太在意。。。。这种不算是错误。。。junit能过。


抛兄很务实啊,不过测试能过不等于易于维护,所以还是被看做是坏代码。
0 请登录后投票
   发表时间:2008-05-20  
fins 写道
给那小子安排个任务, 让他读一读下面这篇帖子以及回复 然后让写个读后感
帖子如下:
http://www.iteye.com/topic/193218


轮回是这样产生的,呵呵
0 请登录后投票
   发表时间:2008-05-20  
你确定你没问题,而且你的其他小弟同你一条心,那么想办法把他踢掉。
不值得白养一张嘴。
0 请登录后投票
   发表时间:2008-05-20  
freej 写道

抛兄很务实啊,不过测试能过不等于易于维护,所以还是被看做是坏代码。


应该只属于多复制了无用代码。我觉得这种代码也不是什么大问题,因为这里重复赋值也不会出现问题,就是会让后来维护的人员迷糊。

不知道pmd,findbugs这些检测工具能不能检测出这种无用的重复代码。

PS:这个,我觉得完全靠人管理,没法解决这个兄弟的问题,他写的代码毕竟是可运行的。
但是呢,如果追求完美的话,不如指定一些checkstyle规则,然后用pmd,findbugs加强代码质量检测,写出违反规则的代码考虑是记录一下,还是发回去重写啊。
不过这样的话,你眼前清净了,手下的同志们可就要诉苦了。
0 请登录后投票
   发表时间:2008-05-20  
xyz20003 写道
freej 写道

抛兄很务实啊,不过测试能过不等于易于维护,所以还是被看做是坏代码。


应该只属于多复制了无用代码。我觉得这种代码也不是什么大问题,因为这里重复赋值也不会出现问题,就是会让后来维护的人员迷糊。

不知道pmd,findbugs这些检测工具能不能检测出这种无用的重复代码。

PS:这个,我觉得完全靠人管理,没法解决这个兄弟的问题,他写的代码毕竟是可运行的。
但是呢,如果追求完美的话,不如指定一些checkstyle规则,然后用pmd,findbugs加强代码质量检测,写出违反规则的代码考虑是记录一下,还是发回去重写啊。
不过这样的话,你眼前清净了,手下的同志们可就要诉苦了。



如果都养成了好习惯的话,那写出干净的代码是很自然的事,不会觉得苦。我觉得好的习惯是培养的重点,这样有利于整个开发维护过程的顺利进行。

pmd和findbugs我都没用过,有空真的学学习了。
0 请登录后投票
   发表时间:2008-05-20  
抛出异常的爱 写道
bulargy 写道
miracle9i 写道
斗胆问一句,俺、俺没发现这段有啥不正常的啊

额的个神呐~~~
resourceTypeList=resourcesManager.findResourceType();  
this.setResourceTypeList(resourceTypeList);  

得到了值后,又把自己set给自己了~~~~你不觉得奇怪么~~~~汗一个先~~

这个不算什么。。。。
这种错。。。很常见。。。
不用太在意。。。。这种不算是错误。。。junit能过。

额的神啊,我又发现了无数让我又想笑又想哭的代码了~~~~
例如:
自己拼hql 然后人云亦云的抄别人用占位符传参数的代码
修改公用类,不通知其他人
一个东西有树和列表2种都可以操作,为了做树了就把用列表操作的给删掉了,导致错误
让他模糊查询,他用"="号
有多个查询条件,他对所有的组合进行if else
还有那命名。。。。XxxId的意思竟然能够是Xxx不能为空
我彻底的崩溃了,他老人家现在回学校答辩去了,一个人写的代码,我们3帮他改bug,前天统计出43个。。。。今天又n个
他要是回来了,还是好好回炉几个月,谁的面子我都不卖了~~~
0 请登录后投票
   发表时间:2008-05-20  
fins 写道
给那小子安排个任务, 让他读一读下面这篇帖子以及回复 然后让写个读后感
帖子如下:
http://www.iteye.com/topic/193218

呵呵,这招够损啊~~~
0 请登录后投票
   发表时间:2008-05-20  
pikachu 写道
你确定你没问题,而且你的其他小弟同你一条心,那么想办法把他踢掉。
不值得白养一张嘴。

我也想,可我没权利~~~
0 请登录后投票
   发表时间:2008-05-20  
xyz20003 写道
freej 写道

抛兄很务实啊,不过测试能过不等于易于维护,所以还是被看做是坏代码。


应该只属于多复制了无用代码。我觉得这种代码也不是什么大问题,因为这里重复赋值也不会出现问题,就是会让后来维护的人员迷糊。

不知道pmd,findbugs这些检测工具能不能检测出这种无用的重复代码。

PS:这个,我觉得完全靠人管理,没法解决这个兄弟的问题,他写的代码毕竟是可运行的。
但是呢,如果追求完美的话,不如指定一些checkstyle规则,然后用pmd,findbugs加强代码质量检测,写出违反规则的代码考虑是记录一下,还是发回去重写啊。
不过这样的话,你眼前清净了,手下的同志们可就要诉苦了。

其实也没追求完美,就是能写出“正确点”“正常点”“规范点”的代码
团队代码规范文档java的和js的都订好了,之前都让他们看过。
我估计他也是看过就忘干净了,让他去读读《think in java》补补基础,读读《重构》改善改善代码质量,至今未读。。。我都放弃了。。。。
0 请登录后投票
   发表时间:2008-05-20  
freej 写道
xyz20003 写道
freej 写道

抛兄很务实啊,不过测试能过不等于易于维护,所以还是被看做是坏代码。


应该只属于多复制了无用代码。我觉得这种代码也不是什么大问题,因为这里重复赋值也不会出现问题,就是会让后来维护的人员迷糊。

不知道pmd,findbugs这些检测工具能不能检测出这种无用的重复代码。

PS:这个,我觉得完全靠人管理,没法解决这个兄弟的问题,他写的代码毕竟是可运行的。
但是呢,如果追求完美的话,不如指定一些checkstyle规则,然后用pmd,findbugs加强代码质量检测,写出违反规则的代码考虑是记录一下,还是发回去重写啊。
不过这样的话,你眼前清净了,手下的同志们可就要诉苦了。



如果都养成了好习惯的话,那写出干净的代码是很自然的事,不会觉得苦。我觉得好的习惯是培养的重点,这样有利于整个开发维护过程的顺利进行。

pmd和findbugs我都没用过,有空真的学学习了。

对呀,养成好一点的代码习惯,最好是团队内大家的代码习惯有一定的默契,那样才能提高工作效率和质量,也便于维护。
0 请登录后投票
论坛首页 招聘求职版

跳转论坛:
Global site tag (gtag.js) - Google Analytics