php - Gemme formular intastning ved validering - Udvikleren.dk

Tags: php validering mysql formular

Bruger #17071 @ 19.02.12 02:06

Halløjsa

Jeg fumler med validering af en formular til opdatering af allerede indtastede oplysninger. Jeg har følgede formular med validering, og som gemmer de indtastede oplysninger i en MySQL database. Dette virker helt fint og valideringen giver de forventede oplysninger om fejl, så som manglende indtastning:

PHP kode

 [code lang="PHP"]<?php
     error_reporting(0);
 
     $myconn = mysql_connect("localhost", "root", "") or die(mysql_error());
     mysql_select_db("my_database", $myconn) or die(mysql_error());
 ?>
 
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
 <head>
     <title>Create Record</title>
 
     <script type="text/javascript">
     <!--
     function GoToURL()
     {
     window.location="record_form_create.php";
     }
     //-->
     </script>
 </head>
 
 <body>
 
 <h1>Create Record</h1>
 
 <form action="record_form_create.php" method="post">
 
 <?php
     $ValidateForm = $_POST['ValidateForm'];
 
     if ($ValidateForm == "Yes") {
 
     $My_Column_1 = $_POST['My_Column_1'];
     $My_Column_2 = $_POST['My_Column_2'];
 
     if ($My_Column_1 == "") {
     ($Error_My_Column_1 = "Yes");
     }
 
     if ($My_Column_2 == "") {
     ($Error_My_Column_2 = "Yes");
     }
     }
 
     if (($ValidateForm == "")
     or ($Error_My_Column_1 == "Yes")
     or ($Error_My_Column_2 == "Yes")) {
 ?>
 
 <p>My Column 1 <?php if ($Error_My_Column_1 == "Yes") echo "Error"?>
 <br /><input name="My_Column_1" style="width: 100px" type="text" value="<?php echo $My_Column_1?>" /></p>
 
 <p>My Column 2 <?php if ($Error_My_Column_2 == "Yes") echo "Error"?>
 <br /><input name="My_Column_2" style="width: 100px" type="text" value="<?php echo $My_Column_2?>" /></p>
 
 <input name="submit" type="submit" value="SUBMIT" />
 <input name="reset" onclick="GoToURL();" type="reset" value="RESET" />
 
 <input name="ValidateForm" type="hidden" value="Yes" />
 </form>
 
 </body>
 </html>
 
 <?php
     } else {
 
     mysql_query("INSERT INTO my_table (My_Column_1, My_Column_2) VALUES ('$My_Column_1', '$My_Column_2')") or die(mysql_error());
 
     mysql_query($query);
 
     mysql_close($myconn);
 
     header("Location: records.php");
     }
 ?>

[/code]

... men altså, i nedenstående eksempel har jeg problemer med for det første at eet felt husker de indtastede oplysninger, når der er fejl i et andet felt, og for det andet at valideringen ikke giver de forventede oplysninger om fejl, så som manglende indtastning:

PHP kode

 <?php
     error_reporting(0);
 
     $Column_ID = mysql_real_escape_string($_GET['Column_ID']);
 
     $myconn = mysql_connect("localhost", "root", "") or die(mysql_error());
     mysql_select_db("my_database", $myconn) or die(mysql_error());
 
     $result = mysql_query("SELECT * FROM my_table WHERE Column_ID='$Column_ID'") or die(mysql_error());
 
     $row = mysql_fetch_array($result);
 ?>
 
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
 <head>
     <title>Edit Record</title>
 
     <script type="text/javascript">
     <!--
     function GoToURL()
     {
     window.location="record_form_edit.php?Image_ID=<?php echo $Column_ID?>">;
     }
     //-->
     </script>
 </head>
 
 <body>
 
 <h1>Edit Record</h1>
 
 <form action="record_form_edit.php?Column_ID=<?php echo $Column_ID?>" method="post">
 
 <?php
     $My_Column_1_Form = $_POST['My_Column_1'];
     $My_Column_1_Database = $row['My_Column_1'];
 
     $My_Column_2_Form = $_POST['My_Column_2'];
     $My_Column_2_Database = $row['My_Column_2'];
 
     $ValidateForm = $_POST['ValidateForm'];
 
     if ($ValidateForm == "Yes") {
 
     $My_Column_1 = $row['My_Column_1'];
     $My_Column_2 = $row['My_Column_2'];
 
     if ($My_Column_1 == "") {
     ($Error_My_Column_1 = "Yes");
     }
 
     if ($My_Column_2 == "") {
     ($Error_My_Column_2 = "Yes");
     }
     }
 
     if (($ValidateForm == "")
     or ($Error_My_Column_1 == "Yes")
     or ($Error_My_Column_2 == "Yes")) {
 ?>
 
 <p>My Column 1 <?php if ($Error_My_Column_1 == "Yes") echo "Error"?>
 <br /><input name="My_Column_1" style="width: 100px" type="text" value="<?php if (isset($_POST['submit'])) echo $My_Column_1_Form;
 else
 echo $My_Column_1_Database?>" /></p>
 
 <p>My Column 2 <?php if ($Error_My_Column_2 == "Yes") echo "Error"?>
 <br /><input name="My_Column_2" style="width: 100px" type="text" value="<?php if (isset($_POST['submit'])) echo $My_Column_2_Form;
 else
 echo $My_Column_2_Database?>" /></p>
 
 <input name="submit" type="submit" value="SUBMIT" />
 <input name="reset" onclick="GoToURL();" type="reset" value="RESET" />
 
 <input name="ValidateForm" type="hidden" value="Yes" />
 
 </form>
 
 </body>
 </html>
 
 <?php
     } else {
 
     $My_Column_1 = $_POST['My_Column_1'];
     $My_Column_2 = $_POST['My_Column_2'];
 
     $query = "UPDATE my_table SET My_Column_1='$My_Column_1', My_Column_2='$My_Column_2' WHERE Column_ID='$Column_ID'";
 
     mysql_query($query);
     mysql_close($myconn);
 
     header("Location: records.php");
     }
 ?>

