do-while-false循环是常见的吗？

Question

前一阵子我改变了我处理c风格错误的方式.

我发现很多代码看起来像这样:

int errorCode = 0;

errorCode = doSomething();
if (errorCode == 0)
{
   errorCode = doSomethingElse();
}

...

if (errorCode == 0)
{
   errorCode = doSomethingElseNew();
}

但最近我一直这样写:

int errorCode = 0;

do
{       
   if (doSomething() != 0) break;
   if (doSomethingElse() != 0) break;
   ...
   if (doSomethingElseNew() != 0) break;
 } while(false);

我已经看到很多代码在出现错误后没有执行任何内容,但它始终以第一种方式编写.有没有其他人使用这种风格,如果你不这样做,为什么？

编辑:只是为了澄清,通常这个构造使用errno否则我会int在打破之前将值赋值给.此外,通常还有更多代码,而不仅仅是if (error == 0 )子句中的单个函数调用.但是,要考虑很多好点.

Answer 1

如果您使用的是C++,只需使用异常即可.如果您使用的是C,那么第一种风格效果很好.但如果你确实想要第二种风格,只需使用gotos - 这正是gotos真正是最清晰的结构的情况.

    int errorCode = 0;

    if ((errorCode = doSomething()) != 0) goto errorHandler;
    if ((errorCode = doSomethingElse()) != 0) goto errorHandler;
      ...
    if ((errorCode = doSomethingElseNew()) != 0) goto errorHandler;

    return;
errorHandler:
    // handle error

是的,可能是糟糕的,并且每次调用后的异常或显式错误处理可能会更好,但是比使用另一个构造来尝试和模拟它们要好得多.使用gotos还可以为特定错误添加另一个错误处理程序:

    int errorCode = 0;

    if ((errorCode = doSomething()) != 0) goto errorHandler;
    if ((errorCode = doSomethingElse()) != 0) goto errorHandler;
      ...
    if ((errorCode = doSomethingElseNew()) != 0) goto errorHandlerSomethingElseNew;

    return;
errorHandler:
    // handle error
    return;
errorHandlerSomethingElseNew:
    // handle error
    return;

或者,如果错误处理更多的是"展开/清理你所做的"变种,你可以像这样构造它:

    int errorCode = 0;

    if ((errorCode = doSomething()) != 0) goto errorHandler;
    if ((errorCode = doSomethingElse()) != 0) goto errorHandler1;
      ...
    if ((errorCode = doSomethingElseNew()) != 0) goto errorHandler2;

errorHandler2:
    // clean up after doSomethingElseNew
errorHandler1:
    // clean up after doSomethingElse
errorHandler:
    // clean up after doSomething
    return errorCode;

这个习惯用法为您提供了不重复清理代码的优势(当然,如果您使用的是C++,RAII将更加干净地覆盖清理代码.

Answer 2

第二个片段看起来不错.你有效地重新发明了.

任何阅读第一种代码风格的人都会立即知道发生了什么,第二种风格需要更多检查,因此从长远来看会使维护更加困难,因为没有真正的好处.

编辑,在第二种风格,你已丢弃错误代码,所以你不能采取任何纠正措施或显示信息性的消息,记录有用的东西等....

Answer 3

第一种风格是经验丰富的眼睛同时出现的模式.

第二个需要更多的思考 - 你看它并看到一个循环.你期待几次迭代,但是当你读完它时,这个心理模型会被粉碎......

当然,它可能有用,但编程语言不仅仅是告诉计算机做什么的一种方式,它们也是将这些想法传达给其他人的一种方式.

Answer 4

我认为第一个可以让您更好地控制如何处理特定错误.第二种方式只告诉你发生了错误,而不是它在哪里或是什么.

当然,例外优于两者......

Answer 5

简洁,紧凑,易于快速阅读？

怎么样:

if ((errorcode = doSomething()) == 0
&&  (errorcode = doSomethingElse()) == 0
&&  (errorcode = doSomethingElseNew()) == 0)
    maybe_something_here;
return errorcode; // or whatever is next