[ljungi]

Hallo Leute!

Ich bins mal wieder

Ich sitze hier und brüte über folgendem Problem.
Ein Kunde von mir will ein Sortierungsverfahren nach folgendem Verfahren haben:

text | input type=text
text | input
text | input

In diesen Input feldern steht nun die Sortierungszahl.

Nehmen wir das Beispiel

1
2
3
4
5
6
7
8

Nun will er die 1 auf die 8 und die 8 auf die 3 und die 4 auf die 5 verschieben, und möchte dazu nur die Zahlen ändern:

8
2
3
5
4
6
7
3

So sähe das dann aus.

Nun soll das Script automatisch erkennen welche Zahl geändert wurde.

Soweit ok, ich habe 1 hidden-Feld mit "altsort", also der alten Position.


Ich hänge irgendwie über diesem Code fest:

PHP-Code:

$movedata = count($_POST['sort']);
        if($movedata != 0) {
            for($i=0;$i<$movedata;$i++) {
                //Erst mal sehen, welche geaendert wurden... 
                if($_POST['altsort'][$i] != $_POST['sort'][$i]) {
                    //Zunaechst schauen obs das schon mal in der DB gibt.
                    $ex = $db->query("SELECT sort FROM ipse_presse WHERE sort = '".$_POST['sort'][$i]."' && id != '".$_POST['upid'][$i]."' && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                    $c = $db->numRows($ex);
                    if($db->numRows($ex) != 0) {
                        if($_POST['sort'][$i] == ($_POST['altsort'][$i]-1)) {
                            $db->query("UPDATE ipse_presse SET sort = 999999 WHERE sort = '".$_POST['sort'][$i]."' && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                            $db->query("UPDATE ipse_presse SET sort = sort-1 WHERE sort = '".$_POST['altsort'][$i]."' && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                            $db->query("UPDATE ipse_presse SET sort = '".$_POST['altsort'][$i]."' WHERE sort = 999999 && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                        }elseif($_POST['sort'][$i] == ($_POST['altsort'][$i]+1)) {
                            $db->query("UPDATE ipse_presse SET sort = 999999 WHERE sort = '".$_POST['sort'][$i]."' && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                            $db->query("UPDATE ipse_presse SET sort = sort+1 WHERE sort = '".$_POST['altsort'][$i]."' && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                            $db->query("UPDATE ipse_presse SET sort = '".$_POST['altsort'][$i]."' WHERE sort = 999999 && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                        }else {
                            $db->query("UPDATE ipse_presse SET sort = sort+1 WHERE sort > '".$_POST['sort'][$i]."' && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                            $db->query("UPDATE ipse_presse SET sort = sort+1 WHERE sort = '".$_POST['sort'][$i]."' && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                            $db->query("UPDATE ipse_presse SET sort = '".$_POST['sort'][$i]."' WHERE id = '".$_POST['upid'][$i]."'");                    
                        }
                    }else {
                        //Ansonsten einfach nur verschieben...
                        $db->query("UPDATE ipse_presse SET sort = '".$_POST['sort'][$i]."' WHERE id = '".$_POST['upid'][$i]."'");                    
                    }
                }
            }
        } 


Nun verschiebt er aber nicht korrekt.


Ergebnis:

2
3
4
5
6
8
9
10

Ist meine Lösung ansatzweise schon mal gut, oder gibts da einen Trick?

Danke schon mal!

Niko

[mepeisen]

Ich scheitere schon daran, dein Beispiel zu verstehen....

Mit dem "1 auf 8 verschieben" meinst du, dass bei eins der Wert 8 überschrieben wird, 8 aber an seiner Position erhalten bleibt, richtig?

Dass müsste doch folgendes rauskommen oder?
8 (1)
2
3
5 (4)
5 (wieso steht da bei dir im Beispiel eine 4?!?!?)
6
7
3 (3)

[WarrenFaith]

Mein Verstand scheitert dabei auch. Macht für mich 0 Sinn. Hast du vielleicht ne Page wo man so halbwegs was ansehen kann um es zu verstehen?

[ljungi]

WarrenFaith du hast eine PM in einer Minute in deinem Posteingang mit den Zugangsdaten und der URL.

Für alle noch mal:

Alte Ordnung:

1
2
3

Neue Ordnung, gewünscht:

1->3
2->1
3->2

D.h. der User gibt in das Feld wo 1 drin steht ne 3 ein. Damit wird das auf die 3. Position verschoben.
Das gleiche mit der Option 2. Und 1.

Hoffe ich hab das nun einigermaßen verständlich rüber gebracht.

Wer sich das noch mal online ansehen will, schreibt hier bitte. Möchte nicht öffentlich die Daten hergeben

[ljungi]

So, habs nun hinbekommen.

Nach etlichen durchdenkungen mit Papier und Stift hab ich die Logik nun kapiert.

Hier der Quellcode

PHP-Code:

                        if($_POST['sort'][$i] == ($_POST['altsort'][$i]-1)) {
                            $db->query("UPDATE ipse_presse SET sort = 999999 WHERE sort = '".$_POST['sort'][$i]."' && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                            $db->query("UPDATE ipse_presse SET sort = sort-1 WHERE sort = '".$_POST['altsort'][$i]."' && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                            $db->query("UPDATE ipse_presse SET sort = '".$_POST['altsort'][$i]."' WHERE sort = 999999 && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                        }elseif($_POST['sort'][$i] == ($_POST['altsort'][$i]+1)) {
                            $db->query("UPDATE ipse_presse SET sort = 999999 WHERE sort = '".$_POST['sort'][$i]."' && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                            $db->query("UPDATE ipse_presse SET sort = sort+1 WHERE sort = '".$_POST['altsort'][$i]."' && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                            $db->query("UPDATE ipse_presse SET sort = '".$_POST['altsort'][$i]."' WHERE sort = 999999 && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                        }elseif($_POST['sort'][$i] > ($_POST['altsort'][$i]+1)) {  
                            $db->query("UPDATE ipse_presse SET sort = sort-1 WHERE sort <= '".$_POST['sort'][$i]."' && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                            $db->query("UPDATE ipse_presse SET sort = sort+1 WHERE sort < '".$_POST['altsort'][$i]."' && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                            $db->query("UPDATE ipse_presse SET sort = '".$_POST['sort'][$i]."' WHERE id = '".$_POST['upid'][$i]."'");                    
                        }elseif($_POST['sort'][$i] < ($_POST['altsort'][$i]+1)) {
                            $db->query("UPDATE ipse_presse SET sort = sort+1 WHERE sort >= '".$_POST['sort'][$i]."' && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                            $db->query("UPDATE ipse_presse SET sort = sort-1 WHERE sort > '".$_POST['altsort'][$i]."' && rubrik = '".$_POST['rubrik'][$i]."' && kategorie = '".$_POST['kategorie'][$i]."'");
                            $db->query("UPDATE ipse_presse SET sort = '".$_POST['sort'][$i]."' WHERE id = '".$_POST['upid'][$i]."'");                    
                        } 


Grüße
Niko

Update: < und > vertauscht gehabt. Diese Version geht nu endgültig

... und gefeuert wirst du trotzdem mit dem code. fristlos.
http://de.wikipedia.org/wiki/SQL_Injection

grüße
axo

[ljungi]

Der Code wird davor auf Angriffe überprüft.