被同事气死了

他的代码为什么不是他自己调试？

润

@wd 因为他去别的项目组了，我留下来殿后

同一段代码到处复制，愣是不抽象出一个工具类，结果导致逻辑修改的时候到处找，我不止一次跟项目经理说，重构吧，项目经理说没时间，凑活用

项目最开始的原班人马大部队已经去进行别的项目的开发了，但是现有的项目需要有开发继续维护，我就是那个留下来的人，经过长时间的迭代，代码已经不满足现有的需求，甚至还成为了阻碍，急需一次重构升级，但是没有时间搞，就这么拧巴着，最终，今天晚上爆发了

算了，摆烂吧，下班回家自己玩自己的项目的时候上点心就行了

代码和你有一个能跑就行[doge]

我的同事临离职前写了好多 bug.

您确定是重构？不是重头开始？技术债，哪那么容易还，尤其是加需求的时候，让原来开发的人过来都想跑路，更何况你

@imzcg2 别扎了

他还在项目组？那不是继续留后患？
贴脸输出：代码截图，发邮件，抄送领导

@liangkang1436 #5 我司的项目历来如此。

去别的项目组继续祸害

没有人教吗？谁 merge 的？ CI 怎么过的 review 怎么过的？

有没有可能你们公司奖励这种你唾弃的代码呢？ 至少应该跟团队谈谈 看看大家是怎么想的

发帖是靠通灵嘛

@ericls 功能是没问题的，只是维护起来很麻烦，前期为了项目进度，领导也默许了

@liangkang1436 吸取经验教训吧 下次用这个为理由 让团队争取给自己更多时间去优化代码结构 理由就是为了以后更好维护

能跑吗？能跑就行

团队负责人不管吗？

格式化上 eslint ，代码问题上 code review 啊

为了项目进度 = 没有能力重构
都是借口而已，写个漂亮的业务代码能花多少时间？一直写垃圾代码就是没有那个能力而已，见多了

这根本不是问题啊,他代码垃圾你就多排时间呗.时间够就重写.跟领导要足够多的时间就行.
2 个小时的需求就要 2 天. 1 天的要 5 天 这还是事吗

@buffzty 我就是这么干的，但是心里和能力还是不够强大。每天有效工作时间 2 小时，看着那堆东西就难以下手。

楼主说的不是我的同事吗，哈哈哈

还 review 个毛线啊人都走了，有时间有能力的话，你就大刀阔斧的改吧

我现在状况跟你差不多。但是比你还要惨一点：
1.很多没有被调用的重复垃圾代码。（怀疑是凑行数）
2.明显暴露在线上的 undefined 错误。
关键是，因为原来的人全去做其他东西了，我是做移动端被拉进来顶前端的，现在除了多要时间也不知道该怎么办了。

变量用拼音，还不如用中文呢！哈哈哈哈

这有什么办法呢。等你跳多几家公司你就发现每家公司都会有这样的代码

摆烂吧

屎山才是常态 符合热力学第三定律

大家重构完测试人员不是还需要重新测试一遍吗，如果业务逻辑复杂的话跑不了自动化，这属不属于给人家增加工作量啊

@micean #22

“
为了项目进度 = 没有能力重构
都是借口而已，写个漂亮的业务代码能花多少时间？一直写垃圾代码就是没有那个能力而已，见多了
”

把我心里话用语言描述出来了，一直不知道怎么用语言表述~

3000 行算什么，我们公司好多项目一页 js 1 万行到 3 万行，全部不复用，相似模块从来不提出去，一个函数重复写 10 遍，就因为传值不同，命名大多是词不对意的英文或者不知名的简写，map 从来不 key index ，代码中充斥着 console.log()，控制台 100 多个警告和报错，只是代码能跑，索尼、荣耀、海尔、等大厂都在用我们这套系统，对了，代码还没做压缩...当然没人想搞那屎山一样的代码，变相的保护的源码...

不懂就问，为啥不能用拼音啊，虽然我没用，我甚至都想用汉字

又不是不能用…… 重构不如重写。

拼音又不是看不懂

摆烂，找领导摊牌：这是坨屎，宁看怎么办？你不管我也不管了。

政企有些事务意译英文也是无法理解的，还不如英译用拼音

@beisilu 如果你每天有效工作时间 2 小时,那就至少要 5 天排期.或者十天也行.让领导知难而退 还让之前的人改,或者让你重写. 我遇到屎山都是要么不改 要么重构

@stroh 想想我都头疼

@root01 拼音首字母缩写

不能润，还是建议自己精简下，因为估计以后还会有坑

你们组老大咋想的呢？
之前留下来的能跑的代码，我看了一会儿就觉得要重构。给老大看了，也觉的是要重构
然后我理清业务逻辑后，直接重写了，花了一个多星期吧

@IvanLi127 你没做 2g 的项目吧？各种专业名词真的顶不住。。

这种代码 CR 肯定过不了

我怀疑你再说我

@micean 说的对

接屎山都是这样的感觉 最有效的做法是找个人把屎山丢出去 实在丢不出去只能重构 降低心智负担

变量用拼音可以用英语不好解释，用首字母缩写就过分了。。。

一般这种情况，PM 也是屎，只顾当前自身利益不管产品长期发展，纵容时间紧凑就压缩质量，是在造成后患，反正出了问题也不是他 PM 的问题，把矛盾转嫁到执行层。

不会是我吧

非杠，其实我觉得这种情况，自己可能也需要反思一下，为什么我会碰到这样的同事

@feather12315 大实话

前两天刚碰上个字段名叫 childs 的。见怪不怪。

这有啥，你那是没见过在注释里撒谎的

- 变量命名用拼音，不用英语
因为不知道用啥英文。。。例如代币新代币，维护列表运维列表

- 不分离代码，一个 JS 写 3000 多行，一个类 2000 多行
- 到处复制粘贴
半天时间完成一个功能没时间优化，怎么简单怎么来

- 拒绝格式化，不改
下单子花时间优化代码

@Bongzyu #56 哈哈哈哈看来有故事

我的建议是赶紧润

润啊

@arthas2234 一般领导只关心怎么让项目继续平稳的运行下去
即使上一个人天坑 也只会安排你赶紧填上

快跑.jpg

哪里的公司，怎么招进去的

跟我司代码如出一辙

有些木匠懂得榫卯，有些熟悉钉子。家具都能立起来，只有木头，知道不同滋味。

这个主题配合头像简直完美哈哈哈哈哈哈哈

那你是没见过我之前的一个跑路的同事是用随机数来表示报表的其中一个百分比的数据，我看到代码的时候人都傻了

@ghost024 你永远不知道你的同事写代码的时候脑子里在想什么

@Bongzyu 笑死，居然还有撒谎的，细说。

@liangkang1436 他写了注释，说因为这个统计的数不准确,所以直接 0.8+0.2*随机数，保证数据好看，但是就是对不上:)

@ghost024 66666966

我同事也是
加功能就上千行的代码复制粘贴出一个新文件改一小点。
vue 组件里的样式不写 scope 也不写组件内的父级类名直接改 ui 库组件的样式。
处理冲突看都不看直接应用自己的。

我想打屎她

又不是不能用

这不是你同事的错！只能说贵司没有建立质量保证体系。