Smartere måde at tjekke felter på

fx

<?php
$errors = array();
$fejl = 3; //antal forskellige felter
if(isset($_POST['user'])){ // der er trykket på submit, og "user" findes
  if(trim($_POST['user']) == '') $errors['user'] = 'Ikke udfyldt!';
  else {
    // tjek om brugernavn er optages, hvis ja sæt $errors['user'] til 'Optaget'
    // hvis ikke, så lav $fejl--;
  }
}
else $errors['user'] = "Ikke udfyldt!";

if(isset($_POST['paswd'])) { // form sendt, og "paswd" findes
  if(trim($_POST['paswd']) == '') $errors['paswd'] = 'Ikke udfyldt!';
  else $fejl--;
}
else $errors['paswd'] = "Ikke udfyldt!";

if(isset($_POST['name'])) { // form sendt, og "name" findes
  if(trim($_POST['name']) == '') $errors['name'] = 'Ikke udfyldt!';
  else $fejl--;
}
else $errors['name'] = "Ikke udfyldt!";

if(!$fejl){ // ingen fejl!
  $user = $_POST['user'];
  $paswd = $_POST['paswd'];
  $name = $_POST['name']:
  // gør noget - fx indsæt i db
}

// formen

echo '<form ....>';
echo 'User: <input type="text" name="user">';
if(isset($errors['user'])) echo $errors['user'];

echo '<br><br>Paswd: <input type="password" name="paswd">';
if(isset($errors['paswd'])) echo $errors['paswd'];

echo '<br><br>Name: <input type="text" name="name">';
if(isset($errors['name'])) echo $errors['name'];
?>

---

men, det ville være nemmere med en klasse der generede formen - så kunne den aut. udylde value="", hvis der var blevet sendt!

Morten

Jeg prøver lige at arbejde lidt med det. Har sikkert en masse spørgsmål om lidt ;)

Det synes jeg nu godt nok opfører sig underligt. Så snart jeg kommer ind på siden står der ud for felterne. Ikke udfyldt! Og det er på sin vis også korrekt, men har ikke ligefrem haft chancen for det.

Således ser scriptet ud nu:
<?php
$errors = array();
$fejl = 3; //antal forskellige felter
if(isset($_POST['user'])){ // der er trykket på submit, og "user" findes
  if(trim($_POST['user']) == '') $errors['user'] = 'Ikke udfyldt!';
  else $fejl--;
   
  }

else $errors['user'] = "Ikke udfyldt!";

if(isset($_POST['paswd'])) { // form sendt, og "paswd" findes
  if(trim($_POST['paswd']) == '') $errors['paswd'] = 'Ikke udfyldt!';
  else $fejl--;
}
else $errors['paswd'] = "Ikke udfyldt!";

if(isset($_POST['name'])) { // form sendt, og "name" findes
  if(trim($_POST['name']) == '') $errors['name'] = 'Ikke udfyldt!';
  else $fejl--;
}
else $errors['name'] = "Ikke udfyldt!";

if(!$fejl){ // ingen fejl!
  $user = $_POST['user'];
  $paswd = $_POST['paswd'];
  $name = $_POST['name'];

print "Og det sættes nu i db'en!";

}

// formen

echo "<form action='register.php' method='post'>";
echo "User: <input type='text' name='user'>";
if(isset($errors['user'])) echo $errors['user'];

echo "<br /><br />Paswd: <input type='password' name='paswd'>";
if(isset($errors['paswd'])) echo $errors['paswd'];

echo "<br /><br />Name: <input type='text' name='name'>";
if(isset($errors['name'])) echo $errors['name'];
print "
<input type='submit' name='submit' value='Register'>
</form>
";
?>

Men det ser ud til at virke, bortset fra at der ikke skal stå Ikke udfyldt! Så snart man kommer ind på siden.

Og de korrekte felter slettes forresten også hvis der er et felt der ikke er korrekt.

Kan du ikke komme med det klasse eksempel du snakkede om?

du skal bare slette dem der står

