Klassen nach Bedingung laden

[Reality]

Hi,
ich habe ein Problem.
Ich möchte mein Entpackungprogramm sagen, dass wenn die Datei mit zip endet, er ZipOutputStream laden soll und bei gzip GZIPOutputStream.

Nur leider geht das nicht. Eine neue Klasse möchte ich deswegen nicht schreiben.

Liebe Grüße
Reality

 

[thE_29]

?

wieso nimmst nicht den String Operator endsWith("zip") mache das und bei gzip mache das!

und mit File lässt du dir den Dateinamen als string zurückbirngen (getAbsolutePath()) und dann endswith..

 

[Reality]

Klar, den String übergebe ich mit endsWith(), aber das funktioniert eben nicht:

//Pseudo-Code
if(file.toString().endsWith("zip")){
ZipInputStream zip = new ZipInpuStream(new FileInputStream(file));
}

else if(file.toString().endsWith("gz")){
GZIPInputStream zip = new GZIPInpuStream(new FileInputStream(file));
}

 

[thE_29]

prob mal statt File.toString();
File.getAbsolutePath()

 

[Reality]

Das hat damit nichts zu tun. Ich habe auch mal witzweise folgendes ausprobiert:
Wenn Zahl == 0, dann lade Gzip, ansonsten Zip.

Liebe Grüße
Reality

 

[thE_29]

ich würds mal probieren
mach einfach mal ein System.out.println(file.toString);
und dann System.out.println(file.getAb...());

und schau was er dir ausgibt vielleicht hats da schon was!

 

[Reality]

Gibt beides dasselbe aus.

Liebe Grüße
Reality

 

[thE_29]

und hört da eines mit zip auf ?

am besten ist du postest den code...

 

[Reality]

import java.io.*;
import javax.swing.JOptionPane;
import java.util.zip.*;

class DeComprimize {
  File files[];
  GUI frame;
  
  DeComprimize(File files[], GUI frame) throws IOException{
    this.files = files;
    this.frame = frame;

    decomprimize();
  }

  private void decomprimize() throws IOException{

    for(byte i = 0; i < files.length; i++){
      if (files[i].exists() && files[i].toString().endsWith("gz")) {
        int laenge = files[i].toString().length() - 3;
        String name = files[i].toString().substring(0, laenge);

        byte buffer[] = new byte[6000];
        int gelesen;

        FileOutputStream output = new FileOutputStream(name);
        FileInputStream input = new FileInputStream(files[i]);
        if(files[i].toString().endsWith("zip"))
          GZIPInputStream gzipIn = new GZIPInputStream(input);

        else
          ZipInputStream gzipIn = new ZipInputStream(input);

        while ( (gelesen = gzipIn.read(buffer, 0, buffer.length)) != -1)
          output.write(buffer, 0, gelesen);

        JOptionPane.showMessageDialog(frame,
                                      files[i].toString() + " erfolgreich entpackt");

        output.close();
        gzipIn.close();
        input.close();
      }
      
      else
        JOptionPane.showMessageDialog(frame, "Falsches Format!");
    }
  }
}

 

[thE_29]

wie soll der jemals in die untere Bedingung kommen wenn sowas erfüllt werden muss?

if (files.exists() && files.toString().endsWith("gz"))

?? glaube kaum das .zip und .gzip mit gz aufhören

 

[Reality]

Hi,
gzip endet mit gz.
Ich habe etwas Daten Chaos, ich ändere ständig etwas.
Es müsste so heissen:

if (files[i].exists() && files[i].toString().endsWith("gz") || files[i].toString().endsWith("zip"))

Aber das ändert nichts an der Tatsache, dass es nicht funktioniert.
Immer kommt die Fehlermeldung:
Keine Anweisung in Zeile bla. ';' in Zeile bla erwartet.

Du kannst ja mal selbst das komplette Projekt compilieren:

GUI:

import javax.swing.*;
import java.io.*;
import java.awt.event.*;

public class GUI extends JFrame implements ActionListener{

  JPanel contentPane = new JPanel();
  JMenuBar mBar = new JMenuBar();

