吐槽 5 年经验年薪 20W+前端同事写的屎山代码

鄙人前端一枚，几个月前入职了一家传统行业公司，技术栈都比较陈旧，前端开发还是 jquery 手写 dom 的开发方式，我进来之后写的新项目都是使用 vue 写的，但不维护以前的代码。现在有个同事要离职了，就把他以前写的老项目都交给我维护了，我打开一看被这代码秀的一愣一楞的，细细看了之后头皮发麻，手写 dom 拼接 html，大量行内样式夹杂其中：

这位同事年薪 20W+，5 年经验，之前写的都是这种代码，没用过 vue 之类的新技术，nodejs 和 npm 环境还是我帮他装的，我进来之后也教了他一些 vue 的东西，后面在他在新写的 vue 项目中也负责了一些功能，我后来 build 项目的时候报了一个警告，说是有个文件超过了 500Kb，我打开他提交的文件一看，乖乖，一个 vue 文件他洋洋洒洒的写了一万多行，体积达到了 600 多 Kb，细看之后，我又被他的骚代码秀到了：

写的什么玩意？这是对一个布尔值取反的操作，这老哥先判断值是真的还算假的，如果为真则改为假，如果为假则改为真。我寻思这不是 a=!a 一行代码就能搞定的事吗？后面又随便翻了下，类似的代码出现了好多次。

这同事的 5 年经验是实打实的 5 年经验，一年经验用 5 年的那种，他的技术你要说他烂我还有点想反驳，前面两张图那种代码你们猜他是用什么编辑器写的？你们绝对想不到，他是用 nodepad++写的，没有代码高亮，没有错误提示的 nodepadd++，用 nodepad++能写出这样的代码，我只能称之为神人。

后面又随便翻了下，又翻出来类似这样的，嗯，这其实是一行代码：

接手这样的代码是真的难受，今夜之后，这世界上又多了一个伤心的人。

fengmumu

2020-12-17 14:37:29 +08:00

@Hoshinokozo 前面同意，后面不是很同意，算是每个人不同的看法吧，比如一个长表单拆分成多个子表单，当时的业务是表单之间相互不关联，每个子表单自治，然后后面某几个之间有依赖关系，这时候改起来就很难受了，甚至要重写就算不重写也是各种传递和引用，还不如一直写在一起呢，
小声逼逼：不认为拆分成多个小文件是很好的事情，如果是做一个抽象的功能，把功能点拆分，然后引用比较合理，如果和业务挂钩，早早拆分，不但不利于后面人的维护也不是利于后面功能扩展（你只能给你预见到的功能留口子，但是你架不住产品提骚需求啊）。所以我总觉得和业务相关的少做优化，拆分的话尽可能的少，一切以后面人可以方便理解维护为主，代码的优雅处于第二位