[php][mysql] Przesuwanie bloków menu Opcje

[kociupk]

mam problem z przesuwaniem blokow menu mam o to taki kod php

[PHP] pobierz, plaintext 
<?php 
if($op == 'gora_kat') { 
	$zapytanie = "SELECT * FROM menu_kategorie WHERE pozycja < '".$_GET['id']."' ORDER BY pozycja DESC LIMIT 1"; 
	$query = mysql_query($zapytanie) or die(mysql_error()); 
	$dane = mysql_fetch_array($query); 
	 
	echo 'nowe '.$nowe_id =  $dane['pozycja']; 
	echo 'stare '.$_GET['id']; 
	$zmien_n_id = "UPDATE menu_kategorie SET pozycja = '".$_GET['id']."' WHERE pozycja = '".$nowe_id."' LIMIT 1"; 
	$query_n_id = mysql_query($zmien_n_id) or die(mysql_error()); 
	 
	$zmien_a_id = "UPDATE menu_kategorie SET pozycja = '".$nowe_id."' WHERE pozycja = '".$_GET['id']."' LIMIT 1"; 
	$query_a_id = mysql_query($zmien_a_id) or die(mysql_error()); 
	 
	if(!$query_n_id && !$query_a_id) { 
		echo 'Nie można zmienic pozycji kategorii!'; 
	} else { 
		echo 'ok'; 
	} 
} 
if($op == 'dol_kat') { 
	$zapytanie = "SELECT * FROM menu_kategorie WHERE pozycja > '".$_GET['id']."' ORDER BY pozycja ASC LIMIT 1"; 
	$query = mysql_query($zapytanie) or die(mysql_error()); 
	$dane = mysql_fetch_array($query); 
	 
	echo 'nowe '.$nowe_id =  $dane['pozycja']; 
	echo 'stare '.$_GET['id']; 
	$zmien_n_id = "UPDATE menu_kategorie SET pozycja = '".$nowe_id."' WHERE pozycja = '".$_GET['id']."' LIMIT 1"; 
	$query_n_id = mysql_query($zmien_n_id) or die(mysql_error()); 
	 
	$zmien_a_id = "UPDATE menu_kategorie SET pozycja = '".$_GET['id']."' WHERE pozycja = '".$nowe_id."' LIMIT 1"; 
	$query_a_id = mysql_query($zmien_a_id) or die(mysql_error()); 
	 
	if(!$query_n_id && !$query_a_id) { 
		echo 'Nie można zmienic pozycji kategorii!'; 
	} else { 
		echo 'ok'; 
	} 
} 
?>
[PHP] pobierz, plaintext 

no i niby wszystko dziala ok ale sie bloki menu nie przesuwaja http://koteluk.pl/sklep_internetowy/panel/lista_menu.php

[tommy4]

bo najpierw zmieniasz nowe id na stare id, a potem zmieniasz wszystkie wiersze, gdzie jest stare id na nowe id, czyli skutek jest taki, że zamieniasz w dwoch wierszach.

[kociupk]

a mi sie wydaje ze to innaczej robie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) najpierw zmieniam aktualne id na nowe (czyli nastepne albo poprzednie po aktualnym id)

[PHP] pobierz, plaintext 
<?php
$zmien_n_id = "UPDATE menu_kategorie SET pozycja = '".$nowe_id."' WHERE pozycja = '".$_GET['id']."' LIMIT 1";	 $query_n_id = mysql_query($zmien_n_id) or die(mysql_error());
?>
[PHP] pobierz, plaintext 

a pozniej zmieniam poprzednie albo nastepnie id na id aktualne czyli na $_GET['id'] ale to jakos nie dziala ;/

[PHP] pobierz, plaintext 
<?php
$zmien_a_id = "UPDATE menu_kategorie SET pozycja = '".$_GET['id']."' WHERE pozycja = '".$nowe_id."' LIMIT 1";	 $query_a_id = mysql_query($zmien_a_id) or die(mysql_error());
?>
[PHP] pobierz, plaintext 

raz dziala a raz nie;/

Ten post edytował kociupk 30.05.2007, 16:52:52

[Kicok]

a pozniej zmieniam poprzednie albo nastepnie id na id aktualne czyli na $_GET['id']

Nie zamieniasz "poprzedniego albo następnego id", tylko "poprzednie albo następne id + to, które zamieniłeś tym pierwszym zapytaniem".
Jeśli `pozycja` to klucz podstawowy tej tabeli, to skopałeś strukturę. A jeśli nie, to tu masz proste rozwiązanie twojego problemu:


1. Pobierz pozycję przesuwanego elementu

[SQL] pobierz, plaintext 
SELECT pozycja FROM menu_kategorie WHERE (id = $_GET['id'])
[SQL] pobierz, plaintext 

2. Pobierz id i pozycję poprzedniego/następnego elementu bazując na wyniku pierwszego zapytania:

[SQL] pobierz, plaintext 
SELECT id, pozycja AS `pozycja2` FROM menu_kategorie WHERE (pozycja < $pozycja) ORDER BY pozycja DESC LIMIT 1
[SQL] pobierz, plaintext 

3. Jeśli zapytanie nie zwróciło wyniku, to znaczy że dany element jest pierwszy/ostatni i nie robisz nic. W przeciwnym wypadku zamieniasz oba elementy pozycjami:

[SQL] pobierz, plaintext 
UPDATE menu_kategorie SET pozycja = $pozycja2 WHERE ( id = $_GET['id'] )
UPDATE menu_kategorie SET pozycja = $pozycja WHERE ( id = $id )
[SQL] pobierz, plaintext 

[Sopnet]

Jeśli `pozycja` to klucz podstawowy tej tabeli, to skopałeś strukturę.

Klucz podstawowy jest kluczem unikalnym wiec struktury by sobie nie skopal. Co innego ze teraz ma podwojne wpisy w polu pozycja. Wyjsciem byloby nadanie klucza uniklanego na ta pozycje. Ale tera jest za pozno.

A tak ogolnie to tak jak w poprzednich postach:
W momencie drugiego uaktualniania masz dwa wiersze z pozycja=$_GET['id']. Co z tego ze robisz limit 1 w zapytaniu, skoro mysql moze raz zgadnac o ktory tobie chodzilo a raz nie? Warto zawsze na trzezwo przesledzic co skrypt robi. Zalozenie miales sluszne, ale skrypt robi cos innego...