  JButton btEntpacken = new JButton("Entpacken");
  JButton btPacken = new JButton("Packen");
  JMenu datei = new JMenu("Datei");
  JMenuItem decomp = new JMenuItem("EntPacken");
  JMenuItem comp = new JMenuItem("Packen");
  JMenuItem close = new JMenuItem("Schließen");
  JComboBox method = new JComboBox();

  public GUI(){
    contentPane.add(btEntpacken);
    contentPane.add(btPacken);
    contentPane.add(method);

    btEntpacken.addActionListener(this);
    btPacken.addActionListener(this);
    decomp.addActionListener(this);
    comp.addActionListener(this);
    close.addActionListener(this);

    setJMenuBar(mBar);
    setContentPane(contentPane);

    datei.add(decomp);
    datei.add(comp);
    datei.add(close);

    method.addItem("gzip");
    method.addItem("zip");

    mBar.add(datei);
  }

  public void actionPerformed(ActionEvent ae){
    File files[];

    if(ae.getSource() == close)
      System.exit(0);

    else{
      JFileChooser fd = new JFileChooser();
      fd.setMultiSelectionEnabled(true);
      int action = fd.showOpenDialog(this);

      if(action == JFileChooser.APPROVE_OPTION){ //falls nicht abgebrochen wurde...
        files = fd.getSelectedFiles();

        if(ae.getSource() == comp || ae.getSource() == btPacken){
          try {
            new Comprimize(files, "GZIP", this);
          }
          catch (IOException e) {}
        }

        else if(ae.getSource() == decomp || ae.getSource() == btEntpacken){
          try{
            new DeComprimize(files,  this);

          }catch(IOException e){}
        }
      }
    }
  }

  public static void main(String args[]){
    JFrame frame= new GUI();
    frame.setSize(300, 100);
    frame.setTitle("Das ultimative Entpackungsprogramm");

    frame.addWindowListener(new WindowAdapter(){
    public void windowClosing(WindowEvent e){
      System.exit(0);
    }
    });

    frame.setVisible(true);
    frame.setResizable(false);
  }
}

/*class OnlyZIP implements FileFilter{

  public boolean accept(File file){

    return file.toString().toLowerCase().endsWith(".gz");
  }
}*/

comprimize:

import java.io.*;
import javax.swing.JOptionPane;
import java.util.zip.*;

class DeComprimize {
  File files[];
  GUI frame;
  
  DeComprimize(File files[], GUI frame) throws IOException{
    this.files = files;
    this.frame = frame;

    decomprimize();
  }

  private void decomprimize() throws IOException{

    for(byte i = 0; i < files.length; i++){
      if (files[i].exists() && files[i].toString().endsWith("gz")) {
        int laenge = files[i].toString().length() - 3;
        String name = files[i].toString().substring(0, laenge);

        byte buffer[] = new byte[6000];
        int gelesen;

        FileOutputStream output = new FileOutputStream(name);
        FileInputStream input = new FileInputStream(files[i]);
        if(files[i].toString().endsWith("zip"))
          GZIPInputStream gzipIn = new GZIPInputStream(input);

        else
          ZipInputStream gzipIn = new ZipInputStream(input);

        while ( (gelesen = gzipIn.read(buffer, 0, buffer.length)) != -1)
          output.write(buffer, 0, gelesen);

        JOptionPane.showMessageDialog(frame,
                                      files[i].toString() + " erfolgreich entpackt");

        output.close();
        gzipIn.close();
        input.close();
      }
      
      else
        JOptionPane.showMessageDialog(frame, "Falsches Format!");
    }
  }
}

Es ist noch nicht ganz fertig, darum ist die JCombox momentan nicht funktionsfähig. Es soll dafür gedacht sein, in welchem Format es komprimieren soll.

Liebe Grüße
Reality

 

[thE_29]

Dein If spinnt, das ist es

if(files[i].toString().endsWith("zip"))
{
          GZIPInputStream gzipIn = new GZIPInputStream(input);
}

        else
{          ZipInputStream gzipIn = new ZipInputStream(input); 
}

 

[Reality]

Es kommt dann immernoch eine Fehlermeldung:

