Class: Formulier Generator & Validator

Door Thijs X, 20 jaar geleden, 7.866x bekeken

Features:
- Genereren van een formulier
- Valideren van een formulier

Update
- Validatie ingebouwd
- setActie() aan toegevoegd ( in formulier klasse )
- Alternatief gebruikt voor filter_var mocht de PHP versie geen PHP 5.2 zijn

Tips en verbeteringen zijn altijd welkom!

Voorbeeld: http://php.ferket.net/formulier.php

Gesponsorde koppelingen

PHP script bestanden

  1. class-formulier-generator-en-validator

 

Er zijn 22 reacties op 'Overig'

PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen
Vincent
Vincent
20 jaar geleden
 
0 +1 -0 -1
Mooi gemaakt :-)
Arian Stolwijk
Arian Stolwijk
20 jaar geleden
 
0 +1 -0 -1
Ik weet niet of jullie Formhandler kennen, maar die is zeer uitgebreid en heel handig.;)
Thijs X
Thijs X
20 jaar geleden
 
0 +1 -0 -1
Ja ik Formhandler wel maar ik maak toch het liefst alles zelf ;) Ik denk dat de meeste mensen hier hetzelfde doen.
Om er van te leren en/of alleen de juiste functies erin te hebben die je nodig hebt.
- -
- -
20 jaar geleden
 
0 +1 -0 -1
@Thijs:
Inderdaad, ik maak dingen ook graag zelf.
Jelmer -
Jelmer -
20 jaar geleden
 
0 +1 -0 -1
Wat reacties op je OOP-stijl:
- Een klasse "Velden" noemen is niet echt handig, eerder verwarrend. 1 object representeert toch niet meerdere velden. Ik zou hier trouwens een interface voor gebruiken, aangezien je de methods die je in deze abstracte klasse definieert toch weer herdefinieert.

- In je klasse "Veld" hanteer je als eerste parameter, dat het type moet representeren een nummertje. Is het niet handiger om hier een constante voor te gebruiken in je code? Dan kan je tenminste zien wat voor soort veld je nu eigenlijk aan hebt gemaakt.

- Daarbij, waarom kan Veld van 4 verschillende typen zijn, terwijl bijvoorbeeld Select, Textarea, wat ook velden zijn, een aparte klasse hebben? Ik zou uitgaan van voor ieder soort veld een aparte klasse met al dan niet een bovenliggende klasse voor deze 4 velden die veel op elkaar lijken. (Dus stel, je maakt voor type="text" een klasse, dan extend je die voor type="password" weer)

- Zou het niet slim zijn om de veld-klassen zelf verantwoordelijk te houden voor het tekenen van het veld? Nu staat alle HTML in de klasse Formulier. Stel dat ik nu een eigen veld zou willen toevoegen, dan zou ik Formulier ook weer moeten aanpassen. Daarnaast zou ik voor alle gebruikte variabelen een accessor moeten maken (aangezien jij niet werkt met public properties) terwijl wanneer ik uit zou gaan van wat meer zelfstandige Veld-klassen ik alleen maar een draw() method nodig zou hebben.

- Persoonlijke ervaring: Gebruik bij voorkeur 'protected' voor eigenschappen die van de klasse zelf zijn en de eigenschappen definiƫren. Op die manier kan je je bijvoorbeeld heel gemakkelijk een klasse extenden omdat je al toegang hebt tot alle benodigde eigenschappen. Ik denk dat je alleen maar eigenschappen die worden aangemaakt door de methods van een klasse, en verder nooit buiten de klasse zullen komen private hoeven te zijn. Bijvoorbeeld een $cache-array of een counter hoe vaak een bepaalde method is aangeroepen. Van die eigenschappen maken alleen de methods in die ene klasse gebruik, en alleen zijzelf.

- Voor jouw InvoerException bestaat al een Exception: InvalidArgumentException.

Verder, zou het niet mooi zijn om ook meteen de validatie in je setje klassen mee te nemen?
Thijs X
Thijs X
20 jaar geleden
 
0 +1 -0 -1
Jelmer bdankt voor je uitgebreide reactie!!
Echter zijn sommige dingen niet helemaal duidelijk.

- Wat bedoel je precies met een interface, welke classes zou jij zelf maken?

