Problem mit $_GET

[McCry]

Hallo,
wie man sicherlich schon gelesen hat habe ich ein Problem mit $_GET.
Ich glaube zumindest, dass es daran liegt.

Also:
Ich habe mir einen Blog gemacht und bin gerade dabei eine Kommentarfunktion hinzuzufügen.
In phpmyadmin hab eich eine extra Tabelle für die Kommentare gemacht. Dort gibt es:
id, artikelid, name, text, datum
mein Code:

include "include/design/db_connect.php";
        
        $cName = $_POST['name'] ;
        $cText = $_POST['text'];
        $aID = $_GET["bid"];
        $cDate = date("Y.m.d");
        
        if($cName == "" or $cText == "")
        {
            echo "Du hast nicht alle Felder ausgefüllt.";
        } else {
            $eintragen = mysql_query("INSERT INTO comments VALUES ('', '{$aID}', '{$cName}', '{$cText}', '{$cDate}')");
            echo "Dein Kommentar wurde gespeichert.";
        }
        mysql_close($verbindung);

Die URL lautet z.B.: ritualist.bplaced.net/index.php?page=blog&bid=7
Dort hat bid den Wert 7. Wenn ich jetzt aber einen Testkommentar poste, hat dieser
anschließend in der Tabelle den Wert 0.
kann mir jemand sagen, woran das liegt?
Bis dahin schonmal Danke
LG McCry

 

[bdt600]

Vermutlich ist die Tabellenspalte auf Int konfiguriert, aber du übergibst eine Zeichenkette.

 

[McCry]

Und auf was muss ich sie dann ändern?

 

[bdt600]

Wenn es eine Ganzzahl sein soll, dann stimmt die Datenbank, aber dein Script nicht, oder eben andersrum. Das kannst nur du wissen.

Und so wie es aussieht, hast du bei dem Datum auch varchar oder ähnliches, warum nicht den Typ date? Dafür gibt es das ja extra.

Und noch ein Tipp: übernehme nie Daten von außen, ohne sie abzusichern, sprich, durch mysql_real_escape_string jagen.

 

[McCry]

In meiner db ist das ein INT. Und 7 ist doch eine Integer Zahl oder?
Und was müsste ich denn dann im Script ändern?

 

[bdt600]

Und 7 ist doch eine Integer Zahl oder?

Aber nicht

'{$aID}'

 

[McCry]

Und was ist im Code eine Integer?

Ach liegt das an den '' ?

 

[bdt600]

include "include/design/db_connect.php";

if(empty($_POST['name']) ||  empty($_POST['text']))
{
   echo "Du hast nicht alle Felder ausgefüllt.";
}
else
{
    $query = "INSERT
                INTO
                   `comments`
                SET
                  `aid` = "   . mysql_real_escape_string($_GET["bid"]) . ",
                  `name` = '" . mysql_real_escape_string($_POST['name']) . "',
                  `text` = '" . mysql_real_escape_string($_POST['text']) . "',
                  `datum`=now()";

    mysql_query($query)
       or die ("MySQL-Error: " . mysql_error());
}

mysql_close($verbindung);

 

[McCry]

Muss ich da noch etwas ändern oder sollte es auch so funktionieren?
Bei mir gehts nämlich nicht.

MySQL-Error: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ' `author` = 'hjhj', `text` = 'hjhjhj', ' at line 5

 

[bdt600]

Da ich deine Anpassungen nicht kenne, kann ich auch nichts dazu sagen.

 

[McCry]

das Formular sieht so aus:

<form action="index.php?page=blog" method="post">
   Name:</br>
   <input type="text" name="name"></br>
   Kommentar:</br>
   <textarea id="area" name="text"></textarea></br>
   <input type="submit" name="commentOk" value="Senden"></br>
</form>

 

[bdt600]

Du hast doch auch den PHP-Teil angepasst :(

Und wo soll dann im Script $_GET["bid"] herkommen?

 

[McCry]

Sorry, aber ich habe jetzt langsam keine Ahnung mehr, worauf du hinaus willst.

Was muss ich ändern, damit '{$aID}' eine Integer Zahl ist?

 

[bdt600]

Das Script, was ich hier gepostet habe, konntest du ja nicht 1:1 übernehmen. Wahrscheinlich hast du beim anpassen einen Fehler gemacht. Und beachte, dass bid garnicht an das Script per Get weitergegeben wird, sondern nur page. Von bid steht da nix beim Formular

 

[McCry]

Und wo soll dann im Script $_GET["bid"] herkommen?

Das kommt aus der URL.
hier kannst du dir mal ansehen was ich damit meine : http://ritualist.bplaced.net/index.php?page=blog
Es gibt eine Liste mit den Blogeinträgen, und wenn man auf einen der Links Klickt, kommt man zu dem Eintrag.

 

[bdt600]

Das kommt aus der URL.

Nö, tut es nicht. Das Formular sendet per Post name, text und commentOK und per Get kommt page rüber.

 

[McCry]

Neinnein. Es gibt auch ein $_GET["bid"]. Nur der Teil des Codes ist nicht in dem dabei, was ich gepostet habe.

 

[McCry]

Nö, tut es nicht. Das Formular sendet per Post name, text und commentOK und per Get kommt page rüber.

Aber wenn ich auf einen der links klicke, steht in der URL auch bid=7 oder 6

 

[bdt600]

<form action="index.php?page=blog" method="post">

Wo steht da was von bid??????

Wenn du das Formular abschickst, geht dir bid verloren!!!

 

[McCry]

Achso. Woher soll ich denn wissen, das du noch vom Formular redest.