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

还招人吗？遇到这样的老大真得是幸福啊

不懂，我一般就是直说，也建议直说。

code style 的话，最好设立规范，然后用自动代码检查。

如果是算法不合理，没考虑到当数据很大会有较大耗时或性能消耗。这就是与工作能力相关了。一般来说，能提供职业技术能力的建议，程序员一般还是会乐于接受的。（来不来得及马上改可能存在排期问题）

如果确实是态度不好，那应该反省一下招聘了。招聘和试用阶段为什么没有发现这种情况。

如果是 N 年老员工，可能就要看看彼此利益干系，实在不行就只能曲线处理，关键任务不交给他。

把 coding standard 弄出来，按照 coding standard 改变量名。 对效率的要求在 non-functional requirement 中提出来。 如果这些都有，但还是“没用心”，就直接当面说；如果这些没有，就是团队管理的问题。

希望老大直接说，只要说的中肯有帮助

命名规则、代码规范什么的最好有一个文档的吧～如果哪里逻辑复杂又没加注释、或者命名让人根本看不出这个是干嘛的，后面的人维护起来也很头疼，肯定要点出来的。而且这都属于习惯的问题，没什么接受不了

如果是具体到逻辑、效率上的，我还要感谢老大帮我上了一课好嘛～
当然有些模块划分或者功能实现上我和老大有点各持己见，偶尔会有争论，其实最后还是学习到了

如果你是我老大，我希望能直接跟我说，没必要顾及太多，虽然有可能尴尬，但我更希望得到提高

想要有这样的老大

别人我不知道，反正如果人家指出我的代码里有问题，我一般都是先听明白，再确认。
如果真的有问题我还感觉蛮高兴的，因为这样下次就不会再犯了。

私底下明说就是了，目的是为了让程序更好，如果连优化建议都听不进去这人心胸就太小了。太委婉反倒耽误事。

我希望老大直接跟我说 这样我自己也会提升

@Anshi 相关规范工具是哪些 求个列表~ 目前用的是 IDE 自带的

@fhefh 路过，这个貌似要看写啥，C++有 cppcheck, js 有 eslint, 用 lint 做关键字一般可以搜到些~

@akagi 3Q 现在在用在 嘿嘿~

下属身体很壮么

这还需要委婉

楼主怎么定义委婉……有内情？

只是个代码问题，我跟我老板说，都不用特别注意的。

为什么要委婉，直接说啊

亲，你这坨代码写的真好，就是有那么一丢丢的问题。
楼主是要这样么。。。
毛线啊，leader 的职责就是支出代码问题啊。。

其实吧，都说直接说好。但是真轮到自己头上指不定怎么炸毛呢。。
lz 这个话题，我也经常碰到来着，但是我看着不爽还不让我提，你来当 leader 啊。。。

记得 SVN 的时候 貌似有个 SVN blamed
不过我没用过 SVN 所以……