Question

Sometimes I have to write code that alternates between doing things and checking for error conditions (e.g., call a library function, check its return value, keep going). This often leads to long runs where the actual work is happening in the conditions of if statements, like

if(! (data = (big_struct *) malloc(sizeof(*data)))){
    //report allocation error
} else if(init_big_struct(data)){
    //handle initialization error
} else ...

How do you guys write this kind of code? I’ve checked a few style guides, but they seem more concerned with variable naming and whitespace.

Links to style guides welcome.

Edit: in case it’s not clear, I’m dissatisfied with the legibility of this style and looking for something better.

Answer 1

Though it pains me to say it, this might be a case for the never-popular goto. Here’s one link I found on on the subject: http://eli.thegreenplace.net/2009/04/27/using-goto-for-error-handling-in-c/