- 1
 - 2
 - 3
 - 4
 - 5
 - 6
 - 7
 
if ($condition) {
  // если условие верно, сохраняем признак верности
  $options[ $m ] = '';
} else {
  // если неверно
  $options[ $m ] = 'none';
}
                                    Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 3
+3
if ($condition) {
  // если условие верно, сохраняем признак верности
  $options[ $m ] = '';
} else {
  // если неверно
  $options[ $m ] = 'none';
}
                                    
            Почему-то программист решил, что в случае верного условия нужно сохранять не TRUE, не 1, не что-то иное, а пустую строку. 
А если условие ложно - строку 'none'
        
+153
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, если вторая дата позже первой.
+58
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;
                                    Обнаружил в корпоративном коде. )