PHP gevorderde |
|
Nee, daar maak je je topic inderdaad niet voor. Maar het is wel een heel veel gemaakte fout. Je script wordt er namelijk veel slomer door en dat wil jij denk ik niet..
Maar als jij eigenwijs wilt zijn, ik vind het best hoor.
Zet eerst je variabelen maar eens buiten quotes en gebruik addslashes() (want wat gebeurt er als je hallo'hoi of iets dergelijks intypt?).
En waarom maak je de variabele $aantal aan als je hem maar 1 keer gebruikt? Een variabele maak je aan als je hem meer dan 2 keer gebruikt...
En waarom gebruik je dit:
<?php
if($_POST['zoek'])
?>
<?php if($_POST['zoek']) ?>
$_POST['zoek'] bestaat niet altijd, dus die geeft een foutmelding (in de vorm van een notice)..
Dit zou al beter zijn:
<?php
if(isset($_POST['zoek'])) // er werd op de knop GEKLIKT, als je op enter drukt gebeurt er niks
?>
<?php if(isset($_POST['zoek'])) // er werd op de knop GEKLIKT, als je op enter drukt gebeurt er niks ?>
Maar wat nou als er een leeg formulier wordt gepost? (denk aan checkboxes, en ja: Die krijg je nog wel 's hoor, maak je daar maar geen zorgen over).
Inderdaad, dan krijg je het formulier na het posten weer te zien... En dan kun je dus niet gaan zoeken of weet ik wat.
En ook als je op enter drukt, dus in plaats van op de knop te klikken, krijg je het formulier weer te zien..
Met deze manier van posten lukt het wel:
<?php
if($_SERVER['REQUEST_METHOD'] == 'POST') // er was een post
?>
<?php if($_SERVER['REQUEST_METHOD'] == 'POST') // er was een post ?>
|