login  Naam:   Wachtwoord: 
Registreer je!
Scripts > PHP > Overige scripts > [PHP5] MyForm V1.0


Reacties op het script [PHP5] MyForm V1.0

Offline  ikkedikke
Gepost op: 11 februari 2006 - 11:56
PHP expert



Mooi gedaan, alleen een dingetje:
regel 113:
isset($size) && !empty( $size ) && is_numeric( $size ) === true ) ? $FormData .=

$size is altijd gedefinieerd, daarom zet je er een standaard waarde voor..
en het is misschien makkelijker om gewoon te checken of de waarde 1 is of een string. disabled="disabled" is toch ook goed?
(TRUE == (bool)$maxLength) ? $FormData .= ' maxLength='.$maxLength : '';
Nu wordt een lege string of het getal 0 (al dan niet in een string) als false herkend en de rest true..

Offline  Richard
Gepost op: 15 februari 2006 - 14:32
Crew algemeen



Die docblocks die je hebt zien er wel leuk uit, maar ze kloppen van geen kant natuurlijk;)

ten eerste zie ik al protected vara .... , terwijl ze er staan als publics, daarnaast staan er ook nog veel te veel.

bij je methods heb je in de description alleen de functienaam () staan, terwijl je daar een description moet geven. ook mis ik overal de return param.

hiernaast mis is ook nog de page-level docblock... tja, als je het mooi met phpDocumentor wil laten zien, heb je nog veel te doen;)

Offline  Stijn
Gepost op: 15 februari 2006 - 19:25
PHP expert



Maak er een tutorial over , want heb geen idee hoe je dat dan zou moeten doen. Van die protected is waar, maar dat was nog klad, mijn excuses hiervoor dus 

Offline  Stijn
Gepost op: 13 maart 2006 - 19:43
PHP expert



heb een bugje gevonden bij de functie 'EndForm();'
Heb de code aangepast. Sorry voor deze laattijdige melding 

edit

de constructor heeft een wijzig want bij het verzenden van data geeft hij gewoon NULL door en nu niet meer srry voor deze nalatigheid

Offline  Richard
Gepost op: 13 mei 2006 - 13:21
Crew algemeen



Citaat:
[I][B]stijn1989[/B] schreef op 15 Februari 2006 - 19:25[/I]

Maak er een tutorial over , want heb geen idee hoe je dat dan zou moeten doen. Van die protected is waar, maar dat was nog klad, mijn excuses hiervoor dusĀ 


http://manual.phpdoc.org

moeilijk he;) google...


Enkel aanvullende informatie, vragen en antwoorden op vragen zijn welkom.
 
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.022s