je kan je parameters mss beter in een andere volgorde zetten.
bvb:
parameter 2 (hostname) zet je achter parameter 3 (filename). Filename is iets dat je overal nodig hebt, hostname niet. Zo spaar je dus bij methode 2 en 3 die lege parameter al uit.
Vervolgens, bij direct2 en bij ftp heb je 2 parameters in je functie: hostname en ftpserver; Bij ftp laat je de 1ste leeg, bij direct2 de 2de. Waarom laat je ftpserver niet weg en gebruik je bij ftp ook nite die hostname parameter.
Persoonlijk ben ik trouwens een voorstander van waardes retourneren op het einde van een functie ipv er global scopes van te maken!
//edit:
nog een opmerking. Iets als dit:
if ($ftpget==false) {
kan volgens mij beter/Veiliger in de vorm van:
if ($ftpget === false) {
Je controleert dan of ftpget echt wel op FALSE is gezet
Onbekend
Gepost op: 19 mei 2008 - 10:17
Verbannen
Bedankt voor de tips, ik zal er nog eens naar kijken.