Checkbox

[aboutme]

Hi,

ich habe folgendes Problem:

Der User hat eine Freundesliste und soll seine besten Freunde durch Ankreuzen der Checkbox markieren. Doch es funktioniert nicht richtig.. Ich hab keine Ahnung wo mein Fehler liegt. Kann mir bitte jemand helfen?

$anzahl = @mysql_result(mysql_query("select id from rockdudes_friends order by id desc limit 0,1"),0,0);

    for($x = 0; $x <= $anzahl; $x++)
    {
        if($_POST[$x])
        {
                mysql_query("UPDATE rockdudes_friends SET best='1' WHERE uid = '".$dude['id']."' AND fid = '".mysql_real_escape_string($_POST[$x])."'");
                mysql_query("UPDATE rockdudes_friends SET checked='1' WHERE uid = '".$dude['id']."' AND fid = '".mysql_real_escape_string($_POST[$x])."'");
        }
    }

  $best="UPDATE rockdudes_friends SET best='0' WHERE uid = '".$dude['id']."' AND checked = '0'";
  $friend = mysql_query($best);
  
  $best="UPDATE rockdudes_friends SET checked='0' WHERE uid = '".$dude['id']."'";
  $friend = mysql_query($best);
  
$done = '<div id="done"><h3>.: done!</h3><p>Friends goufen geännert.</p></div>'; 

[martin_duisburg]

woher kommt $dude['id']."

warum machst du das?


[php]
WHERE uid LIKE $dude['id']
[php]

[aboutme]

das ist die Userid des angemeldeten Users. Die wird in einer anderen Datei abgefragt und includet

[martin_duisburg]

und warum

WHERE uid LIKE $dude['id'] ?

nicht WHERE uid = $dude['id']?

[aboutme]

ok jetzt ist es =

aber das ändert ja nichts am script...

[martin_duisburg]

und was funzt jetzt genau nicht...

keine der Abfrage nur ein Teil der Abfragen?

[aboutme]

die user die markiert sind werden nicht als markiert in die DB geschrieben

[martin_duisburg]

also dieser Teil hier?

 for($x = 0; $x <= $anzahl; $x++)
    {
        if($_POST[$x])
        {
                mysql_query("UPDATE rockdudes_friends SET best='1' WHERE uid = '".$dude['id']."' AND fid = '".mysql_real_escape_string($_POST[$x])."'");
                mysql_query("UPDATE rockdudes_friends SET checked='1' WHERE uid = '".$dude['id']."' AND fid = '".mysql_real_escape_string($_POST[$x])."'");
        }
    } 

[aboutme]

genau.. aber das komische ist. Sonst hat der immer funktioniert. Das versteh ich nicht.

[$_Baker]

Zeig doch mal den Teil, der das Formular aufbaut.

[martin_duisburg]

was enthält $_POST[$x]

wird die Schleife überhaupt durchlaufen?

was enthält $anzahl?

und warum schickst du zwei Querys ab wenn du den selben Datensatz veränderst?

 mysql_query("UPDATE rockdudes_friends SET best='1',checked='1'  WHERE uid = '".$dude['id']."' AND fid = '".mysql_real_escape_string($_POST[$x])."'"); 

[aboutme]

is gut hab den Fehler gefunden. Danke

    for($x = 0; $x <= $anzahl; $x++)
    {
        if($_POST[$x])
        {
                mysql_query("UPDATE rockdudes_friends SET best='1' WHERE id = '".mysql_real_escape_string($_POST[$x])."'");
                mysql_query("UPDATE rockdudes_friends SET checked='1' WHERE id = '".mysql_real_escape_string($_POST[$x])."'");
        }
    } 

so stimmts.

Danke

[zaubatrik]

Hi,

warum machst du aus den zwei Querys nicht eine?

mysql_query("UPDATE rockdudes_friends SET checked='1', best='1' WHERE id = '".mysql_real_escape_string($_POST[$x])."'");

Ist doch völlig unnötig da zwei abzuschicken!

[H2O]

Ausserdem macht mysql_real_escape_string() für nummerische Werte wenig Sinn. Besser int_val() benutzen und in der Abfrage die Anführungszeichen entfernen:

$sql = "
    UPDATE 
        rockdudes_friends 
    SET 
        checked = 1, 
        best = 1
    WHERE
        id = " . int_val($_POST[$x]);
$res = mysql_query($sql) or exit("Fehler: " . mysql_error() . "<br />Abfrage: $sql<br />";
... 

[$_Baker]

OffTopic:

---

Die Funktion int_val möchste ich mal sehn

---