[MySQL][PHP]Zmiana Kolejności wpisów Opcje

[konradkw]

Bardzo banalna zmiana kolejności tylko nie do końca działa jak trzeba ..

Możecie podpowiedzieć czy widzicie tutaj błąd?

[PHP] pobierz, plaintext 
 
if (isset($_POST['up'])) 
 
{ 
	$id = $_POST['id'];
	$k = $_POST['kolejnosc'];
	$kup = ($k + 1);
	$kdown = ($k - 1);
 
	 $zmien = mysql_query("UPDATE `baza` SET `kolejnosc` = '$kup' WHERE `id` = '$id'");	
 
	if ($zmien) 
	{
	$zmienn = mysql_query("UPDATE `baza` SET `kolejnosc` = '$k' WHERE `kolejnosc` = '$kup'");	
	}
 
}
 
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 
if (isset($_POST['down'])) 
 
{ 
	$id = $_POST['id'];
	$k = $_POST['kolejnosc'];
	$kup = ($k + 1);
	$kdown = ($k - 1);
 
	 $zmien = mysql_query("UPDATE `baza` SET `kolejnosc` = '$kdown' WHERE `id` = '$id'");	
 
	if ($zmien)
 
	{
	$zmienn = mysql_query("UPDATE `baza` SET `kolejnosc` = '$k' WHERE `kolejnosc` = '$kdown'");
	}
}
 
 
[PHP] pobierz, plaintext 

Powyższy kodzik działa tylko połowicznie.. zmienia $zmienn dziala ale $zmien sie nie wykonuje lub sie wykonuje ale nie zamienia tego co trzeba.
to będzie jakiś banalny błąd ale nie widzę go jakoś...


wszystkie wartości się poprawnie przekazują - sprawdzałem kilka razy.

[CuteOne]

$zmien = mysql_query("UPDATE `baza` SET `kolejnosc` = '$kup' WHERE `id` = '$id'") or die(mysql_error());

[konradkw]

próbowałem - niema błędów.

[CuteOne]

To podstaw testowe dane do zapytania np.
$zmien = mysql_query("UPDATE `baza` SET `kolejnosc` = '1000' WHERE `id` = '1'");

i sprawdź czy pójdzie.

[konradkw]

sprawdzilem tak i sie zmienia.

[b4rt3kk]

Zauważ, że może zmieniasz w kółko to samo:

[PHP] pobierz, plaintext 
$zmien = mysql_query("UPDATE `baza` SET `kolejnosc` = '$kup' WHERE `id` = '$id'");
[PHP] pobierz, plaintext 

tutaj warunkiem jest ID, więc nie ma problemu, nie ma drugiego takiego samego. Ale tutaj:

[PHP] pobierz, plaintext 
$zmienn = mysql_query("UPDATE `baza` SET `kolejnosc` = '$k' WHERE `kolejnosc` = '$kup'");
[PHP] pobierz, plaintext 

warunek sprawdza pole kolejnosc, a istnieją już dwie krotki o takiej samej kolejności. Więc może tak być że zmieniasz najpierw przykładowo kolejność krotki o id 18, z 9 na 10, a później znów z 10 na 9.

[konradkw]

no racja tak na to nie popatrzyłem ... masz może pomysł jak to obejść? może odwrócić kolejność wykonywania? lub sprawdzić id do zmiany w $zmienn i wtedy zmienic?

[b4rt3kk]

Pierwszą czynnością jaką powinieneś zrobić powinno być pobranie id sąsiedniego pola (wg kolejności) i z tym pod jakąś zmienną możesz już dokonywać UPDATE. Właściwie, to prawda jest taka, że robisz UPDATE na wszystkie pola, które spełniają warunek kolejnosc=$kolejnosc, bo nie masz LIMIT na końcu.

Ten post edytował b4rt3kk 23.11.2012, 00:12:57

[konradkw]

Dalem tak:

[PHP] pobierz, plaintext 
 
if (isset($_POST['up'])) 
 
{ 
	$id = $_POST['id'];
	$k = $_POST['kolejnosc'];
	$kup = ($k + 1);
	$kdown = ($k - 1);
 
	$kaled = mysql_query ("SELECT * FROM `baza` WHERE `kolejnosc` = '$kup' LIMIT 1");
	while ($ked = mysql_fetch_array($kaled, MYSQL_ASSOC)) { $aa = $ked['id']; }
 
	$zmien = mysql_query("UPDATE `baza` SET `kolejnosc` = '$kup' WHERE `id` = '$id' LIMIT 1") or die(mysql_error());	
 
	if ($zmien) 
	{
		$zmienn = mysql_query("UPDATE `baza` SET `kolejnosc` = '$k' WHERE `id` = '$aa' LIMIT 1");
	}
 
}
 
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 
if (isset($_POST['down'])) 
 
{ 
	$id = $_POST['id'];
	$k = $_POST['kolejnosc'];
	$kup = ($k + 1);
	$kdown = ($k - 1);
 
	$kaled = mysql_query ("SELECT * FROM `baza` WHERE `kolejnosc` = '$kdown' LIMIT 1");
	while ($ked = mysql_fetch_array($kaled, MYSQL_ASSOC)) { $aa = $ked['id']; }
 
	$zmien = mysql_query("UPDATE `baza` SET `kolejnosc` = '$kdown' WHERE `id` = '$id'") or die(mysql_error());	
 
	if ($zmien)
 
	{
		$zmienn = mysql_query("UPDATE `baza` SET `kolejnosc` = '$k' WHERE `id` = '$aa'");
 
	}
}
 
[PHP] pobierz, plaintext 

I działa jak trzeba

Dziękuję serdecznie!

Zawsze dobrze takie "świeże spojrzenie" wykorzystać - za długie patrzenie na ten sam nawet banalny kod często do niczego nie prowadzi .. tylko nerwy psuje (IMG:style_emoticons/default/smile.gif)

Ten post edytował konradkw 23.11.2012, 00:27:42