Jeg har tidligere brugt en lignende metode i ASP med to variabler, een for databaseoplysninger og een for formularoplysninger, for at det enkelte felt kunne "maintain state" og det virkede helt fint.

Er der nogen, der kan give en hjælpende hånd?

7 svar postet i denne tråd vises herunder
1 indlæg har modtaget i alt 4 karma

Sorter efter stemmer Sorter efter dato

Bruger #13559 @ 19.02.12 08:49

198

Alle de fejlmeddelelser du nævner, kommer når der mangler én eller flere af værdierne i arrayet $_POST. Hvis du laver et GET request vil alle værdier mangle, og hvis du ikke udfylder alle værdier ved et POST request får du også fejl. Hvis du ikke regner med at ha' mere end én form på hver side, kan du helt droppe "ValidateForm" feltet og istedet tjekke om det pågældende request er POST

PHP kode

 if ($_SERVER['REQUEST_METHOD'] == 'POST') {
 
 }

Derudover skal du sikre dig at begge 'My_Column_#' er udfyldt. Det kan du f.eks. gøre ved

PHP kode

 $My_Column_1 = isset($_POST['My_Column_1']) ? $_POST['My_Column_1'] : "" ;

Hvis du bliver træt af at gentage ovenstående logik kan du lave en lille hjælperfunktion

PHP kode

 function get_post_var($key, $default="") {
     return isset($_POST[$key]) ? $_POST[$key] : $default ;
 }

og så kan du skrive

PHP kode

 $My_Column_1 = get_post_var('My_Column_1');

Indlæg senest redigeret d. 19.02.2012 08:50 af Bruger #13559

Bruger #17015 @ 19.02.12 03:23

614

Måske er jeg bare træt, men jeg er ikke sikker på at jeg forstår hvad fejlen er... :-)

Hvis du ikke allerede har gjort det, så prøv at kommentere din error_reporting(0); linje ud (tilføj // i starten af linjen) så viser den de fejl der evt kunne være. Det ville være en stor hjælp.

Bruger #17071 @ 19.02.12 04:10

Hej Gustav

Jeg kan da sagtens vise dig fejlmeddelelserne. I det script, som jeg har problemer med (som opdaterer), er der følgende:

Notice: Undefined index: My_Column_1 in record_form_edit.php on line 36
Notice: Undefined index: My_Column_2 in record_form_edit.php on line 39
Notice: Undefined index: ValidateForm in record_form_edit.php on line 42
Notice: Undefined variable: Error_My_Column_1 in record_form_edit.php on line 63
Notice: Undefined variable: Error_My_Column_2 in record_form_edit.php on line 68

I det script, der opretter (det øverste, og som rent faktisk virker efter min hensigt), er der også fejlmeddelelser som følger:

Notice: Undefined index: ValidateForm in record_form_create.php on line 30
Notice: Undefined variable: Error_My_Column_1 in record_form_create.php on line 51
Notice: Undefined variable: Error_My_Column_2 in record_form_create.php on line 54

Så skal det jo heller være nogen hemmelighed at detterhersens PHP er nyt for mig endnu, så jeg takker for hvad hjælp du kan give :-)

Bruger #17015 @ 19.02.12 04:42

614

Undefined index og undefined variable betyder at du bruger hhv en værdi i et array eller en variabel som ikke er sat på forhånd. Ikke en kritisk fejl.

Får du ikke en anden fejl når du submitter din form i edit filen? Det kan jeg se at du ikke har gjort på denne fejl: Notice: Undefined index: ValidateForm in record_form_edit.php on line 42

Bruger #17071 @ 19.02.12 05:38

Næhh, der er egentligt ikke andre fejl end det...

Der sker det, at valideringen fungerer fint, så længe der er skrevet noget i felterne (under opdateringen), men så snart et felt er tomt, så bliver det tomme felt alligevel accepteret, uden at valideringen virker.

Når jeg så vil gå tilbage og lave endnu en opdatering og udfylde et felt, der er tomt, så får jeg godtnok den ønskede fejlmeddelelse, men jeg kan ikke "submitte" og kommer ikke videre. Er det ikke bare sært?

Jeg kan muligvis midlertidigt gøre scriptet tilgængeligt "live", hvis det er nogen hjælp.

Bruger #9859 @ 19.02.12 08:33

Hej,

php er ikke min stærkeste side, men mener dog at mysql_real_escape_string kræver en connection til database for at virke.

I dit eksempel ser det ud som om du kalder mysql_real_escape_string først, og derefter opretter en connection til mysql.

Bruger #17015 @ 19.02.12 10:32

614

@AH, korrekt - men den tomme streng "" trigger en false værdi når den bliver brugt i boolske statements og notices er ikke fejl - men advarsler. Det betyder ikke noget for om det kan køre eller ej. Normalt kører man alligevel ikke med E_NOTICE i error_reporting... Hvis overhovedet, så i hvert fald kun i logs.

@Jørgen Lehmann, korrekt - mysql_real_escape_string bruger funktionalitet fra MySQL til at escape. Den virker ikke hvis der ikke er connection. Men den burde give en E_Warning - og det ser ikke sådan ud fra de fejl vi har fået.

Indlæg senest redigeret d. 19.02.2012 10:34 af Bruger #17015

Gemme formular intastning ved validering

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler