if || or === 12строк||2строки :что выбрать?

Статус
В этой теме нельзя размещать новые ответы.

AmdY

Пью пиво
Команда форума
Chushkin
не моё личное мнение, а аргументированное мнение.
PHP:
switch($_GET['module']) {
 case 'users':
 case 'info':
 case 'faq':
  // ещё много кейсов
  $module = $_GET['module'];
  break;
 default:
  $module = 'index';
}
гораздо читабельнее, короче и человекопонятнее длинной простыни из 30 кейсов.
PHP:
if (empty($allowedModule[$module])) {
    $module = 'index';
}
if (!isset($allowedModule[$module]) || $allowedModule[$module] != ALLOW_FOR_ADMIN) { // изменив одну строку ввели больший уровень разграничения доступа
    $module = 'index';
}
Если хозяевам этого форума нравится медвеводство, то флаг им в руки - форумов много и уйти на другой не проблема.
Ты абсолютно бесполезный участник форума, обладающий низкой квалификацией и низкой обучаемостью. Ничего полезного на форуме ты не пишешь, твои советы вредны. Ты флеймишь в разделе работа приукрашивая свой уровень и вводишь в заблуждение работодателей. Нет ничего удивительного, что подобные дела пресекались и будут присекаться модератором.
 

AmdY

Пью пиво
Команда форума
Chushkin
вот именно, мы дополнительно отделили мух от котлет, формирование самого масива это отдельная задача, которая может быть как статический файл, так и динамическое формирование на основе базы данных или с учётом прав пользователя. за счёт этого читабельность повысилась ещё на порядок.
 

Redjik

Джедай-мастер
Chuskin === cabalero?'Ничего удивительного':'Как много похожих людей, со схожими мыслями'
 

флоппик

promotor fidei
Команда форума
Партнер клуба
Тема закрыта.

Проблемы личного характера и бессмысленные споры между участниками не являются предметом обсуждения форума.
Обсуждайте их в привате.
 
Статус
В этой теме нельзя размещать новые ответы.
Сверху