Reacties op het script htmlTag

Joost

Gepost op: 31 mei 2009 - 12:47

PHP expert

Prima script an sich, maar ik heb nog wel even de volgende puntjes:

- Je noemt al je parameters iNaam. Dat is naamconventie niet correct, omdat een i voor integer staat. Voor strings bijvoorbeeld gebruik je een s. Dit is natuurlijk niet verplicht, maar maakt het wel zo leesbaar.
- In je constructor zet je je default tagname op NULL, voor strings lijkt me een "" beter. NULL wordt meestal gebruikt voor objecten.
- In je constructor declareer je de attributen attributes en childs op een lege array. Dat kan je beter doen bij de de declaratie van de attributen.
- Misschien zie ik dit fout hoor, maar waarom declareer je text en shottag als private en tagName, attributes en childs als protected?

Verder vraag ik mij het nut af van zo'n klasse af, aangezien je dit sneller zelf kan maken. Dat is sowieso mijn idee van een HTML klasse, maar afgezien van dat, vind ik het wel een goed, netjes gecommentariseerd script.

Stijn

Gepost op: 31 mei 2009 - 23:41

PHP expert

Voeg een methode __clone() toe, dan kan je het volgende doen.

php code - Bekijk de code zonder highlighting - Klap code in

                <?php
//veel HTML instanties
$listitem_2 = clone $listitem;
$listitem_2->setText("...");

$ul->addchilds($...);
            
<?php
//veel HTML instanties
$listitem_2 = clone $listitem;
$listitem_2->setText("...");
 
$ul->addchilds($...);
 

jvs

Gepost op: 01 juni 2009 - 13:34

HTML beginner

Hey mensen bedankt voor de reacties ik ga er zeker rekening mee houden en de nodige aanpassingen maken maar zit nu volop in examens dus zal iets voor daarna zijn! iig bedankt!

Enkel aanvullende informatie, vragen en antwoorden op vragen zijn welkom.