[GRATIS] Contact script!

Ik heb het ooit besteld bij iemand, maar bijna niks mee gedaan!
Je mag het gratis gebruiken:

- Als je de Copyright laat staan!
- Zonder doorverkoop rechten!

<!--
Copyright: Sander C! ZONDER doorverkoop rechten! Gratis @one2xs
-->
<style>
form input, form select, form textarea {
padding: 5px;
color: #222;
border: 1px solid #ccc;
border-right:1px solid #ddd;
border-bottom:1px solid #ddd;
font-family: Arial, Helvetica, sans-serif;
font-size: 12px;
border-radius: 5px;
moz-border-radius: 5px;
webkit-border-radius: 5px;
}

form input button {
background: #1FAABE;
color: #ffffff;
font-size: 12px;
font-family: Tahoma, Geneva, sans-serif;
</style>

[COLOR= #0000BB]<?php[/COLOR]
[URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][URL="http://www.php.net/isset"]isset[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]submit[/COLOR][COLOR= #007700]])){[/COLOR] 
[COLOR= #0000BB]$naam[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]naam[/COLOR][COLOR= #007700]];[/COLOR] 
[COLOR= #0000BB]$email[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]email[/COLOR][COLOR= #007700]];[/COLOR] 
[COLOR= #0000BB]$bericht[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]bericht[/COLOR][COLOR= #007700]];[/COLOR]
[URL="http://www.php.net/if"]if[/URL][COLOR= #007700](![/COLOR][COLOR= #0000BB]$naam[/COLOR] [COLOR= #007700]&& ![/COLOR][COLOR= #0000BB]$email[/COLOR] [COLOR= #007700]&& ![/COLOR][COLOR= #0000BB]$bericht[/COLOR][COLOR= #007700]){[/COLOR] 
[URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"<b>U heeft niet alles correct of volledig ingevuld!</b><br><br>"[/COLOR][COLOR= #007700]; 
}[/COLOR][URL="http://www.php.net/else"]else[/URL][COLOR= #007700]{[/COLOR] 
[COLOR= #0000BB]$mail[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #DD0000]"hierjoumail@joumail.com"[/COLOR][COLOR= #007700];[/COLOR] 
[COLOR= #0000BB]$subject[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #DD0000]"Contact van $naam."[/COLOR][COLOR= #007700];[/COLOR] 
[COLOR= #0000BB]$datum[/COLOR] [COLOR= #007700].=[/COLOR] [URL="http://www.php.net/date"]date[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]d/m/Y H:i:s[/COLOR][COLOR= #007700]);[/COLOR]
[COLOR= #0000BB]$inhoud[/COLOR] [COLOR= #007700].=[/COLOR] [COLOR= #DD0000]"Naam: "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]naam[/COLOR][COLOR= #007700]].[/COLOR][COLOR= #DD0000]"
"[/COLOR][COLOR= #007700];[/COLOR]
[COLOR= #0000BB]$inhoud[/COLOR] [COLOR= #007700].=[/COLOR] [COLOR= #DD0000]"Email: "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]email[/COLOR][COLOR= #007700]].[/COLOR][COLOR= #DD0000]"
"[/COLOR][COLOR= #007700];[/COLOR]
[COLOR= #0000BB]$inhoud[/COLOR] [COLOR= #007700].=[/COLOR] [COLOR= #DD0000]"Bericht: "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]bericht[/COLOR][COLOR= #007700]].[/COLOR][COLOR= #DD0000]"
"[/COLOR][COLOR= #007700];[/COLOR]
[COLOR= #0000BB]$inhoud[/COLOR] [COLOR= #007700].=[/COLOR] [COLOR= #DD0000]"Verstuurd op "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$datum[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]" via het IP-Adress "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$_SERVER[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]REMOTE_ADDR[/COLOR][COLOR= #007700]].[/COLOR][COLOR= #DD0000]"

"[/COLOR][COLOR= #007700];[/COLOR]
[URL="http://www.php.net/mail"]mail[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$mail[/COLOR][COLOR= #007700],[/COLOR] [COLOR= #0000BB]$subject[/COLOR][COLOR= #007700],[/COLOR] [COLOR= #0000BB]$inhoud[/COLOR][COLOR= #007700]);[/COLOR]
[URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]"<b>Dit was een test, maar is naar nergens verzonden omdat er geen mail is ingegven!</b><br><br>"[/COLOR][COLOR= #007700]; 
} 
}[/COLOR]
?>

<form method="post">
Uw naam:<br><input type="text" style="width: 200px" name="naam"><br><br>
Email:<br><input type="text" style="width: 200px" name="email"><br /><br>
Zet hier uw bericht neer over wat u ons wilt contacteren:<br><textarea id="bericht" name="bericht" rows="8" style="width: 400px;"></textarea><br><br>
<input type="submit" name="submit" value="Verzenden"></form>

Indien je het wilt met doorverkoop rechten en zonder copyright, kan je het kopen voor 0,01 punt in dit topic: http://www.one2xs.com/threads/39571

Vragen en klachten mogen hier. Alle Reacties welkom!

- Mail-header injection is mogelijk. Je bent erg vatbaar om dit als spam-doorstuurservice te laten gebruiken.

- Tevens mis ik controle op de mail() functie. Mensen die dit bij mail()-blockende providers zoals Woelmuis, zullen niet blij worden.

- En isset() als formulier-controle is niet de juiste manier.
Gebruik de juiste manier met $_SERVER[REQUEST_METHOD]=="POST" dus.

- Onnodig variabelen overkopieeren op lijn 27 t/m 29

BREKEN DIE BOEL DUS !
(trouwens, zoiets koop je niet, je zou dan minstens je geld dan terug moeten vragen)

Hmmm, Ik ben het niet helemaal met je eens Aarclay. Het systeem is niet mooi geprogrammeerd en slecht uitgelijnd. Maar ik veranderen mijn POST variablen ook altijd naam vaste.

Je kan isset() wel gebruiken bij submit formulieren als je bijvoorbeeld meerdere submit velden hebt. In sommige browsers werkt het alleen niet met [enter].

In plaats van kritiek hebben kan je hem ook tips geven .

Ik vind trouwens dat de uitlijning wel belachelijk slecht gedaan is. Dus dat moet de ts eerst maar eens gaan oefenen.

$email kan je beter valideren met filter($email, VALIDATE_MAIL) (of zoiets, moet je ff opzoeken).

Voor de rest mis ik uberhaubt goeie foutafhandeling. (ik kan toch geen 352^$%3 heten?)

Wel een mooi gebaar om scripts gratis weg te geven.

Rogier schreef:

In plaats van kritiek hebben kan je hem ook tips geven .

Opbouwende kritiek....

Als je opbouwende kritiek geeft mag je anders best wel wat zeggen over de foutafhandeling en de uitlijning . In plaats van variablen die worden herschreven.

Om eerlijk te zijn ben ik het compleet met Aarclay eens Wat is het nu van variabele overschrijven als je toch niet gebruikt? Scripten gaat ook over efficientie en dit is mooi voorbeeld van inefficientie. Verder nul controle of het werkelijk een email adres is of dat iemand zit te klooien en van het bericht een array maakt. En misschien check dat je maximaal 1 bericht per zoveel secondes mag versturen

Rogier schreef:

Hmmm, Ik ben het niet helemaal met je eens Aarclay.
...
Maar ik veranderen mijn POST variablen ook altijd naam vaste.

Je kan isset() wel gebruiken bij submit formulieren als je bijvoorbeeld meerdere submit velden hebt. In sommige browsers werkt het alleen niet met [enter].

isset() kan je gebruiken, maar zorg gewoon dat je in ieder geval $_SERVER[REQUEST_METHOD] == "POST"gebruikt. Als je meerdere forms hebt kan je dit gebruiken in combinatie met een isset() op de submitknop van het formulier.

Alleen controleren met isset() is gewoon euh..... &^%&^%&*^ Maar dat had ik je geloof ik al eens eerder verteld Rogier.

Waarom
en
? Alleen
is goed. Verder zou ik het wat uitlijnen, en wat foutmeldingen plaatsen zoals Jenner al zegt.

Aarclay schreef:

Eerdere citaten tonen

Rogier schreef:

Hmmm, Ik ben het niet helemaal met je eens Aarclay.

...
Maar ik veranderen mijn POST variablen ook altijd naam vaste.

Je kan isset() wel gebruiken bij submit formulieren als je bijvoorbeeld meerdere submit velden hebt. In sommige browsers werkt het alleen niet met [enter].

isset() kan je gebruiken, maar zorg gewoon dat je in ieder geval $_SERVER[REQUEST_METHOD] == "POST"gebruikt. Als je meerdere forms hebt kan je dit gebruiken in combinatie met een isset() op de submitknop van het formulier.

Alleen controleren met isset() is gewoon euh..... &^%&^%&*^ Maar dat had ik je geloof ik al eens eerder verteld Rogier.

Mij heb je het nooit verteld, ik gebruik al sinds ik kan programmeren server request_method. En heb pas een keer in mijn hele programeer carriere isset() gebruik om form submits te testen. En toen stonden er 5 formulieren. (was admin paneel) Het is alleen geen hele grote fout en met meerdere forms is moet je wel isset() gebruiken. Ik snap niet wat je tegen isset() hebt maar echt heel vervelend is het niet hoor.

En voor de foutafhandeling is het toch wel handig om de POST variables over te schrijven. Het is bijna geen werk, je code wordt er gewoon overzichtelijker van. Je kan meer dingen doe met de variable. Maarja, voor een overzichtelijke code moet je eerst gaan uitlijnen.

Ik zeg alleen, als je dan kritiek geeft. Pak dan de grote fouten aan en niet die kleintjes. Die mail header injectie bijvoorbeeld is veel gevaarlijker. Op regel 35 staat ook nog een fout .= moet = zijn.

De if() op regel 30 werkt niet met alle versies php. Hij controleert de variablen. Maar deze bestaan sowieso. Ookal zijn ze 0. Ik gebruik hier altijd deze functie voor:

[COLOR= #0000BB]<?php[/COLOR]
[URL="http://www.php.net/public"]public[/URL] [COLOR= #007700]function[/COLOR] CheckStandardField[COLOR= #007700]([/COLOR][COLOR= #0000BB]$string[/COLOR][COLOR= #007700]) {[/COLOR]
    [URL="http://www.php.net/return"]return[/URL] [URL="http://www.php.net/preg_match"]preg_match[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]/[^a-zA-Z]/[/COLOR][COLOR= #007700],[/COLOR] [COLOR= #0000BB]$string[/COLOR][COLOR= #007700]);
}[/COLOR]
?>

Zo schrijf je voor elk veld dat je verwacht gewoon een mooie reguliere expressie. Als het veld 5 tekens moet zijn gebruik je: [^a-zA-Z]{5}

Je kan beter kijken of mail ook wel juist is enz en niet weergeven dat je IETS niet hebt ingevuld, maar wat er fout is. Plaats de errors in array en bij 0 voer je script uit anders show je je array. Voor de rest met Aarclay eens, alhoewel in de reactie boven mijne ook wat belangerijke dingen worden opgesomt.

Joeey schreef:

Waarom
en
? Alleen
is goed. Verder zou ik het wat uitlijnen, en wat foutmeldingen plaatsen zoals Jenner al zegt.

Je verhaal over breaks klopt niet.
http://www.w3schools.com/tags/tag_br.asp --> Differences between HTML and xHTML

Ligt er dus aan wat je aan het doen bent

Jordyt schreef:

Eerdere citaten tonen

Joeey schreef:

Waarom
en
? Alleen
is goed. Verder zou ik het wat uitlijnen, en wat foutmeldingen plaatsen zoals Jenner al zegt.

Je verhaal over breaks klopt niet.
http://www.w3schools.com/tags/tag_br.asp --> Differences between HTML and xHTML

Ligt er dus aan wat je aan het doen bent

Alle drie zijn goed.

is HTML

is XHTML en
is beter voor oudere browsers.

Jordyt schreef:

Eerdere citaten tonen

Joeey schreef:

Waarom
en
? Alleen
is goed. Verder zou ik het wat uitlijnen, en wat foutmeldingen plaatsen zoals Jenner al zegt.

Je verhaal over breaks klopt niet.
http://www.w3schools.com/tags/tag_br.asp --> Differences between HTML and xHTML

Ligt er dus aan wat je aan het doen bent

Het beste is gewoon dat je altijd standaard xHTML probeert de coderen...

Sanderc schreef:

Eerdere citaten tonen

Jordyt schreef:

Eerdere citaten tonen

Joeey schreef:

Waarom
en
? Alleen
is goed. Verder zou ik het wat uitlijnen, en wat foutmeldingen plaatsen zoals Jenner al zegt.

Je verhaal over breaks klopt niet.
http://www.w3schools.com/tags/tag_br.asp --> Differences between HTML and xHTML

Ligt er dus aan wat je aan het doen bent

Het beste is gewoon dat je altijd standaard xHTML probeert de coderen...

Ja, klopt maar als je hem door een validator voor html haalt gaat hij zeuren
toevoeging op 28 juni 2013, 16:26:10:
Probleem was dat jij zowel
als
gebruikte, dat is wel raar.

Jordyt schreef:

Eerdere citaten tonen

Sanderc schreef:

Eerdere citaten tonen

Jordyt schreef:

Eerdere citaten tonen

Joeey schreef:

Waarom
en
? Alleen
is goed. Verder zou ik het wat uitlijnen, en wat foutmeldingen plaatsen zoals Jenner al zegt.

Je verhaal over breaks klopt niet.
http://www.w3schools.com/tags/tag_br.asp --> Differences between HTML and xHTML

Ligt er dus aan wat je aan het doen bent

Het beste is gewoon dat je altijd standaard xHTML probeert de coderen...

Ja, klopt maar als je hem door een validator voor html haalt gaat hij zeuren
toevoeging op 28 juni 2013, 16:26:10:
Probleem was dat jij zowel
als
gebruikte, dat is wel raar.

Dat script kocht ik van mrdylan toen ik zelf nog geen hol kon van html,css en PHP etc. Ik heb er later (toen ik zelf nog maar weinig van kon) wat dingen aan toegevoegd...