Говнокод #8320 — PHP — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

PHP / Говнокод #8320

+200

$y = date("Y");
$date = date("H:i");

   echo '<h3>Московское время: <span style="color:silver">';
   if ($date == "06:00") echo "Шесть часов ноль минут";
   if ($date == "06:01") echo "Шесть часов одна минута";
   if ($date == "06:02") echo "Шесть часов две минуты";
   if ($date == "06:03") echo "Шесть часов три минуты";
   if ($date == "06:04") echo "Шесть часов четыре минуты";
   if ($date == "06:05") echo "Шесть часов пять минут";
   if ($date == "06:06") echo "Шесть часов шесть минут";
   if ($date == "06:07") echo "Шесть часов семь минут";
   if ($date == "06:08") echo "Шесть часов восемь минут";
   if ($date == "06:09") echo "Шесть часов девять минут";
   if ($date == "06:10") echo "Шесть часов десять минут";
   if ($date == "06:11") echo "Шесть часов одиннадцать минут";
   if ($date == "06:12") echo "Шесть часов двенадцать минут";
   if ($date == "06:13") echo "Шесть часов тринадцать минут";
   if ($date == "06:14") echo "Шесть часов четырнадцать минут";
   if ($date == "06:15") echo "Шесть часов пятнадцать минут";

/*  Здесь пропускаем около 800 строк подобного кода*/

   if ($date == "17:57") echo "Семьнадцать часов пятьдесят семь минут";
   if ($date == "17:58") echo "Семьнадцать часов пятьдесят восемь минут";
   if ($date == "17:59") echo "Семьнадцать часов пятьдесят девять минут";
   if ($date == "18:00") echo "Восемьнадцать часов ноль минут";
   if ($date == "18:01") echo "Восемьнадцать часов одна минута";
   if ($date == "18:02") echo "Восемьнадцать часов две минуты";
   if ($date == "18:03") echo "Восемьнадцать часов три минуты";
   if ($date == "18:04") echo "Восемьнадцать часов четыре минуты";

/*  Пропустим еще около 600 строк... Кажется закончилось... Хорошо, что в сутках только 24 часа по 60 минут в каждом. */ 

   if ($date == "05:58") echo "Пять часов пятьдесят восемь минут";
   if ($date == "05:59") echo "Пять часов пятьдесят девять минут";
   echo "</span></h3>";

/* Но ведь есть еще года....  */
  echo '<h3>На дворе: <span style="color:silver">';
   if ($y == "2011") echo "Две тысячи одиннадцатый год";
   if ($y == "2012") echo "Две тысячи двенадцатый год";
   if ($y == "2013") echo "Две тысячи тринадцатый год";
   if ($y == "2014") echo "Две тысячи четырнадцатый год";
   if ($y == "2015") echo "Две тысячи пятнадцатый год";
   if ($y == "2016") echo "Две тысячи шестнадцатый год";
   if ($y == "2017") echo "Две тысячи семнадцатый год";
   if ($y == "2018") echo "Две тысячи восемнадцатый год";
   if ($y == "2019") echo "Две тысячи девятнадцатый год";
   if ($y == "2020") echo "Две тысячи двадцатый год";
   if ($y == "2021") echo "Две тысячи двадцать первый год";

/* Подрежем немного шедевр. Похоже планируется поддержка данного ПО аж до середины века */

   if ($y == "2053") echo "Две тысячи пятдесят третий год";
   if ($y == "2054") echo "Две тысячи пятдесят четвертый год";
   if ($y == "2055") echo "Две тысячи пятдесят пятый год";
   echo '</span></h3>';

Данный шедевр алгоритмизации имеет в оригинале около 5000 строк. Я его сильно сократил, но смысл не изменился. Комментарии в коде мои, в оригинале естественно никаких комментариев нет.

Запостил:

RussianSpy, 27 Октября 2011

Комментарии (29) RSS

Vasiliy 27.10.2011 16:00 # +13

Жаль без секунд.
Ответить
Lure Of Chaos 27.10.2011 16:08 # +4

а теперь с датами и вручную
Ответить
Vasiliy 27.10.2011 16:11 # +8

Зато какое пространнство для рефракторинга
Ответить
- SmackMyBitchUp 27.10.2011 18:03 # +2
  
  Скорее для реfuckинга, если уж на то пошло.
  Удалить нахрен.
  Ответить
gegMOPO4 27.10.2011 18:58 # +2

24*60 = 1440 строк. Полсотни на года. 366 на число и месяц (хоть они и пропущены).