- Accessors heb ik express nog niet toegevoegd omdat het alleen nog maar het gereneren van het formulier was, als ik het ga uitbreiden met validatie heb ik dat inderdaad wel nodig.

Ga eens even kijken naar die InvalidArgumentException en hoe ik die het beste kan toepassen

Dus beste is om voor elk soort veld een classe te maken en dmv de __toString methode elke keer te overriden bij bijv: text, password, hidden. Omdat die worden extended van elkaar
PHP Newbie
PHP Newbie
20 jaar geleden
 
0 +1 -0 -1
Ik blijf het vreemd vinden om 468 regels code te moeten gebruiken om een formuliertje van 44 regels te genereren... Kun je dan niet beter gewoon even het formuliertje typen?

Ik zie in dit geval het voordeel van de OOP klasse niet.
Thijs X
Thijs X
20 jaar geleden
 
0 +1 -0 -1
Het is ook niet de bedoeling voor 1 formulier PHP Newbie maar wanneer je vaker in dezelfde applicatie gebruik maakt van formulieren.

Als je nou bijvoorbeeld iets wilt wijzigen in al je formulieren hoef je dat in dit geval maar in 1 classe te doen.
Jelmer -
Jelmer -
20 jaar geleden
 
0 +1 -0 -1
@Thijs: In PHP5 kan je gebruik maken van interfaces. Dit zijn een soort regels, definities die een klasse kan implementeren. Maar wil die klasse dat mogen, dan moet hij voldoen aan alle regels van de interface. Zie deze tutorial.

Je gebruikt wel accessors, namelijk al die getXXX() methods. Wanneer je het tekenen door de klassen zelf laat doen, heb je in principe veel minder van deze methods nodig.

Een select-klasse zou ik dan weer niet laten afstammen van een input-type-field/password-klasse, want deze zijn wezenlijk anders. Maar een text/password-veld zijn vrijwel identiek. Daar zou dit dus prima kunnen.

@PHP Newbie:
In deze opzet is het ook nog niet echt voordelig. Echter, wanneer je er validatie instopt, en het bijvoorbeeld mogelijk maakt om op basis van een Model een formulier te laten genereren kan dit uiteindelijk best handig zijn. Wanneer je dan wijzigingen in je model doorvoert, hoef je niet al je formulieren te bewerken. En bij langere formulieren zal deze methode trouwens ook winnen aangezien je dan validatie, waarde controleren en invullen etc in 1 combineert. Het copy-paste gehalte is normaal enorm bij formulieren, hier niet. Maar voor een klein contact-formulier loont het de moeite inderdaad niet.
Thijs X
Thijs X
20 jaar geleden
 
0 +1 -0 -1
Nieuwe Update.

- Word gebruik gemaakt van een Interface
- Alle HTML word per classe zelf bepaald en dmv de __toString methode naar het formulier gehaald
- Voor elk veld / button een apparte klasse aangemaakt, en de de dichtbijelkaar liggende velden worden extended van elkaar en de __toString methode overrided.
Jelmer -
Jelmer -
20 jaar geleden
 
0 +1 -0 -1
Je mag bij InvalidArgumentException nog best vermelden welk argument fout is, en wat de eventuele oplossing is hoor :)
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
<?php
throw new InvalidArgumentException('$mxHeight moet een integer zijn en liggen tussen 30 en 50');
?>


Je hebt op verschillende plekken de constructor van je klasse die je extend overschreven, maar alleen om vervolgens weer de overschreven variant aan te roepen? Je kan in bijv File & Password best de constructor helemaal weglaten. Hij pakt automatisch die van Text.

Je bent bij Submit nog vergeten je iPrintable te implementeren. Wat dat doet Submit wel, maar het staat niet vermeld.

