EmpireCMS - Сверх гибкий и самый быстрый CMS движок

А сам-то, что, кроме этой китайской хрени ничего освоить так и не смог, сколько ни пытался? Сочувствую.

Какие фишки, создание полей? :D Такие фишки почти у каждого движка есть) У того же битрикса это есть. У ExpressionEngine есть.

- EmpireCMS - Сверх гибкий и самый быстрый CMS движок

- а чем он лучше других?

- не лезьте в эту тему, я просто верю!

(с) Религия, она такая - да

И стало интересно мне, что это за цмс такая.

и нашел я цмс, и открыл ее... и увидел код...

	$newstempstr=str_replace('[!--id--]',$add[id],$newstempstr);
	$newstempstr=str_replace('[!--classid--]',$add[classid],$newstempstr);
	$newstempstr=str_replace('[!--class.name--]',$class_r[$add[classid]][classname],$newstempstr);
	$newstempstr=str_replace('[!--ttid--]',$add[ttid],$newstempstr);
	$newstempstr=str_replace('[!--tt.name--]',$class_tr[$add[ttid]][tname],$newstempstr);
	$newstempstr=str_replace('[!--onclick--]',$add[onclick],$newstempstr);
	$newstempstr=str_replace('[!--userfen--]',$add[userfen],$newstempstr);
	$newstempstr=str_replace('[!--username--]',$add[username],$newstempstr);

И ахренел я от увиденного и закрыл код и больше его никогда не открою

... всмирная исторя, банк "Империал"

там везде такой говнокод. Точнее это одно из самых пряморуких решений

нет, ну чесслово, что мешало сделать вот так?

$search = array(
		'[!--id--]',
 		'[!--classid--]',
		'[!--class.name--]',
		'[!--ttid--]',
		'[!--tt.name--]',
		'[!--onclick--]',
		'[!--userfen--]',
 		'[!--username--]'
		);
$replace = array(
		$add[id],
		$add[classid],
		$class_r[$add[classid]][classname],
		$add[ttid],
		$class_tr[$add[ttid]][tname],
		$add[onclick],
		$add[userfen],
		$add[username]
		);
$newstempstr=str_replace($search,$replace,$newstempstr);

или как вам 20 штук if-elseif с одним условием.

когда там просто switch - case должен быть без вариантов

какaя mvc/hmvc архитектура?!... забудьте. Это всё придумали трусы, которые боятся копаться в коде.

Объектное программирование? а что это?!

в общем. точно как в демотиваторе, только Индия заменить на китай

Ничё ты не понимаешь. Это же для простоты понимая кода разработчиками :)

Честно говоря, в вашем варианте проще сделать неочевидную ошибку (нарушение строгого порядка следования сложно заметить), также сложнее отлаживать. Хотя ваш код конечно красивше.

Между if-elseif и switch разница минимальна, в питоне вообще switch нет реализуется через elif.

Я не знаю, какого возраста этот проект. Если древний, то это объяснимо.

В любом проекте говнокод можно найти. Вордпрессу уже сколько лет, а до сих пор от него избавиться не может.

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

Впрочем, быть может нам, как европейцам с очевидно отличным менталитетом, не понять китайской школы написания софта.

окай, а вот такой вариант не проще для восприятия?

$replace_to = array(
	'[!--id--]' 		=> $add[id],
 	'[!--classid--]' 	=> $add[classid],
	'[!--class.name--]'	=> $class_r[$add[classid]][classname],
	'[!--ttid--]'		=> $add[ttid],
	'[!--tt.name--]'	=> $class_tr[$add[ttid]][tname],
	'[!--onclick--]'	=> $add[onclick],
	'[!--userfen--]'	=> $add[userfen],
 	'[!--username--]' 	=> $add[username]
	);
		
$newstempstr=str_replace(array_keys($replace_to),array_values($replace_to),$newstempstr);

где-то далеко заплакал милторг

Да, так лучше (надежнее), но с тем, что в оригинале, разница небольшая. Т.е. шило на мыло. Если это нужно для 5-10 строк. Если бы было много преобразований, то ваш вариант был бы однозначно лучше.

borisd, там в оригинале под сотню строк подряд. я просто не стал постить всю телегу