Смена графики в зависимости от времени года

<?php
function current_season() {
$icons = array(
"spring" => "images/spring.png",
"summer" => "images/summer.png",
"autumn" => "images/autumn.png",
"winter" => "images/winter.png"
);

$day = date("z";

$spring_starts = date("z", strtotime("March 21");

$spring_ends = date("z", strtotime("June 20");

$summer_starts = date("z", strtotime("June 21");

$summer_ends = date("z", strtotime("September 22");

$autumn_starts = date("z", strtotime("September 23");

$autumn_ends = date("z", strtotime("December 20");

if($day >= $spring_starts && $day <= $spring_ends ):
$season = "spring";
elseif($day >= $summer_starts && $day <= $summer_ends):
$season = "summer";
elseif($day >= $autumn_starts
&& $day <= $autumn_ends):
$season = "autumn";
else
$season = "winter";
endif;
$image_path = $icons[$season];

echo $image_path;

}
?>

Может кому пригодится. При написание кода я допустил ошибку, которую делает каждый из вас, кто её найдет, получит небольшой подарок на WebMoney

Изм. 1 раз. / Посл. изм.

Денис Павлик 11.03

(22 фев 2015, 10:54)

if($day >= $spring_starts && $day <= $spring_ends )

в конце строки отсутствует двоеточие
Добавлено 22.02.15 в 10:44:19:
да и вообще они там в каждом else отсутствуют.

Да проще свое написать,чем чужое разбирать)

Тьфу блин...это мой косяк. А ошибка всех заключается в другом. Попробуй найти.

Может сразу свой язык программирования изобрести Код написан понятно и разбирать там ничего не нужно.

зачем писать каждый раз date("z" , если и так ясно что $day = date("z";

Денис Павлик, кроме этого и непонятных дат, точнее чисел ничего нет. синтаксических ошибок нету, я бы сразу заметил.

Bastian,

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

Продемонстрирую идею на примере:
<?php
$pieces = explode('.', $image_name);
$extension = array_pop($pieces);
?>
И что это такое? Что хранится в переменной $extension?

Взгляни на этот кусок кода ещё раз, но только с комментарием:
<?php
#Получаем расширение изображения
$pieces = explode('.', $image_name);
$extension = array_pop($pieces);
?>
Теперь для того чтобы понять, что делает этот код с первого взгляда не надо сильно напрягать мозги: у тебя перед глазами комментарий, код, и нет никаких вопросов.