1. Список говнокодов пользователя MikleSmart

    Всего: 3

  2. PHP / Говнокод #24081

    +3

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    if ($condition) {
      // если условие верно, сохраняем признак верности
      $options[ $m ] = '';
    } else {
      // если неверно
      $options[ $m ] = 'none';
    }

    Почему-то программист решил, что в случае верного условия нужно сохранять не TRUE, не 1, не что-то иное, а пустую строку.
    А если условие ложно - строку 'none'

    MikleSmart, 06 Апреля 2018

    Комментарии (2)
  3. JavaScript / Говнокод #13856

    +153

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    function compare_date(start_day, start_hour, start_min, end_day, end_hour, end_min)
    {
      var start_ = start_day.split('-');
      var end_ = end_day.split('-');
    
      if (start_[2] > end_[2]) return false;
      else {
        if (start_[2] == end_[2]) {
          if (start_[1] > end_[1]) return false;
          else {
            if (start_[1] == end_[1]) {
              if (start_[0] > end_[0]) return false;
              else {
                if (start_[0] == end_[0]) {
                  if (parseFloat(start_hour) > parseFloat(end_hour)) return false;
                  else {
                    if (parseFloat(start_hour) == parseFloat(end_hour)) {
                      if (parseFloat(start_min) > parseFloat(end_min)) return false;
                      else {
                        if (parseFloat(start_min) == parseFloat(end_min)) return false;
                        else return true;
                      }
                    }
                    else return true;
                  }
                }
                else return true;
              }
            }
            else return true;
          }
        }
        else return true;
      }
    }

    Откопал функцию-велосипед для сравнения двух дат. Возвращает true, если вторая дата позже первой.

    MikleSmart, 27 Сентября 2013

    Комментарии (0)
  4. PHP / Говнокод #10279

    +58

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    if ($who == 'student') $answerArray['student_id'] = $this->people->id;
    if ($who == 'teacher') $answerArray['teacher_id'] = $this->people->id;
    if ($who == 'student') $answerArray['student_text'] = $post_array['message'];
    if ($who == 'teacher') $answerArray['teacher_text'] = $post_array['message'];
    if ($who == 'student') $answerArray['student_date'] = date('Y-m-d H:i:s');
    if ($who == 'teacher') $answerArray['teacher_date'] = date('Y-m-d H:i:s');
    if ($who == 'student' && $files) $answerArray['student_file'] = $files[0]['name'];
    if ($who == 'teacher' && $files) $answerArray['teacher_file'] = $files[0]['name'];
    if ($who == 'student' && $files) $answerArray['student_file_orig_name'] = $files[0]['orig_name'];
    if ($who == 'teacher' && $files) $answerArray['teacher_file_orig_name'] = $files[0]['orig_name'];
    if ($who == 'student') $answerArray['status'] = 0;
    if ($who == 'teacher') $answerArray['status'] = 1;

    Обнаружил в корпоративном коде. )

    MikleSmart, 15 Мая 2012

    Комментарии (5)