Döngü tabanlı bir dizindeki ilk öğeyi atlamanın net yolu

Ben sıfır tabanlı bir dizi ilk öğeyi atlamak gerekir bir for döngüsü var.

Bunlardan hangisi niyetlerimi daha net gösteriyor?

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

veya

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

İkisinden de nefret ediyorum.

Sihirli sayılar kullanabileceğinizi kim söyledi? Eğer 1 ofsetinden başlayacaksanız, neden 1 ofsetinden başladığınızı bize anlatmaya ne dersiniz? Eşit derecede sihirli bir sıfır eklemek bana hiçbir şeyi açıklamaz.

Bu yük yükü dengesi mi? Bu bir boş sonlandırılmış c dizesine dönüştürdüğünüz bazı pascal dizesi mi? Lütfen bize neler olduğunu anlat.

Üzgünüm, ama kariyerimin büyük bir kısmını bunun gibi anlamsız gizemleri çözdüm ve onlara olan sabrım zayıfladı. Mı iyi adla bir değişken sormak için çok gerçekten?

İyi adla, ilk öğeyi neden atladığımızı açıklayan bir ad kastediyorum. Sadece ilk elementi atladığımızı söyleyen bir şey değil. 1 bana kendi başına olduğunu söyledi.

Kısa cevap: ilk seçenek daha iyidir.

İkinci seçenek sadece gürültü ekler. 0 + 1'in okuyucunun 0 olabileceğini anlamasına yardımcı olması çok düşük bir ihtimaldir, ancak 1'dir. Özellikle tüm dizilerin 0'dan başladığı bir dilde.

Daha önce de belirtildiği gibi, döngünün 0'dan değil 1'den başladığını vurgulamak istiyorsanız, sadece bir yorum ekleyin.

Bize ilk öğeyi atladığınızı söyleme - bunu görebiliriz. Açık olmayan şey neden . Yani .. bağlamdan belli değilse, bize nedenini söyleyin:

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

Veya yorumdan kaçınıyorsanız, şöyle bir şey:

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

Bize dilin nasıl çalıştığını hatırlatmak için yorum ve hile yapmayın .

Örneğiniz yapmacık görünüyor. Gerçek dünya kodunda, döngülerin ikinci dizi öğesinde başlaması gerektiği gerçeği büyük olasılıkla aşağıdaki kod satırlarında açıktır. Örneğin, gerçek kod şöyle görünürse

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

döngünün neden 0 yerine 1'de başladığını açıklığa kavuşturmak için hiçbir açıklama veya "0 + 1" yapısı gerekmez.

Ancak, döngü içindeki kod nedenleri bu kadar açık bir şekilde açıklamazsa (belki array[0]de özel bir anlamı vardır ve geri kalan öğelerden farklı bir şekilde ele alınması gerekir), açıklayıcı bir yorum ekleyin. Ancak bunu yapmadan önce, array[0]bu özel anlama sahip olmaktan kaçınabiliyorsanız iki kez düşünün ve muhtemelen daha iyi bir alternatif olan çevre kodunu yeniden düzenleyin.

Hiç seçenek # 2 görmedim, ama hoşuma gitti. Neden? Seçenek # 1 ile programlayıcının dizilerin 0'dan başlamasını unuttuğunu merak ediyorum. Seçenek # 2, kasıtlı olarak 1'den başladıklarını daha net hale getirir.

Bununla birlikte, her iki durumda da neden öğeyi atladığınıza yorum eklemek için en iyisi.

Veya neden birinden başladığınızı kolayca açıklayabiliyorsanız, bir sabit kullanın. Örneğin, komut satırı bağımsız değişkenlerine bakıyorsanız,

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

Şahsen, YMMV yerine muhtemelen bir yorum kullanırdım.

Birinin ilkinin kafasını karıştıracağından şüpheliyim. Hepimiz bunu yapmak zorundaydık. Öyle ki, ikincisinin kafasını karıştırması çok daha olası. "Neden orada 0+ var? + Operatörünü bir şekilde geçersiz kıldılar mı?"

İyi bir derleyici, ikincisini yine de ilkine çevirecektir, ancak yorumlanmış PHP kullanıyorsunuz gibi görünüyor. Tercüman bu döngüye her girdiğinde, aslında 0 ve 1 eklemesi gerekecek. Çok önemli değil, ama tercümanı neden işe yaratıyor?

Başlangıç ​​noktasını açıklayan bir değişken kullanın.

" Sıfır tabanlı bir dizideki ilk öğeyi atlamanız " gerekir , örneğin:

skipFirstElement = 1;

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

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

Eğer biri sıfırdan başlayan tüm döngülere takıntılıysa, bir devam ifadesi kullanabilirsiniz. Neden normalde atlamayacağınıza göre atladığınıza bir yorum ekleyin.

Yaptığım şey, döngüden önce ilk öğeyi kaldırmak. Gerekirse yeni bir dizi oluşturun. Bir yorumda neden bunu yaptığınızı açıklayın. Ve sonra basit bir foreach yapın.

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

Bu şekilde niyetiniz çok açık.

Daha fazla açıklığa kavuşturmak için, kodu işleri daha net hale getirmek için uygun bir adla bir yönteme sarabilirsiniz.

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

Ancak bunların hepsinin bağlamı hala eksik. Elementlerle ne yapmak istersiniz? Yeni diziyi iade edecek misiniz? Sizden sonraki orijinal ve yeni diziyi önemsiyor musunuz doStuff()?

Her neyse, burada net bir cevap yok ve kodun nasıl okunabilir olacağına karar vermek büyük ölçüde bağlama bağlı.