Ridurre il se-dichiarazione stessa

Question

Questo è il mio se-economico al momento:

if (excel_getValue("A" + i) == "" && 
    excel_getValue("A" + (i + 1)) == "" && 
    excel_getValue("A" + (i + 2)) == "" && 
    excel_getValue("A" + (i + 3)) == "" && 
    excel_getValue("A" + (i + 4)) == "" && 
    excel_getValue("A" + (i + 5)) == "" && 
    excel_getValue("A" + (i + 6)) == "" && 
    excel_getValue("A" + (i + 7)) == "" && 
    excel_getValue("A" + (i + 8)) == "") 

C'è qualche cosa che può ridurre questa affermazione? Dovrei costruire questa if-statement in un ciclo for?

BTW, questo se-dichiarazione, se già in una forloop e utilizza la i del forloop

Answer 1

È possibile utilizzare All:

if(Enumerable.Range(0, 9).All(c => excel_getValue("A" + (i + c)) == "")) { 
} 

Answer 2

C'è un modo semplice per "ridurre" (almeno "abbellire") con un ciclo for ... questo non sarebbe più performante, però, in modo da scegliere:

bool do = true; 
for(var j = 0; j < 9; j++) { 
    if (excel_getValue("A" + (i+j)) != "") { 
    do = false; 
    break; 
    } 
} 
if(do) { /* whatever */ } 

o utilizzando Linq:

if(Enumerable.Range(0, 9).All(x => excel_getValue("A" + (i + x)) == "")) { /* whatever */ } 

Questo potrebbe essere probabilmente ridotto facilmente, ma credo che il punto è che s dovrebbe essere "più carino" e non "migliore" o "complicato". Nulla sarebbe battere il tuo originale in termini di prestazioni :-)

Answer 3

Sì, un ciclo for è un buon modo per semplificare il codice in questo caso. Qualcosa di simile:

bool result = true; 
    for(int j = 0; j < max; j++) 
    { 
     result = result && excel_getValue("A" + (i + j)) == ""; 
    } 

Answer 4

Si potrebbe provare qualcosa di simile:

if (Enumerable.Range(0, 8).All(p => excel_getValue("A" + (i + p)) == ""))