同事不注意代码质量，应该如何沟通

[victorwu34]

一次是根据 id 的集合查询，查询的 api 如果 id 集合比如 3000 个以上就不支持，我以前的代码有错，3000 之外的都被忽略掉，结果同事把代码改成了循环几千次去查表。我跟他沟通这个问题，他说，就是这样，多了就不支持，那你改啊。

还比如，一次他把几十个返给前端实体里面的字段修改了，前端很多接口拿不到这个字段，很多页面报错。这次和他沟通，我说这是你造成的 bug，你自己去改，他说，我这里的比你的 bug 还多，咱们是一个团队，这正常。 还有很多次，都是他瞎写瞎改造成的，每一次都无法沟通，别人也在帮他改错。

已经很尊重他了，也帮助了他，我们没有代码审查，技术经理也从来不问不管，没有任何改善，不知道这种情况应该如何沟通。

[CEBBCAT]

看起来公司里没这个文化，凭你自己也似乎很难改变一个公司。没工作的我这么分析一下：费力不讨好，骑驴找马吧

[victorwu34]

@CEBBCAT 你就是说无法沟通😭😭😂😂

[R4mble]

你这两个例子我都没太看懂诶*罒▽罒*
你是前端，你同事是后端吗

[LifStge]

例子不是代码质量的问题啊 沟通问题 开发人员相处的不融洽...

[boris1993]

这已经不是代码质量问题了，这是团队里进了个二逼

要我说啊，首先，（联合起几个人）找领导报告这事，私下里他胡逼写的时候就损丫的骂丫的

傻逼不值得被尊重

[zgl263885]

有些人就不适合写代码，特别是工作一年之后还是这样的话。但是这种事情又没法跟人家直说，可以跟上级领导反应，劝他做其他岗位工作，说不定做的更好。每个人都有优点和特长，与此人擅长做设计，美工，市场等。

[gramyang]

有两种方法可选：
一是严正秉持谁污染谁治理的策略，发现是他的问题导致的 bug 就让他自己弄。当然前提是你能够收集到足够严谨的证据来证明是他个人的问题导致的（代码截图，聊天信息，邮件记录）。
二是把这个人的各种坑操作都记录下来（同样是代码截图，聊天记录，邮件记录等），然后等待一个时机（比如某一次出现重大事故，不管是不是他造成的都无所谓）。把搜集到的证据全部整理清除，让人一目了然，用一个邮件发给直属领导（记得属上其他看不惯这厮的同事的名字）。

如果上面两种方法都不顶用，那么很可能这哥们是关系户，你斗不过也没必要浪费精力和他斗，跑路吧

[chinesestudio]

快狠准 。写代码是习惯 做好备注 别人看得懂就行 。划水的时候可以优化代码。

[tt67wq]

对他说： 你代码写的像蔡徐坤！

[elgae]

lz 是不是在大兴上班？

[alw]

谁导致的 BUG 谁修，帮忙并不是理所应当的，打死都不帮他修，难道还要我背责任？

[mawenjie]

我想起前公司一个同事兼负责人，他出的方案基本上就是 “就是这样，多了就不支持，那你改啊”。

[meteor957]

还是单独沟通一下吧，脸皮薄的说一次就记住了

[LwZiye]

人善被人欺，直接开怼

[tiedan]

只要没影响到自己工作 其实不用管。

[catcn]

直接改丫的代码

[itfinally23]

这个，在行政上会做代码检查，构建时加个插件，不合格的全部停止构建。

[tairan2006]

这种公司呆着没啥意思啊，建议溜

[zhpech]

溜了吧 没啥相关文化，可能还会给你扣影响团队和谐的帽子

[katsusan]

> 技术经理也从来不问不管
都到这份上了还想啥呢

[q8164305]

赶紧跑路吧，这种人我也遇到过，根本没法合作，后面会憋死你，人家就是把这当工作，而不是爱好，不好指责人家，只能说道不同不相为谋罢了

[lmeteor1101]

没有 leader 吗

[miao1007]

走吧

[specita]

唉，我现在也很难受，搞过两次代码评审，基本就是要求按照阿里的代码规范来，也让去下了个提示插件，结果呢，还是那个样子，idea 已经提示黄色警告了也从来不会去看...沟通了几次，每次都是点头说好...结果一个屌样.....最后自己去改了，还被别人说代码被别人动了....难受得一逼

[Priest]

@specita 用工具来强化代码规范会

[Priest]

@specita 上条信息不小心就发了

用工具来强化代码规范能省不少无意义的争论，可以使用 git 的 pre-commit hook 配合代码格式化工具(Prettier，Black)来做

[v2chou]

没有一点代码洁癖

[Jackietsui]

骑驴找马，公司没有管的你自己的力量根本不够