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

2017 年 12 月 12 日
 xiaojunjor

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

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

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

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

13189 次点击
所在节点    程序员
109 条回复
wdlth
2017 年 12 月 12 日
你们不规定编码规范，不用静态分析么？
l00t
2017 年 12 月 12 日
有需求，或者有要求，你提前说。写完再唧唧歪歪的还没个实锤，就算是领导，底下人也未必服的。编程上很多问题不过是个意见分歧和个人偏好而已。当然如果你三番五次强调过了对方还是再犯，那确实是工作态度有问题。
RaymondYip
2017 年 12 月 12 日
之前说过几次指出来都以为我在骂人，，后面不说了 直接自己看到顺手就改掉舒服多了，就是自己累点
romisanic
2017 年 12 月 13 日
为什么要委婉？
没有由头的话，现在设立一个规则或者要求，让大家遵循，说明的时候就拿这部分代码举例子。
有问题说出来才好进步。
说了不改，那就该换人了。。。
takato
2017 年 12 月 13 日
完全可以找一份和他问题相似的代码，找个非正规场合单独给他看看，不要指出问题。这时候如果对方够聪明，应该能够自己发现一些问题。
如果对方没意识到再考虑评审时候说吧。

个人意见
jsfaint
2017 年 12 月 13 日
大家都在说楼主委婉的问题，我换个思路
如果代码风格问题，用 lint 工具扫了让他改或者参考团队代码风格
如果是性能问题，不上 profiling 就说有性能问题，这样不严谨啊
missdeer
2017 年 12 月 13 日
你是老大你说了算，委婉干什么

一劳永逸的方法是制订编码规范，落实 code review
Adamla
2017 年 12 月 13 日
@T110E5 E5 大哥好
ii4Rookie
2017 年 12 月 13 日
@zhouquanbest 哥，不是你教我这样写的吗？
zjddp
2017 年 12 月 13 日
渴望有这样的老大
zhangfeiwudi
2017 年 12 月 13 日
直接就说了，不用委婉啊
8355
2017 年 12 月 13 日
我觉得先看看是能力不够还是能力够随意乱写
如果能力不够虚心接受那你多教教
诚心乱写就骂人啊..
skylancer
2017 年 12 月 13 日
组织一场 code review 不是最好的办法么..？
scnace
2017 年 12 月 13 日
那么反过来呢 要怎么委婉地说(
hundred
2017 年 12 月 13 日
有什么好委婉的，发工资的时候委婉发了 8 折么
myself659
2017 年 12 月 13 日
代码规范有的话，直接上规范
没有的话通过代码 review 论对错与优劣
T110E5
2017 年 12 月 13 日
@Adamla 是 T29 小兄弟？还是 M103 呀，，，，2333
meathill
2017 年 12 月 13 日
不用委婉，直说即可。

代码规范用 lint 工具，必须通过才能提交。

定期组织 Code Review，新人的代码必须 Code Review 才能合并到 master。
kkzxak47
2017 年 12 月 13 日
直接叫过来指出来啊
m939594960
2017 年 12 月 13 日
我现在特想问一个问题,我作为员工,怎么指出 leader 的代码有一些小问题呢

这是一个专为移动设备优化的页面（即为了让你能够在 Google 搜索结果里秒开这个页面），如果你希望参与 V2EX 社区的讨论，你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://v2ex.ih06.com/t/414064

V2EX 是创意工作者们的社区，是一个分享自己正在做的有趣事物、交流想法，可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX