一个工作5、6年的人都是这样写代码，艹

一起开发项目，我觉得最重要的就是合作了吧。 

你更新他的代码，target目录也提交到svn。类里面经常看到一些警告（没用的import、没用的private方法、一些泛型未指定警告），楼主没有强迫症，但一个警告搞的项目、包、类都是小感叹号真的看着舒服么？  还有他copy代码重来都是连变量名、方法名都不改，只要能跑成功就提交的。 之前吧有经理看着， 有checkstyle这些个工具管着，偶尔也代码检视，还好些，也没见他这多毛病。 现在有个项目他负责， 看他的代码，什么换行，注释很随意就不说了，连个基本的警告都不注意都提交到svn，真心蛋疼。
工作6年了，就每天上班下班，复制粘贴，估计说的就是他这种人。

刚更新了他4个类，3个都有警告， 也没发说， 只能在他这渣渣代码上写自己的，烦！

23 个解决方案

#1

警告看的不舒服也//

#2

虽然我也很痛恨这个代码 不规范的问题，但是根据楼主的描述，应该算是他带着你做项目，那么这个时候，像这种活科恩能够就是你来做了，我也碰到过，当时不能理解，但是现在能体会了~~~~

#3

引用 2 楼 jimmy609 的回复:

虽然我也很痛恨这个代码 不规范的问题，但是根据楼主的描述，应该算是他带着你做项目，那么这个时候，像这种活科恩能够就是你来做了，我也碰到过，当时不能理解，但是现在能体会了~~~~

你是说我要帮他改调警告、把代码调整好这些东西？

你可能认为他很忙，带项目比较累，注意不了这些细节。  
但我不认同你的观点， 我觉得这是个人习惯及态度问题， 我觉得他太随意，对自己没要求。

#4

我就有轻微代码洁癖 最痛恨多余代码 5行能完成的事写了10行 再就是横向代码过长
参数列表超过三个或者异常列表超过三个 还横着排   看到黄色小警告 也烦  变量名
函数名 类名包名 都注意着点 别弄的各应人不拉的

#5

是烦人

#6

原来大家都一样，哈哈

#7

引用 3 楼 zheshigexiaohao 的回复:

Quote: 引用 2 楼 jimmy609 的回复:

虽然我也很痛恨这个代码 不规范的问题，但是根据楼主的描述，应该算是他带着你做项目，那么这个时候，像这种活科恩能够就是你来做了，我也碰到过，当时不能理解，但是现在能体会了~~~~

你是说我要帮他改调警告、把代码调整好这些东西？

你可能认为他很忙，带项目比较累，注意不了这些细节。  
但我不认同你的观点， 我觉得这是个人习惯及态度问题， 我觉得他太随意，对自己没要求。

恩  我的意思是你可以委婉的提一下，如果他没法改变，你就辛苦下改了，这个我是有体会的，以前也很痛恨这种代码，但是等自己带项目了，的确是对于这些不太注重了，我现在写的代码整体上就是不如两年前的规范，一些细节上的东西就是让自己带的人帮着改了~~~~，当然也有那些始终如一的，不过确实我见过很多经理写的代码都是不规范的~~~

#8

工作中没有得到上面的允许不要随便动别人的代码？
你们的这个问题是管理问题。

#9

引用 8 楼 Inhibitory 的回复:

工作中没有得到上面的允许不要随便动别人的代码？
你们的这个问题是管理问题。

一般说来是这样， 刚好我也懒的改。 就是单纯的不爽，影响工作心情。

确实是管理问题， 本来项目组是要求必需有统一的checkstyle、code template、formatter 这些东西，但项目比较多也懒的一个个都配置。现在基本也没有代码review， 初期定的开发规范什么的都差不多忘完了。  

 慢慢的就放羊了。 

#10

处境相同，特别恶心的这种代码，重点是出了事情，责任全丢给你，尼玛，领导了不起( ⊙ o ⊙ )啊！天天一到下班就告诉你活来了，这种领导最恶心

#11

你们做东西会改同一个类吗？应该尽量避免，管好自己的事就好，除非你是他上司。

#12

工作十多年的垃圾我见多了，别见怪

#13

最痛恨在branch上做这些没用的重构，你知道你改了几个变量名，改了几个缩进，加了几个泛型，会对之后其他人做merge造成非常大的影响

#14

没红插就不错了，还有关心黄的干啥。。

#15

这是一个worker的习惯，习惯养不好到哪里都这样。

#16

蛋定！蛋定！做人最重要的是开心，做个有德行有智慧的人。坚持自己的风格，让旁边的人潜移默化受你的影响

#17

有同感啊，我自己的代码我肯定会保证没有任何警告的，不然不舒服呢

#18

引用 4 楼 wanghualeilei 的回复:

我就有轻微代码洁癖 最痛恨多余代码 5行能完成的事写了10行 再就是横向代码过长
参数列表超过三个或者异常列表超过三个 还横着排   看到黄色小警告 也烦  变量名
函数名 类名包名 都注意着点 别弄的各应人不拉的

看你洁癖不是很严重，连标点符号都么有的啦。

#19

引用 13 楼 viszl 的回复:

最痛恨在branch上做这些没用的重构，你知道你改了几个变量名，改了几个缩进，加了几个泛型，会对之后其他人做merge造成非常大的影响

我们不是“主干－分支” 这种开发模式， 我们都是在主干上开发， 一般不修改同一个文件，避免冲突。

#20

我一个刚毕业怎么能说他工作6年的人呢， 开不了口呀。

受着吧。

#21

个人习惯, 有的程序员习惯性的把开发工具的警告功能关闭掉.
所以在他看来是没有警告的. 而你使用的开发工具没有去设置这个所以你看不惯.
一般都是老人喜欢这么做,好比从1.3开始过来的.

#22

呵呵  尽量扩展代码   别修改了   这就是所谓的开闭原则 呵呵。  

#23

难道是在说我