Bruk
error_reporting( E_ALL );
under utvikling av php, da vil den rapportere med alle warning, notices og error. Det er mer "nazi", men det blir i det minste mer korrekt. Å gjøre noe annet er bare idiotisk, da sitter man der uten informasjon om hva som har hendt dersom det er noe galt. Med de klassiske innstillingene vil du ikke få noen spesiell warning dersom du prøver å aksessere en variabel som ikke eksisterer. Eksempelvis $Data og $data.
I tillegg, kan funksjonen
strtotime være grei å bruke, så slipper du tenke på skuddår og slikt.
Kode
$time = strtotime( "+ 1 year" );
I funksjonen din ville jeg deklarert og initialisert variabelen data til -1 og returnert på slutten av koden. På den måten kan du dra en sjekk mot -1 og i det minste avklare om det skjer noe fornuftig/ikke fornuftig i funksjonen. Slik du har den funksjonen nå vil den kunne enten returnere verdi, eller ikke noe som helst. Dette vil kunne klassifiseres som litt dårlig programmeringsskikk.
Kode
function readFromFile($number)
{
$Data = -1;
if($number < 43)
{
$number = strpad($number, 2, "0", STR_PAD_LEFT);
$textfile = "../jobb/{$number}.txt";
$file = fopen("$textfile", 'r');
$Data = fread($file, filesize($textfile));
fclose($file);
}
return $Data;
}
Jeg kan også tipse om å bruke Zend Studio, og lære deg å bruke debuggeren. Zend støtter også web debugging hvis du ikke vil gjøre debuggingen lokalt. Da har du mulighet til å kunne gå steg for steg igjennom koden din (runtime) og se (med f.eks watches) på verdiene på variablene dine. Dette gir det ufattelig mye bedre debuggingsmuligheter og det kan kalles et paradigmeskifte.
Sist endret av m0b; 8. juli 2010 kl. 18:20.