2

Necesito descomprimir una carpeta desde java sin embargo no me funciona la clase

package controlador;
import java.io.BufferedInputStream;  
import java.io.BufferedOutputStream;  
import java.io.FileInputStream;  
import java.io.FileOutputStream;
import java.util.zip.ZipInputStream;

public class Descomprime {  
       public void Descomprimir (String c, String d )throws Exception{

    ZipInputStream entrada = new ZipInputStream(new BufferedInputStream(new FileInputStream(c)));
    if(entrada.getNextEntry()==null)
    {
        int leido;
        byte []buffer = new byte [4758];
        BufferedOutputStream salida = new BufferedOutputStream(new FileOutputStream(d),4758);
        while((leido = entrada.read(buffer, 0, 4758))!=1)
            salida.write(buffer, 0, leido);
            salida.flush();
            salida.close();
            entrada.close();
        }
    }
}

Me arroja un error en la linea

BufferedOutputStream salida = new BufferedOutputStream(new FileOutputStream(d),4758);
    while((leido = entrada.read(buffer, 0, 4758))!=1)
    introducir el código aquí
2
  • 1
    Te falta abrir una llave luego del while ( .... ) {.
    – user227
    el 9 sep. 2016 a las 21:40
  • 1
    También después agregar la llave al while te faltaría una llave '}' para cerrar la clase
    – abrahamhs
    el 9 sep. 2016 a las 22:10

1 respuesta 1

2

Estas confundiendo el flujo Zip (ZipInputStream) con las entradas (ZipEntry). Lo correcto sería guardarse el entry en una variable y actuar cuando es distinto a null, no cuando es igual :

    ZipInputStream flujo = new ZipInputStream(new BufferedInputStream(
                             new FileInputStream(c)));
    ZipEntry entrada = flujo.getNextEntry();
    if ( entrada != null) {

En la línea :

while ((leido = entrada.read(buffer, 0, 4758)) != 1)

entrada.read devuelve el número de bytes leidos o -1 si se ha llegado al final de la entrada. Cuando devuelve -1 esto provoca la excepción en la línea :

            salida.write(buffer, 0, leido);

al pedir que escriba -1 bytes.

Lo correcto sería comparar que no es -1 :

while ((leido = entrada.read(buffer, 0, 4758)) != -1)

También hay que tener en cuenta que un ZIP puede contener carpetas además de archivos. No podemos simplemente tratar todas las entries como si fuesen archivos.

        if (entrada.isDirectory()) {
            File directorio = new File(nombreSalida);
            directorio.mkdir();
        }

Finalmente, un ZIP puede contener varios archivos (entradas o entries), con lo que no basta un if, hay que recorrerlos en un bucle para obtener todos los archivos y descomprimirlos. El siguiente es el código completo :

package controlador;

import java.io.BufferedInputStream;
import java.io.BufferedOutputStream;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.util.zip.ZipEntry;
import java.util.zip.ZipInputStream;

public class Descomprime {

  public void Descomprimir(String ficheroZip, String directorioSalida)
          throws Exception {
    final int TAM_BUFFER = 4096;
    byte[] buffer = new byte[TAM_BUFFER];

    ZipInputStream flujo = null;
    try {
      flujo = new ZipInputStream(new BufferedInputStream(
              new FileInputStream(ficheroZip)));
      ZipEntry entrada;
      while ((entrada = flujo.getNextEntry()) != null) {
        String nombreSalida = directorioSalida + File.separator
                + entrada.getName();
        if (entrada.isDirectory()) {
          File directorio = new File(nombreSalida);
          directorio.mkdir();
        } else {
          BufferedOutputStream salida = null;
          try {
            int leido;
            salida = new BufferedOutputStream(
                    new FileOutputStream(nombreSalida), TAM_BUFFER);
            while ((leido = flujo.read(buffer, 0, TAM_BUFFER)) != -1) {
              salida.write(buffer, 0, leido);
            }
          } finally {
            if (salida != null) {
              salida.close();
            }
          }
        }
      }
    } finally {
      if (flujo != null) {
        flujo.close();
      }

    }
  }
}

Me he tomado la libertad de cambiar el tamaño del buffer por un número redondo (en binario). Pero independientemente del número usado no es recomendable repetirlo por varios sitios del código, mejor definir una constante y utilizar esa constante en el código.

6
  • cuanto lo ejecuto y lo debug no ingresa en la condicion while ((entrada = flujo.getNextEntry()) != null) le estoy pasando la ruta de entrada, el zip contiene dos carpetas y dentro de las carpetas hay dos imagenes el 10 sep. 2016 a las 20:56
  • la ruta que le paso donde esrta el zip es Y:\casaentrada\Lote012016083056.zip y donde descomprime Y:\casaentrada\Lote012016083056 el 10 sep. 2016 a las 21:04
  • En su momento lo probé y ahora lo he vuelto a probar y funciona bien. ¿Es posible que le hayas pasado la ruta "Y:\casaentrada\Lote012016083056.zip"? en ese caso está mal porque \ es para secuencias de escape. Debes usar "Y:\\casaentrada\\Lote012016083056.zip" y lo mismo en la otra ruta. el 10 sep. 2016 a las 21:11
  • realmente asi esta con el \\ o de lo contrario lo toma como error el 11 sep. 2016 a las 15:33
  • Pues no se que problema puedes tener. ¿Qué sistema operativo usas? ¿Qúe te devuelve java -version? ¿Puedes meter todo tu código en un archivo descargable, poniendo también el Lote012016083056.zip con que haces las pruebas y subirlo a algún sitio desde el que me lo pueda descargar? O si es un archivo que no puedes subir por ser confidencial sube otro que hagas tu y en que se produzca el mismo fallo. el 11 sep. 2016 a las 15:57

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.