Upload Class

Ziet er netjes uit, misschien nog een kleine tip door de extensie check op strtolower() uit te voeren?

Toegepast :) Bedankt !

Ja ziet er goed uit, een paar opmerkingen, waarom niet gelijk goed doen in PHP 5? Dit soort dingen: var $maxkb, var $directory en var $extensies horen eigenlijk in een constante. Net als $waarde*1024. Stop die 1024 gewoon in een constante dat is veel beter te onderhouden (stel je voor dat er opeens maar 1000 kb in een mb gaan ;))

@Rik: Er is geen constructor, dat gaat dus niet lukken.

Waarom toch weer gekozen voor een PHP 4 aanpak? Dat is achterhaalde zooi (eol!) en kan niet goed uit de voeten met OOP. Gemiste kans.

Ja die constructor moet je dus even invoegen. Dat is toch veel makkelijker, stel dat je op 2 plekken in je applicatie in 2 verschillende directory's zou willen werken dan moet je dus 2 bijna identieke classes hebben, lijkt me niet handig...

Als ik zin heb wil ik um vnaaf wel even ombouwen

Beste vriendjes en vriendinnetjes, door mijn eigen slordigheid heb ik php 4 gezet, terwijl er eigenlijk php 5 moet staan. Nouja, dat is dan opgelost.

Maar je gebruikt ook werkwijze van php4? Bv. gebruik van 'var' voor de declaratie van variabelen, terwijl je in php5 de scope aangeeft (public, protected, private).

Zet de toegestane extensies in een array. Is misschien iets meer werk, maar wel netter en handiger.

Heb je een voorbeeldje van hoe je hem moet toepassen? Welke methods je waar moet aanroepen?

Trouwens, al die "else return false" stukjes kan je ook weglaten, en helemaal aan het eind een return false plaatsen. Wordt de return true uitgevoerd, dan zal hij de return false niet bereiken. Anders wel, en heb je hetzelfde resultaat in soms tot wel 10 regels minder, en meer overzicht.

Verplaatsen kan ook gewoon met 'rename'. Gaat veel sneller aangezien je de bestanden niet hoeft te kopiëren.