Исходный код
if ((arg(0) == 'node' and arg(1) != 11) or arg(0) != 'node') {
Как можно улучшить исходный код
- По правилам булевой логики выражение можно упростить:
arg(0) != 'node' or arg(1) != 11
. - Очень полезно добавить поясняющие переменные в этот код, потому что непосредственно из нынешней записи выражения не понятно, что тут происходит.
$currentItem = arg(0);
$itemValue = arg(1);
$isNode = $currentItem == 'node';
$isOpen = $itemValue != '11';
if (!$isNode or $isOpen) {
Теория
- Рефакторинг «Введение поясняющей переменной»