如果你作为 leader，怎么委婉的跟下属说他的代码有一些小的毛病？

[xiaojunjor]

某一个模块，业务是实现了的，但是有的地方不是很完美

比如效率低，但是因为模块涉及到的数据量很小，所以不仔细看代码的话，发现不了

比如重复定义变量，方法变量名不规范等等

就是这些我感觉，跟“工作能力”没有太大的关系，而是跟“工作是否用心”有关系的事，怎么去讲，能让对方比较好接受一些，并且知道以后类似的这些东西需要“用点心”去写，而不是单纯的“实现业务”就可以了？

[T110E5]

大家一起搞个 review

[liangdu]

老大，我知道了，下次会注意写好代码的。下周五我想请假去面试。

[doublleft]

委婉？不存在的，我一般都会说这里有颗鼻屎你擦一下

[kiwi95]

你是老大，又不是他是老大。。。老大都还这么谨慎的吗

[tmac]

每天下午下班前固定 code review 半小时。这些都是对大家有帮助的。直说也无妨。

[maemual]

code review 的时候会明确说

[zjuster]

你是他老大，不需要委婉...

[Kmzl]

我希望我老大直说

[SourceMan]

不说他怎么进步？？

[notreami]

直接吐槽，不服来辩。。

[php01]

让他解释下这段代码的逻辑，从他所说的逻辑中提问，比如问“这段逻辑中会不会出现某种情况，如果出现了某种情况的话怎么处理”

[Phariel]

有什么问题直说无妨 大家都是成年人

[discrete]

项目里用 Git 的话用 Codacy 一类的自动检查代码质量，「重复定义变量」这种简单的问题还是能查出来的。这样就不用你明说了。

[sunny352787]

委婉？干嘛要委婉？做的不好直说啊，少给他发工资的时候他会不会找你啊？ review 的时候直接说你这里写的不对，我这边的小崽子们我要求换行都要按我说的来，否则不允许上传，“委婉”这玩意太影响效率了。

[Gonejack]

要有很明确的点，不要简单说这里不好却又没有好的理由。

要尝试做正确示范，不然很容易站着说话不腰疼或者没留意到别人踩的坑。

[Lothar]

用 Github / Gitlab 之类的话，直接在合分支的 PR / MR 里加 Code Review 评论嘛～ 没啥需要委婉的，程序员之间代码上的事情，直接开喷就行。

[qq976739120]

有你这样的老大真幸福

[Anshi]

我希望我老大直说；（有你这样的老大真幸福，还招人吗）

[akagi]

编译级别错误 > 自动化检查 > Code Review > 正面一对一

变量名不规范通常是经验和能力范畴，我自己也还有点儿这个毛病。重点是看到本质原因，并且给出解决方案；如果做不到，对事不对人、语气轻一些，一般也是会感谢指出的人的。

[Anshi]

比如效率低，但是因为模块涉及到的数据量很小，所以不仔细看代码的话，发现不了

比如重复定义变量，方法变量名不规范等等
————

以上这些是我初入职场完全不在意的问题，我觉得就我自己写，自己看，就自己知道就好了。虽然我知道这么写不对，但是懒得改了。这种心态。会这么干估计我不不爱写这份代码，这份业务。

...后来用了相关规范工具，类似的错误会标黄色，或者红色，波浪线等等...我受不了。。就改了。。。