login  Naam:   Wachtwoord: 
Registreer je!
Scripts > PHP > Overige scripts > Update controle


Reacties op het script Update controle

  Onbekend
Gepost op: 17 mei 2008 - 16:26
Verbannen



Wat vinden jullie ervan? Het is mijn eerste script op sima, dus brand maar los 

Offline  Wim
Gepost op: 18 mei 2008 - 22:37
Crew algemeen



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.

Offline  ikkedikke
Gepost op: 20 mei 2008 - 16:12
PHP expert



Je hebt je zogenaamde "exitpoints" over het hele script verspreid liggen.
Dit maakt het niet echt overzichtelijk om te begrijpen of na te lezen.

  Onbekend
Gepost op: 20 mei 2008 - 18:00
Verbannen



ikkedikke schreef:
Je hebt je zogenaamde "exitpoints" over het hele script verspreid liggen.
Dit maakt het niet echt overzichtelijk om te begrijpen of na te lezen.

Wat bedoel je met exitpoints?

Offline  Godlord
Gepost op: 31 mei 2008 - 17:01
PHP gevorderde



Waarschijnlijk de "return false;".


M.v.g. Godlord.


Enkel aanvullende informatie, vragen en antwoorden op vragen zijn welkom.
 
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.023s