Проблема с IF как решить?

Question

[Никифор Макаров]

Проблема заключается в моих кривых руках, написал говнокод - исправляй сам. Чем я и займусь. Спасибо за советы. Тема закрыта.

Answer 1

[Rsa97]

А не лучше этот IF прямо в запрос включить? Если до заполнения значение `date_finish` NULL, то

SELECT `track`, `phone`, `status`, 
       to_days(IFNULL(`date_finish`, curdate())) - to_days(`date_add`) as `diff` 
    FROM `track_tabl` WHERE `id_user` = '$id_user'

Если `date_finish` - пустая строка, то

SELECT `track`, `phone`, `status`, 
       to_days(IF(`date_finish` = '', curdate(), `date_finish`)) - to_days(`date_add`) as `diff` 
    FROM `track_tabl` WHERE `id_user` = '$id_user'

Ну и лучше используйте параметризованные запросы.

Answer 2

[# a r t u r #]

Почитайте https://ru.wikipedia.org/wiki/Model-View-Controller

а то смотреть больно

Где вообще у вас проверка на пустоту diff? и откуда вы берете текущее время?

Comments

[Никифор Макаров]

Спасибо, за материал для прочтения.
"Где вообще у вас проверка на пустоту diff?" у меня проверка на пустоту поля date_finish " $dfs= $date_finish_s->fetchAll(PDO::FETCH_ASSOC);
if(empty($dfs['date_finish'])) ";
"и откуда вы берете текущее время?" - "$select_date = $db->query("SELECT *, to_days(curdate()) - to_days(date_add) as diff FROM track_tabl WHERE id_user='$id_user'");" "curdate" - yandex.ru/yandsearch?text=curdate%20sql&clid=18365...

P.S Не судите строго, это мои начинания.