同事的离谱代码让我想发疯

[0x114514]

楼主是前端，每次看到组里其他人写的代码我都痛苦的要命。随便举几个例子吧：

• 最基本的代码对齐都做不到，例如它们的代码是这样的：

if (isDefault) {
    this.xxx = true;
  this.timer = setInterval(() => {
      if (this.zzz) {
        if (this.ddd) {
          this.box = 24;
            }
    }
  })
}

每次看到这些代码我都快脑溢血发作，现在随便哪个编辑器都能做到自动对齐大括号，我真的怀疑它们在用记事本写代码。

• 喜欢全局重置某个组件库样式，从来不管会不会影响到其他组件样式。例如：

.el-table__fixed-right {
  display: none;
}

每次都是等到测试发现了，然后再回过头来改这个问题。

• 用 AI 漫天拉屎

自从公司的人会用 Trae 之后，每次看到它们提交的 commit 里有 markdown 写的很详细的提交信息之后，我就知道它们又拉了一坨大的。

业务确实实现了，问题也解决了，但是代码质量真的令人发指，比如说，从某个模块中莫名奇妙引入一个根本不存在的虚空方法，然后莫名奇妙的在代码中调用，然后莫名其妙的逻辑根本不会走这里，业务还莫名奇妙实现了。

我怀疑它们真的是从来不 review AI 生成的代码，只要能跑通就行。代价是后续负责修 BUG 的人要一点一点抠这个是什么意思，连 CC 有时候都理不清它们写的是什么。

别问我为什么不和 Leader 沟通，因为它就是带头拉屎的人。

F@@@@@@KKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKK

[noahjsn]

用魔法打败魔法。在屎山上再拉一坨

[strobber16]

你也用 ai 就行了。公司不管你就只能加入

[hrzlvn]

继续拉，继续拉，很多人已经说 ai 写代码就行了，人不用管，哈哈

[lovelyxiaod]

你可以给他一个 agent.md, 至于格式化的问题你就顺手 ctrl + shift + f 吧

[itechify]

ai 写的有时候不对齐

[comeAcross]

打不过就加入[dog]

[Gilfoyle26]

大家一起烂不就行了

[YanSeven]

这种最好了，加入进去一起浑水一起摸，然后自己偷偷学习跳槽。有责任心的人用 AI 会比以前累，没有责任心的就随地拉屎得了。

[DICK23]

husky 用起来啊，检查不过不然提交

[dddd1919]

都选了 AI 生成，还在乎格式干啥？不要去 review AI 的代码

[billbur]

打不过就加入 or 跑路

[crysislinux]

现在我看到像是 AI 生成的 PR 描述我都不想看，找我 review 反手就是一个 approve 。。。

[ashuai]

我教你，点右键，格式化代码
或者你也可以选择，全选 ctrl + a, shift + tab, shift + tab, shift + tab, shift + tab ，commit, push

[xing7673]

公司不出钱用 cursor 或者 cc 么？

[malusama]

添加代码库的 pre-commit ， 上线 GitHub action 检测 lint 不符合都不能直接合并

[importmeta]

有工资就行，能跑就行，可能这些人就是这么想的。

[iamnotcodinggod]

‘它们’很细节

[Saniter]

提交前不会过一下 format 吗

[avenger]

用 lint 啊，不通过不让提交

[106npo]

ai 写的代码有啥好看的，看下文档然后 ai 跑一遍就行了

[Ketteiron]

@avenger #19 上司都带头拉屎了，说这些有啥用。缺的不是解决办法，因为所有解决办法都没用，除非能开掉造成这些现象的开发者，因为不开掉它们绝对无法维护好 CI 。

[imnpc]

AI 特别喜欢瞎编不存在的方法

[MajestySolor]

你只是个打工仔，为啥要考虑这些东西，只要问题不是出在你手上，别人爱怎么搞就怎么搞
自己统筹负责的项目可以精益求精，多人协同的项目就别吹毛求疵了，纯粹是自己给自己找麻烦🤣

