Prosty sposób na pominięcie pierwszego elementu w indeksie opartym na pętli

Mam pętlę for, w której muszę pominąć pierwszy element w tablicy zerowej.

Który z nich pokazuje moje zamiary jaśniej?

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

lub

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

Nienawidzę obu.

Kto powiedział, że możesz użyć magicznych liczb? Jeśli zamierzasz zacząć od przesunięcia 1, powiedz nam, dlaczego zaczynasz od przesunięcia 1. Dodanie równie magicznego zera nic mi nie wyjaśnia.

Czy to jest przesunięcie ładunku? Czy to jakiś ciąg pascal, który konwertujesz na zakończony zerem łańcuch c? Powiedz nam, co się dzieje.

Przepraszam, ale zmarnowałem wiele z mojej kariery, dekodując takie bezsensowne tajemnice, a moja cierpliwość do nich osłabła. Czy zmienna o przyzwoitej nazwie jest naprawdę zbyt wymagająca?

Przez dobre imię rozumiem imię, które wyjaśnia, DLACZEGO pomijamy pierwszy element. Nie jest to coś, co po prostu mówi, że pomijamy pierwszy element. 1 powiedział mi to samo.

Krótka odpowiedź: pierwsza opcja jest lepsza.

Druga opcja po prostu dodaje hałas. Jest bardzo mało prawdopodobne, że 0 + 1 pomaga czytelnikowi zrozumieć, że mogło to być 0, ale jest to 1. O wiele bardziej prawdopodobne, że zostanie zaintrygowany na chwilę i odciągnie uwagę od tego, o co chodzi w pętli. Zwłaszcza w języku, w którym wszystkie tablice zaczynają się od 0.

Jak już wspomniano, jeśli chcesz podkreślić fakt, że pętla zaczyna się od 1, a nie 0, po prostu dodaj komentarz.

Nie mów nam, że pomijasz pierwszy przedmiot - możemy to zobaczyć. Nie jest oczywiste, dlaczego . Więc .. jeśli nie jest to oczywiste z kontekstu, powiedz nam dlaczego:

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

Lub, jeśli masz awersję do komentarzy, coś takiego:

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

Nie używaj komentarzy i podstępów, aby przypominać nam, jak działa ten język.

Twój przykład wygląda na wymyślony. W prawdziwym kodzie fakt, że pętle muszą zaczynać się od drugiego elementu tablicy, najprawdopodobniej wynika z następujących linii kodu. Na przykład, jeśli prawdziwy kod wygląda tak

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

nie byłoby żadnego wyjaśnienia ani konstrukcji „0 + 1”, aby wyjaśnić, dlaczego pętla zaczyna się od 1 zamiast 0.

Jeśli jednak kod wewnątrz pętli nie wyjaśnia przyczyn w tak oczywisty sposób (być może array[0]ma specjalne znaczenie i musi być traktowany inaczej niż pozostałe elementy), dodaj komentarz wyjaśniający. Ale zanim to zrobisz, zastanów się dwa razy, czy możesz uniknąć array[0]tego specjalnego znaczenia, i zreorganizuj otaczający kod, co prawdopodobnie byłaby lepszą alternatywą.

Nigdy nie widziałem opcji # 2, ale podoba mi się. Dlaczego? Z opcją nr 1 zastanawiałbym się, czy programista zapomniał, że tablice zaczynają się od 0. Opcja nr 2 wyjaśnia, że ​​celowo zaczynają od 1.

To powiedziawszy, najlepiej w obu przypadkach dodać komentarz, dlaczego pomijasz element.

Lub, jeśli możesz łatwo opisać, dlaczego zaczynasz od jednego, użyj stałej. Na przykład, patrząc na argumenty wiersza poleceń, coś podobnego

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

Osobiście prawdopodobnie po prostu użyłbym komentarza YMMV.

Wątpię, czy ktokolwiek byłby zdezorientowany tym pierwszym. Wszyscy musieliśmy to zrobić. Do tego stopnia, że ​​drugi jest znacznie bardziej zagmatwany. „Dlaczego jest tam 0+? Czy w jakiś sposób zastąpiły one operatora +?”

Przyzwoity kompilator i tak zamieni drugi na pierwszy, ale wygląda na to, że używasz PHP, które jest interpretowane. Więc za każdym razem, gdy interpreter natrafi na tę pętlę, będzie musiał dodać 0 i 1. Nie jest to wielka sprawa, ale po co zmuszać tłumacza do pracy?

Użyj zmiennej, która wyjaśnia punkt początkowy.

Musisz „ pominąć pierwszy element w tablicy zerowej ”, na przykład:

skipFirstElement = 1;

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

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

Jeśli ktoś ma obsesję na punkcie wszystkich pętli zaczynających się od zera, można użyć instrukcji kontynuacji. Dodaj komentarz, dlaczego pomijasz, ponieważ normalnie nie.

Chciałbym usunąć pierwszy element przed zapętleniem. Utwórz nową tablicę, jeśli potrzebujesz. Wyjaśnij w komentarzu, dlaczego to robisz. A potem po prostu zrób prosty foreach.

$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
}

W ten sposób twoje zamiary są całkowicie jasne.

W celu dalszego wyjaśnienia można zawinąć kod w metodę o odpowiedniej nazwie, aby wszystko było bardziej zrozumiałe.

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

Jednak w tym wszystkim wciąż brakuje kontekstu. Co chcesz zrobić z elementami? Czy zwrócisz nową tablicę? Czy zależy Ci na oryginale i nowej tablicy po tobie doStuff()?

W każdym razie nie ma tutaj jasnej odpowiedzi, a decyzja, jak uczynić kod czytelnym, zależy w dużej mierze od kontekstu.