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

    • --i++ // оператор подергивания
      Ответить
      • оператор + и - могут быть перегружены, и делать какой-то сайд-эффект (например отправлять электронную почту).
        К сожалению, в PHP нет перегрузки операторов.
        Ответить
    • теплый ламповый replace + sql injection
      Ответить
      • sql injection'а нет - это точно проверяется в других частях.
        но реализация REPLACE'а просто великолепна :)
        Ответить
        • >> это точно проверяется в других частях.
          ух ты, еще и размазанная по всему коду логика работы с бд
          Ответить
    • это напомнило мне о умножении нуля на налл)))
      Ответить
    • return true;! - причем - в любом случае!
      Ответить

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