Bunu yazmanın daha iyi yolu?

Question

Bunu yazmanın daha iyi (yani daha okunaklı) bir yolu var mı?

if (isset($input_vars['directive']) && $input_vars['directive'] == 'edit') { 

Answer 1

Maalesef değil. Bu kodu bir işlevle sarın ve buna her ihtiyaç duyduğunuzda bunu çağırabilirsiniz.

function compareArrayItem($array, $key, $value) { 
    return isset($array[$key]) && $array[$key] == $value; 
} 

if (compareArrayItem($input_vars, 'directive', 'edit')) { 
    // Do something 
} 

Ancak bu bana göre bir anlam ifade etmiyor (ve orijinal kodunuzdan daha az okunabilir). Ya da hata raporlama seviyesini E_NOTICE içermeyecek şekilde azaltabilirsiniz, böylece ilk ifadeye ihtiyacınız olmaz.

error_reporting(E_ALL^E_NOTICE); 

if ($input_vars['directive'] == 'edit') //... 

Ancak bunu yalnızca kodunuzu kısaltmak için yapmanız önerilmez.

Senin yerinde olsaydım, sadece yalnız bırakırım. Olduğu gibi iyidir.

Answer 2

Aşağıdakiler her zaman aynı sonucu verir.

if($input_vars['directive'] == 'edit'){ 

Bunun nedeni, eğer ayarlanmamışsa, eğer 'edit' değilse, 'edit' ise, o zaman onun setidir.

Bu durum bir bildirim döndürür; ancak bu özelliği, PHP kurulumunuzdan bu özelliği devre dışı bırakabilirsiniz.

if(@$input_vars['directive']=='edit') { 

biraz daha okunabilir olur ve direktif yoksa o uyarıları üretmez:

Answer 3

derdim.

Answer 4

$ input_vars ['directive'] 'ı birden fazla değere karşı test edeceğinizi tahmin edeceğim (aksi halde, neden sadece $ input_vars [' edit ']' de saklanan basit bir boole sahip olmuyorsunuz veya benzer?). Aynı zamanda bu testleri bir diğerinden sonra yaptığınızı tahmin ediyorum (eğer 'X' düzenleme ',' display 'Y ise).

Böyle bir durumda, isset() testini bir if ifadesine koyun ve diğerlerini bunun içine yerleştirin (anahtar/durum akışı kötü bir seçim olmaz). $input_vars izin değerler kümesinin bilinen ve söz çekleri biryere olan ise şu yardımcı işlevi

Answer 5

yapacağız:

function item ($array, $key) { 
    if (isset ($array [$key])) 
     return $array [$key]; 
    else 
     return NULL; // Or use whatever is more appropriate 
}

Sonra orijinal kod

benzeyecek şekilde değiştirilebilir

if (item ($input_vars, 'directive') == 'edit') { ... 

Bu yalnızca daha okunabilir değil, aynı zamanda çoğaltma işlemini de kaldırır: her iki dizi değişkeni ve anahtarı yalnızca bir kez görünür.