Modo chiaro per saltare il primo elemento in un indice basato su loop

Ho un ciclo for in cui devo saltare il primo elemento in un array a base zero.

Quale di questi mostra le mie intenzioni più chiaramente?

for($i=1 ; $i < count(array) ; $i++){
    array[$i];
}

o

for($i=0+1 ; $i < count(array) ; $i++){
    array[$i];
}

Odio entrambi.

Chi ha detto che potresti usare numeri magici? Se inizi con un offset di 1, che ne dici di dirci PERCHÉ stai iniziando con un offset di 1. Aggiungere uno zero ugualmente magico non mi spiega nulla.

È questo l'offset del payload? È una stringa pasquale che stai convertendo in una stringa c terminata con null? Per favore, dicci cosa sta succedendo.

Mi dispiace, ma ho sprecato gran parte della mia carriera decodificando misteri inutili come questo e la mia pazienza per loro si è esaurita. È una variabile con un nome decente davvero molto da chiedere?

Per nome decente intendo un nome che spiega PERCHÉ stiamo saltando il primo elemento. Non qualcosa che dice semplicemente CHE stiamo saltando il primo elemento. L'1 me lo ha detto da solo.

Risposta breve: la prima opzione è migliore.

La seconda opzione aggiunge solo rumore. È molto improbabile che 0 + 1 aiuti il ​​lettore a capire che avrebbe potuto essere 0 ma è 1. Molto più probabilmente sarà perplesso per un breve momento e distratto da ciò che riguarda il loop. Soprattutto in una lingua in cui tutte le matrici iniziano da 0.

Come altri menzionato, se si desidera sottolineare il fatto che il ciclo inizia da 1, non da 0, è sufficiente aggiungere un commento.

Non dirci che stai saltando il primo oggetto: possiamo vederlo. Ciò che non è ovvio è il perché . Quindi .. se non è ovvio dal contesto, dicci perché:

// array[0] is just a header
for($i=1 ; $i < count(array) ; $i++){
    array[$i];
}

Oppure, se sei contrario ai commenti, qualcosa del tipo:

$lastHeaderIndex = 0;
for($i = $lastHeaderIndex + 1 ; $i < count(array) ; $i++){
    array[$i];
}

Non usare commenti e trucchi per ricordarci come funziona la lingua.

Il tuo esempio sembra inventato. Nel codice del mondo reale, il fatto che i loop debbano iniziare dal secondo elemento dell'array è molto probabilmente evidente dalle seguenti righe di codice. Ad esempio, se il codice reale appare così

for($i=1 ; $i < count(array) ; $i++){
    array[$i-1]=array[$i];
}

non ci sarebbe alcuna spiegazione o costrutto "0 + 1" necessario per chiarire perché il ciclo inizia da 1 anziché 0.

Tuttavia, se il codice all'interno del ciclo non spiega i motivi in ​​modo così ovvio (forse array[0]ha un significato speciale e deve essere trattato in modo diverso rispetto agli elementi rimanenti), quindi aggiungere un commento esplicativo. Ma prima di farlo, pensa due volte se puoi evitare di avere array[0]questo significato speciale e riorganizza il codice circostante, che probabilmente sarebbe la migliore alternativa.

Non ho mai visto l'opzione n. 2, ma mi piace. Perché? Con l'opzione n. 1 mi chiedo se il programmatore abbia dimenticato che le matrici iniziano da 0. L'opzione n. 2 chiarisce che stanno deliberatamente iniziando da 1.

Detto questo, meglio in entrambi i casi aggiungere un commento sul motivo per cui stai saltando l'elemento.

Oppure, se riesci a descrivere facilmente perché stai iniziando da uno, usa una costante. Ad esempio, se si esaminano gli argomenti della riga di comando, qualcosa del genere

define ('FIRST_REAL_ARGUMENT', 1);
for ($i=FIRST_REAL_ARGUMENT; ...)

Personalmente, probabilmente userei solo un commento, YMMV.

Dubito che qualcuno sarebbe confuso dal primo. Abbiamo dovuto farlo tutti. Tanto che è molto più probabile che il secondo confonda. "Perché c'è uno 0+ lì? Hanno in qualche modo ignorato l'operatore +?"

Un compilatore decente trasformerà comunque il secondo nel primo, ma sembra che tu stia usando PHP, che viene interpretato. Quindi ogni volta che l'interprete raggiunge quel loop, dovrà effettivamente aggiungere 0 e 1. Non è un grosso problema, ma perché fare in modo che l'interprete faccia il lavoro?

Utilizzare una variabile che spiega il punto iniziale.

Devi " saltare il primo elemento in un array a base zero ", quindi ad esempio:

skipFirstElement = 1;

for($i=$skipFirstElement ; $i < count(array) ; $i++){
    array[$i];
}

//We are skipping the first element because...    
if ($i==0)  
{    continue;      } 

Se uno è ossessivo con tutti i loop che iniziano da zero, è possibile utilizzare un'istruzione continue. Aggiungi un commento al motivo per cui stai saltando poiché normalmente non lo faresti.

Quello che farei è rimuovere il primo elemento prima di eseguire il looping. Crea un nuovo array, se necessario. Spiega in un commento perché lo stai facendo. E poi fai una semplice ricerca.

$arrayCopy = $array; // in case you don't want to touch the original array
array_shift($arrayCopy); // removing first element because of X reason.
foreach($arrayCopy => $element) { 
    // do stuff
}

In questo modo il tuo intento è perfettamente chiaro.

Per chiarire ulteriormente è possibile racchiudere il codice in un metodo con un nome appropriato per rendere le cose più chiare.

function doStuffToAllButTheFirst($array) { // this copies the original array, so there are no sideffects
    array_shift($array);
    foreach($array => $element) { // do stuff }  
}

Tuttavia tutto ciò manca ancora del contesto. Cosa vuoi fare con gli elementi? Restituirai il nuovo array? Ti interessa l'originale e il nuovo array dopo di te doStuff()?

Comunque, non c'è una risposta chiara qui, e decidere come rendere il codice leggibile dipende molto dal contesto.