Как прибавить к текущей дате дни?

Question

[Deman1]

Здравствуйте, подскажите как добавить дни к дате для sql, добавить надо к date_end только количество дней может быть разное 3, 15, 20,55 и тд, как сделать чтобы запрос сразу генерировался с количеством дней?

$sql = "INSERT INTO `order` SET ";
$sql .= "user_id = '" . (int)$user_id . "', ";
$sql .= "status = '0', ";
$sql .= "date = NOW()";
$sql .= "date_end = ????????????";
$db->query($sql);

Answer 1

[Михаил]

Во-первых, никогда, не собирайте SQL-запрос по кускам с помощью конкатенации строк. Это очень, очень, ОЧЕНЬ, плохой способ, за который тысячи разработчиков поплатились взломом сайтов через SQL-injection. Используйте параметры запроса.

Во-вторых, у вас в запросе ошибка: INSERT и SET не могут использоваться совместно.

В-третьих, проще всего воспользоваться функцией DATE_ADD, передавая ей количество дней как параметр, например так:

$pdo->prepare("INSERT INTO `order`(user_id, date, date_end) VALUES(:userId, :date, DATE_ADD(:date, INTERVAL :days DAY))")
	->execute(["userId" => 100,
			   "date" => date("Y-m-d"),
			   "days" => 10
			  ]);

Полный пример

Обратите внимание: phpize.online предоставляет объект $pdo. Вам в вашем коде придется его инициализировать вручную, либо понять как использовать параметры вместе с тем объектом $db, который у вас есть. Если это Laravel - то он тоже умеет аналогичным образом подставлять параметры.

Comments

[Akina]

Михаил,

у вас в запросе ошибка: INSERT и SET не могут использоваться совместно.

Судя по бэктикам в запросе - это MySQL. А в нём INSERT и SET прекрасно сочетаются. См. https://dev.mysql.com/doc/refman/8.0/en/insert.html, вариант 2.

[Михаил]

Akina, оу... Спасибо за исправление. Не знал про это извращение в MySQL. Но все равно лучше так не делать - меньше проблем будет при необходимости перейти на другую БД.

[Akina]

Михаил, вот не считаю это извращением. Более того, считаю странным, что в стандарте нет этого синтаксиса - он прекрасно согласуется с синтаксисом UPDATE, а также делает текст запроса очень наглядным, потому что чётко показывает соответствие полей и выражений присвоения. например, в этом варианте синтаксиса такая хня, как несовпадение количества полей с количеством значений, просто невозможна - а это обычная ошибка начинающих (и не только) программистов.

[Михаил]

Akina, возможно это было бы и удобно. Но его нет в стандарте, а значит - увы... Мне приходится работать с разными БД (последнее время обычно с постгрес), поэтому все что не в стандарте я буду использовать только если иначе никак не получится.

Answer 2

[Newto]

$plus_days = 3;
$sql = "INSERT INTO `order` SET ";
$sql .= "user_id = '" . (int)$user_id . "', ";
$sql .= "status = '0', ";
$sql .= "date = NOW()";
$sql .= "date_end = NOW() + INTERVAL {$plus_days} DAY";
$db->query($sql);

Comments

[Михаил]

SQL injection, welcome!! Я уж не говорю о том что INSERT и SET - это несочетаемо.

Не знаете сами - так хоть не учите других.

[Newto]

Во первых, SET стоит в изначальном запросе ТСа. SET и INSERT в MySQL вполне себе сочетаемы. Во вторых, я обозначил переменную plus_days в коде, ДО sql запроса, определив ей определенное значение. О какой инъекции тогда вы говорите? Если уж переменная будет приниматься в скрипт извне, то это уже дело автора, как он её будет обрабатывать. Если же речь о самом способе построении запроса через конкатенацию строк, то это к данному вопросу не имеет отношения, я не пытаюсь учить ТСа тому, о чем он не спрашивает, я лишь дополнил его изначальный запрос. Поэтому ваши претензии мне непонятны.

Answer 3

[Akina]

В случае MySQL просто прибавить соотв. INTERVAL:

INSERT INTO `order`
SET ...
    date_end = CURRENT_DATE + INTERVAL :days_to_add DAY

В других СУБД - использовать функцию DATEADD().