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 юзаю

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

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

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