同事的代码写得烂怎么办

新入职一家公司，负责review别人的代码，发现一个同事的代码写得很烂，应该怎么办呢? 跟他说了怎么写，他就是不听，反复说两三次他才加上去，搞了一星期了，今天我approve了他的代码，但是质量感觉还是一般。我不想耽误太多时间在这上面了，也不想拖进度，他的代码也不是很重要，没到production，以后还会改动的。我这么做好吗？经理看到了觉得我没有认真review？我怎么做比较好啊

老同志代码质量让你这个新和尚不满意 这不是他的问题

是manager和其他老同志的问题

desperatedman 发表于 2020-06-26 10:11:46
好吧，可是我已经approve了，我是不是太nice了？另外总是这么来来回回的，也耽误我的时间啊

你可以先干自己的活 每天下班前再给他review一次 每天不超过一次

我会直接跟老板说 对事不对人 我们组的code quality辣鸡，但不说是谁，take a step back 说对改进组里代码质量的办法 比如怎么改进你们的code review mechanism etc

补充内容 (2020-6-27 11:48):
我至少自己是这么做的 最后各方面反应都还不错 仅供参考

不approve 一直跟他扯皮 他比你急

foreveryounger 发表于 2020-6-27 01:05
不approve 一直跟他扯皮 他比你急

好吧，可是我已经approve了，我是不是太nice了？另外总是这么来来回回的，也耽误我的时间啊

foreveryounger 发表于 2020-6-27 01:17
你可以先干自己的活 每天下班前再给他review一次 每天不超过一次

好吧，以后就这么干，即使这样也觉得耽误时间. 现在不知道经理怎么看我啊，还会不会给我安排code review的活啊？

写下comments 不改不approve

这得看大环境了。。
我们org普遍写的烂 大家就随便approve完事了 只要不出错就行
如果这是普遍现象 就别拖着他了- - 说不定他还控诉你拖他的ddl

如果是全组code都烂或者水过review，block一个人没用的。需要让老板意识到代码质量是blocking issue，全组慢慢提高要求
如果是单一的outlier，可以block这个人，加和老板打个招呼。因为ta大概率会跟老板说你要求太高影响了ta进度