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

[BruceLe]

鄙人前端一枚，几个月前入职了一家传统行业公司，技术栈都比较陈旧，前端开发还是 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++能写出这样的代码，我只能称之为神人。

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

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

[Tink]

又不是不能用

[BruceLe]

@billjobs 我想表达的是他的工作经验和技术水平达不到这个薪资的水平，他实际薪资 25W+，月薪 2W 多了，月薪 2W 只会 jquery 就有点离谱

[BruceLe]

@ianva 就是这样的喽，不过不要吐槽一些参数的命名，那些奇奇怪怪风格不一的参数名称都是后端定义的

https://i.loli.net/2020/12/16/H6ulbWwQKm7jC1Z.png

[ianva]

上个开源项目看看吧，这代码未来挺危险的，一个文件超过 300 行时，组件和数据的交互层，业务逻辑层都在一起，看不到数据的模型，业务数据和交互状态交织在一起

[anguiao]

这个 if-else 可还行，严重怀疑是故意的，正常人写不出这样的东西。

[duola]

反正公司不敢炒了他。

[BruceLe]

@ianva 没啥危险不危险的，这就是一个用户列表页面，增删查改那些功能，几百行代码就搞定，后面不会有什么大的改动了

[ianva]

楼主还是虚心一点，这个重构也许是个轮回

[cmostuor]

不得不说这么多的 if else 看得我要崩溃 github 在垃圾的代码都比这要好看且性能好

[caaaalabash]

建议别重构。。。

[mxT52CRuqR6o5]

你让 1 年同样技术水平的新手绝对做不完这个需求的，1w 行的单文件代码 1 年经验的绝对 hold 不住的
5 年是实打实的 5 年，只不过是用 0 编程思想歪技术路线不适合的开发工具也能顺利的把需求实现的实打实 5 年经验

[mxT52CRuqR6o5]

就好比一颗长了 5 年的树，你是竖着长的，他是横着长的

[henvm]

我觉得楼主可以 吐槽别人 吐槽 5 年经验写的屎山代码，但是不能说人家不配这个年薪 20W+，难道楼主不该反思自己要价太低了吗？人家的工资是自己谈的自己慢慢涨的。

[henvm]

我觉得吐槽别人多年经验没学到东西或者没有长进可以，但是不能去吐槽人家拿的工资多少问题。如果自己觉得别人不如我而工资又高过我不爽，这个及时你拿到公司台面上说，公司降了他工资也不会涨你工资，即时涨了这种直接损人利己的利益还是算了。应该反思自己这么多年来是不是工资要低了。

[henvm]

@billjobs 你说到了重点了，可能人家刚进去工资也不高，20W+是慢慢涨上去的，而且这 20W+ 其实不多，很多一线 IT 公司做了 5 年远远高于这个数目的不要太多。

[20015jjw]

看吐

[ByZHkc3]

过份了

[q447643445]

别人看你的代码也是这种感觉.

[anxiousPumpkin]

代码能写成这样，说明本身就敷衍了事不负责任。就这种人楼上还有人在为他辩护，yygq 楼主，真是惊了。

[akira]

用 notepad 能写的这么整齐也是不容易了啊。。。

能实现业务的代码才是好代码