Witam.

Specyfikacja sytuacji:

Elementy w moim systemie menu są sortowane według pola 'order' od zera w górę, w panelu administracyjnym jest opcja przesuwania ich w górę oraz i w dół. Oto funkcja, która służy mi do przykładowego przesunięcia danego elementu tablicy "w górę":

[PHP] pobierz, plaintext 
<?php
function up_menu_item() {
  global $sql;
  $menuitemid = GetPageArgument('menu_item_id', 0); // biorę id itemu do przesunięcia z $_GET
 
  $sql->query('SELECT `order` FROM `###menuitems` WHERE `id`='.$menuitemid); // pobieram aktualną wartość pola 'order'
  if ($sql->num_rows() < 1) return; // jeśli nie ma takowego, przerywam
  $row = $sql->fetch_array();
  $baseitem_order = $row['order'];
 
  $sql->query('SELECT `id` FROM `###menuitems` WHERE `order`='.($baseitem_order-1)); // pobieram id jednego pola "wyżej"
  if ($sql->num_rows() < 1) return; // jeśli nie ma takowego, przerywam
  $row = $sql->fetch_array();
  $otheritem_id = $row['id'];
 
  $sql->query('UPDATE `###menuitems` SET `order`=`order`-1 WHERE `id`='.$menuitemid); // zamieniam ordery
  $sql->query('UPDATE `###menuitems` SET `order`=`order`+1 WHERE `id`='.$otheritem_id);
}
?>
[PHP] pobierz, plaintext 

Specyfikacja rezultatu:

Działa bez zarzutu.

Specyfikacja problemu:

Cholera, tutaj są aż cztery zapytania x)

Pytanie:

Ma ktoś pomysł, jak zamienić te order'y szybciej i za pomocą mniejszej liczby zapytań?

Witam.

Wykombinowałem kod przy pomocy ktorego robi sie to jednym zapytaniem. Nic nie psuje jesli np. wiersz juz jest na samej gorze a ty mimo wszystko bedziesz go chcial przesunac wyzej. Nic nie musisz sprawdzac. Przetestowyany i dziala ale ... tak niefortunnie wybrales pole 'order' slowo zarezerwowane ze trzeba je brac w cudzyslowia i trzeba robic dodatkowe aliasy. Dla mojej testowej tabeli (categories) kod wyglada nastepujaco.

[SQL] pobierz, plaintext 
UPDATE categories c, ( (SELECT categories_id, ord.categories_order
                        FROM categories, (SELECT categories_order
                                          FROM categories WHERE categories_order < (SELECT categories_order
                                                                    FROM categories WHERE categories_id = 6
                                                                    LIMIT 1  )
                                          ORDER BY categories_order DESC LIMIT 1  ) AS ord
                        WHERE categories_id = 6)
                       UNION
                       (SELECT c1.categories_id, ord.categories_order
                        FROM categories c1, (SELECT categories_order
                                             FROM categories WHERE categories_id = 6) AS ord
                        WHERE c1.categories_order < (SELECT categories_order
                                                     FROM categories WHERE categories_id = 6
                                                     LIMIT 1  )
                        ORDER BY c1.categories_order DESC LIMIT 1  )) AS nowe_dane
SET c.categories_order = nowe_dane.categories_order
WHERE c.categories_id = nowe_dane.categories_id
[SQL] pobierz, plaintext 

Wszystkie wystapienia "6" musisz zastapic swoja zmienna $menuitemid (przyklad jest dla kategorii o id 6)
categories -> ###menuitems
categories_id -> id
categories_order -> order

Przerobka nie powinna sprawiac klopotow a tylko troche gimnastyki bo tak jak mowilem musisz wszystko apostrofowac (tzn nazwa tabeli i kolumna order) a przy moich nazwach nie trzeba

pozdrawiam

Ps. jesli mialbys problem z przerobka daj znac, pomoge
jesli bys nei mogl odnalesc sie w tym kodzie to moge wytlumaczyc krok po kroku

