Почему неправильно работает функция времени-даты?

Проблема здесь в том, как интерпретируются диапазоны дней для знаков, которые пересекают начало и конец года, как у Козерога и Водолея. В этих случаях сравнение дней в году (z — порядковый номер дня) требует особого подхода, потому что date('z') для 19 января (примерно 19-й день в году) будет меньше, чем для 31 декабря (примерно 365-й день).

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

Обновим условие для Козерога и Водолея, а также добавим проверку для тех, кто родился на пограничных датах:

PHP:

1. <?php
2. // ...
3.  
4. if (isset($_GET['birthday'])) {
5.     if ($_GET['birthday'] === '') {
6.         echo 'Вы не выбрали дату рождения!';
7.     } else {
8.  
9.         $currentYear = date('Y');
10.         $birthday = (int)date('z', strtotime($_GET['birthday'])) + 1;
11.  
12.         switch (true) {
13.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 3, 21, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 4, 19, $currentYear))):
14.                 echo "Овны: {$horoscopes['Овны']}";
15.                 break;
16.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 4, 20, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 5, 20, $currentYear))):
17.                 echo "Тельцы: {$horoscopes['Тельцы']}";
18.                 break;
19.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 5, 21, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 6, 20, $currentYear))):
20.                 echo "Близнецы: {$horoscopes['Близнецы']}";
21.                 break;
22.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 6, 21, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 7, 22, $currentYear))):
23.                 echo "Раки: {$horoscopes['Раки']}";
24.                 break;
25.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 7, 23, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 8, 22, $currentYear))):
26.                 echo "Львы: {$horoscopes['Львы']}";
27.                 break;
28.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 8, 23, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 9, 22, $currentYear))):
29.                 echo "Девы: {$horoscopes['Девы']}";
30.                 break;
31.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 9, 23, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 10, 22, $currentYear))):
32.                 echo "Весы: {$horoscopes['Весы']}";
33.                 break;
34.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 10, 23, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 11, 21, $currentYear))):
35.                 echo "Скорпионы: {$horoscopes['Скорпионы']}";
36.                 break;
37.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 11, 22, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 12, 21, $currentYear))):
38.                 echo "Стрельцы: {$horoscopes['Стрельцы']}";
39.                 break;
40.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 12, 22, $currentYear)) || $birthday <= (int)date('z', mktime(0, 0, 0, 1, 19, $currentYear))):
41.                 echo "Козероги: {$horoscopes['Козероги']}";
42.                 break;
43.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 1, 20, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 2, 18, $currentYear))):
44.                 echo "Водолеи: {$horoscopes['Водолеи']}";
45.                 break;
46.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 2, 19, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 3, 20, $currentYear))):
47.                 echo "Рыбы: {$horoscopes['Рыбы']}";
48.                 break;
49.         }
50.     }
51. }

В этом варианте знак Козерога определяется корректно: сначала проверяется, попадает ли день в диапазон с 22 декабря по 31 декабря, либо с 1 по 19 января. Водолей тоже определяется корректно.


Ввот основные изменения:

1. Замена условий на switch (true): Вместо switch ($birthday), используется switch (true), что позволяет делать проверки с логическими выражениями (условиями) внутри каждого case, а не только на равенство значений.
   
   
2. Корректировка условий для Козерога и Водолея:
   
   
   • В случае Козерога: проверяется, находится ли день рождения в диапазоне с 22 декабря по 31 декабря или с 1 по 19 января:
     

PHP:

1. case ($birthday >= (int)date('z', mktime(0, 0, 0, 12, 22, $currentYear)) || $birthday <= (int)date('z', mktime(0, 0, 0, 1, 19, $currentYear))):
2.     echo "Козероги: {$horoscopes['Козероги']}";
3.     break;

Здесь используется оператор || (логическое ИЛИ), чтобы охватить оба периода (конец и начало года).

В случае Водолея: знак определяется, если день рождения находится в интервале с 20 января по 18 февраля. Так как даты не пересекают границы года, условие для Водолея остается стандартным:

PHP:

1. case ($birthday >= (int)date('z', mktime(0, 0, 0, 1, 20, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 2, 18, $currentYear))):
2.     echo "Водолеи: {$horoscopes['Водолеи']}";
3.     break;

Эти изменения позволяют правильно интерпретировать дни рождения, которые лежат на границе года, как в случае с Козерогом и Водолеем.
--- Добавлено ---
Замена switch ($birthday) на switch (true) — это ключевое изменение. Оно позволяет проверять логические выражения внутри каждого case, а не только точное соответствие значению переменной $birthday.

В итоге, весь блок switch начинает работать как цепочка if...else if, что позволяет корректно обрабатывать диапазоны дат и решает проблему с Козерогом и Водолеем.

 

