Care este modul corect de a converti de la o buclă la o buclă în timp ce?

Redarea cu valoarea indicelui în timp ce într - o structură de buclare este predispus la probleme, indiferent de structura looping este .

Nu va conta dacă este o buclă sau o buclă în timp ce, punctul este va indexatorul în cele din urmă să vă conducă la a lua o decizie de reziliere bucla?

Dacă sunteți sigur că sunteți indexor va provoca în cele din urmă starea de ieșire să fie atins, atunci este tot ceea ce ar trebui să fie preocupat, să nu fie de a utiliza un timp sau un timp.

Cel mai simplu mod de a face acest lucru ar fi să nu se transforme într-o buclă în timp, cum ar fi de mai jos.

for (int i = from; i < to; ) {
  // do some code (I don't know exactly what, it is subject to change)
  i += rand()*10;
}

Mi se pare că poate fi mai ușor și mai ușor de citit pentru al converti la:

while (condition == true) {
   // do stuff
   // set the condition flag appropriately
}

și, astfel, se separă terminarea buclei de incrementarea variabilă.

Dacă văd o buclă cu o verificare de limită ( de exemplu i < limit) , aș tinde să se presupună că există o variabilă care este modificată într - un (rezonabil) mod coerent. Nu există nici un motiv pentru care nu se poate face ceea ce vrei, dar mi - ar inclina spre un comportament mai ușor de citit și mai mult de așteptat.

Pentru a răspunde la întrebarea cu privire la care codul aș alege; Am alege codul de mai mult timp. Este mult mai ușor de citit prima bucla (mai mult). Și da, eu pot citi de-al doilea, dar chiar dacă aveți o mulțime de experiență trebuie să se uite de două ori să știe ce face asta bucla. Plus compilatorul va optimiza codul suficient de bine.

De ce deranjez cu bucle stupide , atunci când se poate face același lucru (și mult mai mult!) Cu uber-puternic goto?

i = fro;
my_loop:
//all your stuff here
i++;
if (i < to) goto my_loop;

Dacă sunteți unul dintre acei programatori cu inima slabe care diminuează goto, atunci puteți încerca cu acest lucru:

i = fro;
while(1) {
    //your stuff here
    if (++i < to) break;
}