Consiglio: errori e miglioramento del codice

[Razorblade]

Ciao,
per migliorare il tuo codice potresti pensare di creare una tua libreria che legga il file excel e ti restituisca i dati.

Per libreria intendo un oggetto a se stante (in un file a parte, ad esempio ExcelDataReader), a cui passare il file e che ti restituisca i dati nel formato in cui ti servono, per esempio un oggetto o un array.

La parte della visualizzazione è un passaggio successivo.

Gli oggetti PHPExcel potrebbero essere incapsulati all'interno della tua libreria.

Inoltre non fidarti dell'esistenza di chiavi all'interno di un array, verifica sempre se esistono le chiavi che ti aspetti e con i dati che ti aspetti, inoltre verifica se il file è esistente, leggibile e del formato corretto.

Crea uno o più oggetti Exception per la tua libreria, es. ExcelDataReaderException, questo può semplicemente estendere Exception nativa di Php, puoi lanciare questa eccezione all'interno della tua libreria in caso di eccezione ( ad esempio, il foglio che dovrebbe esistere nell'excel non è presente).

Questo un possibile esempio

try {
  $excelReader = new ExcelDataReader($filepath);
  $data = $excelReader->getData();
} catch(ExcelDataReaderException $e) { 

/* Blocco catch multiplo nel caso in cui sia necessaria 
una differente gestione delle eccezioni in base all'eccezione lanciata */


} catch(ExcelDataReaderException $e) {

} catch(Exception $e) {

}


// visualizzazione dei dati