Sostituzione di if concatenati con operazioni in cortocircuito

Questo è un problema molto banale: Esistono quattro funzioni booleane: a(), b(), c() e d(). Voglio continuare a chiamarli in ordine fino a quando il primo restituisce true. invece di fare la tradizionaleSostituzione di if concatenati con operazioni in cortocircuito

if(!a()) { 
    if(!b()) { 
     if(!c()) { 
      d(); 
     } 
    } 
}

if(!a() && !b() && !c()) d();

ho pensato di scrivere l'espressione come una valutazione in corto circuito.

(a() || b() || c() || d());

Ma non ho mai visto questo test eseguito in questo modo in un codice C/C++. C'è qualche problema con questo approccio che mi manca?

Grazie.

fonte

2013-03-21 user666412

Sono in grado di vedere * cinque * funzioni booleane. –

@chris il o non cambia il significato: d() viene eseguito solo se tutte le altre funzioni sono false. – Loghorn

Vorrei consigliarlo contro una prospettiva di manutenzione. Potrebbe non essere immediatamente ovvio alle persone che cosa fa il codice quando vedono la versione cortocircuitata (se è una preoccupazione per te). Il fatto che dovessi chiedere del comportamento di SO dovrebbe essere un suggerimento su cosa accadrà quando altre persone leggeranno il tuo codice. – Michael

Il codice che hai scritto è valido. d() verrà valutato solo se altre funzioni booleane restituiscono false.

Tuttavia, la valutazione in cortocircuito è meno leggibile e soggetta a errori dal punto di vista della manutenzione, poiché un lettore potrebbe non capirlo in modo banale.

fonte

2013-03-21 21:30:42 meyumer

perché è soggetto a errori? – user666412

Hai sbagliato. 'd()' viene valutato se NO OTHER restituisce true. Quale immagino rende il vostro punto sulla leggibilità. –

Grazie Ben. Era di fretta. – meyumer

Sono equivalenti, ma la soluzione di valutazione cortocircuitata potrebbe essere meno leggibile, specialmente se i nomi delle funzioni sono lunghi o accettano parametri.

fonte

2013-03-21 21:29:46 Loghorn

Sostituzione di if concatenati con operazioni in cortocircuito

risposta

Problemi correlati