Warum trägt er nicht ein?

  • Warum trägt er nicht ein?

    Hallo.
    Kann mir bitte jemand helfen, warum er bei diesem Skript nicht in die DB einträgt?
    Hab schon ewig lange gesucht, mit anderen Skripts verglichen aber ich denk anscheinden zu kompliziert...

    Quellcode

    1. <?php
    3. $verbindung = mysql_connect("localhost", "rezensionen" , "***")
    4. or die("Verbindung zur Datenbank konnte nicht hergestellt werden");
    6. mysql_select_db("rezensionen") or die ("Datenbank konnte nicht ausgewählt werden");
    8. $user = $_POST["user"];
    9. $titel = $_POST["titel"];
    10. $aufgabe = $_POST["aufgabe"];
    11. $datum_aufgabe = $_POST["datum_aufgabe"];
    13. if($datum_aktuell == "" OR $user == "" OR $titel == "" OR $aufgabe == "" OR $datum_aufgabe == "")
    14. {
    15. echo "Eingabefehler. Bitte alle Felder korrekt ausfüllen. <a href=\"eintragen.php\">Zurück</a>";
    16. exit;
    17. }
    19. $ergebnis = mysql_query("SELECT user, titel, aufgabe, datum_aufgabe FROM aufgabe1);
    20. $anzahl = mysql_num_rows($ergebnis);
    22. if($anzahl == 0)
    23. {
    24. $entry = "INSERT INTO aufgabe1 (user, titel, aufgabe, datum_aufgabe) VALUES ('$user', '$titel', '$aufgabe', '$datum_aufgabe')";
    25. $eintrag = mysql_query($entry);
    27. if($eintrag == true)
    28. {
    29. echo "Eintrag erstellt. <br> <a href=\"aufgaben.php\">Zu den Aufgaben</a><br />\n";
    30. echo "<br />";
    31. echo "<br />";
    32. echo "Eintrag erstellt. <br> <a href=\"index.php\">Zum Index</a>";
    33. }
    34. else
    35. {
    36. echo "Fehler beim Eintragen";
    37. }
    38. }
    40. else
    41. {
    42. echo "Fehler";
    43. }
    45. ?>
    47. </body>
    48. </title>
    Alles anzeigen


    Die HTML-Datei wäre:

    Quellcode

    1. <html>
    2. <head>
    3. <title>Neues Loginsystem</title>
    4. </head>
    6. <body background="grafiken/bg.gif">
    8. <center>
    10. <table bgcolor="#ffffff" border="3">
    11. <tr><td>
    12. <center><font size="6" face="Arial">&nbsp;&nbsp;&nbsp;Tragen Sie hier Ihre Aufgaben ein, die Sie erledigen müssen&nbsp;&nbsp;&nbsp;</font>
    13. </tr></td>
    14. </table>
    16. <br />
    17. <br />
    19. <table bgcolor="#ffffff" border="3">
    20. <tr><td>
    22. <form action="eintragen.php" method="post">
    24. <table border="0">
    25. <tr><td width="250">
    27. Ihr Name:&nbsp;</td><td>
    28. <input type="text" size="24" maxlength="30"
    29. name="user">
    30. </td>
    31. </tr>
    32. <tr>
    33. <td>
    35. Titel der Aufgabe:&nbsp;</td><td>
    36. <input type="text" size="24" maxlength="50"
    37. name="titel">
    38. </td>
    39. </tr>
    40. <tr>
    41. <td>
    43. Beschreibung der Aufgabe:&nbsp;</td><td>
    44. <input type="text" cols="30" rows="12" size="24" maxlength="50"
    45. name="aufgabe">
    46. </td>
    47. </tr>
    48. <tr>
    49. <td>
    51. Datum, um die Aufgabe zu erledigen:&nbsp;</td><td>
    52. <input type="text" size="24" maxlength="30"
    53. name="datum_aufgabe">
    54. </td>
    55. </tr>
    56. <tr>
    57. <td>
    58. </table>
    62. <br /><br />
    63. <center>Danke für den Eintrag.</center>
    64. <br />
    65. <center><input type="submit" value="Aufgabe eintragen"></center>
    66. </form>
    68. </tr></td>
    69. </table>
    71. <a href="aufgaben.php">Zu den Aufgaben</a>
    73. </center></center>
    75. </body>
    76. </html>
    Alles anzeigen


    Danke ;)
    Nach langer Zeit wieder ein bisschen als Programmierer tätig

  • Jetzt zeigt er die Meldung von Zeile 43 ("Fehler") an...
    Außerdem hab ich in Zeile 14

    Quellcode

    1. if($datum_aktuell == "" OR $user == "" OR $titel == "" OR $aufgabe == "" OR $datum_aufgabe == "")

    das $datum_aktuell rausgenommen, da das ja ncht definiert ist, oder?
    Nach langer Zeit wieder ein bisschen als Programmierer tätig

  • Auch ich das schonmal irgendwem gesagt hab: Das direkte Eintragen von Werten in die Datenbank ist gefährlich (SQL-Injections). Du solltest die Strings vor dem Eintragen in die Datenbank escapen :!:
    ~ mfg SeBa

    Ich beantworte keine PMs zu Computer-/Programmierproblemen. Bitte wendet euch an das entsprechende Forum.

    [Blockierte Grafik: http://i.creativecommons.org/l/by-sa/3.0/80x15.png]

  • der html code ist nicht eingerückt, den hab ich mir nicht angeschaut
    aber es zählt sowieso nur das was ankommt

    ich nehme an mit der Abfrage willst du erreichen, dass die selben Daten nicht 2x abgeschickt werden, oder?

    dann musst du das auch so formulieren:

    Quellcode

    1. SELECT user, titel, aufgabe, datum_aufgabe FROM aufgabe1
    2. WHERE user = '$user'
    3. AND titel = '$titel'
    4. AND aufgabe = '$aufgabe'
    5. AND datum_aufgabe = '$datum_aufgabe';


    Über Sinnigkeit lässt sich jedoch streiten. Schöner wäre ein sauberer Primärschlüssel.
    Würde dir das quakenet Tutorial (Punkt 34) empfehlen oder eben ein PHP/MySQL Buch

  • Tut mir leid, aber wir haben Verständigungsschwierigkeiten. Ich will, dass ich 4 Sachen eingebe und er diese in die Datenbank einträgt. Fertig. Mehr will ich nicht. Das hat schon einmal mit dem Code funktioniert, nur jetzt geht es nicht mehr. Ich habe mich als hilfesuchend hier her gewendet...
    Nach langer Zeit wieder ein bisschen als Programmierer tätig

  • Wenn du keine Überprüfung machen willst, dann kannst du die ganze Bedingung auch weglassen. Das wären #20 bis #24 und #35 bis #38

    Ich sag ja nur, dass du mit deinem aktuellen Code die Anzahl aller Einträge überprüfst.
    Und in Abhängigkeit derer machst du dann einen Eintrag (falls die ganze Tabelle leer ist) oder du gibst eben einen Fehler aus (wenn schon irgendetwas drinne steht)

    Das ist das, was dein code sagt.

  • Das müsste dann so aussehen, oder?

    Quellcode

    1. <?php
    3. $verbindung = mysql_connect("localhost", "rezensionen" , "***")
    4. or die("Verbindung zur Datenbank konnte nicht hergestellt werden");
    6. mysql_select_db("rezensionen") or die ("Datenbank konnte nicht ausgewählt werden");
    8. $user = $_POST["user"];
    9. $titel = $_POST["titel"];
    10. $aufgabe = $_POST["aufgabe"];
    11. $datum_aufgabe = $_POST["datum_aufgabe"];
    13. if($datum_aktuell == "" OR $user == "" OR $titel == "" OR $aufgabe == "" OR $datum_aufgabe == "")
    14. {
    15. echo "Eingabefehler. Bitte alle Felder korrekt ausfüllen. <a href=\"index.html\">Zurück</a>";
    16. exit;
    17. }
    18. $entry = "INSERT INTO aufgabe1 (user, titel, aufgabe, datum_aufgabe) VALUES ('$user', '$titel', '$aufgabe', '$datum_aufgabe')";
    19. $eintrag = mysql_query($entry);
    21. if($eintrag == true)
    22. {
    23. echo "Eintrag erstellt. <br> <a href=\"aufgaben.php\">Zu den Aufgaben</a><br />\n";
    24. echo "<br />";
    25. echo "<br />";
    26. echo "Eintrag erstellt. <br> <a href=\"index.html\">Zum Index</a>";
    27. }
    28. else
    29. {
    30. echo "Fehler beim Eintragen";
    31. }
    32. ?>
    Alles anzeigen


    Das funkt aber leider nicht. Dann bringt er die Fehlermeldung aus Zeile 16...

    Edit: Problem gelöst. Ich hatte im "if" das $datum_aktuell noch drin...
    Nach langer Zeit wieder ein bisschen als Programmierer tätig