Errore I/O

[Alex'87]

ZipFile Zf;
try {Zf = new ZipFile (file);}
catch (ZipException e) {Zf=null;}
catch (IOException e1) {Zf=null;};
System.out.println("Decomprimo il file "+Zf.getName());

Male.
In caso di eccezione tu metti Zf a null e vai avanti come se nulla fosse.
Peccato che poi tu faccia Zf.qualcosa e se Zf è null ottieni NullPointerException.

In questo caso ha poco senso gestire l'eccezione, poco o nulla puoi fare per rimediare.

La chiusura degli stream va fatta in finally.

[Squaloattacca]

ZipFile Zf;
try {Zf = new ZipFile (file);}
catch (ZipException e) {Zf=null;}
catch (IOException e1) {Zf=null;};
System.out.println("Decomprimo il file "+Zf.getName());

Male.
In caso di eccezione tu metti Zf a null e vai avanti come se nulla fosse.
Peccato che poi tu faccia Zf.qualcosa e se Zf è null ottieni NullPointerException.

In questo caso ha poco senso gestire l'eccezione, poco o nulla puoi fare per rimediare.

La chiusura degli stream va fatta in finally.

L'avevo notato; in effetti nella nuova versione ho messo un Zf di default.
Messa la chiusura in finally dopo l'ultimo catch.
Ma ora ho notato che mi hai tolto il flush(). Dopo aver finito, non dovrei anche svuotare il buffer?