Errore I/O

[Squaloattacca]

Salve
Mi sto appena affacciando al mondo di java.
In questo momento, sto creando un'applicazione a console, che cerca il file.zip passato come argomento, ne elenca i file e li estrae.
Il codice è:

import java.util.*;
import java.util.zip.*;
import java.io.*;

class extract {
    public static void main (String [] f) throws IOException {
        ZipFile Zf;
        try {Zf= new ZipFile (f[0]);}
        catch (ZipException e) {Zf=null;}
        catch (IOException e1) {Zf=null;};
        System.out.println("Decomprimo "+Zf.getName());

        Enumeration files = Zf.entries();
        while(files.hasMoreElements ()) {
            ZipEntry one = (ZipEntry) files.nextElement();
            System.out.println ("Estraggo "+one+" da "+Zf.getName());
            System.out.println("Dimensione compresso: "+one.getCompressedSize());
            System.out.println("Dimensione non compresso: "+one.getSize());
            System.out.println("CRC: "+one.getCrc());
            System.out.println("Modificato in data: "+one.getTime());
            
            InputStream in = Zf.getInputStream(one);
            FileOutputStream out= new FileOutputStream(one.getName());

            for (int ch=in.read();ch!=-1;ch=in.read()) {
                out.write(ch);
                out.close();
                in.close();
            }
        }
    }
}

A parte il fatto che non riesco a capire le condizione dell'ultimo ciclo for (l'ho preso da una guida ), ho un errore di input/output su ogni zip che decido di estrarre. Tutto il resto dello script funziona (elenca i files, le dimensioni compresso e non, e il CRC), ma l'estrazione fallisce. Per completezza riporto l'output:

Exception in thread "main" java.io.IOException: Stream closed
at java.util.zip.InflaterInputStream.ensureOpen(Unkno wn Source)
at java.util.zip.InflaterInputStream.read(Unknown Source)
at extract.main(extract.java:25)

Qualcuno sa spiegare cosa sto sbagliando?

[minomic]

Ciao,
"Stream closed" mi fa pensare che tu chiami il metodo close su qualche stream e poi tenti nuovamente di utilizzarlo. In particolare quel ciclo for mi sembra strano: fai un write e poi un close. All'iterazione successiva da per forza errore. Probabilmente il codice che hai visto era senza graffe e tu le hai aggiunte in maniera errata: le due close vanno fuori dal ciclo for.

[Squaloattacca]

Ciao,
"Stream closed" mi fa pensare che tu chiami il metodo close su qualche stream e poi tenti nuovamente di utilizzarlo. In particolare quel ciclo for mi sembra strano: fai un write e poi un close. All'iterazione successiva da per forza errore. Probabilmente il codice che hai visto era senza graffe e tu le hai aggiunte in maniera errata: le due close vanno fuori dal ciclo for.

Grazie, ho risolto, e grazie alle tue spiegazioni ho anche capito di più cosa stavo facendo.

[Alex'87]

Che modo inefficiente di estrarre uno zip... non puoi leggere a buffer?

[Squaloattacca]

Che modo inefficiente di estrarre uno zip... non puoi leggere a buffer?

Come dicevo, sto giusto iniziando.
Va beh, immagino di poter cominciare con:

BufferedReader br = new BufferedReader(new InputStreamReader(in));

E poi, nel ciclo for avrò

ch=br.read()

Sbaglio?

[LeleFT]

Sbaglio?

La risposta alla domanda "Sbaglio?" è "sì"... BufferedReader è un aggeggo che serve a leggere stream di "caratteri". Quello che devi fare tu è leggere stream di byte (quindi, byte secchi, senza alcuna interpretazione di caratteri, che potrebbe falsare la lettura).

Quello che intendeva dire Alex è che non ha molto senso leggere (e scrivere) un byte alla volta... conviene usare un array di byte (di una certa dimensione, generalmente multipli di 1024) in modo da "velocizzare" la lettura.


Ciao.

[Squaloattacca]

La risposta alla domanda "Sbaglio?" è "sì"... BufferedReader è un aggeggo che serve a leggere stream di "caratteri". Quello che devi fare tu è leggere stream di byte (quindi, byte secchi, senza alcuna interpretazione di caratteri, che potrebbe falsare la lettura).

Quello che intendeva dire Alex è che non ha molto senso leggere (e scrivere) un byte alla volta... conviene usare un array di byte (di una certa dimensione, generalmente multipli di 1024) in modo da "velocizzare" la lettura.


Ciao.

Ho notato: mi corrompe i file XD sto consultando la documentazione per trovare la soluzione. In base a cosa decido la dimensione del buffer?

EDIT: per il momento sto adottando la seguente:

BufferedInputStream br = new BufferedInputStream(in);
BufferedOutputStream bw = new BufferedOutputStream(out);

dove in e out sono definiti nel codice originario. Non so che fare a quel ciclo for adesso

[LeleFT]

Ho notato: mi corrompe i file XD sto consultando la documentazione per trovare la soluzione. In base a cosa decido la dimensione del buffer?

Generalmente "a tuo gusto". Diciamo che solitamente si vedono in giro valori che vanno dai 4 agli 8 KB (quindi 4 * 1024 o 8 * 1024), per tenersi "leggeri".


Ciao.

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