Heaven, jesteś moim pozytywnym bożyszczem :*

W mojej wersji wygląda to w ten sposób:

[SQL] pobierz, plaintext 
UPDATE ###menuitems c, ( (SELECT `id`, ord.`order`
                        FROM ###menuitems, (SELECT `order`
                                          FROM ###menuitems WHERE `order` < (SELECT `order`
                                                                    FROM ###menuitems WHERE `id` = '.$menuitemid.'
                                                                    LIMIT 1   )
                                          ORDER BY `order` DESC LIMIT 1   ) AS ord
                        WHERE `id` = '.$menuitemid.')
                       UNION
                       (SELECT c1.`id`, ord.`order`
                        FROM ###menuitems c1, (SELECT `order`
                                             FROM ###menuitems WHERE `id` = '.$menuitemid.') as ord
                        WHERE c1.`order` < (SELECT `order`
                                                     FROM ###menuitems WHERE `id` = '.$menuitemid.'
                                                     LIMIT 1   )
                        ORDER BY c1.`order` DESC LIMIT 1   )) AS nowe_dane
SET c.`order` = nowe_dane.`order`
WHERE c.`id` = nowe_dane.`id`
[SQL] pobierz, plaintext 

I działa bez zarzutu. Dziękuję bardzo . Jednak, jeśli miałbyś minimalne chęci, mógłbyś przynamniej pobieżnie wytłumaczyć, jakim mechanizmem to działa? Paru rzeczy nie mogę skojarzyć, a chcę się doskonalić :]

