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

[xiaojunjor]

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

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

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

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

[zenxds]

code review 直接说

[Moorj]

不需要委婉+1

我之前给我的女售后和女运营讲解一些要点的时候，得跪着说，不然她总以为我找茬，是故意欺负她

“是是是，你说的对，你是领导，都听你的”

就很气

[sm0king]

有什么委婉的，直接说呀

[mengzhuo]

直接说，或者 review 的时候说

[Mikey]

委婉？那还是 leader ？

[magic3584]

我的主管会直接给我指明，我觉得挺好，能学到东西

[wingoo]

都是直接说的
一般会在 merge 的时候看下

[yongzhong]

@xomix 赞同你的看法

[loryyang]

看到直说，对事不对人呀。你既然是 leader，本身就有这个权力和义务来协助提升小弟的能力
你如果不说，就是你的失职了

[fengwei23]

直说比较好，或者进行 code review，我是希望老大说出来的

[chztv]

作为 leader，不需要委婉，这是你的义务。

[uleh]

这还需要“委婉”么。。
熟一点的一般会说：这什么玩意儿，重写！
不太熟的会说：这里有 xx 问题，你改下

[hustlike]

pull request 可以加 comments

[13036101641]

有可能你不说，他不知道，现在说了，以后他做 TL 了，恰好遇见一个这样子的下属，他也会再去教下去

[GuangTsang]

带着你的解决方案去和他谈，用事实证明你的观点，你的方案比他的更有效率

[sunsmooth]

要让他能感觉到你是在帮他，而不是故意在找茬。

[codermagefox]

哇,楼主,我巴不得被带我的大佬喷.
被喷,说明有进步的空间,说明代码确实烂.
只要不涉及父母,人身攻击都没问题啊(没错我就是抖 M
事实上被喷了以后确实长记性快.

[nickel123666]

居然还要委婉，我的 leader 在指点我代码的时候都是说“你也好意思写这种代码哦”；
“这里！怎么办吧，要不要改？”
“为什么这里 xxxx ”；
哇，感觉被骂得好爽。。

[xxbutoo]

真为了你的小弟好 直接喷他
最好周围没人的时候

[sensui7]

直接讲哪里可以有问题