转载

观点：完美代码只是一个幻想

引言

完美主义者最大的特点就是过度追求一件事情的完美，他们看什么东西都不会完全满意，因此经常陷入深深的矛盾之中，殊不知这个世界上根本就没有绝对的完美，将精力投注到工作中、生活中各个方面，努力改善，乐此不疲。至于吗？程序员中的完美主义者又会怎样呢？

许多程序员文化是建立在完美代码的理想上：代码不仅能够运行，而且也必定是干净、优雅的。我们以巧妙地构建解决难题的对策为傲。然而这种完美主义可能不利于团队的成功，因为完美主义常常导致个人分歧。

然而能得到所有人公认的完美代码标准并不存在。对于完美代码，每个人都有一个略微不同的审美观点，这意味着我们每个人都有自己的想法：什么样的代码看起来完美。如果我们都是由完美主义来驱动——希望我们的代码看起来像我们想要的样子，那么我们最终会与队友发生分歧，因为我们每个人互相反对，只是为了让代码库看起来像我们所想看到的样子。

伦敦一位资深程序员 Daniel Irvine 就在博客分享了他对完美代码的看法以及对策。

当我成长为一个程序员时，我发现有一些小技巧，可以让团队避免因为追求完美代码而发生冲突。下面就让我们来看一看。

不要被教条束缚

对代码库的唯一要求就是，它是可用的。通过一个简单的方法来验证，如果它经过完全覆盖测试并通过，就可以证明是可用的。除此之外，每个测量都是主观的。

当你阅读其他人的代码，不要去想如果是你写的话会怎样。不要试图在你脑海中重写这段代码，让它存在就是它的方式。

减少你对代码设立的标准

用制表定位键（Tab）还是空格键（Space）？两个还是四个空格？为你的左括号设置同一行呢，还是另起一行呢？我不知道如果只有一个单一的编程语言的话，是不是就不会有这种争论？解决这个问题的标准方法就是使用书面编码标准，这会为团队的代码带来一致性。

不幸的是，很难形成完整的编码标准。总是会有灰色区域导致了潜在的分歧，如命名、模式、对象建模技术等。

而且，他们团队定下的规则有时会引火烧身。

我曾经所在的团队，对编码标准有过如下规则：“任何函数不得超过7行代码”。事后看来，这个规则，还不如没有。虽然我仍然赞同这个观点，但这一规则还是激起了很多混乱和争辩。团队成员在编码的时候不能全心投入，因为需要总是想着这个规则。还有人则走向另一个极端，永远拒绝接受它。总之，我们团队花了大量时间和精力，来保持遵守这个规则。

试想，那些时间如果用来结对编程或是一起改进代码该有多好。所有的规则都有一定的代价，并且，尽管规定事无巨细，团队成员们仍然可能在其它事情上存在分歧。

虽然我仍然尽量保持编写简短的函数和方法——通常少于七行——但我不会让它们成为必须遵守的规则。

让代码库成为自己的标准，而不是写出什么规则。

不要被Pull Request套牢

我通常会迅速合并Pull Request（略：PR），即使它对代码有很大的改动。这样做有两个原因。第一是等待PR修改可能阻塞其它同伴的工作流。第二点更微妙，基本代码保持可延展性是非常重要的：这意味着，要时刻准备接受修改。“完美PR”文化阻碍了这一点。它促进了代码在主分支是“黄金”，并不应该再次改变的概念。如果我们转而允许不完美代码进入主分支，那我们会鼓励更高的变化率。团队成员会学会问自己：“我看的代码足够干净吗？”

这有点违背直觉：允许主分支写入不完美代码可以提升项目质量。

那么，审查PR的更好的方法是什么？

我的策略是这样的。我首先通读整套变更，标注任何可能重要的事情。然后优先给出反馈，最多给三条修改建议给出意见。其它的就不管了。

我很少就编码风格问题进行评论，比如多出的空格，或者没有很好缩进的参数列表。如果代码是可延展的，有人可能在以后会清理它。同时，这些风格问题并不会妨碍程序的运行。