0

Хочу больше интерактивности на свой проектик.
Добавил это:

function season()
{

$season;

$curdate = date("d.m.Y", strtotime("now"));

$summer_start = "01.06.".date("Y"); // 01 июня текущего года
$summer_end = "31.08.".date("Y"); // 31 августа текущего года

$spring_start = "01.03.".date("Y"); // 01 марта текущего года
$spring_end = "31.05.".date("Y"); // 31 мая текущего года

$autumn_start = "01.09.".date("Y"); // 01 сентября текущего года
$autumn_end = "30.11.".date("Y"); // 30 ноября текущего года

$winter_start = "01.12.".date("Y"); // 01 декабря текущего года
$winter_end = "28.02.".date("Y"); // 28 февраля текущего года

if (strtotime($curdate) < strtotime($winter_start) and strtotime($curdate) >  strtotime($winter_end))

    {
        $season = $season;
    }
else
    {
        $season = "- зима! Низкие температуры! Низкие цены!";
    }

if (strtotime($curdate) > strtotime($spring_start) and strtotime($curdate) <  strtotime($spring_end))

    {
        $season = "весна - цены тают!";
    }
else
    {
        $season = $season;
    }


if (strtotime($curdate) > strtotime($summer_start) and strtotime($curdate) <  strtotime($summer_end))

    {
        $season = "лето - цены прячутся в тень!";
    }
else
    {
        $season = $season;
    }

if (strtotime($curdate) > strtotime($autumn_start) and strtotime($curdate) <  strtotime($autumn_end))
    {
        $season = "осень и у нас осенний ценопад!";
    }
else
    {
        $season = $season;
    }

print "<b>".$season."</b>";
}

Но оно ничего не выводит. Вывод делаю season(). Что в коде не так?

6
  • 1
    Может вы еще расскажете что хотелось бы выводить?
    – u_mulder
    Commented 1 сент. 2015 в 18:31
  • to: u-mulder А из текста после кода это разве не понятно?
    – I_CaR
    Commented 1 сент. 2015 в 18:59
  • 1
    А зачем вы вообще проверяете день и год? Может достаточно проверить месяц или не, это слишком просто?
    – u_mulder
    Commented 1 сент. 2015 в 19:13
  • Ну... тут вы правы (на счёт месяца), просто движок скрипта брас со старого прожекта, там ловлю дату Дня победы и НГ.--- А код исправил. Подставил <= и >= всё заработало. Указывал просто первое число, вот и не показывало. Попробую сейчас даты откинуть - спасибо за идею!
    – I_CaR
    Commented 1 сент. 2015 в 19:45
  • 2
    $season = $season; это простите что вообще такое? Commented 2 сент. 2015 в 6:33

3 ответа 3

2

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

$curdate = date("d.m.Y", time());

$summer_start = "01.06.".date("Y"); // 01 июня текущего года
$summer_end = "31.08.".date("Y"); // 31 августа текущего года

$spring_start = "01.03.".date("Y"); // 01 марта текущего года
$spring_end = "31.05.".date("Y"); // 31 мая текущего года

$autumn_start = "01.09.".date("Y"); // 01 сентября текущего года
$autumn_end = "30.11.".date("Y"); // 30 ноября текущего года

$winter_start = "01.12.".date("Y"); // 01 декабря текущего года
$winter_end = "28.02.".date("Y"); // 28 февраля текущего года

if (strtotime($curdate) >= strtotime($winter_start) and strtotime($curdate) <=  strtotime($winter_end))
    {
        $season = "- зима! Низкие температуры! Низкие цены!";
    }

if (strtotime($curdate) >= strtotime($spring_start) and strtotime($curdate) <=  strtotime($spring_end))

    {
        $season = "весна - цены тают!";
    }

if (strtotime($curdate) >= strtotime($summer_start) and strtotime($curdate) <=  strtotime($summer_end))
    {
        $season = "лето - цены прячутся в тень!";
    }

if (strtotime($curdate) >= strtotime($autumn_start) and strtotime($curdate) <=  strtotime($autumn_end))
    {
        $season = "осень и у нас осенний ценопад!";
    }
echo $season ;
8
  • C Зимой запутался!!! Там период (НГ - переход года меня с мыслей сбил)
    – I_CaR
    Commented 1 сент. 2015 в 19:02
  • НЕТ!!!! Зима по прежнему не работает! В ней ведь переход года.
    – I_CaR
    Commented 1 сент. 2015 в 19:09
  • ушел от компа :( ставьте везде elseif, зиму оставьте последней - что останется :)
    – splash58
    Commented 1 сент. 2015 в 19:13
  • Всё заработало с моим кодом! Просто не указал "<=" и ">=" И ставил 1 число когда проверял, поэтому-то и не выводило! Спасибо! Ваш ответ дал толчок к повторному копанию в коде и ошибка(недочёт) был найден!
    – I_CaR
    Commented 1 сент. 2015 в 19:43
  • В високосный год удивитесь..
    – Sergiks
    Commented 1 сент. 2015 в 19:48
2

При желании весь скрипт можно упростить:

$result = floor(date('n') / 3);
if ($result == 1) {
    // это весна
} elseif ($result == 2) {
    // это лето
} elseif ($result == 3) {
    // это осень
} else {
    // это зима
}
1
  • Люблю свой код, так как каждую заковырку знаю. А тут как-то разбираться надо. Вот тут как-то понятнее и тоже коротко: rotorweb.ru/dizajn/smena-dizajna-po-vremenam-goda.html
    – I_CaR
    Commented 1 сент. 2015 в 20:19
0

просто в первом коде надо было все таки функцию season() в конце вызвать. И $season = $season в else лишнее.

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.