Программируем на CakePHP

Я конечно понимаю что язык PHP простой и в нем многое допустимо. Понимаю, что на свете куча новичков, которые еще только учатся программировать. Вполне осознаю также что я и сам не спец. Но всегда когда я встречаю в коде ТАКОЕ меня коробит. А, блин, попадается с пугающей регулярностью.

Цитата из кода самописной панели администрирования сайта одного очень крупного завода Московской области:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19

if ($month=='1'){
	$month='январь';
}
if ($month=='2'){
	$month='февраль';
}
if ($month=='3'){
	$month = 'март';
}
if ($month=='4'){
	$month='апрель';
}
if ($month=='5'){
	$month='май';
}
if ($month=='6'){
	$month='июнь';
}
//далее пропущено, ибо тоже самое...

Как нетрудно догадаться, это часть кода перевода номера месяца в его название. Нет слов. Автор не осилил даже конструкцию else if, я уж не говорю вынести это диво в функцию — оно у него там откопировано в каждый файл админки.

Дорогие горе-программисты, чудо-администраторы и прочие самодеятели!

Перевести номер месяца в его название можно вообще без всяких условий, когда ж наконец соображать начнете? Если номер месяца у вас в переменной $month, и в ней же надо получить название месяца, то пишите хотя бы так:

1
2

$months = array('январь', 'февраль', 'март', 'апрель', 'май', 'июнь', 'июль', 'август', 'сентябрь', 'октябрь', 'ноябрь', 'декабрь');
$month = $months[$month-1];

Чувствуете разницу?

Теги: php, оптимизация, ошибка, юмор