[luckyliu1926]

啥 ai 也不会这么弱智啊，还是人的问题

[300]

两家公司几十个同事，连我只有两个人会格式化代码后上传 ╮(╯▽╰)╭

这家公司也很 nb 。相同的业务代码能写十几份几乎一样的（ nb 的点就在于他们每次写的思路一样，流程大差不差，变量名每次都有点不同）

明明有代码规则自己不遵守，当我我复制粘贴他们的代码，他们还会提意见😁说明他们还是能看懂的

能怎样呢，打不过就加入。公司现在的风气已经变了，以前是有多少事做多少。现在是你把事情全做完了（或者阶段做完了）说明你没发挥价值

[kjstart]

你们没 code review 吗? 没 eslint 检查吗? 没 ut 覆盖率检查吗?
我们有上述问题根本点不了 merge, ut 不到 80%也不能 merge.
每个 repo 都有一组 default reviewer, 至少要拿到两个 approve 才能 merge.

而且我们前端的开发工具改成类似 scratch 那种拖拽的了, 写 code 的机会都很少了.

[300]

@300 格式化很简单，IDEA 这种直接在 commit 设置里选一下就是都不去做，我都提醒了他们也没用

@malusama 我们有 lint ，sb 的是老板自己会 push/merge lint 失败的代码
然后攒一堆找一个人（经常是我）说你跑一下 format.sh 修复一下

反正格式化一下也算我在工作了，只不过其中会伴随一些操蛋的事情罢了

[bruce0]

代码格式化,我写 go 的,以前组内只有我一个人格式化后提交,一度让我很崩溃,代码同步下来,我改动一行,格式化后,提示几十上百行修改.
后来我写了一个 svn hook 的脚本,所有提交的.go 文件,必须能通过 fmt 检查才能提交,否则禁止提交.这个问题算是解决了

[NGGTI]

让他们在 push 前，review 下或者，让 AI 自己 review 下就能解决大部分了。

[yhxx]

@ashuai 你格式化了下次这里出问题 git blame 可就是你的了

[BenHunDun]

第 1 个, 格式化问题, 如果有人牵头弄一下, 其实提交的时候可以限制, 还算比较容易达成. 项目提供一些自动化命令以及规定统一代码规范.
第 2 个, 更觉更多就是要 code review 了.

[0x114514]

@kjstart 有，没人遵守，写的代码规范更是没人看

[0x114514]

@MajestySolor 我在努力让自己接收这一切，给钱就好，随便它们整吧。

[0x114514]

@bruce0 我记得之前用 VSCode 写 go ，保存时会强制自动格式化，其他 IDE 肯定也有这个功能吧， 好奇你的同事们怎么做到的

[0x114514]

@YanSeven 你说的很对，浑水摸鱼，自己提升自己好了。

[darkengine]

Leader 自己都拉，你不拉倒显得你不解风情了 /doge

[moons]

如果其他人代码写的太好，加上 ai 的辅助，效率就会很高
你又不是领导，效率太高，普通员工就容易失业
所以，应该感谢其他人写的代码一般般，并祈祷多一些容易排查的 bug 。

[treblex]

我喜欢自动 format ，但是容易把别人的垃圾代码修改记录变成我的，就不敢用了，只能偶尔局部 format

[Lush]

配置一下 commit 的时候执行一下 prettier 项目格式就统一了

[MYDB]

不要技术强迫症，接受任何风格的代码，只针对功能和 bug 交流

[RedNax]

我们 prettieer 配置直接写进 eslint ，eslint 又有 lint-staged 和 husky 把关，CI 上还有 lint 步骤，不管哪种情形都不会让 format 不对的代码 merge 进去的。

另外 vscode 配置一个 format on save ，用 eslint 来 format ，谁都省事。

[anonymous00]

