Лень в 14 строчек кода

4 Апр
2012

Давно я не встречал достойного говнокода!
Есть некий класс, у него статичный метод который возвращает список месяцев в году. Да, простой массив в 12 элементов со списком названий месяцев.
Вот такой кусок реализации был найден в коде

function month_list () {
	$data = array();
	$ts = mktime(0, 0, 0, 1, 1, 2009);
	for ($i = 1; $i <= 12; $i++) {
		$data[date('n', $ts)] = date('F', $ts);
		$ts = $ts + 31*24*60*60;
	}
	return $data;
}

 
Что именно употреблял программист до коли неизвестно, но мне кажется что программисту казалось что 7 строчек, тела функции, выглядит круче чем 14. Ибо проще, быстрее, нагляднее сделать так

function month_list () {
	$data = array();
	$data[1] = 'January';
	$data[2] = 'February';
	$data[3] = 'March';
	$data[4] = 'April';
	$data[5] = 'May';
	$data[6] = 'June';
	$data[7] = 'July';
	$data[8] = 'August';
	$data[9] = 'September';
	$data[10] = 'October';
	$data[11] = 'November';
	$data[12] = 'December';
	return $data;
}

Либо он просто ленился написать лишние 7 строчек кода.
Не ленитесь, блеать!

2 комментария к записи Лень в 14 строчек кода

Avatar

Adel Shigabutdinov

12th Апрель 2012 в 06:48

Вот тут я б с тобой не согласился 🙂
1. Глядя на нижний листинг за секунду становится понятно что этот код делает
2. Верхний код стоит 62 операции, нижний код — 13 операций
3. тогда уж так:
function month_list($months=array())
{
for ($i=1; $i<=12; $i++)
$months[] = date("F", mktime(0, 0, 0, $i, 1, 98));

return $months;
}

Avatar

Сергей

12th Апрель 2012 в 09:10

Адель, как бэ верхний листинг — кусок кода который был найден на просторах нашего проекта 🙂
Второй листинг — как бы я его написал 🙂
Обновил пост, на всякий случай)

Оставить комментарий или два

Наверх
Rambler's Top100