Nu je die interface gebruikt, kan je die ook meteen gebruiken om bij Formulier::add af te dwingen dat je alleen printbare elementen kan toevoegen:
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
<?php
    public function add(iPrint $object) { ...
?>
Thijs X
Thijs X
20 jaar geleden
 
0 +1 -0 -1
Wederom bedankt voor je reactie Jelmer,

Ga gelijk even aanpassen!

Dat je die constructor weg mocht laten wist ik niet eens, aangezien dat bij Java ook niet mag.

Edit:

Geupdate!
Bjorn Vennema
Bjorn Vennema
20 jaar geleden
 
0 +1 -0 -1
Mooie class!

Misschien ga ik alleen nu een domme vraag stellen: Wat is het nut van het gebruik van een interface? De functie __toString() wordt toch gwn overal gebruikt? Hoezo wordt die dan eerst toegekend aan een interface?
Jelmer -
Jelmer -
20 jaar geleden
 
0 +1 -0 -1
@Bjorn: De interface wordt weer genoemd in Formulier::add. Dit dwingt af dat alleen objecten die de interface implementeren als argument gegeven kunnen worden. Klassen die de interface implementeren moeten een method genaamd __toString hebben. Zo weet je dus zeker dat alle objecten die via Formulier::add zijn toegevoegd een method genaamd __toString hebben.
Bjorn Vennema
Bjorn Vennema
20 jaar geleden
 
0 +1 -0 -1
Ahh ok, klinkt wel logisch. Bedankt voor de verheldering. Als ik nog 1 vraag moet stellen dan. Ik weet wat PHP_EOL doet, maar wat doet TAB?
Thijs X
Thijs X
20 jaar geleden
 
0 +1 -0 -1
Tab maakt gewoon een tab in je broncode aka \t, alleen met enkele quotes ( ' ) werkt dat niet vandaar dat ik hem even define bovenaan tussen dubbel quotes ( " ).
Thijs X
Thijs X
20 jaar geleden
 
0 +1 -0 -1
Nieuwe update:
- Validatie ingebouwd
- setActie() toegevoegd aan formulier classe
En nog paar kleine aanpassingen

Voorbeeld: http://php.ferket.net/formulier.php
Klystr
klystr
20 jaar geleden
 
0 +1 -0 -1
Echt een mooi script Thijs, netjes volgens de styleguide ;-)

Ik heb alleen nog 1 klein dingetje, en je moet het zien als opbouwend kritiek...klassecommentaar/methodecommentaar is ervoor bedoelt dat iemand zonder naar de code te kijken weet wat de code doet...met andere woorden, als jij

@return String $dikketamme

in je commentaar hebt staan, dan weet degene die de documentatie uit PHPDoc leest nog niet wat er gebeurt. Dus, klein puntje daar...maak er iets van als

@return String Name of textfield

Verder geen op/aanmerkingen, ziet er echt netjes uit!
Thijs X
Thijs X
20 jaar geleden
 
0 +1 -0 -1
@ klystr

Bedankt voor de opmerking! Ik zal het wel veranderen naar het volgende:
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
@return String $html, hier dan een omschrijving over de variabele
Citroen Anoniem Graag
Citroen Anoniem Graag
19 jaar geleden
 
0 +1 -0 -1
Groot beveiligings lek:

Als je het formulier faket (dmv Tamper Data, ff-extension) en sommige _verplichte_ velden niet mee laat sturen, dan merkt hij niet op dat er een _verplicht_ veld mist...
Klaasjan Boven
Klaasjan Boven
19 jaar geleden
 
0 +1 -0 -1
Thijs

Wat ik niet terugvind is de functie juistWachtwoord. Jij vraagt in je voorbeeld slechts een keer om het wachtwoord terwijl normaal gesproken het wachtwoord twee keer ingegeven moet worden om een evt typefout te tackelen. Als je dat kan inbouwen heb je volgens mij een mooie uitbreiding
PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen
Mark moes
mark moes
19 jaar geleden
 
0 +1 -0 -1
Aangezien ik deze class wil gebruiken in combinatie met een loginsysteem, ben ik bezig met een extra validate function voor het vergelijken van de wachtwoorden. Dit is de functie:

Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
6
7
<?php
    public function isGelijk ( $sWaarde ) {
        if($_POST['wachtwoord'] !== $_POST['retypewachtwoord']) {
            return false;
        }
    }

?>


Ik wou dit zo gaan gebruiken:
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
<?php
    $oForm
->add( new Password('Wachtwoord:', 'wachtwoord', 40), array('minLengte', 5), 'isGelijk' );
    $oForm->add( new Password('Wachtwoord:', 'retypewachtwoord', 40), array('minLengte', 5), 'isGelijk' );
?>


Maar het werkt nog niet echt. Wie kan me helpen? t zou een mooie toevoeging zijn aan dit systeem.

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

Inhoudsopgave

  1. class-formulier-generator-en-validator

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.