mailform werkt niet

Status
Niet open voor verdere reacties.

Barry_127

Gebruiker
Lid geworden
20 jan 2003
Berichten
478
kan iemand naar mijn mail form kijken en verbeteren wat fout is?

heb hem even toegevoegd met de extensie .txt ;)
 

Bijlagen

  • mailform.txt
    3 KB · Weergaven: 38
probeer deze eens
persoonlijk krijg ik een foutmelding bij het commando mail()...maar ik denk dat dat komt vanwege mijn instellingen...onlangs pc geformatteerd, dus...
maar ik denk dattie verder zo wel moet werken

enige verbeteringen:
- constant <form> en </form> ? :8-0: dat moet je niet doen bij een formulier
- <input type=submit value="..." method="post"> dat method="post" moet in de <form>, zeker niet in een <input>
- $_POST["site"] werkt niet als je geen invoerveld hebt met de naam "site"
- die controle op of de velden leeg zijn pakte je niet goed aan...als alle velden leeg zijn, krijg je bijv. alle foutmeldingen achter mekaar...ik heb het nu tot 1 foutmelding gevoegd, is wat makkelijker...

goeie dingen:
- je gebruikt if(!empty($_POST))...goed voluit geschreven, hou dit aan
- je gebruikt goeie onderscheidingen tussen pure output & php
- je ordening is vrij goed

succes d'r mee :)
 

Bijlagen

  • mailform.txt
    1,9 KB · Weergaven: 21
Laatst bewerkt:
hm raar die zin (regel 24, daar ligt de fout) had ik eerder nog niet gezien
maar verwijder regel 24 eens, dat is de regel met:
PHP:
  $header= "Wrom: CJVTLBXFGGMEPYOQKED
als er niet nog meer verrassingen zijn, hoort-ie het dan te doen
 
Geplaatst door acid007
PHP:
$header= "Wrom: C";
zoiets misschien...?
ja zo klopt de code wel ja (duh :p) maar ik gebruik $header niet, dus waarom zou ik hem erin zetten ?
kijk, als Barry dat per se wil hebben, ok hou dan maar...maar zo niet, haal dan maar weg...neemt alleen maar ruimte in beslag
 
Header is nodig als je html mails wil versturen, toch?.. lijkt me wel goede optie..
 
Status
Niet open voor verdere reacties.
Terug
Bovenaan Onderaan