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

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

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

有的 v 友可能误会我想表达的意思了，可能在有些 v 友眼里 20W 年薪不算什么，只能招到这样的人，我想表达的是他的技术达不到 5 年经验和这个薪资应有的水平，他实际薪资 25W+，月薪 2W 多，月薪 2W 只用过 jquery 而且写出这样的代码就有点离谱

leader 已经决定给我时间重构了，不过我不会看他的 shit 代码逻辑，看的脑壳疼，只能靠逆向重写一遍了，我的简历以后会添加这样一条经历，重构过一个 1W 多行的祖传屎山项目。
那些说风凉话的人，我祝你们也接手这样的代码，与这样的人做同事。引用一句 v 友的话：就这种人 v2 还有一堆人为他辩护，可见码农这个行业已经堕落到何种程度了

[Loserzhu]

吐槽代码烂谁都会，写好代码确很难。顺便，同情一下楼主。

[pecopeco]

跟我的遭遇差不多，同样一个文件一万多行，不过我还是咬咬牙给重构了，历时接近一个月

[dreasky]

以后接手你代码的人也是和你一样想法的

[weyou]

等 5 年以后楼主再看看今天自己写的代码，估计心情是一样一样的

[James369]

你的选择就是：做一只屎克螂，或者继续往上拉一砣。。

[18758036350]

吐槽不牛逼，接手重构真大佬

[SupperMary]

笑死

[zhuangzhuang1988]

能用就行.
能给公司赚钱 更好.

[BruceLe]

@James369
@18758036350 继续堆屎山是不可能的，已经和 leader 反应了，leader 表示会给我时间重构

[DuXing]

FE：面试的时候也没问这些，问了"http 三次握手”、“一个浏览器从输入地址到打开页面都经历了什么”，我回答的都挺好。

[teawithlife]

贵司肯定是按照代码行数定年薪的～～

不过话说最后一张图，vue + element-ui 不是只能这么写么？难道有其他写法？

[zhengfan2016]

哈哈哈哈心疼 lz😂

[joshua7v]

没有代码高亮 没有错误提示
一样可以写代码 而且我见过的就挺多这样的
当然他们多数都是很早期的程序员了

工作的代码很难不变成屎山

[malloc2free]

有次去参加 hackathon，组到个用类 word 的文本编辑器写前端 h5 的大哥，白底黑字
同样没用框架，同样没有代码高亮错误提示

一言难尽，最难过的 24 小时

[BruceLe]

@teawithlife 那张图我是吐槽的是代码的可阅读性，连个换行没有，不同的配置项像下图这样换一下行，可阅读性不是提高很多？

[Kasumi20]

不知道楼上嚼什么, 我思量但凡上过 6 年级, 也不会连取反都不会写

[scys]

培训班的风格

[darknoll]

@teawithlife 说明 eslint 都不会用啊

[dji38838c]

this.showDescribition


Describition 是什么单词

[Bijiabo]

根据我的经验，这种人去阿里比较适合。

[BruceLe]

@dji38838c 大佬你又发现了新的吐槽点，我寻思应该是 description

[CEBBCAT]

@Livid 要不换个节点？比如职场话题？或者生活？

[weifan]

前人栽树，后人就凉了...

[VDimos]

是不是 kpi 用代码行数考核的？

[aaronlam]

。。。这真的很随意

[tomychen]

这大佬是 C 语言转过来的吧

[Akat3uki]

就很离谱

[Austin2035]

jQuery 就一定不好了么？ vue 就很先进了？

[ryougifujino]

边看边笑

[hoyixi]

非科班出身，直接从事前端开发的， 很多都没用最基本的编程素养的。 甚至有些连逻辑思维都够呛。
前两年前端泡沫，不少人冲着薪资就入行了。一个前端，一个 iOS， 一个 Python， 都是比较好”混“的~

[love]

这种就是没有追求的码农，写多少年代码就是一个水平，对编程没兴趣的人干这行就是这样，这种人到处都是

[BruceLe]

@lookcos 我有说过 jquey 不好？没看到他写的 vue 代码一样 shit 吗？

[billjobs]

5 年 20W+本来就说明水平不行啊，写成这样很奇怪吗

[ianva]

想看楼主自己的代码，就是好奇

[user8341]

是不是没有 code review ？

[gbin]

这代码写得确实辣鸡，楼上没必要辩解。

[BruceLe]

@ianva vue+eslint 风格的代码，没什么好看的，你真想看的话我也可以晒

[elfive]

所以说，技术最不值钱。
技术钻研再透彻，写出的代码最牛逼，也只是个为别人（老板、甲方）写代码的。

[ianva]

@BruceLe 晒一下嘛，Code Review 都会有成长

[fuis]

毕竟这个钱也就招这样的人

