Reacties op het script Image Upload Class + Thumbnail

Er wordt ook niet gecontroleerd op mime-types. Ik weet dat ermee gerommeld kan worden, maar het is over het algemeen toch wel een sterke, extra maatregel. Deze zou ik, als ik jou was, wel toevoegen.

[..quote..]

Je weet dat de browser de mimetype meestuurt, en het dus *niet* te vertrouwen is?

Iets wat helemaal tegen het OOP principe ingaat zijn global variabeles, omdat er anders afhankelijkheden in je applicatie ontstaan. Jij gebruikt deze toch in je klasse, in de methode upload(), namelijk in de vorm van $_FILES. Het is beter $_FILES mee te geven als parameter.

Er wordt ook niet gecontroleerd op mime-types. Ik weet dat ermee gerommeld kan worden, maar het is over het algemeen toch wel een sterke, extra maatregel. Deze zou ik, als ik jou was, wel toevoegen.

Ik zou je methode thumbnail ook willen verdelen in 2 functies, namelijk ook nog eentje die de GD bewerkingen uitvoert en een eventuele array met meldingen teruggeeft. Dit is beter voor de modulariteit van je klasse, evenals voor de herbruikbaarheid en aanpasbaarheid.

Je hebt nu een paar attributen public, deze kan je beter protected/private maken
met een setter (__set()). Waarom dit beter is staat wel beschreven in deze tutorial: http://www.site...t_OOP#kop5

Heb de Tut nogmaals doorgelezen, maar ik zie nog niet helemaal het nut hiervan in, ik maak namelijk eigen soortgelijke set-functies.

@JeXuS & Arie2Zero: zoals ik al vermeld kan dit er prima bij als extra check. Maakt het toch weer wat robuuster, ook al is hij niet 100% correct.

[..quote..] Wat daar staat beschreven komt duidelijk naar voren in jouw klasse, namelijk, ik kan het attribuut size ook de waarde 'abcdefghijk'. Hierdoor kan de klasse zijn werk niet goed doen, immers, er is geen geldige size beschikbaar. Met een simpele setter die controleert of het wel een integer is, is je klasse direct een stuk robuuster.

[..quote..]Waarom zal je een check doen of de waarde niet numeriek is? Deze class heeft dat niet nodig. Waarom zal je een class gaan gebruiken om verkeerde waarden te gaan gebruiken waar het mogelijk is? Als je waarde niet numeriek is dan zal het automatisch de integer 0 krijgen, dus kan je niet uploaden, maar ik vind dit gewoon onzin. Ik ga toch ook niet checken of de x-value wel numeriek is? Een class moet je gewoon juist gebruiken en niet allemaal onzin dingen gaan declareren, want daarvoor is de class niet gemaakt. Jij als "webmaster" geeft de max-value aan, daar zit verder geen user-input bij te kijken. Dus de controle vind ik compleet overbodig in deze class.

[..quote..]Dat kan jij allemaal best vinden, maar de integriteit van je klasse wordt daardoor echt minder. Tuurlijk gooi je er ook een check of de x-value een integer is. En wie zegt dat je de klasse niet gaat hergebruiken in een applicatie waarin de user _wel_ zelf kan bepalen wat de maximale upload size is (denk aan CMS)? Je beredeneert hier gewoon hetzelfde alsof je in MySQL een string in een integer kolom giet, dat het dan toch wel een 0 wordt? Dat is een belachelijke gedachte. Dit is helemaal buiten de OO methodiek om. Je weet nooit in wat voor situaties je deze klasse gaat hergebruiken. Zeggen dat een klasse 'gewoon juist moet gebruikenen niet allemaal onzing dingen gaat declareren' gaat natuurlijk nergens over, eerder gewoon domme luiheid van de programmeur itself. PEBCAK, problem exists between chair and keyboard.