Lexperts Contact Form

Waarom zoveel code voor een simpele contactform? Overigens is het gebruik van shorttags afgeraden.

Voor de rest is het wel een leuk opzetje voor mensen die een contactformulier willen.

Zoveel code?
En waarom worden shorttags afgeraden?

Waarom chaptcha, zeer gebruiksonvriedelijk. Probeer te zoeken naar een ander beveiligingsoptie (bijvoorbeeld een limiet op het aantal te versturen berichten).

- Je controle op e-mail is niet goed, hij accepteert alles met een @ en een punt;
- Bij een foutmelding wordt de keuze uit de select niet onthouden.

Volgens mij klopt het gedeelte van regel 28 tot en met regel 38 niet (kan je dan niet gewoon beter direct die hele variabele leeg gooien? Want dat is alles wat je d'r doet).
Het stuk van regel 86 tot en met regel 122 klopt ook al niet, aangezien ' ' (dus een spatie) niet als empty wordt gezien.
Ook overschrijf je daar steeds 'het fout-bericht'.
Je gebruikt nog eregi, dat is slecht en oud. Die gaat eruit, gebruik liever preg_match.
Je telefoonnummer controle klopt ook niet, ik mag blijkbaar van jou gewoon 9*10^99 cijfers invoeren. Verder zou ik ook niet aan de opmaak gaan zitten klooien, maar gewoon aangeven hoe mensen hun nummer in moeten voeren en kijken of het volgens die manier klopt.
Regel 126 tot en met regel 132 is ook niet goed. Er is namelijk gewoon een constante PHP_EOL die een nieuwe regel geeft en ook nog altijd de goede.
Alle data die je verzend in het mailtje (regel 154) doe je met str_replace. Aan het begin van het script dacht ik dat je netjes gebruik zou maken van sprintf, maar dat doe je dus jammer genoeg niet. Sprintf is netter en volgens mij ook sneller in gebruik.
ARG!!! DIT SCRIPT IS GEVAARLIJK! Mail injections! Geef nooit de gebruiker de mogelijkheid om in de headers van een mailtje te klooien en nog wat dingen, google maar: php mail injections.
Het is beter om geen short-tags te gebruiken aangezien die meestal uitstaan.

Edit: Verder vind ik ook dat je best wel erg aan het strooien bent met @ (foutonderdrukking). Het is beter om de fouten af te handelen dan ze te onderdrukken. En sowieso staan die op een aantal rare plekken vind ik.

En vertel eens waarom jou script zo super geweldig goed is, dat iedereen hem zou moeten gebruiken. Zie het nu d'r nog niet zo in. Ik denk dat mensen beter simpelere scripts kunnen pakken.

Fout helemaal fout.
SHORTTAGS ZIJN TEN DODE OPGESCHREVEN; ze zijn vies; bah en eng --- weet iemand trouwens of ze in 6 uiteindelijk worden geschrapt ---


Enig idee wat je hier doet? misschien kan je beter het één en ander afvangen ipv te negeren

ohjah mag ik lekker zelf mijn server instellingen bepalen

Ik zal morgen het een en ander aanpassen en toelichting geven op alle vragen.

ik wou hem gebruiken maar nu ik weet dat hij gevaarlijk is.. als hij aangepast is neem ik hem wel denk ik :D