Refactor code ohne goto

Question

Ich bin ein wenig verwirrt. Wie kann diese Aussage ohne goto refaktoriert werden?

if(first_expression) 
{ 
    // .... 
    if(second_expression) 
    { 
     // .... 
    } 
    else 
    { 
     goto jump_out; 
    } 
} 
else 
{ 
    jump_out: 
    // .... 
} 

Ist das ein guter Weg?

bool expressionFailed = false; 
if(first_expression) 
{ 
    if(second_expression) 
    { 
     // .... 
    } 
    else 
     expressionsFailed = true; 
} 
else 
    expressionsFailed = true; 

if(expressionsFailed) 
{ 
    // .... 
} 

Ich kann && nicht verwenden, da Code nach if(first_expression) Bedarf ausgeführt werden, wenn second_expression = false

Answer 1

Sie sind auf dem richtigen Weg. Ich würde es vereinfachen zu:

bool succeeded = false; 
if(first_expression) 
{ 
    if(second_expression) 
    { 
     succeeded = true; 
     // .... 
    } 
} 

if(!succeeded) 
{ 
    // .... 
} 

Answer 2

Sie können es wie folgt umschreiben:

if(first_expression) 
{ 
    // .... 
    if(second_expression) 
    { 
     // .... 
    } 
} 
if(!first_expression || (first_expression && !second_expression)) 
{ 
} 

Answer 3

Ich weiß nichts über Ihren Code aber Sie könnten das tun:

void on_error() 
{ 
    //error stuff 
} 

int main() 
{ 
    if(first_expression) 
    { 
     if(second_expression) 
     { 
      // .... 
     } 
     else 
      on_error(); 
    } 
    else 
     on_error(); 
} 

oder noch besser:

void job() 
{ 
    if(first_expression) 
    { 
     if(second_expression) 
     { 
      // .... 
     } 
     else 
      throw; 
    } 
    else 
     throw; 
} 

int main() { 
    try 
    { 
    job(); 
    } 
    catch() 
    { 
    // err stuff 
    } 
    return 0; 
}