PHP PDO не обновляет данные в MySQL

@colobaev · Регистрация: 02.05.2014

Студворк — интернет-сервис помощи студентам

Приветствую форумчане. Не могу понять, почему следующий код не обновляет в MySQL данные

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
public function edit($ignored=Array())
    {
        global $registry;
        
        $vars = get_class_vars(get_class($this));
        $vars = get_defined_vars();
        foreach ($vars["this"] as $key => $v)
        {
            if ($key == "error" || $key == "id" || $key == "name_module" || in_array($key, $ignored)) continue;
            $data[':'.$key] = $v;
            $val .= ", $key=:$key";
        }
        $val[0] = $val[1] = '';
        
        $mod = htmlspecialchars(trim("mod_" . $this->name_module . "_category"));
        $query = "UPDATE $mod SET $val WHERE id=:id";
        $query = $registry->get("db")->prepare($query);
        $query->bindParam(':id', $this->id);
        foreach ($data as $key => $val)
        {
            $query->bindParam($key, $val);
        }
        
        $query->execute();
        if ($query->rowCount() == 0)
        {
            $this->error = 1;
            return false;
        }
        
        $this->error = 0;
        return true;
    }

Если вручную добавить в запрос вместо $var то, что храниться в этой переменной, то запрос успешно выполняется.
errorInfo выдает следующую информацию:
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

@Блеф · 25.07.2017, 15:53

приведите текст sql запроса с переменой

@colobaev · 25.07.2017, 20:36 **[ТС]**

SQL
1
UPDATE mod_pages_category SET name=:name, archive=:archive, rights=:rights, uri=:uri WHERE id=:id

@Вилки Выкидные · 26.07.2017, 13:33

Судя по тому что мы читаем в мануале по bindParam: ...переменная привязывается по ссылке...

То по логике цикл должен выглядеть примерно так:

PHP
1
2
3
4
foreach ($data as $key => &$val)
{
    $query->bindParam($key, $val);
}

@colobaev · 26.07.2017, 13:50 **[ТС]**

Неа, не работает. Приведу еще раз код, в котором добавлю переменную с полным запросом $q, которая успешно выполняется. В эту переменную я записал то, что вывел print'ом из $query.

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
public function edit($ignored=Array())
    {
        global $registry;
        
        $vars = get_class_vars(get_class($this));
        $vars = get_defined_vars();
        foreach ($vars["this"] as $key => $v)
        {
            if ($key == "error" || $key == "id" || $key == "name_module" || in_array($key, $ignored)) continue;
            $data[':'.$key] = $v;
            $val .= ", $key=:$key";
        }
        $val[0] = $val[1] = '';
        
        $mod = htmlspecialchars(trim("mod_" . $this->name_module . "_category"));
        $query = "UPDATE $mod SET $val WHERE id=:id";
        $q = "UPDATE mod_pages_category SET name=:name, archive=:archive, rights=:rights, uri=:uri WHERE id=:id";
        $query = $registry->get("db")->prepare($q);
        $query->bindParam(':id', $this->id);
        foreach ($data as $key => $val)
        {
            $query->bindParam($key, $val);
        }
        
        $query->execute();
        if ($query->rowCount() == 0)
        {
            $this->error = 1;
            return false;
        }
        
        $this->error = 0;
        return true;
    }

@Вилки Выкидные · 26.07.2017, 13:56

colobaev, в $data у Вас массив именованных параметров со значениями ?

распечатайте перед вторым циклом

PHP
1
echo '<pre>' . print_r($data, true) . '</pre>';

@colobaev · 26.07.2017, 16:04 **[ТС]**

Всё верно

Array
(
[:name] => NewName
[:archive] => 123
[:rights] => 123
[:uri] => test
)

Добавлено через 2 часа 4 минуты
Поправил код, работает. Спасибо кто откликнулся =)

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
public function edit($ignored=Array())
    {
        global $registry;
        
        $vars = get_class_vars(get_class($this));
        $vars = get_defined_vars();
        $value = "";
        foreach ($vars["this"] as $key => $val)
        {
            if ($key == "error" || $key == "id" || $key == "name_module" || in_array($key, $ignored)) continue;
            $data[$key] = $val;
            $value .= "$key=:$key, ";
        }
        $value = substr($value, 0, -2);
        $mod = htmlspecialchars(trim("mod_" . $this->name_module . "_category"));
        $query = $registry->get("db")->prepare("UPDATE $mod SET $value WHERE id = :id");
        $data["id"] = $this->id;
        $query->execute($data);
        if ($query->rowCount() == 0)
        {
            $this->error = 1;
            return false;
        }
        
        $this->error = 0;
        return true;
    }

@Вилки Выкидные · 26.07.2017, 16:25

Сообщение от colobaev

Поправил код

я кстати тоже так PDO юзаю

без биндов, просто массив извлекаю с именованными параметрами и значениями

Новые блоги и статьи Все статьи Все блоги /
Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Programma_Boinc 01.01.2026 Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Сочетание глобально распределённой вычислительной мощности и инновационных. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .
Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .

@Блеф > /dev/null 286 / 119 / 86 Регистрация: 14.05.2014 Сообщений: 767
	25.07.2017, 15:53
	приведите текст sql запроса с переменой 0