Добрый день!
Посмотрите моё решение.
Оставил, то что было для сравнения результата.
Выберите какое понравится.

PHP:

1. <?php
2. error_reporting(-1);
3. // №11
4. echo '<form action="" method="GET">';
5. echo '<input type="date" name="birthday" required >';
6. echo '<input type="submit" value="отправить">';
7. echo '</form>';
8. $horoscopes =
9.     [
10.         'Овны' => 'найдите в себе желание радоваться этой жизни, даже если не очень хочется.',
11.         'Тельцы' => 'сегодня даже работа ответит вам взаимностью.',
12.         'Близнецы' => 'умерьте аппетиты и не выходите за рамки разумного.',
13.         'Раки' => 'вы в отличной физической форме и преодолеете все препятствия.',
14.         'Львы' => 'у вас есть тактика, вот её и придерживайтесь.',
15.         'Девы' => 'кто-то захочет совершить нападение на ваше сердце.',
16.         'Весы' => 'настроение будет на высоте, а этого уже достаточно.',
17.         'Скорпионы' => 'все бонусы этого дня будут поджидать вас вечером.',
18.         'Стрельцы' => 'наслаждайтесь моментом, а о будущем вы ещё успеете подумать.',
19.         'Козероги' => 'соблюдайте сегодня максимально дружелюбный вид.',
20.         'Водолеи' => 'впечатлений, полученых сегодня вам хватит надолго.',
21.         'Рыбы' => 'звёзды дают зелёный свет на новые знакомства.',
22.     ];
23.  
24.    $horoscopesInd = array_keys($horoscopes);
25.  
26. $horoscopesDate =
27. [
28.     ["3-21","4-19"],
29.     ["4-20","5-20"],
30.     ["5-21","6-20"],
31.     ["6-21","7-22"],
32.     ["7-23","8-22"],
33.     ["8-23","9-22"],
34.     ["9-23","10-22"],
35.     ["10-23","11-21"],
36.     ["11-22","12-21"],
37.     ["12-22","1-19"],
38.     ["1-20","2-18"],
39.     ["2-19","3-20"]
40. ];
41.  
42. function getTs($md, $Y){
43.  
44.     $md = str_pad($md, 4, "0", STR_PAD_LEFT);
45.     return strtotime("$Y-$md");
46. }
47. if (isset($_GET['birthday'])) {
48.     $currentYear = $Y = date('Y');
49.     $mdBd = substr($_GET['birthday'], 5);
50.  
51.     foreach($horoscopesDate as $i => $interval)
52.     {
53.         $begin = getTs($interval[0], $Y);
54.         $tsBd = getTs($mdBd, $Y);
55.         if($i==9) $Y++;
56.         $end = getTs($interval[1], $Y);
57.         if(in_array($tsBd, range($begin,$end)))
58.         {
59.             echo $interval[0]," ", $interval[1],"<br>";
60.             $zName = $horoscopesInd[$i];
61.             echo $zName, " ", $horoscopes[$zName];
62.             break;
63.         }
64.     }
65. #########################################################################
66. # Для сранения резeльтата оcтавил Ваш код
67. echo "<hr>";
68.  
69. $currentYear = date('Y');
70.         $birthday = (int)date('z', strtotime($_GET['birthday'])) + 1;
71.  
72.         switch (true) {
73.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 3, 21, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 4, 19, $currentYear))):
74.                 echo "Овны: {$horoscopes['Овны']}";
75.                 break;
76.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 4, 20, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 5, 20, $currentYear))):
77.                 echo "Тельцы: {$horoscopes['Тельцы']}";
78.                 break;
79.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 5, 21, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 6, 20, $currentYear))):
80.                 echo "Близнецы: {$horoscopes['Близнецы']}";
81.                 break;
82.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 6, 21, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 7, 22, $currentYear))):
83.                 echo "Раки: {$horoscopes['Раки']}";
84.                 break;
85.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 7, 23, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 8, 22, $currentYear))):
86.                 echo "Львы: {$horoscopes['Львы']}";
87.                 break;
88.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 8, 23, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 9, 22, $currentYear))):
89.                 echo "Девы: {$horoscopes['Девы']}";
90.                 break;
91.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 9, 23, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 10, 22, $currentYear))):
92.                 echo "Весы: {$horoscopes['Весы']}";
93.                 break;
94.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 10, 23, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 11, 21, $currentYear))):
95.                 echo "Скорпионы: {$horoscopes['Скорпионы']}";
96.                 break;
97.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 11, 22, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 12, 21, $currentYear))):
98.                 echo "Стрельцы: {$horoscopes['Стрельцы']}";
99.                 break;
100.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 12, 22, $currentYear)) || $birthday <= (int)date('z', mktime(0, 0, 0, 1, 19, $currentYear))):
101.                 echo "Козероги: {$horoscopes['Козероги']}";
102.                 break;
103.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 1, 20, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 2, 18, $currentYear))):
104.                 echo "Водолеи: {$horoscopes['Водолеи']}";
105.                 break;
106.             case ($birthday >= (int)date('z', mktime(0, 0, 0, 2, 19, $currentYear)) && $birthday <= (int)date('z', mktime(0, 0, 0, 3, 20, $currentYear))):
107.                 echo "Рыбы: {$horoscopes['Рыбы']}";
108.                 break;
109.         }
110. ################################################################################
111. }

