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.

[Squaloattacca]

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.

Perfetto. Ora non mi resta che risolvere l'enigma di quel ciclo

[Squaloattacca]

Ok, per gli utenti scrivo come ho pensato di risolvere. Ora l'ultima parte appare così:

InputStream in = Zf.getInputStream(one);
            BufferedInputStream br = new BufferedInputStream(in);
            FileOutputStream out = new FileOutputStream(one.getName());
            BufferedOutputStream bw = new BufferedOutputStream(out);
            byte [] ba = new byte [4 * 1024];
            int bubi = -7;
            while ((bubi = br.read(ba))>0) {
                bw.write(ba, 0, bubi);
            }
            bw.flush();
            bw.close();
            br.close();

Sembra funzionare a meraviglia, e spero che il codice sia buono