Нужно ли проверка INSERT, UPDATE, DELETE?

Question

[Ibishka]

if (mysqli_query($connect, "UPDATE ...")) {
           // ....
} else $data['msg'] = 'Something went wrong. Please try again';

Или это лишнее?

Comments

[Gennady S]

Нужно для кого? Вам? База данных не выполнит запрос в двух случаях: неверные данные пользователя (и их необходимо проверить перед записью в базу) или системная исключительная ситуация, вопрос уже администрирования СУБД. О первом, видимо, пользователя стоит проинформировать, о втором — уведомить скорее себя.

[FanatPHP]

Gennady S, не бывает этой ошибки из-за неверных данных пользователя

[Gennady S]

FanatPHP, попробуйте в целое число записать строку или превысить допустимые диапазоны.

[FanatPHP]

Gennady S, это ошибка не пользовтеля, а программиста

[Gennady S]

FanatPHP, бессмысленное пояснение. «Пользовательские данные» как сущность вполне определенное и конкретное понятие. Это точка. И можно добавить, что к ним относится и попытка взлома, на которую, в свою очередь, программист может быть и не в силах повлиять (не допустить ошибок, но потерпеть крах).

Answer 1

[FanatPHP]

Хороший вопрос, ответ на который знают очень немногие.
Чтобы на него ответить, надо обладать опытом как программиста, так и пользователя веб-сайтов. А это уникальное сочетание. Большая часть пользователей РНР не обладает ни тем ни другим, очень небольшая часть каким-то одним, и только единицы - и тем и другим.

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

Поэтому

• вместо колупания с проверками каждого запроса просто говорим драйверу БД чтобы кидал ошибку
  
• пишем простой обработчик ошибок, который
  
  1. логирует ошибку для программиста
     
  2. выставляет правильный код НТТР ответа
     
  3. пишет что-то юзеру если он напрямую запрашивает страницу, а не аяксом
     
  
  
  

Соответственно, если у тебя запрос аяксом, то проверяешь код ответа и пишешь 'Something went wrong. Please try again' яваскриптом.