代码优化的问题,不能瞎改。

前段时间招了个刚毕业的,面试的时候感觉什么都会,什么spring,事务,多线程并发,数据结构,算法,设计模式,前端闭包原型链,,,,三轮面试官结论一致:牛b! 然后就来了,安排做权限,说的头头是道,做的也挺快,大家感慨果然少年英才不同凡响,过了仨月因为某些原因离职了,这期间感觉表现良好,虽然其有时候问的问题比较低级,比如eclipse导入项目一直报错不会修复等,我对他的唯一直觉是代码可能有问题,因为一个公共用的js文件从原来的300行左右一下激增到了800行,而且修改了一些原来的代码,比如ajax返回的code原来就0跟1,后来他加上了-1,-2跟2,几个状态啥意思也没说,也没通知任何人,我找他说过此问题,该同事解释g#v%/@#=+设计等等就应该这么做,反正听着高大上我没通读代码也不好反驳,就这么着吧。接下来招人接管代码继续开搞,出问题了,新人妹子表示看不懂代码,于是乎老衲出手帮忙整理代码逻辑,这一看简直气炸了。我们代码都是传统的controller,service,dao三层按模块在不同包下,这货自己建了个文件夹,也没分层所有的东西自己另起了一套都在一个文件夹下,里边各种封装继承共22个类,花三四天才整理完真正有用的8个类,其中3个还是自己写的异常类(其实项目框架有异常处理,这里他自己来了一套),代码方法内注释都是段落注释,类内变量声明也没放到类的前边而是到处散落导致一个变量多次声明,三个不同名字方法实际功能完全一致,controller跟接口方法没有注释,类的命名各种繁琐,大概是7.8个单词并排吧,方法明也反向操作各种简化,就是三五个字母,都不知是啥缩写,,,总之代码继承关系复杂,类内代码也比较混乱,好在总算弄了个差不多,没在妹子面前丢脸。然后前端,当时我希望能有一把刀,打车去砍死他。我怀疑这货不是写代码而是花式秀技术,一个简单的保存用户他用for in遍历保存,一个用户保存了一次,这要有10万用户,,,而且还自己改写了个ajax方法调用通用ajax方法,这里边自己实现了一个类似于extend的功能,这就完了?no,no,没有秀你一脸怎能结束呢,这货又来了个deferred来处理ajax回调,还来了个Map类来各种倒腾数据,至于前边说的公共js文件代码激增,是因为他把自己认为可重用的代码全扔了过去,对了,他做的功能,整体风格还跟项目不太一致,,,,这么折腾下来,代码普通人根本看不懂,关键是效率也很低,,,本来打算重构他前端代码的,看了下只整理了一部分放弃了,因为太乱了,估计还不如重新开发,前端用了也差不多一周时间,最后实在心累不想搞了,没bug就先凑合着吧,等明年前端重构。 说白了,这事儿就是没定期代码审核的锅,要早审核代码何至于后边这么折腾。送那位老兄一句话,认真点,我相信以你的能力三年后能成大神,但在你的成神之路上,请尽量少祸祸你的那些兄弟,保重!




多频传媒

猜你喜欢

转载自blog.csdn.net/TENCENTSYS/article/details/89013243