吐槽老员工代码
java web 开发。

1.在完全没有必要的地方，使用ThreadLocal传递变量。
2.很长很长的函数，动不动就上百行，甚至好几百行。
3.很少写注释。
4.一个函数兼任多个功能，用版本号来区分，返回的对象的数据结构是完全不一样的。
5.在一个请求中，动不动就启动一个新线程。
6.修改别人的代码，习惯性Ctrl + Shift + F，代码格式化。
7.try、if、for、while，深度达7、8层的嵌套。

这样的代码，我个人看来是“丑陋的”，“难以维护的”。

但就是这样的代码，竟然在工作了也快十年的员工写出来的。


------解决方案--------------------
楼主傻了
------解决方案--------------------
构建系统时用checkstyle控制一下
------解决方案--------------------
我现在也在干擦屁股的活。。



------解决方案--------------------
也许只是编程习惯，也许是让人觉得他很NB，楼主可以调教一下
------解决方案--------------------
EXTJS 一个函数几百行很正常~
------解决方案--------------------
前一阶段看过一篇文章说的是说是关于看别人代码问题！写的很好，看了之前我只会吐槽和重构自己的代码！别人的东西你不知道的情况太多
------解决方案--------------------
请示经理，重构，不然系统会一致烂下去的。
------解决方案--------------------
个人编码习惯，同一个系统，有可能不同人做不同模块，完全不同风格的代码。可以考虑重构，提高代码质量
------解决方案--------------------
框架设计好，核心确定，其他的业务逻辑代码难看点，一般公司不会太叫这个真。如果真的忍无可忍，那就每天做下code review，及时纠正因习惯产生的代码问题
------解决方案--------------------
每个称职的技术工作者都有追求完美的天性。
个人认为对于这种不够完美的代码，如果黑盒测试已经通过，在不影响工作进度的情况下做适当的调优。
清晰的代码结构代表着清晰的思路，坚持自己的编码风格，并持之以恒，终有一天会轻而易举的解决各种技术问题。
------解决方案--------------------
重构也要看项目的类型，产品项目可以在产品升级的时候，考虑重构。工程项目，要慎重考虑
------解决方案--------------------
java web 开发。