Как выполнить такой запрос?

Question

[animr]

Друзья, помогите разобраться с запросом:

$sql = "UPDATE table SET Sort = :Sort + 1 WHERE Sort = (SELECT Sort FROM table WHERE Sort > :Sort)";

Написал функцию чтоб исполнялся запрос:

public static function SortProduct($value){
$db = Db::getConnection();
$sql = "UPDATE table SET Sort = :Sort + 1 WHERE Sort = (SELECT Sort FROM table WHERE Sort > :Sort)";
$result = $db->prepare($sql);
$result->bindParam(':Sort', $value, PDO::PARAM_INT);
return $result->execute();
}

Поле Sort уникальное и числовое.
Функция должна увеличивать все числа в таблице в поле Sort на +1, начиная со следующего числа.

Answer 1

[Илья]

По порядку:
SET Sort = :Sort + 1
То есть вы увеличивается значение не на +1, а на :Sort (отправленный параметр) + 1, так и должно быть?
-
Ну а по сути в остальном все должно работать, в чем вопрос то? Если вылетает ошибка: какая ошибка вылетает?

Comments

[animr]

Не выполнятся этот запрос... и ошибок нет

[Илья]

animr: если нет ошибок, значит нет строк удовлетворяющий условию.

[animr]

Вот функция где я его вызываю:

public static function updateProductById($id, $options) {
Prod::SortProduct($_POST['Sort']);

$db = Db::getConnection();

// Текст запроса к БД
$sql = "UPDATE table SET
Name = :Name,
Article = :Article,
Price = :Price,
WHERE id = :id";

$result = $db->prepare($sql);
$result->bindParam(':id', $id, PDO::PARAM_INT);
$result->bindParam(':Name', $options['Name'], PDO::PARAM_STR);
$result->bindParam(':Article', $options['Article'], PDO::PARAM_STR);
$result->bindParam(':Price', $options['Price'], PDO::PARAM_INT);
return $result->execute();
}

что тут не так?

[animr]

а строк там полно удовлетворяющих этим условиям

[Илья]

animr: ну и на хер этот код? Вы думаете что если в вашей функции все нормально, то где то выше косяк?? Возьмите, руками прямо в базу напишите запрос:

UPDATE table SET Sort = 666 + 1 WHERE Sort = (SELECT Sort FROM table WHERE Sort > 666)

Вместо "666" число которые вы посылаете в запросе, и посмотрите произойдет что-то или нет.
Когда я говорил, что поля "нет строк удовлетворяющий условию", я говорил про SQL а не про PHP!

[Илья]

animr: если сам SQL запрос работает корректно, то проверяйте что у вас приходит в функцию в качестве $value. Если все ок, то подавайте в суд на PHP, вас нае...обманули!

[Илья]

animr: вероятнее всего вам нужно будет добавить LIMIT 1 в запрос и все

[animr]

ты понял что я хочу получить ?

[Илья]

animr: я понял что ты не можешь объяснить что тебе надо, и тебе лень сделать:
1. проверку запроса напрямую к базе;
2. проверку входных параметров функций;
3. хоть немного пошевелиться.

[animr]

Умник, ты тут умничаешь или помогаешь? если я разбирался бы в этом я сюда не писал. Я только начинаю это все учить. Если помогаешь помогай, иначе не трать ни свое не мое время.

[Илья]

animr: я умничаю? Я тебе пишу ЧТО нужно сделать, ты в ответ шлешь ... КОД, непонятно зачем. Если ты не знаешь КАК сделать то ЧТО я написал, так ты спроси КАК это сделать.

[animr]

если ты помогаешь то слушай, я объясню тебе что мне нужно. В базе есть поле Sort, в ней уникальные значения числа. Это поле позволяет сортировать товары на сайте по возрастанию. Когда я хочу поднять товар вперед, я прописываю там число например 5, и все остальные товары у которых значение поля больше или равно 5ти, увеличиваются как "значение + 1" . Теперь понятно ?

[animr]

я проверил что ты говоришь, и понял что не правильно делаю...

[animr]

потому прислал код

[Илья]

animr: ты скинул код который ВЫШЕ функции, там ничего с базой не делается и поэтому смысла в этом коде нет.
Что приходит в функцию которая выполняет запрос? Проверить можно:

public static function SortProduct($value){
	var_dump($value); die();
	$db = Db::getConnection();
	$sql = "UPDATE table SET Sort = Sort + 1 WHERE Sort IN (SELECT Sort FROM table WHERE Sort >= :Sort)";
	$result = $db->prepare($sql);
	$result->bindParam(':Sort', $value, PDO::PARAM_INT);
	return $result->execute();	
}

После запроса, у тебя на экране появится все что успел вывести скрипт и значение которое приходит в фукнцию.
В функции я также поправил SQL: теперь у тебя у ВСЕХ записей, у которых Sort больше переданного значения, увеличиться на +1.

[Илья]

animr: короче косяк был в SQL потому что в PHP все верно (если параметр в функцию приходит нормально)

[Илья]

animr: в свете последних событий, вот такой должен быть SQL:

UPDATE table SET Sort = Sort + 1 WHERE Sort >= :Sort";

[animr]

Спасибо, сейчас проверю и отпишусь.

[animr]

Не работает, входное значение неправильное

[Илья]

animr: NULL или что-то другое? Если NULL - то значит вы либо не отправляете запрос методом POST (проверьте какой метод указан в тэге, должно быть ну и собственно в форме должно быть поле с именем 'Sort'), либо некорректные данные отправляете.

[animr]

Я редактирую например какую нибудь строчку с id = 20, у него значение числа поля сортировки равна 200, хочу его переместить на 7 место, и число 200 изменил на 7, и все строчки у которых значение поля Sort > 7 увеличиваются на "значение поля + 1" там у кого старое значение 7 становится 8, где 8 = 9, где 9= 10 и т.д. КАК такое реализовать.????

[animr]

туда попадает значение редактируемого товара по id

[Илья]

animr: вот примерно так, если какая то ошибка будет, то у тебя выскочить исключение.

<?php

public static function updateProductById($id, $options) {
	$db = Db::getConnection();
	try {
		$db->beginTransaction();
		$db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); // для выброса исключений
		$sort = (int) $_POST['Sort'];
		/*
		 * Первый запрос (в SQL добавил сорт, или он в другом месте меняется)
		 */
		$sql = "UPDATE table SET Name = :Name, Article = :Article, Price = :Price, Sort = :Sort WHERE id = :id";
		$result = $db->prepare($sql);
		$result->bindParam(':id', $id, PDO::PARAM_INT);
		$result->bindParam(':Sort', $sort, PDO::PARAM_INT);
		$result->bindParam(':Name', $options['Name'], PDO::PARAM_STR);
		$result->bindParam(':Article', $options['Article'], PDO::PARAM_STR);
		$result->bindParam(':Price', $options['Price'], PDO::PARAM_INT);
		$result->execute();
		/*
		 * Второй запрос (правки остальных индексов)
		 */
		$sql = "UPDATE table SET Sort = Sort + 1 WHERE Sort > {$sort}";
		$db->exec($sql);
		//
		$db->commit();
	}
	catch (Exception $e) {
		$db->rollBack();
		throw $e;
	}
	return true;
}