23,405
社区成员
发帖
与我相关
我的任务
分享吐槽老员工代码
java web 开发。
1.在完全没有必要的地方，使用ThreadLocal传递变量。
2.很长很长的函数，动不动就上百行，甚至好几百行。
3.很少写注释。
4.一个函数兼任多个功能，用版本号来区分，返回的对象的数据结构是完全不一样的。
5.在一个请求中，动不动就启动一个新线程。
6.修改别人的代码，习惯性Ctrl + Shift + F，代码格式化。
7.try、if、for、while，深度达7、8层的嵌套。
这样的代码，我个人看来是“丑陋的”，“难以维护的”。
但就是这样的代码，竟然在工作了也快十年的员工写出来的。
1.在完全没有必要的地方，使用ThreadLocal传递变量。
2.很长很长的函数，动不动就上百行，甚至好几百行。
3.很少写注释。
4.一个函数兼任多个功能，用版本号来区分，返回的对象的数据结构是完全不一样的。
5.在一个请求中，动不动就启动一个新线程。
6.修改别人的代码，习惯性Ctrl + Shift + F，代码格式化。
7.try、if、for、while，深度达7、8层的嵌套。
这样的代码，我个人看来是“丑陋的”，“难以维护的”。
但就是这样的代码，竟然在工作了也快十年的员工写出来的。
...全文
请发表友善的回复…
发表回复
limin_ding 2013-09-13
- 打赏
- 举报
重构也要看项目的类型，产品项目可以在产品升级的时候，考虑重构。工程项目，要慎重考虑
forgetsam 2013-09-13
- 打赏
- 举报
java web 开发。
寒山石 2013-09-11
- 打赏
- 举报
每个称职的技术工作者都有追求完美的天性。
个人认为对于这种不够完美的代码，如果黑盒测试已经通过，在不影响工作进度的情况下做适当的调优。
清晰的代码结构代表着清晰的思路，坚持自己的编码风格，并持之以恒，终有一天会轻而易举的解决各种技术问题。
可乐罐 2013-09-11
- 打赏
- 举报
框架设计好，核心确定，其他的业务逻辑代码难看点，一般公司不会太叫这个真。如果真的忍无可忍，那就每天做下code review，及时纠正因习惯产生的代码问题
花木兰1闪21A 2013-09-10
- 打赏
- 举报
前一阶段看过一篇文章说的是说是关于看别人代码问题！写的很好，看了之前我只会吐槽和重构自己的代码！别人的东西你不知道的情况太多
剑神一笑 2013-09-10
- 打赏
- 举报
EXTJS 一个函数几百行很正常~
踩踩一哥 2013-09-10
- 打赏
- 举报
也许只是编程习惯，也许是让人觉得他很NB，楼主可以调教一下
姜小白- 2013-09-10
- 打赏
- 举报
个人编码习惯，同一个系统，有可能不同人做不同模块，完全不同风格的代码。可以考虑重构，提高代码质量
赵增光 2013-09-10
- 打赏
- 举报
请示经理，重构，不然系统会一致烂下去的。
不关橙猫猫事的哦 2013-09-08
- 打赏
- 举报
我现在也在干擦屁股的活。。
无聊找乐 2013-09-08
- 打赏
- 举报
构建系统时用checkstyle控制一下
aaaabbbccd9876 2013-09-07
- 打赏
- 举报
楼主傻了