Встретил сегодня такую проверку на наличие не пустого массива:
|
1 |
if (!isset($inventoryData[0])) { return null;} |
Это можно оптимизировать вот так:
|
1 |
if (!count($inventoryData)) { return null;} |
Или вот так
|
1 |
if (!$inventoryData) { return null;} |
элегантно и просто !
…
Знаете ли вы что оператор break в конструкции switch case не обязателен ?
Вот такой вот код:
|
1 2 3 4 5 6 7 8 9 10 11 12 13 |
switch($req = $_REQUEST['controller']) { case 'users': $this->$req(); break; case 'statistics': $this->$req(); break; case 'settings': $this->$req(); break; case 'content': $this->$req(); break; case 'lists': $this->$req(); break; } |
можно оптимизировать вот таким образом:
|
1 2 3 4 5 6 7 8 9 10 |
switch($req = $_REQUEST['controller']) { case 'users': case 'statistics': case 'settings': case 'content': case 'lists': $this->$req(); break; } |
условия будут по очереди проверяться как если бы вы сделали
|
1 2 3 |
if ($req == 'users' || $req == 'statistics' || ...) { ... } |
Таким образом нам не обязательно на разные условия с одним действием,
…
Свежие комментарии