nalim
Новичок
Чем лечить людей которые пишут такие коды.
Постепенно я все больше и больше занимаюсь
командной разработкой.
Причем возрастает постоянно как количество
людей в команде, так и разных проектов,
где используются различные технологии и фреймфорки.
Так один из новых программистов недавно
на поставленную задачу доработки функционала сайта
выдал мне отлично оформленный класс
со всеми наворотами PHP5
почти без изъянов по архитектуре.... не считая того
что прямо внутри методов он использовал
echo
или прямой вывод HTML.
Но недавно я встретил нечто действительно поразительное.
Привожу пример кода, на мой взгляд антиестественного.
Поясните плиз:
откуда исторически могло взяться такое "изящное"
решение
и имеет ли оно право на существование?
Постепенно я все больше и больше занимаюсь
командной разработкой.
Причем возрастает постоянно как количество
людей в команде, так и разных проектов,
где используются различные технологии и фреймфорки.
Так один из новых программистов недавно
на поставленную задачу доработки функционала сайта
выдал мне отлично оформленный класс
со всеми наворотами PHP5
почти без изъянов по архитектуре.... не считая того
что прямо внутри методов он использовал
echo
или прямой вывод HTML.
Но недавно я встретил нечто действительно поразительное.
Привожу пример кода, на мой взгляд антиестественного.
Поясните плиз:
откуда исторически могло взяться такое "изящное"
решение
и имеет ли оно право на существование?
PHP:
function ins_p_ret($to_close = 2, $message = '', $params = "") {
switch ($to_close) {
case 0:
if ( !isset($params) || $params == "" ) {
$res = '<P>';
} else {
$res = '<P ' . $params . '>';
}
break;
case 1:
$res = '</P>';
break;
case 2:
if ( !isset($params) || $params == "" ) {
$res = '<P>';
} else {
$res = '<P ' . $params . '>';
}
$res .= $message . '</P>';
}
return $res;
}
function ins_href($to_close = 2, $href = '', $name = '', $params = "") {
if (constDirectLinks == 1) {
$href = ereg_replace('\index.php', '', $href);
$href = ereg_replace('\?', '', $href);
$href = ereg_replace('\=', '_', $href);
$href = ereg_replace('\&', '/', $href);
$href = '/'.$href.'/';
}
switch ($to_close) {
case 0:
if ( !isset($params) || $params == "" ) {
$res = '<a href="' . $href . '">';
} else {
$res = '<a href="' . $href . '" ' . $params . '>';
}
$res .= $name;
ins_e($res);
break;
case 1:
ins_e('</a>');
break;
default:
if ( !isset($params) || $params == "" ) {
$res = '<a href="' . $href . '">';
} else {
$res = '<a href="' . $href . '" ' . $params . '>';
}
$res .= $name . '</a>';
ins_e($res);
break;
}
}
