doppio se la condizione, un modo per farla breve

Question

ho frequenty hanno a che fare qualcosa di simile,

if(condition1) { 
     if {condition2) { //this condition is repeated again below 
      //dosomething here 1 code 1 
     } 
     else{ 
      //dosomething here 2 code 2 
     } 
    } 
    else { 
     if {condition2) { //same if condition as above 
      //dosomething here 3 code 3 
     } 
     else{ 
      //dosomething here 4 code 4 
     } 
    } 

In sostanza, solo il controllo di se (condizione2) è ripetuto in entrambi i casi, e dosomething è diversa in tutto 4 posti cioè code1, code2, code3, code4 sono tutti codici diversi.

Quindi, c'è comunque per renderlo compatto e leggibile O questo va bene?

Grazie.

Dopo aver esaminato le modifiche e aver esaminato le risposte, mi chiedo ora se questa domanda abbia senso. Mi sento stupido ora.

Answer 1

Per il vostro caso specifico, l'alternativa migliore è:

non si applica più - valido per il caso in cui dosomething qui 1 era lo stesso con dosomething qui 3

if (c2) 
{ 
} 
else if (c1) 
{ 
} 
else 
{ 
} 

Per i casi meno semplici , è possibile raggruppare le condizioni insieme:

if (c1 && c2) 
{ 
} 
else if (c1 && !c2) 
{ 
} 
else if (!c1 && c2) 
{ 
} 
else if (!c1 && !c2) 
{ 
} 

anche se non so se questo è più leggibile.

Se sono necessarie più condizioni, ho visto il codice come questo:

do { 
    if (c1) 
    { 
     //.... 
     break; 
    } 
    if (c2) 
    { 
     //.... 
     break; 
    } 
    //..... 
} while (false); 

Answer 2

nuova risposta AS DOMANDA È REDATTA

if (condition1 && condition2) 
{ 
    //dosomething here 1 code 1 
} 
else if (condition1 && !condition2) 
{ 
    //dosomething here 2 code 2 
} 
else if (!condition1 && condition2) 
{ 
    //dosomething here 3 code 3 
} 
else 
{ 
    //dosomething here 4 code 4 
} 

SEGUENTE CODICE non è più valido AS DOMANDA HA CAMBIATO!

lo farei:

if(condition2) 
{ 
    if(condition1) 
    { 
    //dosomething here 
    } 
    else 
    { 
    //dosomething here 2, 
    //which might be different than what we are doing above 
    } 
} 

Questo è per la tua domanda specifica, per problemi più complessi la soluzione potrebbe essere diversa