楼主是前端，每次看到组里其他人写的代码我都痛苦的要命。随便举几个例子吧：
- 最基本的代码对齐都做不到，例如它们的代码是这样的：
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
 |
1
noahjsn 12 小时 2 分钟前
用魔法打败魔法。在屎山上再拉一坨
|
 |
2
strobber16 12 小时 0 分钟前 via Android
你也用 ai 就行了。公司不管你就只能加入
|
 |
3
hrzlvn 11 小时 58 分钟前
继续拉，继续拉，很多人已经说 ai 写代码就行了，人不用管，哈哈
|
 |
4
lovelyxiaod 11 小时 57 分钟前
你可以给他一个 agent.md, 至于格式化的问题你就顺手 ctrl + shift + f 吧
|
 |
5
itechify PRO ai 写的有时候不对齐
|
 |
6
comeAcross 11 小时 34 分钟前 via Android
打不过就加入[dog]
|
 |
7
Gilfoyle26 11 小时 31 分钟前
大家一起烂不就行了
 |
 |
8
YanSeven 11 小时 23 分钟前
这种最好了，加入进去一起浑水一起摸，然后自己偷偷学习跳槽。有责任心的人用 AI 会比以前累，没有责任心的就随地拉屎得了。
|
 |
9
DICK23 11 小时 23 分钟前
husky 用起来啊，检查不过不然提交
|
 |
10
dddd1919 11 小时 13 分钟前
都选了 AI 生成，还在乎格式干啥？不要去 review AI 的代码
|
 |
11
billbur 11 小时 11 分钟前
打不过就加入 or 跑路
|
 |
12
crysislinux 11 小时 7 分钟前
现在我看到像是 AI 生成的 PR 描述我都不想看，找我 review 反手就是一个 approve 。。。
|
 |
13
ashuai 10 小时 59 分钟前
我教你，点右键，格式化代码
或者你也可以选择，全选 ctrl + a, shift + tab, shift + tab, shift + tab, shift + tab ，commit, push |
 |
14
xing7673 10 小时 56 分钟前
公司不出钱用 cursor 或者 cc 么？
|
 |
15
malusama 10 小时 52 分钟前
添加代码库的 pre-commit ， 上线 GitHub action 检测 lint 不符合都不能直接合并
|
 |
16
importmeta 10 小时 40 分钟前
有工资就行，能跑就行，可能这些人就是这么想的。
|
 |
17
iamnotcodinggod 10 小时 38 分钟前
‘它们’很细节
|
 |
18
Saniter 10 小时 31 分钟前 via Android
提交前不会过一下 format 吗
|
 |
19
avenger 9 小时 37 分钟前
用 lint 啊，不通过不让提交
|
 |
20
106npo 9 小时 26 分钟前 via Android
ai 写的代码有啥好看的，看下文档然后 ai 跑一遍就行了
|
 |
21
Ketteiron 9 小时 25 分钟前
@avenger #19 上司都带头拉屎了，说这些有啥用。缺的不是解决办法，因为所有解决办法都没用，除非能开掉造成这些现象的开发者，因为不开掉它们绝对无法维护好 CI 。
|
 |
22
imnpc 9 小时 21 分钟前
AI 特别喜欢瞎编不存在的方法
|
 |
23
MajestySolor 9 小时 14 分钟前
你只是个打工仔，为啥要考虑这些东西，只要问题不是出在你手上，别人爱怎么搞就怎么搞
自己统筹负责的项目可以精益求精，多人协同的项目就别吹毛求疵了，纯粹是自己给自己找麻烦🤣 |
 |
24
luckyliu1926 8 小时 50 分钟前
啥 ai 也不会这么弱智啊，还是人的问题
|
 |
25
300 8 小时 40 分钟前
两家公司几十个同事，连我只有两个人会格式化代码后上传 ╮(╯▽╰)╭
这家公司也很 nb 。相同的业务代码能写十几份几乎一样的（ nb 的点就在于他们每次写的思路一样，流程大差不差，变量名每次都有点不同） 明明有代码规则自己不遵守，当我我复制粘贴他们的代码，他们还会提意见😁说明他们还是能看懂的 能怎样呢，打不过就加入。公司现在的风气已经变了，以前是有多少事做多少。现在是你把事情全做完了（或者阶段做完了）说明你没发挥价值 |
 |
26
kjstart 8 小时 39 分钟前
你们没 code review 吗? 没 eslint 检查吗? 没 ut 覆盖率检查吗?
我们有上述问题根本点不了 merge, ut 不到 80%也不能 merge. 每个 repo 都有一组 default reviewer, 至少要拿到两个 approve 才能 merge. 而且我们前端的开发工具改成类似 scratch 那种拖拽的了, 写 code 的机会都很少了. |
|
27
300 8 小时 34 分钟前
|
 |
28
bruce0 8 小时 22 分钟前
代码格式化,我写 go 的,以前组内只有我一个人格式化后提交,一度让我很崩溃,代码同步下来,我改动一行,格式化后,提示几十上百行修改.
后来我写了一个 svn hook 的脚本,所有提交的.go 文件,必须能通过 fmt 检查才能提交,否则禁止提交.这个问题算是解决了 |
 |
29
NGGTI PRO 让他们在 push 前，review 下或者，让 AI 自己 review 下就能解决大部分了。
|
 |
31
BenHunDun 7 小时 52 分钟前
第 1 个, 格式化问题, 如果有人牵头弄一下, 其实提交的时候可以限制, 还算比较容易达成. 项目提供一些自动化命令以及规定统一代码规范.
第 2 个, 更觉更多就是要 code review 了. |
 |
33
0x114514 OP @MajestySolor 我在努力让自己接收这一切，给钱就好，随便它们整吧。
|
 |
36
darkengine 6 小时 33 分钟前
Leader 自己都拉，你不拉倒显得你不解风情了 /doge
|
 |
37
moons 6 小时 9 分钟前
如果其他人代码写的太好，加上 ai 的辅助，效率就会很高
你又不是领导，效率太高，普通员工就容易失业 所以，应该感谢其他人写的代码一般般，并祈祷多一些容易排查的 bug 。 |
 |
38
treblex 5 小时 47 分钟前
我喜欢自动 format ，但是容易把别人的垃圾代码修改记录变成我的，就不敢用了，只能偶尔局部 format
|
 |
39
Lush 5 小时 16 分钟前 via Android
配置一下 commit 的时候执行一下 prettier 项目格式就统一了
|
 |
40
MYDB 4 小时 47 分钟前
不要技术强迫症，接受任何风格的代码，只针对功能和 bug 交流
|
 |
41
RedNax 3 小时 59 分钟前
我们 prettieer 配置直接写进 eslint ，eslint 又有 lint-staged 和 husky 把关，CI 上还有 lint 步骤，不管哪种情形都不会让 format 不对的代码 merge 进去的。
另外 vscode 配置一个 format on save ，用 eslint 来 format ，谁都省事。 |
 |
42
anonymous00 2 小时 5 分钟前
直接让我想起曾经老生常谈的技巧：把个人问题上升为集体问题。
加入/融入，“锦上添花”，耐心等待。 不过，如果你的 team 整体是这样，说明你…不属于那里，看你的耐受度或其他机遇吧，你的 leader 无心尽责，可能也是在混日子或骑驴找马（尽管当前在国内不容易），有机会私下聊的话，顺带吐槽两句，试探一下。 |
Select Language