c - 使用 C switch 语句进行错误处理

Question

考虑这个在执行实际工作之前检查错误的 C 结构:

int function(struct Context *context,struct Connection *conn)
{
    int retval;

    switch(0)
    {   
        case 0:
            retval = BUFFER_INACTIVE;
            if(conn->mSocket == -1) 
                break;
            retval = BUFFER_FULL;
            /* Is there enough room to add ? */
            if((context->mMaxBufferSize - conn->mSendPacketLength) < aPacketLength)
                break;

            /* Is the send packet buffer half sent? */
            if(conn->mSendPacketLength > 0 && conn->mSendPacketPos != conn->mSendPacket)
                break;

            /* Do some work here */
            retval = BUFFER_DONE;
    }
    /* Do some things before returning */
    printf("%d",retval);
    return retval;
}

您认为这是可读的吗？使用 goto 或堆叠 if() 的替代方案会更好吗？

Answer 1

我从未见过 switch 解决方案，但我做过这样的事情:

do {
    err = func();
    if( err ) break;
    err = func2();
    if( err ) break;
    ...
} while( 0 );
if( err ) {
   // handle errors
}

但是那和这之间的真正区别是什么:

err = func();
if( err ) goto done;
err = func2();
if( err ) goto done;
...
done:
if( err ) {
   //handle errors;
}

第一个只是第二个重写以避免使用关键字 goto，我认为 goto 解决方案更具可读性。我花了一段时间，但我设法说服自己 goto 并非总是是邪恶的。

最后，如果可能的话，我更喜欢只使用 if 语句，因为它使代码更具可读性，但在必要时也可以使用 goto。