{% else-1 %}
(22 фев 2015, 11:15) (0/0) [0]
donttime, Отсутствует комментирование.
Многие разработчики не могут написать даже самое простое, что есть в любом языке программирования - комментарий.
Подавляющее количество кода, который я когда-либо видел, содержит незначительное количество комментариев. Для того чтобы написать хороший комментарий уходит достаточно времени, но оно окупается когда тебе необходимо что-то изменить в функционале. Так же в случае рефакторинга, отсутствие описания кода может привести к его недееспособности.

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

Взгляни на этот кусок кода ещё раз, но только с комментарием:
<?php
#Получаем расширение изображения
$pieces = explode('.', $image_name);
$extension = array_pop($pieces);
?>
Теперь для того чтобы понять, что делает этот код с первого взгляда не надо сильно напрягать мозги: у тебя перед глазами комментарий, код, и нет никаких вопросов.
(22 фев 2015, 10:52) (0/0) [0]
bibilink, Может сразу свой язык программирования изобрести * Код написан понятно и разбирать там ничего не нужно.
(22 фев 2015, 10:51) (0/0) [0]
donttime, Тьфу блин...это мой косяк. А ошибка всех заключается в другом. Попробуй найти.
(22 фев 2015, 10:23) (0/0) [0]
<?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 из 1