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

    +162

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    public function save()
    {
        $query = "SELECT * FROM `users` WHERE `id` = '$this->userId'";
        $result = $this->db->select($query);
        if (!empty($result)) {
            $query = "UPDATE `users` SET `email` = '$this->email', `password` = '$this->pass', `name` = '$this->name', `age` = '$this->age', `date_birth` = '$this->dateBirth', `sex` = '$this->sex', `aboute` = '$this->about' WHERE `id` = '$this->userId'";
        } else {
            $query = "INSERT INTO `users` VALUES('$this->userId','$this->email','$this->pass','$this->name','$this->age','$this->dateBirth','$this->sex','$this->about')";
        }
        $this->db->query($query);
    }

    Запостил: rtfm, 03 Ноября 2010

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

    • ava hdkeeper 03.11.2010 14:48 # 0

      Брутальный ActiveRecord.
      Ответить

    • ava Morgan 04.11.2010 00:54 # −2

      здесь нужен REPLACE INTO!
      // как всегда ваш .....
      Ответить

      • ava istem 04.11.2010 01:17 # 0

        replace не обновит индекс...
        Ответить

      • ava rtfm 08.11.2010 10:47 # 0

        REPLACE INTO здесь не нужен.
        как было сказано выше, REPLACE не обновляет индексы, а это плохо.
        здесь нужен
        INSERT INTO ...
        ON DUPLICATE UPDATE ...
        Ответить

    • ava guest 27.01.2017 22:56 # 0

      vanished
      Ответить

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