Symbol kann nicht gefunden werden: Variable gzipIn in Klasse DeComprimize

Diese Fehlermeldung kommt 2x.

Liebe Grüße
Reality

PS: Bei einer Zeile kann man doch if auch ohne Klammern schreiben. Wieso hat es Probleme gemacht?

 

[thE_29]

weil du ein new gehabt hast und das waren 2, weil du zuesrst eine Variable anlegst und dann new sagst!

        InflaterInputStream gzipIn = null;
        if(files[i].toString().endsWith("zip"))
          gzipIn = new GZIPInputStream(input);
        else
          gzipIn = new ZipInputStream(input);

und er regt sich auf weil du deine gzipIn nur in den ifs angelegt hast und er es dann draußen nicht benutzen kann! so wie schleifen variablen halt

 

[Reality]

Ha, danke!

 

[Reality]

Scheiße.
Ich merke gerade, dass sich nur von meinem Programm gepackte Dateien wieder entpacken lassen, weil ich eben beim Entpacken die alte Endung wieder herstelle, in dem ich das gz oder zip weglasse. (davor hiess es ja blabla.txt.gz)
Und es kann auch nicht komprimierte dateien entpacken, die mehrere dateien in sich haben.

Liebe Grüße
Reality

 

[Reality]

Warum erstellt er mir jetzt leere zip-Files, wenn der Typ zip ist?

import java.io.*;
import java.util.zip.*;
import javax.swing.JOptionPane;

class Comprimize{

  File files[];
  GUI frame;
  String type;

  Comprimize(File files[], String type, GUI frame) throws IOException{
    this.files = files;
    this.type = (type.equals("zip"))? "zip" : "gz";
    this.frame = frame;

    comprimize();
  }

  private void comprimize() throws IOException{
    byte buffer[] = new byte[6000];
    int gelesen = 0;

    for(byte i = 0; i < files.length; i++){
      if (files[i].exists() && files[i].isFile()) {
        FileInputStream input = new FileInputStream(files[i]);
        FileOutputStream output = new FileOutputStream(
            files[i].toString() + "." + type);
        DeflaterOutputStream gzipOut;

        if(type.equals("zip"))
          gzipOut = new ZipOutputStream(output);

        else
          gzipOut = new GZIPOutputStream(output);
        
        while ( (gelesen = input.read(buffer, 0, buffer.length)) != -1)
          gzipOut.write(buffer, 0, gelesen);

        gzipOut.close();
        output.close();
        input.close();

        JOptionPane.showMessageDialog(frame,
                                      files[i].toString() + " erfolgreich gepackt");
      }

      else {
        JOptionPane.showMessageDialog(frame, "Fehler!",
                                      files[i].toString() + "ist ungültig",
                                      JOptionPane.ERROR_MESSAGE);
      }
    }
  }
}

Liebe Grüße
Reality

 

[thE_29]

debugs mal durch und schau nach ob das while überhaupt was zurückliefert und ob er überhaupt das durchgeht

 

[Reality]

Werd davon nicht schlau:
http://mitglied.lycos.de/masterchan/debugger.JPG

Liebe Grüße
Reality

 

[Beni]

Siehst du die 8 Knöpfe direkt unter "Meldungen"? Klickt die mal durch, einer davon zeigt den Wert der verschiedenen Variablen an (natürlich nur, wenn das Programm läuft, und an einem Breakpoint angelangt ist).

 

[Reality]

int gelesen = 10;
und byte buffer[]
[0] = 31;
[1] = -117;
[2] = 8;
der Rest ist 0;

Bei gzipOut steht da:
java.util.ZipOutputStream
written: long = 0

Ich denke, das war mal das wichtigste...

Liebe Grüße
Reality

 

[Reality]

Vielleicht sollte ich erwähnen, dass nicht mal der Dialog auftaucht, der sagt, dass die Datei erfolgreich gepackt wurde, obwohl dieser Befehl in der for-Schleife enthalten ist.

Liebe Grüße
Reality

 

[Reality]

Dann geb ich halt Java die Schuld, dass es nicht klappt. Bäh!