同事的代码写得不太规范，读起来有点痛苦，如何委婉地向同事表达自己的想法？

[batman2010]

入职新公司快一周了，一直在读同事的代码，但是同事的代码写得不规范，驼峰命名、行内空格这些都不常见到。想和同事说说，又不想伤了和气，毕竟大家刚认识，相处也还不错。 LZ 不太会说话，所以请教下各位 v 友，对于 LZ 这种情况，如何表达自己的想法为好？

[FrankFang128]

1. 只重构你当前项目的代码，只重构交给你维护的代码
2. 没有「用一个星期来重构」这种事情，你必须「每时每刻都在重构」
3. 没有单元测试就不要乱重构。
4. 改代码风格不叫重构！

[kneep]

应该要被 review 的人拒掉

[muzuiget]

稳定压倒一切啊懂不懂。

[Nausicaa]

除非很熟，否则别找不自在

[shenqi]

弄个 mr ，合上去就行。

[ys0290]

辞职

[batman2010]

@shenqi 请问 mr 是什么啊？

[hpeng]

别说，代码都是小事，我也是刚入职，我就不说我们开发都在主干上，今天这周内网构建整天出问题，不想说话了。

[caserest]

直接说啊 看到这个我突然想到我上一个公司有人用用的变量名字是$woaibeijingtiananmen

[batman2010]

@hpeng 是啊，明明知道问题所在，但说也不是，不说也不是，这种感觉很是累心。

[batman2010]

@caserest 我是新入职的，技术也不是特别强那种，说话没有分量啊，直接说容易让别人有想法。

[osborn]

评价别人代码，除非有碾压别人的实力，否则，要非常非常非常委婉。

[frozen2013]

建议你不要直接和当事人说。

如果你指得不规范仅仅是没有“驼峰命名、行内空格”，作为一个新人直接说出来，资历比你深同事十有八九会认为你小题大做出风头，不一定理会你。
如果你强迫症忍不住，要么向组长提代码规范的重要性，要么在团队会议上自告奋勇自己动手改。

做技术都很实在，但自尊心不一定低。

[young]

都很随意怎么办, 复制粘贴, 改个参数...

[cxbig]

这个属于 TL 领导力问题，团队推行一个代码标准，严格执行。
比方说我们团队的 PHP 代码使用 PSR-*规范 http://www.php-fig.org/
代码提交汇总前会经过同行代码评审，不通过的打回改好再合并。