直接让我想起曾经老生常谈的技巧：把个人问题上升为集体问题。
加入/融入，“锦上添花”，耐心等待。

不过，如果你的 team 整体是这样，说明你…不属于那里，看你的耐受度或其他机遇吧，你的 leader 无心尽责，可能也是在混日子或骑驴找马（尽管当前在国内不容易），有机会私下聊的话，顺带吐槽两句，试探一下。

[wnpllrzodiac]

@YanSeven 跳槽也许去的也是满地拉屎的。现在编程界可能已经大部分都是拉屎了。工匠精神的开发已经被劣币驱逐良币了。开发开发越来越像做销售了。ppt 好看就行了，功能有问题糊弄获取就行。反正又不是自己用。

[cslive]

git hook 用起来

[94]

@lovelyxiaod #4 ，然后一看 Blame ，一堆修改的 committer 就变成你了

---

如果 OP 你是项目前端的负责人，那么就写几个 husky 脚本，在每次提交和推送的时候做一些简单的格式化自动执行和 lint&ts 校验，校验不通过直接让别人提交和推送就行了。
如果你不是负责人大家都是同级的大头兵，就很简单了直接不看别人的代码。你也没办法要求别人按照你的意愿去做，眼不见心不烦。

遇到有 BUG 谁是 Owner 谁去改，如果你接盘了心情好有时间就重写，不开心了就💩上雕花。

[bruce0]

@0x114514 #34 golang,需要配置一下,onAction 还是啥,后来我专门写了一个文档,让他们用起来

[systemGuest]

1 、你还未认识到不同人的思维习惯基本无法统一
2 、你认为你是标准
3 、世人都得遵守你的标准

[xcqnndc]

格式化统一一下就行了

[leegradyllljjjj]

都在撤硕里就别嫌弃别人拉的屎臭了

[Kiriri]

@lovelyxiaod 那可不行，顺手格式化了之后，人家第一眼看到的修改记录就是格式化的那个人了，以后出问题了还会被找。

[beyondstars]

好日子还在后头呢，以后就不是碳基同事的代码让你发疯了，是硅基同事的代码让你发疯，而且以 10x 、20x 于现在的速度生成。

[sodesga]

多大点事，git hook

[dengji85]

中国程序员代码写得再好都没有价值，程序员的上级很少有继续走技术路线的，那些管理者就是靠关系 or 机遇转了管理，其技术水平可能就是个初级，有些技术还可以的，转管理后也不再搞技术了，所以代码写的在好，其实没人会认识到的，花大量去封装和优化，累死自己，还不如去贡献开源项目

[irisdev]

这代码看着想笑

[hi2hi]

eslint 能够解决很多问题；；；算了，你也上 AI ，让 AI 对抗

[zhengfan2016]

@94 没啥用的，我们有个 monorepo 的大仓，有 pre-commit 的 lint 检测，招了个新人，它提交的代码其他人同步之后全部没法正常 commit ，报他代码各种 lint 错误，因为聪明的他早就把.git/hooks/pre-commit 删掉啦，其他人想提交还得帮他修 lint 问题

[zhengfan2016]

@Kiriri 如果你有实权，git hook 加个 pre-commit 格式化吧，可以统一格式化，大头兵的话就憋着不按格式化代码按钮最小化改动改其他人的代码

[94]

@zhengfan2016 #56 ，这种删除的操作是需要直接警告的，为什么会被允许？而且还能够被合并到主线影响到其他人。

[yanxu4780]

这不是写代码的日常吗？ hhhh

[echoZero]

结果导向 ，同事 ai 一把梭，出了问题在解决就是了，并且也不一定是他处理。
最终你同事月度/季度 KPI 还会加一条 用 AI 快速解决需求问题。 领导直接评优秀

[tangguo]

打不过就加入呗，他用 AI 拉屎后续修 BUG 继续拉

[Lowlife]

我一直觉得，能用就行，你就说能不能用吧？用户又不需要看代码，😓