powiedzmy ze mamy tabele categories (przytocze ta moja wersje a w niej wiersze

chodzi nam o zamienienie przy tych dwoch wierszach wartosci kolumny categories_order

mozna to zapisac tak:

[SQL] pobierz, plaintext 
UPDATE categories c, (SELECT 1 AS categories_id, 14 AS categories_order
                      UNION
                      SELECT 2 AS categories_id, 13 AS categories_order) AS nowe_dane
SET c.categories_order = nowe_dane.categories_order
WHERE c.categories_id = nowe_dane.categories_id
[SQL] pobierz, plaintext 

czyli:

[SQL] pobierz, plaintext 
SELECT 1 AS categories_id, 14 AS categories_order
UNION
SELECT 2 AS categories_id, 13 AS categories_order
[SQL] pobierz, plaintext 

tworzy nam to tabele z 2 wierszami

i nastepnie to wpada w update i dla categories_id = 1 jest ustawiane categories_order na 14 a dla categories_id = 2 categories_order ustawiane na 13

czyli mamy update 2 wierszy za pomoca jednego zapytania ale potrzebowalibysmy jeszcze jednego zapytania na pobranie danych do tego zapytania
my chcemy zrobic zapytanie gdzie jedyna wiadoma jest id kategori ktora chcemy przesunac w gore o jeden czyli catgories_id='2'

na poczatku utworzymy zapytanie ktore zwroci wiersz *(2) z powyzszej tabeli czyli id = 2 i order = 13

[SQL] pobierz, plaintext 
SELECT categories_id, ord.categories_order
FROM categories, (SELECT categories_order
                  FROM categories WHERE categories_order < (SELECT categories_order
                                            FROM categories WHERE categories_id = 2
                                            LIMIT 1  )
                  ORDER BY categories_order DESC LIMIT 1  ) AS ord
WHERE categories_id = 2
[SQL] pobierz, plaintext 

zaczynajac od zapytania najbardziej zagniezdzonego

[SQL] pobierz, plaintext 
SELECT categories_order
FROM categories WHERE categories_id = 2
LIMIT 1
[SQL] pobierz, plaintext 

to zwroci nam categories_order dla naszej przesowanej kategori id = 2 czyli 14 i to wstawiam do zapytania mniej zagniezdzonego

[SQL] pobierz, plaintext 
SELECT categories_order
FROM categories WHERE categories_order < (14)
ORDER BY categories_order DESC LIMIT 1
[SQL] pobierz, plaintext 

to znowy zwroci nam najwiesza wartosc categories_order mniejsza od 14 czyli 13

czyli ogolnie cale zapytanie wyglada mniejwiecej tak

[SQL] pobierz, plaintext 
SELECT categories_id, ord.categories_order
FROM categories, 13 AS ord
WHERE categories_id = 2
[SQL] pobierz, plaintext 

no i to da wynik w postaci tabeli z jednym wierszem

polowe mamy teraz jeszcze wiesz z id = 1 i order = 14

[SQL] pobierz, plaintext 
SELECT c1.categories_id, ord.categories_order
FROM categories c1, (SELECT categories_order
                     FROM categories WHERE categories_id = 2) AS ord
WHERE c1.categories_order < (SELECT categories_order
                             FROM categories WHERE categories_id = 2
                             LIMIT 1  )
ORDER BY c1.categories_order DESC LIMIT 1
[SQL] pobierz, plaintext 

a wiec zaczynajac od zapytania zagniezdzonego

[SQL] pobierz, plaintext 
SELECT categories_order
FROM categories WHERE categories_id = 2
[SQL] pobierz, plaintext 

to zwraca nam categories_order = 14 wiec zapytanie wyglada teraz tak

[SQL] pobierz, plaintext 
SELECT c1.categories_id, ord.categories_order
FROM categories c1, 14 AS ord.categories_order
WHERE c1.categories_order < (SELECT categories_order
                             FROM categories WHERE categories_id = 2
                             LIMIT 1  )
ORDER BY c1.categories_order DESC LIMIT 1
[SQL] pobierz, plaintext 

[SQL] pobierz, plaintext 
SELECT categories_order
FROM categories WHERE categories_id = 2
LIMIT 1
[SQL] pobierz, plaintext 

to zwroci nam categories_order dla naszej przesowanej kategori id = 2 czyli 14 (w sumie to nie potrzebenie bo bylo to zrobione w poprzednim zapytaniu i mozna to znalesc pod aliasem ord.categories_order wiec na koncu napisze krotsza wersje tego calego zapytania)

[SQL] pobierz, plaintext 
SELECT c1.categories_id, ord.categories_order
FROM categories c1, 14 AS ord.categories_order
WHERE c1.categories_order < 14
ORDER BY c1.categories_order DESC LIMIT 1
[SQL] pobierz, plaintext 

to cale zapytanie zwroci nam id wiersza ktory ma byc zamieniony z naszym wybranym wierszem czyli 1 i categories_order 14

teraz te 2 selecty laczymy za pomoca union i wstawiamy do update i to wszystko
mam nadzieje ze te wypociny chociaz troche sa zrozumiale

tak wiec wersja troszke krotsza

[SQL] pobierz, plaintext 
UPDATE categories c, ( (SELECT categories_id, ord.categories_order
                        FROM categories, (SELECT categories_order
                                          FROM categories WHERE categories_order < (SELECT categories_order
                                                                    FROM categories WHERE categories_id = 6
                                                                    LIMIT 1 )
                                          ORDER BY categories_order DESC LIMIT 1 ) AS ord
                        WHERE categories_id = 6)
                       UNION
                       (SELECT c1.categories_id, ord.categories_order
                        FROM categories c1, (SELECT categories_order
                                             FROM categories WHERE categories_id = 6) AS ord
                        WHERE c1.categories_order < ord.categories_order
                        ORDER BY c1.categories_order DESC LIMIT 1 )) AS nowe_dane
SET c.categories_order = nowe_dane.categories_order
WHERE c.categories_id = nowe_dane.categories_id
[SQL] pobierz, plaintext 

Jestem niezmiernie wdzięczny za wytłumaczenie... i pełen podziwu za to, że Ci się chciało