Удачи!

 

Ну, изначально вы сделали неверный выбор что с чем сравнивать. Порядковый номер дня в году с интервалом дат знаков Зодиака чисто логически неудобно.
В PHP наиболее удобным является сравнение дат в Юникс формате
Отсюда наиболее удобным будет привести даты интервалов и дня рождения к одному году, взяв например 1970

Ну, тогда уже и моё. Вспомню, помнят ли ручки

PHP:

1. <?php
2. date_default_timezone_set('UTC');
3.  
4. $birthday = '02-29'; // m-d
5.  
6.  
7. function ret_udate($d)
8. {
9.         $date  = date_create('1970-'.$d);
10.     return date_format($date, 'U');
11. }
12.  
13. function ret_zodiac($bd)
14. {
15.     $ubd       = ret_udate($bd);
16.     $horoscope = array (
17.                         array ('Козероги',  '01-01', '01-19'),
18.                         array ('Водолеи',   '01-20', '02-18'),
19.                         array ('Рыбы',      '02-19', '03-20'),
20.                         array ('Овны',      '03-21', '04-19'),
21.                         array ('Тельцы',    '04-20', '05-20'),
22.                         array ('Близнецы',  '05-21', '06-20'),
23.                         array ('Раки',      '06-21', '07-22'),
24.                         array ('Львы',      '07-23', '08-22'),
25.                         array ('Девы',      '08-23', '09-22'),
26.                         array ('Весы',      '09-23', '10-22'),
27.                         array ('Скорпионы', '10-23', '11-21'),
28.                         array ('Стрельцы',  '11-22', '12-21'),
29.                         array ('Козероги',  '12-22', '12-31'),
30.                         );
31.  
32.         foreach ($horoscope as $zodiac)
33.     {
34.             if(ret_udate($zodiac[1]) <= $ubd && $ubd <= ret_udate($zodiac[2]))
35.             {
36.                     return $zodiac[0];
37.             }
38.     }
39. }
40.  
41. $zodiac_sign = ret_zodiac($birthday);
42.  
43. $horoscopes = array (
44.                      'Овны'      => 'найдите в себе желание радоваться этой жизни, даже если не очень хочется.',
45.                      'Тельцы'    => 'сегодня даже работа ответит вам взаимностью.',
46.                      'Близнецы'  => 'умерьте аппетиты и не выходите за рамки разумного.',
47.                      'Раки'      => 'вы в отличной физической форме и преодолеете все препятствия.',
48.                      'Львы'      => 'у вас есть тактика, вот её и придерживайтесь.',
49.                      'Девы'      => 'кто-то захочет совершить нападение на ваше сердце.',
50.                      'Весы'      => 'настроение будет на высоте, а этого уже достаточно.',
51.                      'Скорпионы' => 'все бонусы этого дня будут поджидать вас вечером.',
52.                      'Стрельцы'  => 'наслаждайтесь моментом, а о будущем вы ещё успеете подумать.',
53.                      'Козероги'  => 'соблюдайте сегодня максимально дружелюбный вид.',
54.                      'Водолеи'   => 'впечатлений, полученых сегодня вам хватит надолго.',
55.                      'Рыбы'      => 'звёзды дают зелёный свет на новые знакомства.'
56.                      );
57.  
58. echo $zodiac_sign . ' : ' . $horoscopes[$zodiac_sign];
59. ?>

 

Добрый день!

Всё ещё на PHP 5.3 ?
Вы таже сравниваете не порядковый номер дня в году, а timestamp. Это правильное решение.
Мне не понравилось, что Вы для того чтобы не прописовать логику
повторили в массиве $horoscopes запись для 'Козероги' с разными датами.
Это в данном случае лишнее

PHP:

1. date_default_timezone_set('UTC');

Удачи!

 

Приводится к одному году.
А с "прописыванием" к двум.

Не везде

Нет. Люблю жабоскриптеров побесить