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

[AmdY]

Chushkin
не моё личное мнение, а аргументированное мнение.

switch($_GET['module']) {
 case 'users':
 case 'info':
 case 'faq':
  // ещё много кейсов
  $module = $_GET['module'];
  break;
 default:
  $module = 'index';
}

гораздо читабельнее, короче и человекопонятнее длинной простыни из 30 кейсов.

if (empty($allowedModule[$module])) {
    $module = 'index';
}
if (!isset($allowedModule[$module]) || $allowedModule[$module] != ALLOW_FOR_ADMIN) { // изменив одну строку ввели больший уровень разграничения доступа
    $module = 'index';
}

Если хозяевам этого форума нравится медвеводство, то флаг им в руки - форумов много и уйти на другой не проблема.

Ты абсолютно бесполезный участник форума, обладающий низкой квалификацией и низкой обучаемостью. Ничего полезного на форуме ты не пишешь, твои советы вредны. Ты флеймишь в разделе работа приукрашивая свой уровень и вводишь в заблуждение работодателей. Нет ничего удивительного, что подобные дела пресекались и будут присекаться модератором.

 

[AmdY]

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

 

[Redjik]

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

 

[Вурдалак]

Chushkin, ты в курсе что такое расширяемость кода?

 

[флоппик]

Тема закрыта.

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