[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 能写的这么整齐也是不容易了啊。。。

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

[ligiggy]

吐槽完了，心情好点了吗？有这种人的存在，也是再提醒你，技术一天天在更新，写代码就要不断的学习。

[xrr2016]

哈哈哈，这也太坑了吧

[zhangdawei]

年薪不管，吐槽代码就好了，

[HannibaI]

@anxiousPumpkin 吃惊+1，全部 B 了，一群 SB

[KouShuiYu]

代码写的烂还要被接收的人鞭尸😂

[abelmakihara]

这确实写的很烂啊 反正我是不能接受
像这种又没什么特别的逻辑
还滥用$forceUpdate 也很奇葩了吧
除了 angularjs 的时候被迫经常使用强制刷新
vue/react/angular2 以后没用过

[method]

他是烂。六字真经。

[method]

@billjobs 你得到了它。20W+ 在校招生里也是偏低的。

[xiaomu8]

你理解的工资定义跟你老板理解的工资定义是不一样的，毕竟发工资的是老板
所以也没啥配不配的事，人家实现了老板的价值。

[sunwang]

维护这种代码相当难受，同情楼主

[jsjgjbzhang]

跟我一个同事差不多，复杂业务做不了，简单业务代码写的既漂亮又精简，还时不时拿出来闻一下

[clxtmdb]

虽然不是前端，但是还要同情一波（不过先等我笑完）

[2379920898]

前有 996，后有狮山代码。。你说是公司的错，还是程序员职业操守的错呢

[JamesR]

是不是干外包的？外包的关键是干活要快。

[fend]

我笑了 希望各位 hxd 永远见不到这种辣鸡代码

[YoRolling]

在深圳么，在深圳让我来， 给我同样的薪水我可以写出不那么屎的代码，至少咱也 "精通" vuejs 常规操作。 手动狗头

[tianhei826]

害，当你花很多时间看懂前任写的一堆面向过程的代码后，你也不想把业务逻辑抽出来。就干脆让屎更屎一些。
见过 5W 多行的 SP 吗其中还调用的 package 看到你崩溃

[Stlin]

看着真糟心，不知道楼主要不要重构，同情一波

[leeguo]

又不是不能用

[litujin1123]

@henvm 楼主就是觉得别人比他菜，然后工资高他很多而不爽，毕竟第一份工作 2k

[fengmumu]

插眼，想看一下这个改进后的代码

[supuwoerc]

懂了 我立马去让领导给我涨到年薪 20W

[whitehack]

竟然敢重构...只能说胆子真肥

[crackhopper]

这种代码写法，猜测你们公司是不是按照代码行数论绩效？哈哈。

[cidde]

年薪与代码技术无关，成长和陪伴才是年薪的决定性因素~

[bk201]

又不是不能用
打工人何苦难为打工人

[lovecy]

我感觉我写的前端代码和这位兄弟差不多。。。。。。虽然我是小后端吧。。

[ciddechan]

估计后面还有兼容性的问题

[zjsxwc]

楼主被坑的好惨，什么仇什么怨，居然要维护这种代码。


🤓

[Jiangyf]

都是打工人，没必要；
在成本和时间的限制下，大环境就是这样，能用就行；
LZ 可以选择技术氛围好的环境，不过除了头部核心部门，基本上都在写业务代码。但最起码环境到位，这样一来对自己编程思想，习惯、风格都有好的影响吧；

[no1xsyzy]

@teawithlife 按行数定年薪还不每个配置项换一行（笑哭

[mxT52CRuqR6o5]

你看他一个取反写的这么复杂，其实是因为他不会取反，不知道有这种运算
5 年学习了用最少的技术特性也能实现各种各样不同的简单的复杂的需求的丰富经验

[red2dog]

过分了啊

[PomeloChuan]

好代码写起来很难，烂代码写起来也不简单。字符串拼接的 html 可以使用 pug 这样的模板引擎，然后重复的展示隐藏之类的可以抽象化一下

[sybb]

嘿 类似经历！！！！趁机吐槽一下！！！！！！

我的新领导，前端负责人。刚从鹅厂出来的 30 多岁老程序员了，年薪肯定不用说， 但是写的代码真的是得说一说了！！

以前没写过 vue react ts 这类不能说是新东西的框架和规范， 罗列以下几点：

- 用的 sublime，不装 eslint tslint prettier 等代码规范工具 每次我们拉代码下来 都是一堆黄色警告，之前在我们的委婉要求下装了 eslint， 说是项目里全是报错，又给卸了。。。
- 不会 ts，一个 any 一把梭！！！
- 代码里要设置 key 作为标识总是用中文作为 key 值，比如 user.type = “管理员” 。
- 想要把某块功能的右键菜单封装一下，结果一个方法写了 1000+行(配置的数据类型全是 any，配置的数据格式还不一样。。)
- git 只会用 sourcetree， 曾多次在主仓库覆盖我们的代码。
- git commit message 不规范 ，经常写"no message",甚至还写过”what the fuck“。。
- git 打 tag 用的是中文。

[wwk]

其实首先得确定这是他写的。一个项目长期迭代，维护者可能更换过很多任。另外就老代码而言，能跑是最重要的。重构需要的成本很高，除了本身开发的成本，时间成本（延误正常迭代），需要测试支持（全方面的回归测试），业务成本（大规模重构可能影响正常业务）。


接手过单文件 1w 行的组件...表示根本不敢大刀阔斧的重构，只夹在敢趁着正常版本迭代，有测试支持的情况下，进行部分重构，除了自己得反复验证，还得求着测试大哥顺便多测测某个功能点。到离职都没重构完。。。交接的估计也得吐槽我，哈哈～

[whypool]

意思就是这样的辣鸡代码也配 20 万？

[sosilver]

这可能就是大佬吧，糊一万行也可以 hold 住

[agdhole]

建议换 angular，用框架来强行规范