抱着想写好质量代码的心，然后遇到一个完全不管代码质量的人合作，真是糟心

[857681664]

项目框架已经搭好用 springboot3 和 JPA ，然后他居然在 repository 里写了个 insert 方法，而且还是用 nativeSQL ，当然可能人家没用过 JPA ，那也算了。

然后一个发邮件的功能，他说调试了一天才搞定，然后我一看他的代码，在发邮件的函数里用 java 自带的 http client ，每发一封就 new 一个，就算不用 sprigbooot 的 rest template ，好歹也用一下 okhttpclient 吧，而且也不用 bean 管理一下。然后发邮件有个验证码机制，有有效期之类的，我本来的想法是用单独的表管理所有验证方式，比如注册验证邮件，改密码验证邮件，还有大概率手机验证，2FA 之类的，然后他直接把 token 和过期时间存 user 表里了。

还有很多小细节，比如全局捕获异常也不设置 http 状态码，直接默认都 200 （项目定的 rest 风格），有 loback 也不用，异常直接 printstack ，也从不管已经存在的代码，比如已经有 util 包和 Util 类了，还要新建 utils 包和 Util 类，异常码也是，明明有个 user_not_found 了，还要在一个他写的重复判断用户是否存在的逻辑里加一个 no_email 的错误码，反正我也不想管了，把我负责的 api 弄稳妥就行。

[sagaxu]

说句扎心的，说不定你工资还没他高

[mobpsycho100]

人和代码有一个能跑就行

[Pending]

分清楚工作和个人爱好/喜欢。

[Pending]

@Pending 喜欢 -> 喜好。

[imzhoukunqiang]

工作就是工作，完成任务就好

[bmpidev2019]

让他用 ChatGPT 重构代码，质量很容易提上来了

[x2420390517]

@857681664 #4 可能是培训班的

[x2420390517]

@coderluan 那他离职了之后呢

[DzwsGo]

@ebushicao 说了满口答应然后不改认为在找茬，后面就不管了，最近离职了，还得改他的屎

[ColdBird]

关于代码质量可以跟 leader 商量一下引入 code review 机制，然后每两周阻止一次 review 复盘会复盘一下典型的代码，就比如你举得例子

[Jack66]

根据我的经历很正常，公司所谓的负责人什么全栈。代码屎山，什么全栈样样通样样松都是表扬的话。能给的建议是工作除了这点觉得满意就想想怎么做，或者找机会沟通

[chenPiMeiHaoChi]

恭喜你已经开始了思考职业生涯的第一个阶段，下个阶段你就会发现代码写的好坏和能赚到的钱并不完全是正相关，下下个阶段就该探索其他搞钱方向了

[123zouwen]

项目有规范并且规范是什么不是你一个人能决定的, 是 lead 决定的,或者 lead 放权让你负责这一块.
如果 lead 没有这个要求, 那每个人就会按照自己的风格来写, 应该还不会有 code review 了吧.

[gowk]

可以看出 OP 是非常注重代码质量的，如果是我，我会非常喜欢和你这样的程序员共事，一起讨论代码和设计，一起推进项目做出让用户用起来感觉非常流畅和舒服的应用。但是往往事与愿违，由于各种原因，很多敷衍了事的代码会充斥在代码库里，确实看了很糟心，非常影响写代码的心情。没别的办法，努力提升自己，离开这个环境，进入更有素养更注重代码质量的团队。

「凡是能真正把他人怎么阅读代码放在心上的软件工程师，即使一开始不具备非常好的设计技巧，随着时间的推移，也能很快学会这些技巧。心中有他人是非常重要的意识和素养。」——《软件设计：从专业到卓越》

[gitdoit]

你俩分好工, 谁也别动谁的代码, 把他当黑盒子就行了. 两个人对接的时候做好防御; 其他的管那么多干嘛

[woodfizky]

leader 的问题。

如果找 leader 反馈能改善下，可以忍忍。

如果不行或者没有 leader ，快跑，争取以后自己成为 leader 。

[aken29]

@857681664 #10 同感

[hellomsg]

@857681664 #20 那就是这种人没法合作。默默擦屁股吧

[akakidz]

很明显就是能力不行啊，不知道楼上为啥还要帮这种人讲话。如果是工作经验不足的新人，OP 可以指导一下，如果是三年五年的老人，尽快跟领导反馈吧，不然等他离职后这个屎山谁愿意接？

[echoZero]

能力的参差而已，我们公司很明显， 但是 leader 只要交付，不在乎这些。KPI 有重构这一项 只能说工作不饱和