Onderwerp: PHP emailformulier probleem

Beste WHTers,

Ik ben bezig met een PHP emailformulier; hij werkt prima op 1 ding na :
de velden realname , email en boodschap moeten verplichte velden
zijn. Als ik de velden niet invul wordt het formulier toch verzonden en
wordt er ook geen foutmelding weergegeven. Heeft iemand een idee
wat er mis is ?

PHP code :

PHP Code:

<?php
if (!isset($_POST['realname']) || !isset($_POST['email']) || !isset($_POST['boodschap'])) {
    echo 'U heeft niet alle verplichte velden ingevuld!';
    exit;
}

$realname = htmlspecialchars($_POST['realname']);
$email = htmlspecialchars($_POST['email']);
$boodschap = htmlspecialchars($_POST['boodschap']);
$adres = htmlspecialchars($_POST['adres']);
$pcode = htmlspecialchars($_POST['pcode']);
$woonpl = htmlspecialchars($_POST['woonpl']);
$telnr = htmlspecialchars($_POST['telnr']);
$emailok = htmlspecialchars($_POST['emailok']);


$tijd = time();
$datum = strftime('%d/%m/%y %H:%M', $tijd);
$ip = getenv('REMOTE_ADDR');
$message = $realname.' met het e-mailadres '.$email.' en het IP '.$ip.' stuurde op '.$datum.' het volgende bericht:

Naam : '.$realname.'
Email : '.$email.'
Adres : '.$adres.'
Postcode : '.$pcode.'
Woonplaats : '.$woonpl.'
Telefoon : '.$telnr.'
E-mailing : '.$emailok.'
____________________________________
'.$boodschap.'
------------------------------------';

mail('info@mijndomein.nl', 'Website vraagformulier', $message, 'From: '.$email);

echo 'Bedankt! U krijgt bericht via email.';
?>

Code:

if (empty($_POST['realname']) || empty($_POST['email']) || empty($_POST['boodschap'])) {
    echo 'U heeft niet alle verplichte velden ingevuld!';
    exit;
}

isset kijkt of de waarde bestaat. Bestaan doet 'ie altijd, alleen er zit niks in.
empty kijkt of de waarde leeg is of niet.

Ik raad je wel aan een gebruikersvriendelijke en betere invoer-beveiliging te gebruiken.

Beste Ahmed,

Hartelijk dank, het werkt !

Je zegt "Ik raad je wel aan een gebruikersvriendelijke en betere invoer-beveiliging te gebruiken."

Ik weet erg weinig van PHP, is er iets mis met de invoerbeveiliging en gebruiksvriendelijkheid ?

Nogmaals dank !

je controleert niet of verplichte velden, zoals een e-mailadres, inhoud bevatten. En in het geval van het e-mailadres controleer je niet of het een geldige e-mailnotatie is. Zoals je het nu hebt kan ik het veld leeglaten en "test" als adres invullen.

Bij je if-statement controleer je alleen of er slechts 1 van de velden bestaat: realname, email of boodschap. Als er een punt staat of spatie in de boodschap dan is dit al voldoende om de if-statement over te slaan.

Graag gedaan!

Wat dinges zegt.

Een snelle fix zou zijn om drie losse if-statements te maken met daarin een exit. Voor de email kun je dan gelijk een reguliere expressie gebruiken (zie google) om te valideren of het een echt emailadres is. Dan krijg je if (empty($email) OR !isEmail($email)) { exit; }

Zoiets. Met gebruikersvriendelijkheid bedoel ik een optische weergave van wat er vergeten is in te vullen aan de hand van een rood lijntje o.i.d.

En eventueel geen exit, maar het formulier nogmaals weergeven met datgene ingevuld wat men wel ingevuld had. Bijv. een value="<?php print htmlentities($_POST['boodschap']); ?>" /> in je input.

Denk dat 'ie dat al doet, vandaar de exit. Formulier is waarschijnlijk standaard zichtbaar. Maar inderdaad goed idee dat laatste.

Dank voor de suggesties allen; ga er zeker mee aan de slag !
PHP is best leuk eigenlijk

En als je nou op het formulier nog wat javascript loslaat kan je meteen het mailadres checken.

Javascript:

function formCheck() {

if (document.form.naam.value==""){
alert("Naam?");return false;
}

var emailExp = /^[\w\-\.\+]+\@[a-zA-Z0-9\.\-]+\.[a-zA-z0-9]{2,4}$/;
if(!(document.form.emailadres.value.match(emailExp ))){
alert("E-mail is leeg of onjuist?");
return false;
}

if (document.form.bericht.value==""){
alert("Bericht?");return false;
}

if (document.form.bericht.value.length > 5000){
alert("Uw bericht is te lang, " + document.form.bericht.value.length + " karakters. Maximum is 5000 karakters.");return false;
}

}


HTML formulier:

<form name="form" action="contact2.php" method="post" onSubmit="return formCheck()" >

<table border="0" cellpadding="3" cellspacing="0">
<TR>
<TD>Naam:</TD>
<TD><input type="text" class="forminput2" name="naam" size="20" maxlength="150" style="WIDTH: 300px" /></TD>
</TR>
<TR>
<TD>E-mailadres:</TD>
<TD><input type="text" class="forminput2" name="emailadres" size="20" maxlength="150" style="WIDTH: 300px" /></TD>
</TR>
<tr valign="top">
<td>Bericht:</td>
<td><textarea class="textareainput" name="bericht" rows="7" cols="42" style="width: 300px" ></textarea></td>
</tr>
<tr>
<td>&nbsp;</td>
<td><br /><INPUT class="forminput2" TYPE="submit" VALUE="Verzend" /></td>
</TR>
</TABLE>
</form>

Welkom op dit forum, Gekko.
Maar ik denk dat TS na anderhalve maand het probleem inmiddels al heeft opgelost.

In additie, dit soort geintjes moet je altijd serverside checken, niet enkel clientside( javascript )

Uiteraard.