今天浏览网页的时候，看到了别人发的这样一个帖子。我刚加入一家新公司。在codereview的时候，leader对自己的代码做了一些修改，程序员觉得很多地方没有必要。看完以上帖子，你有什么感想？我的看法是：第一，能遇到这样的领导真好。先不讨论leader的修改。有的地方要不要，看领导就知道了。这些小地方，我会帮你做一些修改，codereview，可以说leader很负责。领导的改变和你的改变哪个更规范？不好说，不过我个人觉得leader的改动确实很规范，至少是这样。二是我认为领导的一些修改确实是没有必要的。比如：在上面我画红框的地方，我把setVisible换成了show，其实是没有必要的，但是我觉得leader的那个更容易让人理解和识别。上面常量的名字也接受了，leader也修改了。其实，确实是大可不必。还有一个地方比如：a.do1()a.do2()，leader改成了a.do1.do2()，可能不需要，但是leader的这个修改可以让代码更简洁看着更方便。在维护代码和更新迭代方面，真的是让你一目了然，一目了然，方便整个团队工作的管理和交接。其实作为一个团队，首先要看整个团队有没有代码协议和规范，变量、常量、方法函数的命名是怎么规定的，如果这个团队有代码协议，就是这样制定的命名规则。我们仍然应该遵循这个规则来命名。想一想：一个团队的领导下有十几个人。你是让leader适应十几个人的风格，还是让十几个人统一到leader的风格？代码风格和规范统一，有利于整个团队的代码开发。维护和交接有利于代码管理和升级。这就需要团队有代码规范。欢迎大家关注我的微信公众号：非知名程序员如：上述程序员对领导的修改不满意。首先查看团队中是否有代码规范。代码规范是如何规定命名的？如果是这样，如果你没有按照规范使用它，那就是你的问题。如果没有规范，那么你可以和领导谈。团队应该制定一个规则，如果有规则，我以后就按照这个规定来写，这样也可以缓解问题。领导代码审查的工作量。(http://godcoder.me/)代码审查为什么需要代码审查？提高质量，及早发现潜在缺陷和BUG，降低事故成本。促进团队内部知识共享，提高团队整体水平。对于审稿人来说，审稿过程也是一个思维重构的过程。帮助更多人了解系统。其实我觉得codereview不仅仅是领导的问题。每天花一个小时review别人的代码，对团队中的每个人来说也是非常有好处的。学习别人写代码的优点。毕竟审稿过程也是审稿人思维重构的过程。欢迎大家关注我的微信公众号：非大名鼎鼎的程序员。此外，整个团队必须有明确的代码规范和协议。好处是codereview本该是重要的事情，而不是花在这些不规范的命名上，让法规来约束大家。codereview最重要的是提高代码质量，发现潜在的缺陷和bug，发现项目模块中不合理的地方，比如：关键的系统模块，业务比较复杂的模块，缺陷率高的模块，etc.***，我想说截图上的leader确实水平很高。在命名方面，确实非常规范。虽然可能有些过于真实和夸张，但确实值得学习。