if(isset(..){
  ...
}
else ...

til

if(isset(...)){
  ...
}

---

du kan selv lege med noget

<input type="text" name="user"<?php if isset($_POST['user']) echo ' value="' . $_POST['user'] . '"'; ?>

her er det dog nemmere at lave en klasse til det - eller en anden form for funktion, så du ikke skal skrive helt så meget

Morten

Okay, tror jeg er ved at forstå det nu.
Men jeg er lidt i tvivl om denne:

$errors = array();
$fejl = 3; //antal forskellige felter
if(isset($_POST['user'])){ // der er trykket på submit, og "user" findes
  if(trim($_POST['user']) == '') $errors['user'] = 'Ikke udfyldt!';
  else {
    // tjek om brugernavn er optages, hvis ja sæt $errors['user'] til 'Optaget'
    // hvis ikke, så lav $fejl--;
  }
}
else $errors['user'] = "Ikke udfyldt!";

Jeg er i tvivl om den $fejl--; der og hvordan du vil have jeg skal sætte $errors['user'] til 'Optaget'.

Vil du prøve se om du kan få det her til at passe sammen med:

if(mysql_result(mysql_query("SELECT COUNT(*) FROM registration WHERE user='$user'"),0) == 1) {
    print "The Desired username is not available!<br />";
}

Kan ikke gennemskue det. Sætter self point op.

Behøver self ikke forgå på den måde. Men kan ikke lige se mig ud af det.

if(isset($_POST['user'])){ // der er trykket på submit, og "user" findes
  if(trim($_POST['user']) == '') $errors['user'] = 'Ikke udfyldt!';
  else {
    if(mysql_result(mysql_query("SELECT COUNT(*) FROM registration WHERE user='$user'"),0) == 1)
    {
      $errors['user'] = 'Optaget';
    }
    else
    {
      $fejl--;
    }
  }
}

det var da ikke så svært - vel ;)

Morten

Nej okay.. :)

Kan bare ikke gennemskue $fejl--; funktion.

Har aldrig set --; før.

-- = træk et fra tallet (det samme som $tal = $tal - 1;)

++ = læg et til tallet

Morten

Fair nok :)

Men det virker nu stadig ikke. Der er dog ingen fejl beskeder:

$errors = array();
$fejl = 1; //antal forskellige felter
if(isset($_POST['login'])){ // der er trykket på submit, og "user" findes
  if(trim($_POST['login']) == '') $errors['login'] = 'empty';
  else {
  if(mysql_result(mysql_query("SELECT COUNT(*) FROM registration WHERE login='$login'"),0) == 1)
    {
      $errors['login'] = 'In use';
    }
    else
    {
      $fejl--;
    }
  }
}
else $errors['login'] = "Field Username is required!";



if(!$fejl){ // ingen fejl!
  $login = $_POST['login'];

print "Smid i db.";

}

?>


<form action="register.php" method="post">
*Username:<br>
<input type="text" name="login" class="inputfield" <? if(isset($_POST['login'])) print ' value="' . $_POST['login'] . '"'; ?>><br>
<input type="submit" name="submit" value="Register" class="inputbutton">
</form>

Den siger ikke noget om ikke udfyldte felter mere. Og siger ikke hvis brugernavn er optaget. Men den siger dog Smid i db hvis bruger navn ikke er optaget. Lidt hjælp? :)

[koden over dette]

<form action="register.php" method="post">
*Username:<br>
<input type="text" name="login" class="inputfield" <? if(isset($_POST['login'])) print ' value="' . $_POST['login'] . '"'; ?>><br>
<?php if(isset($errors['login'])) echo $errors['login']; ?>
<input type="submit" name="submit" value="Register" class="inputbutton">
</form>

Morten

Ahh.. Kunne være jeg skulle vågne op. Mange tak skal du have.

Jeg skal til at lave mere error checking på andre felter senere. Der kommer nok flere spørgsmål :)

Du får tak!

Smid svar btw :)

min harddisk gik ned - undskyld ventetiden - nu har jeg næsten mit Gentoo oppe at køre (det er sku' fedt)

Morten