maintainability - 强制使用牙套

Question

作为我不久前写的代码标准文档的一部分，我强制要求“您必须始终对循环和/或条件代码块使用大括号，即使(尤其是)它们只有一行。”

例子:

// this is wrong
if (foo) 
    //bar
else 
    //baz
while (stuff)
    //things

// This is right.
if (foo) {
    // bar
} else {
    // baz
}
while (things) {
    // stuff
}

如果你不支持单行，然后有人评论它，你就有麻烦了。如果你不支持单行，并且缩进在其他人的机器上显示不一样......你有麻烦了。

所以，问题:有充分的理由说明这将是一个错误的或不合理的标准吗？对此进行了一些讨论，但没有人能比“感觉难看”给我更好的反驳。

Answer 1

我能提供的最好的反驳论点是，空间占用的额外行减少了您一次可以看到的代码量，而您一次可以看到的代码量是决定容易程度的一个重要因素就是发现错误。我同意你给出的包含大括号的原因，但是在多年的 C++ 中，我只能想到一个结果是我犯了一个错误，而且我没有充分的理由跳过大括号反正。不幸的是，我无法告诉您是否看到这些额外的代码行在实践中是否有帮助。

我可能更有偏见，因为我喜欢在相同缩进级别匹配大括号的对称性(以及将包含的语句隐含分组为一个执行 block ) - 这意味着始终添加大括号会增加很多行项目。