- 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;
Обнаружил в корпоративном коде. )