Lexperts Contact Form

Door Lex , 14 jaar geleden, 4.440x bekeken

Script is te downloaden op: http://www.lexperts.nl/free/lexperts-contact-form/

Voor de mensen die liever copy/paste doen.
Zorg er voor dat je drie extra mappen aangemaakt worden (font, image, icon):

De volgende fonts moet in de map "font" te vinden zijn:
http://www.lexperts.nl/free/lexperts-contact-form/demo/font/arial.ttf
http://www.lexperts.nl/free/lexperts-contact-form/demo/font/verdana.ttf

De volgende afbeelding moet in de map "image" te vinden zijn:
http://lexperts.nl/free/lexperts-contact-form/demo/image/form_bg.jpg

De map image moet nog een map "icon" erbij komen en hierin moet de volgende afbeelding te vinden zijn:
http://lexperts.nl/free/lexperts-contact-form/demo/image/icon/accept.png

Voorbeeld: http://www.lexperts.nl/free/lexperts-contact-form/

Gesponsorde koppelingen

PHP script bestanden

  1. lexperts-contact-form

 

Er zijn 8 reacties op 'Lexperts contact form'

PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen
Zero XT
Zero XT
14 jaar geleden
 
0 +1 -0 -1
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.
Lex
Lex
14 jaar geleden
 
0 +1 -0 -1
Zoveel code?
En waarom worden shorttags afgeraden?
Steen
steen
14 jaar geleden
 
0 +1 -0 -1
Waarom chaptcha, zeer gebruiksonvriedelijk. Probeer te zoeken naar een ander beveiligingsoptie (bijvoorbeeld een limiet op het aantal te versturen berichten).
Elwin - Fratsloos
Elwin - Fratsloos
14 jaar geleden
 
0 +1 -0 -1
- 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.


14 jaar geleden
 
0 +1 -0 -1
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.


14 jaar geleden
 
0 +1 -0 -1
Quote:
Overigens is het gebruik van shorttags afgeraden.

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

Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
@ini_set('session.use_cookies', '1');
@ini_set('session.use_trans_sid', 'false');
    
@session_set_cookie_params(0, '/');
@session_start();

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
Lex
Lex
14 jaar geleden
 
0 +1 -0 -1
Ik zal morgen het een en ander aanpassen en toelichting geven op alle vragen.
PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen
Maurice
maurice
13 jaar geleden
 
0 +1 -0 -1
ik wou hem gebruiken maar nu ik weet dat hij gevaarlijk is.. als hij aangepast is neem ik hem wel denk ik :D

Om te reageren heb je een account nodig en je moet ingelogd zijn.

Inhoudsopgave

  1. lexperts-contact-form

Labels

  • Geen tags toegevoegd.

Navigatie

 
 

Om de gebruiksvriendelijkheid van onze website en diensten te optimaliseren maken wij gebruik van cookies. Deze cookies gebruiken wij voor functionaliteiten, analytische gegevens en marketing doeleinden. U vindt meer informatie in onze privacy statement.