Как обновить строку в базе данных?

Question

[Дмитрий Поляков]

Пытаюсь обновить строку в кастомной таблице, но ничего не обновляется, ошибок не выдает. Видимо я что т оне так делаю?

$wpdb->update(TBL_PERSON, $personPostOldData, $personPostData);

Пробовал писать sql запрос в phpMyAdmin, отвечает, что 0 строк задействовано, или что то такое.

UPDATE `wp_person_post_data` SET `autor_id` = '1', `post_id` = 153, `teach_start` = '', `address` = '', `email` = '', `phone` = '', `url` = '', `fb` = '', `vk` = '', `yutu` = '' WHERE `autor_id` = '1' AND `teach_start` = '' AND `address` = 'тест' AND `email` = '' AND `phone` = '' AND `url` = '' AND `fb` = '' AND `vk` = '' AND `yutu` = '' AND `post_id` = 153

Пока решил вопрос удалением старой строки и вставкой новой, но это явный костыль и лишнее обращение к базе. Хотелось бы обойтись меньшими потерями производительности.

Answer 1

[Иван Корюков]

Когда вы делаете update запрос вам не надо указывать данные, которые не изменяются
и не надо указывать в where больше условий чем необходимо для идентификации изменяемой строки. Если в строке есть primary key, то достаточно указать только его.

Comments

[Дмитрий Поляков]

Не известно какие данные обновятся. Пользователь может создать одну запись в категории, потом может ее только обновлять. То есть, мне нужно сравнить текущее значение с новым и есть отличия указать их в запросах SET и WHERE и указать primary key?

[Иван Корюков]

Дмитрий Поляков, нет, вы, конечно же можете засовывать в базу все данные из формы, не важно изменились они или нет. Но перечислять все данные в where на надо)

А так да - перед записью в БД вы можете взять оттуда старые значения и сравнить с тем что пришло от пользователя.

[Дмитрий Поляков]

Разобрался теперь, спасибо!

А если передавать только обновленные данные в SET, запрос будет быстрее срабатывать? То есть, sql работает как ftp? Для обновления каждого параметра, делает новое подключение или меняет всю строку за одно подключение?

[Иван Корюков]

Дмитрий Поляков,
Есть подключение, есть запрос и есть поля. Вот подключение оно в идеале одно на всё время работы скрипта. Запрос - вот ви пишете Update blah blah blah where blah blah blah; - это будет выполнено за один запрос и не важно сколько там полей.

Ну очевидно что чем меньше вы посылаете в запросе, тем быстрее он доходит и тем меньше делает СУБД. Но если вы не выполняете эти запросы по 10-100 раз в секунду, то не думаю что будет разница. Вы можете экономить на этом две миллисекунды и это не сделает вас счастливым.

Всё же запросы на чтение гораздо тяжелее и выполняются чаще. Кроме того еще неизвестно что лучше - делать один запрос на запись в котором чуть больше данных чем надо или же делать дополнительный запрос на чтение. Точнее известно - делать дополнительный запрос в данном случае хуже.

Answer 2

[pool]

упростите до следующего:

UPDATE `wp_person_post_data` 
SET `autor_id` = '1', `teach_start` = '', `address` = '', `email` = '', `phone` = '', `url` = '', `fb` = '', `vk` = '', `yutu` = '' 
WHERE `post_id` = 153

Answer 3

[Rsa97]

А вы уверены, что у вас есть строка, в которой одновременно выполняются все условия из WHERE?

Comments

[Дмитрий Поляков]

Я ошибся с очередностью значений SET и WHERE в функции $wpdb->update()