如果(条件)继续; 或者if(!condition){...}？(风格偏好)

Question

我知道这是一个风格问题,因此是主观标签.我有一小段代码,有两个嵌套条件.我可以用两种方式对它进行编码,我希望看到更有经验的开发人员认为它应该是什么样子.

风格1:

while (!String.IsNullOrEmpty(msg = reader.readMsg()))
{
    RaiseMessageReceived();
    if (parseMsg)
    {
        ParsedMsg parsedMsg = parser.parseMsg(msg);
        RaiseMessageParsed();
        if (processMsg)
        {
            process(parsedMsg);
            RaiseMessageProcessed();
        }
    }
}

风格2:

while (!String.IsNullOrEmpty(msg = reader.readMsg()))
{
    RaiseMessageReceived();
    if (!parseMsg) continue;

    ParsedMsg parsedMsg = parser.parseMsg(msg);
    RaiseMessageParsed();
    if (!processMsg) continue;

    process(parsedMsg);
    RaiseMessageProcessed();
}

(附带问题:如何在源代码示例中放置空行？)

Answer 1

我更喜欢Style 1 - 带有缩进.

Answer 2

我更喜欢Style 2 - 使用continue语句.

Answer 3

原则上我同意喜欢风格1的大多数人.这就是Steve Mcconnell在"代码完成"中所赞同的 - 说出你的意思,即你是否对条件是真的更感兴趣,而虚假状态是罕见的或不是首选的,然后说明首选版本.

在实践中虽然我经常发现自己使用样式2,因为我喜欢首先清除所有可能的错误/无效状态.在我摆脱了我不感兴趣的所有可能性之后,我可以将心脏代码编写到例程的末尾,而不必经常想知道我是否需要防范某些条件或其他条件.基本上,态度是,摆脱糠,,然后在和平中做真正的工作.

Answer 4

这两个都是假的.不要将赋值放在条件表达式中.

(!String.IsNullOrEmpty(msg = reader.readMsg()))

你只是这样做是因为读者的行为不稳定 - 为什么读者会给你一条非信息表明读完了？这是一个有更好设计的读者的替代方案:

while (reader.HasMessage())
{
  string msg = reader.GetMessage();
  HandleMessage(msg);
}