Contact script , Swift-Mailer

Ga je eigen tutorial volgen.
Dit is onzin.

En , wat is er dan allemaal niet goed ?

Edit: Klein foutje eruit gehaald omdat bannen bij email niet werkte

Hmm, volgens mij is hij nu anders. Maar goed, ik zie het nut van dit script niet echt in aangezien dit ook wel bij swiftmailer staat uitgelegd.
Ook is het zo dat je niet goed kan checken met een regex of een e-mailadres geldig is.
Exit / die is onzin.
trim?
Waar komt $error weg?
Waarom $error op deze manier?
Foutieve html.
Als je html verstuurd moet je ook htmlentities gebruiken, want het is html.
Foutive logica.
Dubbelop zooi.
Tabellen horen niet gebruikt te worden voor forms.
Inline css + vage javascript zooi.

regex controleren: Of hij valid is kan hij inderdaad niet controleren maar hij kan wel controleren of er een goede logica in zit dus zo : [email protected]

Exit:
Nu je het zegt , Ja . Heb hem ook even gewijzigd

Waar komt $error weg?
Kromme zin

Htmlentities
Je moet html kunnen versturen
voor als je bijv iets dikgedrukt wil

Inline css
Om aantegeven of een veld niet ingevuld is

Logica.
Ik vind dit redelijk logisch hoor

Nee, die regex laat mogelijk goede e-mailadressen door. Zoals ik dus zei 'Ook is het zo dat je niet goed kan checken met een regex of een e-mailadres geldig is.'
Dus dat betekend dat je geen regex (niet op een zinnige manier) kan toepassen om te kijken of een e-mailadres geldig is.

De zin 'Waar komt $error weg?' is niet krom. Jij bent zelf krom door zomaar opeens $error te gaan gebruiken. Wat je wilt bereiken snap ik, maar kan veel logischer.

Van mij mogen gebruikers ook geen html mailen. Dus daarom zeg ik dat je ook htmlentities moet gebruiken. Of iets dik gedrukt moet worden hoeft de gebruiker niet te bepalen.

Inline css / javscript is dus echt niet net. Wat als ik nu wil dat mijn border groen moet worden als iets is ingevuld? Dat kan niet makkelijk veranderd worden met jou script.

Logica: Het is toch een onzinnig script dus ga uitleggen hoe je het helemaal moet aanpassen.

Dan maak je er toch groen van

Toch vind ik waar komt $error weg? een kromme zin

tabs gebruiken kan overigens ook

Inline rommel is rommel omdat het niet onderhoudsvriendelijk is.
Waar komt de variabele, die later tot array wordt omgetoverd, genaamd $error vandaan c.q. weg?
Waar is die gedefinieerd als zijnde array?
Waarom op deze onhandige manier?

Ik heb er zelf even $error = array(); aan toegevoegd , deze werkik morgen bij omdat ik nu opmijn ipod zit en geen rar kan openen

@karl ik denk dat je dit ermee bedoelde of niet

Gelieve geen download links toe te voegen, maar het script hier te plaatsen. Een link naar de Swift Mailer library is geen probleem, plaats hier dan gewoon de 'toepassing' van die library.

Doe je dit niet, verwijder ik dit script.

@Chris, Hoera

@Jordi, als je het script gaat plaatsen denk dan aan:
Instellingen, het to adres is staat midden in je code, vaak gebruik je bepaalde adressen op meerdere plekken, het zou handiger zijn om een setting gedeelte te hebben.
Je HTML is geheel invalid. Stop het in een variabele die je aan het einde terug geeft of maak je HTML valid.
Over inline css/javascript. Wat is je reden on het inline te doen. Extern is vele malen makkelijker bij te houden.

Ik heb nu het grootste gedeelte verbeterd :)
Ook karls htmlentries
Html validated voor het grootste gedeelte alleen 2 errors die ik niet opgelost krijg

Ook heb ik wat TJVN mij aanraadde de settings bovenaan de pagina gezet

Verder :
Ik gebruik inline css in onchange omdat ik dus wil dat als er een error is , Het vlak rood gekleurd word en als iemand erop klikt dat het weer normaal word
vandaar

Dan kun je met een onchange de class aanpassen en die vanuit je css stylen.

Waarom zijn de echo's met een hoofdletter? strtolower ;-)

@chris , Automatisme :P