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

    +165

    1. 1
    $result = mysql_query("SELECT `id` FROM `cats` WHERE `id` = (SELECT `pid` FROM `tovary` WHERE `id` = '".$this->id."' LIMIT 1)");

    В один запрос получать id не кошерно.

    Было найдено в этом же проекте - http://www.govnokod.ru/2897

    Запостил: Rpsl, 02 Апреля 2010

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

    • ava pasha 02.04.2010 17:17 # 0

      Вероятно было ещё что-то, потом в ходе переработке кода просто подтерли.
      Ответить

      • ava Rpsl 02.04.2010 17:20 # 0

        Значит обращали внимание на эту строку и нормально поправить не удосужились.
        Ответить

    • ava Death 02.04.2010 17:57 # −2.2

      SELECT c.id
      FROM cats c
      JOIN tovary t ON t.pid = c.id
      WHERE t.id='$this->id'
      Ответить

      • ava Rpsl 02.04.2010 18:01 # +2.2

        SELECT pid AS id
        FROM tovary
        WHERE id = $this->id
        Ответить

        • ava Death 02.04.2010 18:23 # −3.2

          возможно из cats надо будет ещё что-то тянуть ))
          Ответить

          • ava Rpsl 02.04.2010 20:20 # +1.2

            бля, ну если придется, тогда и надо будет думать. "Если" - это не повод делать:

            functionname( $_GLOBAL ) ...
            Ответить

        • ava murz 02.04.2010 18:53 # +0.2

          В cats может и не быть такой id
          Ответить

          • ava Rpsl 02.04.2010 21:11 # 0

            Ну, как бы я считаю, что это уже тонкости проектов. Кроме cats еще могут быть всякие brands и еще куча условий. Но это не повод делать каждый раз джоины таблиц по 100500 тысяч записей. Опять же, повторюсь, что все это относительно и зависит от проекта. Но часто бывает проще вернуть ошибку при отсутствие данных, чем напрягать базу.
            Ответить

    • ava Dreyk 02.04.2010 23:03 # +3.2

      бедные cats, чем эти коты так не угодили всем проектам :D
      Ответить

      • ava guest 05.04.2010 07:05 # +0.2

        Котэ имеют большую популярность в IT.
        Ответить

    • ava guest 06.04.2010 12:42 # −2

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

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