php

ben bazı eski kod (başkasının yazdığı) serinletici ve ben de deyim bu rastladı Kafa karıştırıcı eğerphp

if ($role_data["role_id"] != 1)

IMO okunabilirliği daha kötü ve daha fazla kod. Performans burada bir faktör değil. Beklenen girişi ($ role_data [ "ROLE_ID"]) arasında bir sayı olduğunu

--EDIT--

belirtmek gerekir, 0-5 (dahil)

--MORE INFO--

Kodu ilk defa anımsadım.

$role_id = htmlspecialchars (mysql_real_escape_string ($_GET["role_id"])); 
$role_data = $db->fctSelectData ("core_role" , "`role_id` = '" . $role_id . "'");

Bu rolü oluşturma izinleri almak için gider: Ama burada olup bitenler. Hala bu şekilde yazıldığı neden tam net değilim

if (empty ($role_data["role_id" == 1]))

: ($ _GET parametre üzerinden) ilk etapta $role_id geçersiz verilirse Ancak, dolayısıyla boş bir değerini işaretleyerek, hiçbir şey döndürür

kaynak

2015-07-07 Marius Schär

'if (boş ($ role_data [" role_id "== 1]))'! == 'ise ($ role_data [" role_id "]! = 1)'. Eski kodunuz: 'if (boş ($ role_data [" role_id "== 1]))' -> 'if (boş ($ role_data [0]))' (Çünkü "role_id" == 1' = FALSE ve tür hokkabazlık nedeniyle = 0) – Rizier123

@ Rizier123'in her ikisi de farklıdır. –

Her ikisi de sefil başarısız olurken, sevimli ve gösteriş girişiminde biri. Bu veya toplam kargo-kült programlama –

if (empty($role_data["role_id" == 1]))

çevrilmiş Gets the hattı ...

if (empty($role_data[0]))

... PHP yorumlayıcısı tarafından. mantıklı bir

if (empty($role_data["role_id"]) == 1)

..sort: Bu kod, bir "şaka", komik kesmek veya çizgi şeklinde bir hata olabilir.

kaynak

2015-07-08 10:58:05 OptimusCrime

cevap

İlgili konular