1. PHP / Говнокод #16452

    +155

    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
    $query = "SELECT DATE_FORMAT(`Date`, '%d.%m.%Y') 'dat',
                         DATE_FORMAT(`Date`, '%Y-%m-%d') 'd'
                    FROM Sklad
                   WHERE OrderID = '{$row->ID}'
                     AND OrderStatusID = 6
                     AND Kol < 0";
        
        $qsklad_data = mysql_fetch_object(mysql_query($query));

        if ($row->DateFakt == "00.00.0000") {
           
            $query = "UPDATE Orders
                         SET DateFakt = '{$qsklad_data->d}'
                       WHERE ID = '{$row->ID}'";

            $qupddd = mysql_query($query);
        }

    Паттерн программирования: сначала сделать запрос к БД, а потом проверить, нужны ли нам его результаты. $qsklad_data используется только здесь. И опять же два SQL-запроса там, где мог быть один.

    Запостил: Khvorostin, 31 Июля 2014

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

    • ava Cascader 31.07.2014 12:42 # +1

      тут не код говно, тут мозги говно. Слать запросы и не работать с результатом, это чертовски умно.
      Ответить

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