Schauen ob alle Felder ausgefüllt sind

[Codemaster24]

Hi ihr,

wie gesagt bin ich ja noch neu in dem Thema PHP und so.
Ich betreibe learning by doing.

So sieht mein Script derzeit so aus:

<?php
 
 mysql_connect("localhost","root","123456") or die
 ("Es besteht keine Verbindung zu MySQL");
 mysql_select_db("befragung") or die
 ("Diese Datenbank existiert nicht");
 
 
 $Name = $_POST['Name'];
 $Uhrzeit = $_POST['Uhrzeit'];
 $Grund = $_POST['Grund'];
 
 if ($Name=="" OR $Uhrzeit=="" OR $Grund=="")
 {
     echo "Bitte füllen Sie alle Felder aus";
 }
 else
 {
     echo "Ihr Eintrag wurde gespeichert";
 }
 
mysql_query("INSERT INTO user (Name, Uhrzeit, Grund)
         VALUES ('$Name', '$Uhrzeit', '$Grund')");
 mysql_close();
 

 
 
?>

Mir geht es besonders um den Teil:

 
 if ($Name=="" OR $Uhrzeit=="" OR $Grund=="")
 {
     echo "Bitte füllen Sie alle Felder aus";
 }
 else
 {
     echo "Ihr Eintrag wurde gespeichert";
 } 

Nun!
Egal ob ich meine Felder ausgefüllt habe oder nicht... er schreibt mir die Daten trotzdem in die DB.
Also wenn ich NAME nicht ausfülle sagt er mir zwar es seinicht ausgefüllt. Schreibt aber trotzdem in die DB halt dann OHNE Namen drin...

wie kann man das verhindern?
Also das er dann NICHTS in die DB reinschreibt?

Danke schonmal an euch

[Kropff]

indem du pansen deinen query in den else teil packst.

gruß
peter

[em-!x]

Logisch, wenn dein query außerhalb der Bedingung steht

// zu spät... ich sollte die Groß-/Kleinschreibung links liegen lassen

[Codemaster24]

OK!

So klappts

<?php
 
 mysql_connect("localhost","root","123456") or die
 ("Es besteht keine Verbindung zu MySQL");
 mysql_select_db("befragung") or die
 ("Diese Datenbank existiert nicht");
 
 
 $Name = $_POST['Name'];
 $Uhrzeit = $_POST['Uhrzeit'];
 $Grund = $_POST['Grund'];
 
 if ($Name=="" OR $Uhrzeit=="" OR $Grund=="")
 {
     echo "Bitte füllen Sie alle Felder aus";
 }
 else
 {
     mysql_query("INSERT INTO user (Name, Uhrzeit, Grund)
         VALUES ('$Name', '$Uhrzeit', '$Grund')");
     echo "Ihr Eintrag wurde gespeichert";
 }
 

 mysql_close();
 

 
 
?>

Danke

Wie gesagt. Bin noch neu und mache einfach Learning by doing

[Kropff]

Original geschrieben von Codemaster24
Wie gesagt. Bin noch neu und mache einfach Learning by doing

OffTopic:

---

dann solltest du dir vielleicht erst mal einen anderen nick zulegen

---

gruß
peter

[Codemaster24]

*fg*

Stimmt