吐槽 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++能写出这样的代码，我只能称之为神人。

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

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

[freeminder]

@BruceLe 368 行 res 是 null 看来会报错呀；如果 res 不可能是空，364 给的暗示又不对？

[l00t]

@lights #171 没技术那一行都写不出来。能写出可以实现业务的代码了，那是技术够用的状态了。

[key1088]

感觉在讨论砌墙的活

[wqgogogo]

惊得我去看了下有没有写这种的代码，不过我后端一枚，有时也要写 JS

[howoldareyou2]

怎么了，他用汇编语言写，我更佩服他-->

[xiaomu8]

@aloyuu 我去，这是正主现身了？

[Francost]

"月薪 2W 只用过 jquery 而且写出这样的代码就有点离谱",一句话里充满了酸和鄙视，只用过 jquery，并且写了一手屎代码，居然都值 2w 。。。

[Valii]

为前同事辩护的怕不是自己就是这个水平

[Kirsk]

风气就是这样 你把你重写的贴上来 搬砖工就没话说

[xiaoxinshiwo]

这就是代沟，远古程序猿都这么写

[nanxiaobei]

怎么岗，我还以为是 5 年 200w+ 呢。。。

[melvin]

老板说： 代码啥的不重要 重要的功能好用就行了 [doge]

[rbq123456]

@zhang77555 程序员的薪资不和代码挂钩，应该和什么挂钩呢？业务，那不是产品的事情么？

[ily433664]

这算啥，我还见过 jsp 里面 java 和 html 代码混合编写的

[Reapper]

要不用 jquery 重构一遍？

[leeho]

厉害， 不用 code review 的么？

[mark4zhao]

Describition 。。。

[yuyu2140]

哈哈哈，我司祖传烂代码（ 10 多年前的代码，缝缝补补），时至今日每年还能贡献几百甚至上千万的收入，你信吗？
插一句，码农也是个普通职业，辩护也不是堕落，烂代码谁都接手过，每个人都有写过烂代码的时期，希望能平常心看待这样的问题，经历的多了，就见怪不怪了

[cirzear]

tql

[chungon]

这种屎一样的代码为什么会有这么多人洗？