Automatisch Fotoalbum V3
http://test.i-cey.net/voorbeelden/i-cey.net/?pagina=fotoalbum
Mensen nog op-en-aanmerkingen? Iemand die even de moeite wilt doen om de veiligheid te checken? Is het bijv. nog wenselijk om meer if/elsjes te gebruiken en dus bijv. dingen te gaan checken als het aanmaken van thumbs.
Gewijzigd op 01/01/1970 01:00:00 door Arjan Kapteijn
Zet hem anders even opnieuw in de lib.
Ik vraag in eerste instantie om op-en-aanmerkingen.
if(isset($_GET['afbeelding']) && !empty($_GET['afbeelding']))
en dat geldt ook voor
if(isset($_GET['album']) && !empty($_GET['album']))
Ik heb hem niet helemaal doorgenomen maar het ziet er zeer goed uit
http://test.i-cey.net/voorbeelden/i-cey.net/?pagina=fotoalbum&album=fotoalbum/blaat
wekt namelijk ook nu
Gewijzigd op 01/01/1970 01:00:00 door Klaasjan Boven
- $map een constante maken zodat $map in het script niet meer gewijzigd kan worden.
Beginners tegen zichzelf beschermen en gewoon duidelijkheid.
Je gebruikt hem ook in de functie thumbnail, het kan voor de beginners totaal onduidelijk zijn hoe het met de scope zit.
- Array met toegestane extensies (mime types) maken.
- Je chmod alles naar 777, is dit werkelijk nodig?
- Checken of extensie / mime type toegestaan kan dmv array wat ik hierboven zeg, dan hoef je niet een hele lange if te maken.
- Zoals je zelf al zegt; sommige dingen zou je beter met een extra ifje kunnen doen om een onverwachte aktie te voorkomen.
- Duidelijke var namen: $tmp -> $tmp2 of $map -> $map2 (persoonlijke voorkeur ;))
- Het gebruik van de functie getimagesize() kan je misschien beter vervangen door andere functies. getimagesize() kan onverwachte foutmeldingen veroorzaken.
Verder lijkt het redelijk veilig.