Errore I/O

[Alex'87]

Non so cosa sia Zf e one.

Io lo semplificherei un po' comunque, dando dei nomi decenti alle variabili

BufferedInputStream bis = new BufferedInputStream(Zf.getInputStream(one));
BufferedOutputStream bos = new BufferedOutputStream(new FileOutputStream(one.getName()));

byte[] buffer = new byte[4 * 1024];
int bytesRead;

while ((bytesRead = bis.read(buffer)) != -1) {
    bos.write(buffer, 0, bytesRead);
}

bos.close();
bis.close();

Controllare che la read restituisca qualcosa di strettamente maggiore di zero non è corretto: il valore restituito quando non c'è più nulla da leggere è -1.

Manca inoltre la gestione delle eccezioni.

[Squaloattacca]

Non so cosa sia Zf e one.

Io lo semplificherei un po' comunque, dando dei nomi decenti alle variabili

BufferedInputStream bis = new BufferedInputStream(Zf.getInputStream(one));
BufferedOutputStream bos = new BufferedOutputStream(new FileOutputStream(one.getName()));

byte[] buffer = new byte[4 * 1024];
int bytesRead;

while ((bytesRead = bis.read(buffer)) != -1) {
    bos.write(buffer, 0, bytesRead);
}

bos.close();
bis.close();

Controllare che la read restituisca qualcosa di strettamente maggiore di zero non è corretto: il valore restituito quando non c'è più nulla da leggere è -1.

Manca inoltre la gestione delle eccezioni.

Zf è di tipo ZipFile; nel codice per definirlo avevo fatto così:

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());

Invece one è la ZipEntry corrispondente al file da estrarre:

ZipEntry one = (ZipEntry) list.nextElement ();

che nel codice è in un while che la definisce file per file:

while (list.hasMoreElements()) {}

e

Enumeration list = Zf.entries();

Per semplificare il codice mi pare tu abbia direttamente eliminato le mie variabili di InputStream e FileOutputStream e le abbia messe direttamente nello scrittore e lettore con buffer...mi abituerò

Per quanto riguarda le eccezioni:

try {
BufferedInputStream bis = new BufferedInputStream(Zf.getInputStream(one));
BufferedOutputStream bos = new BufferedOutputStream(new FileOutputStream(one.getName()));

byte[] buffer = new byte[4 * 1024];
int bytesRead;

while ((bytesRead = bis.read(buffer)) != -1) {
    bos.write(buffer, 0, bytesRead);
}

bos.close();
bis.close();
}
} catch (ZipException e) {System.out.println("Si è verificato un errore. Estrazione fallita");}
catch (IOException e) {System.out.println("Errore di Input/Output. Estrazione fallita");};

va bene così?

[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?

[Alex'87]

Zf di default? A sua volta non lancia eccezioni? O_o

(Zf è un altro nome da cambiare: le variabili iniziano sempre con la minuscola)

flush viene già chiamato da close()

[Squaloattacca]

Zf di default? A sua volta non lancia eccezioni? O_o

(Zf è un altro nome da cambiare: le variabili iniziano sempre con la minuscola)

flush viene già chiamato da close()

Per zf (minuscolo a sto punto) di default intendo un file che magari conosco già. Oppure dovrei interrompere lo script, magari con un return qualcosa?

[minomic]

Ciao,
potresti visualizzare un messaggio di errore e poi, dato che sei anche nel main, fare System.exit(-1) per terminare il programma. Questo non fa eseguire il blocco finally però, dato che comunque chiudi il programma, non dovrebbe essere un problema.

[Squaloattacca]

Ciao,
potresti visualizzare un messaggio di errore e poi, dato che sei anche nel main, fare System.exit(-1) per terminare il programma. Questo non fa eseguire il blocco finally però, dato che comunque chiudi il programma, non dovrebbe essere un problema.

Il messaggio di errore al posto di un file zip di default intendi?