Где ещё 3000 строк?!
Ответить
- RussianSpy 27.10.2011 19:09 # 0
  строка вида
  
  if ($y == "2011") echo "Две тысячи одиннадцатый год";
  
  в оригинале выглядела примерно так
  
  if ($y == "2011") { echo "Две тысячи одиннадцатый год"; }
  
  Плюс там еще код связанный с говноверсткой, обработка дат занимает около 50 строк, еще здоровенный кусман посвящен работе с месяцами, ну и конечно замеры времени выполнения - как же без них-то...
  
  if ($d=="01") { echo "Первое"; } if ($d=="1") { echo "Первое"; } if ($d=="02") { echo "Второе"; } if ($d=="2") { echo "Второе"; } /* много строк */ if ($m=="02") { echo "Февраля"; } if ($m=="2") { echo "Февраля"; } if ($m=="03") { echo "Марта"; } if ($m=="3") { echo "Марта"; }
  Ответить
  - gegMOPO4 27.10.2011 19:34 # +3
    
    Ух ты, а я думал, что будет if ("$d/$m"=="02/03") ...
    Ответить
guest 28.10.2011 05:14 # +4

Да, и граммар-наци отчаянно негодует! "пятдесят"? "восемьнадцать"?

Хотя, чего ещё ждать от такого кодера
Ответить
TheHamstertamer 28.10.2011 10:56 # +3

MAXIMUM INDUS
Ответить
Em1ss1oN 28.10.2011 15:17 # 0

Не знаю пыха, поэтому вопрос - как сделать иначе?
Ответить
- SmackMyBitchUp 28.10.2011 15:26 # +2
  
  Попал ты, паря
  Ответить
- guest 28.10.2011 18:54 # +2
  
  убить себя нахуй
  Ответить
  - Em1ss1oN 28.10.2011 18:57 # −1
    
    PHP-программист во мне сдох не родившись, так что условие выполнено. Что дальше делать?
    Ответить
- bot-minurast 28.10.2011 19:03 # 0
  
  Автор?
  Ответить
  - Em1ss1oN 28.10.2011 19:48 # 0
    
    Я на такое не способен.
    Ответить
- umren 28.10.2011 20:06 # −1
  
  да, действительно. каким образом эта фишка реализуется у топкодера? потому что действительно может быть такой запрос заказчика )
  Ответить
  - bot-minurast 28.10.2011 20:28 # 0
    
    навскидку: создать массив [единицы, десятки...]
    Ответить
  - guest 28.10.2011 22:05 # +2
    Например, так:
    
    function parse($number, $titles) { $cases = array(2, 0, 1, 1, 1, 2); return $titles[ ($number%100>4 && $number%100<20)? 2 : $cases[($number%10<5)?$number%10:5] ]; } $hrs = array('час', 'часа', 'часов'); $mins = array('минута', 'минуты', 'минут'); for ($h=0;$h<24;$h++) { for ($m=0;$m<60;$m++) { echo parse($h, $hrs) . ", " . parse($m, $mins) . "\n"; } }
    Ответить
    - guest 28.10.2011 22:07 # 0
      
      тем же методом и цифры
      Ответить
    - guest 29.10.2011 05:50 # +1
      
      [just for fun]
      (define (literal hour min) (format #f "~R hour~P ~R minute~P" hour hour min min))
      (literal 9 51)
      $30 = "nine hours fifty-one minutes"
      
      правда говнобритосский
      Ответить
      - kyzi007 07.11.2011 10:34 # −1
        
        А если еще найти готовую либу то вообще мир перевернется)
        Есть же терпеливые люди на свете...
        Ответить
  - guest 29.10.2011 01:12 # +1
    
    Write code that generates the above code!
    Ответить
- saab 01.11.2011 14:11 # 0
  
  сцепить части строк.
  Ответить
LazyPiG 28.10.2011 17:20 # +2

Хотелось бы увидеть код данного человека по переходу на Зимнее/Летнее время ))) (Даже если date() хватает время сервера :D )
Я теперь понял почему его все таки отменили =)))
Ответить
Nemoden 28.10.2011 17:28 # +4
```
$timestamp = time();
// ну шо? Поехали....
if ($timestamp == '1319815513') {
    ...
```
Хотя, я думаю ЭТОТ справился бы.
Ответить
- zloiia 30.10.2011 02:28 # +1
  
  Это просто божественно! Эмоции хлещут через край.
  Ответить
guest 25.11.2014 13:28 # 0

Может он просто сгенерил это)
Ответить
- TarasB 25.11.2014 13:55 # +1
  
  Ага, только при написании генератора для этого кода надо решать точно такую же задачу, как и при написании самого кода.
  Ответить
  - bormand 25.11.2014 14:03 # 0
    
    Можно сделать quine!
    Ответить

Говнокод: по колено в коде.

PHP / Говнокод #8320

Комментарии (29) RSS

Добавить комментарий