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

    +169

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    public function Add()
{
$db->query ('DELETE FROM ' . $this->_table . ' WHERE clientId=' . $this->clientId . ' AND memberId=' . $this->userId);
$db->query ('INSERT INTO ' . $this->_table . ' SET clientId=' . $this->clientId . ', memberId=' . $this->userId);
return true;
}

    без комментариев

    Запостил: rtfm, 28 Июля 2010

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

    • ava bugmenot 28.07.2010 13:10 # +6

      --i++ // оператор подергивания
      Ответить

      • ava Анонимус 28.07.2010 15:59 # 0

        оператор + и - могут быть перегружены, и делать какой-то сайд-эффект (например отправлять электронную почту).
        К сожалению, в PHP нет перегрузки операторов.
        Ответить

    • ava Анонимус 28.07.2010 14:44 # 0

      теплый ламповый replace + sql injection
      Ответить

      • ava rtfm 28.07.2010 14:48 # 0

        sql injection'а нет - это точно проверяется в других частях.
        но реализация REPLACE'а просто великолепна :)
        Ответить

        • ava Анонимус 28.07.2010 14:49 # −2

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

    • ava 3.14159265 28.07.2010 15:34 # −1

      это напомнило мне о умножении нуля на налл)))
      Ответить

    • ava Lure Of Chaos 28.07.2010 21:07 # +2

      return true;! - причем - в любом случае!
      Ответить

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