Reacties op het script [PHP5] MyForm V1.0
|
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.. |
|
|
|
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;) |
|
|
|
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 |
|
|
|
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 |
|
|
|
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. |
|
|
|