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

[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 弄稳妥就行。

[destiny0114]

很好的 OP ，可是我 junior 遇不到像 OP 这样的程序员，我个人喜欢人家批评我的代码

[hakr]

@macaodoll #35 哈哈哈哈笑死

[ExplodingFKL]

无所谓了，代码有提交记录，除了问题不被甩锅就行

[857681664]

@destiny0114 我很乐意跟大伙讨论代码，包括团队里其他前端同事后端同事，沟通都挺愉快，有后端同事不太懂 JPA 的 entity ，我也会告诉他怎么写，用什么注解。
而跟这个人甚至都无法好好沟通，跟他说了发邮件的部分我实现完了，让他把他分支的发邮件部分去掉，说了几遍都没明白我意思，然后一直让我把他分支代码拉下来去掉，我真是黑人问号。最后还把他的发邮件代码提交上去，跟我的还冲突了，我还得费劲把他用到发邮件的部分改一下。最绝的是他的邮件配置居然只写在了 dev 配置 yml ，线上配置 yml 里压根没有，这上线了不是也用不了吗

[SuperAllen]

本质就是草台班子么，组织推动标准和推动组织标准是两码子事，把自己的弄好别被甩锅就 o 了

[m319]

确实难受，但毕竟不在你权责范围内，强行掺和这事估计也推不动，说不定还会惹一身骚，别碰他的代码，做好自己的事情就行

[morgan1freeman]

老板只为功能代码付款了，并没有为良好可阅读的代码 进行付费，领导不在乎的化，就这样吧，我们这边代码更乱，我直接在上面加更多的屎，反正后面谁改错了，炸了 关我屁事

[sunznx]

忍忍算了，每个人有自己的生活和想法，做好自己就行了。有些人想的自己工资就这么点，干嘛要卖力干活呢。人家这么想，也不算错。

[leojia]

代入了一下，确实挺糟心的。跟 leader 说合并前先 code review 一下，要么就自己成为 leader 。

[uaoin]

就怕被同化了

[aw2350]

你是领导嘛？那你凭啥要求人家？

[morrieati]

“我本来的想法是用单独的表管理所有验证方式，比如注册验证邮件，改密码验证邮件，还有大概率手机验证，2FA 之类的，然后他直接把 token 和过期时间存 user 表里了”

[morrieati]

supabase 也是这么做的

[ma836323493]

这个还好， 碰到傻缺产品，设计一坨大便，领导还支持， 你不得不开发大便才难受

[sherlockwoo]

关于全局捕获异常也不设置 http 状态码

除了 401 一般都是 200 吧，根据统一响应中的 code 判断

[857681664]

@aw2350 你说的对，所以他的任何代码我都不看了

[857681664]

@morrieati 那 user 表会不会有点膨胀了，而且也不好控制验证过期吧

[857681664]

@sherlockwoo 我们定好的是 restful ，所以会希望有 200 ，400 ，401 ，403 这样的

[llllk]

最难受的是你明知道他写的一坨答辩之后跑路了这坨大便要交到自己手上，但是却无能为力。。

[natamox]

@coolair 你也走