Swing Progressbar Problem

[VGLink]

Hallo Alle zusammen. Ich hab mich jetzt mit der Progressbar beschäftigt und sie möchte einfach nicht ihren Wert ändern. Ich hab eine GUI erstellt wo man den Zielpfad zur Startdatei angibt und einen weiteren Pfad zum Speicherort. Drück man nun auf start, erstellt das Programm einen neuen Thread in dem Rechnungen vollzogen werden und in dem Thread greif ich mit super.PBar auf die progressbar zu und setze den neuen Wert, doch leider tut sich dort nix. Die Bar bleibt bei 0%.

Beispiel zum code:

private void EncryptActionPerformed(java.awt.event.ActionEvent evt) {                                        

        String start;
        String ziel;
        start = Dateipfad.getText();
        ziel = Zielpfad.getText();
        Thread encrypt = new Thread(new Encrypting(start, ziel));
        encrypt.start();

    }

Im Thread wird dann in einer Schleife rechnungen ausgeführt und in der Schleife dann geschaut wie viele Rechnungen es insgesamt sind und bei der wievielten das Programm nun ist. Im 3 Satz wird dann der Prozentwert ausgerechnet und per super.PBar.setValue() der Prozentbar übergeben.Da die Rechnung nicht im EVT Listener ist wird dieser ja nicht blockiert also wo liegt der Fehler? Achso die Encrypting Methode erbt von der Frame Methode.


Hoffe ihr könnt mir helfen.

 

[irgendjemand]

da du nirgends das event selbst verwendest brauchst du dies vom ActionListener eigentlich garnicht mitgeben ...
viel wichtiger wären da referenzen auf die GUI-klasse oder gleich die JProgressBar ...
ich vermute nämlich das du eine andere JProgressBar manipulierst als die die angezeigt wird ...

etwas mehr code könnte sicher zur lösung beitragen

 

[VGLink]

Ok also die main.java mal weggelassen. Das ist die komplette Java Datei des Threads.

package main;

import java.io.File;
import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.InputStream;
import java.io.OutputStream;
import arraybearbeiten.*;
import crypting.*;

public class Encrypting extends Main implements Runnable {

    private File file;
    private long length;
    private String Start = ("");
    private String Ziel = ("");
    private long status = 0;
    private int prozent = 0;

    public Encrypting(String start, String ziel) {
        file = new File(start);
        length = file.length();
        Start = start;
        Ziel = ziel;
    }

    public void run() {
        try {
            try {
                InputStream inputStream = new FileInputStream(Start);

                OutputStream out = new FileOutputStream(new File(Ziel));

                int read = 0;
                byte[] bytes = new byte[10];

                while ((read = inputStream.read(bytes)) != -1) {

                    status += 10;

                    String[] daten = new String[4];
                    daten[0] = ("");
                    daten[1] = ("");
                    daten[2] = ("");
                    daten[3] = ("");
                    String[] vert = new String[1];
                    vert[0] = ("");
                    Arrayvertauschen tausch = new Arrayvertauschen();

                    tausch.Vertausch(bytes, vert);
                    String Keyvert = ("");

                    for (int i = 0; i < vert[0].length(); i++) {

                        Keyvert += vert[0].charAt(i);

                    }

                    Encryptkey cryptkeyvert = new Encryptkey();

                    cryptkeyvert.Crypt(Keyvert, daten);

                    String Bytesreaded = ("");
                    char m;

                    for (int i = 0; i < bytes.length; i++) {

                        Bytesreaded += (char) bytes[i];

                    }
                    Encrypt cryptbytes = new Encrypt();

                    cryptbytes.Crypt(Bytesreaded, daten);

                    byte[] encryptedtextL = new byte[1];
                    encryptedtextL[0] = (byte) (daten[0].length());
                    byte[] encryptedtext = new byte[(daten[0].length())];

                    for (int i = 0; i < (daten[0].length()); i++) {
                        encryptedtext[i] = (byte) (daten[0].charAt(i));
                    }

                    byte[] Key0L = new byte[1];
                    Key0L[0] = (byte) (daten[1].length());
                    byte[] Key0 = new byte[(daten[1].length())];

                    for (int i = 0; i < (daten[1].length()); i++) {
                        Key0[i] = (byte) (daten[1].charAt(i));
                    }

                    byte[] encryptedkeyL = new byte[1];
                    encryptedkeyL[0] = (byte) (daten[2].length());
                    byte[] encryptedkey = new byte[(daten[2].length())];

                    for (int i = 0; i < (daten[2].length()); i++) {
                        encryptedkey[i] = (byte) (daten[2].charAt(i));
                    }

                    byte[] Key1L = new byte[1];
                    Key1L[0] = (byte) (daten[3].length());
                    byte[] Key1 = new byte[(daten[3].length())];

                    for (int i = 0; i < (daten[3].length()); i++) {
                        Key1[i] = (byte) (daten[3].charAt(i));
                    }

                    int laenge = 1;
                    int schreiben = daten[1].length();

                    out.write(Key0L, 0, laenge);
                    out.write(Key0, 0, schreiben);

                    schreiben = daten[0].length();

                    out.write(encryptedtextL, 0, laenge);
                    out.write(encryptedtext, 0, schreiben);

                    schreiben = daten[3].length();

                    out.write(Key1L, 0, laenge);
                    out.write(Key1, 0, schreiben);

                    schreiben = daten[2].length();

                    out.write(encryptedkeyL, 0, laenge);
                    out.write(encryptedkey, 0, schreiben);

                    this.prozent = (int) (((status * 100) / length));
                    super.PBar.setValue(prozent);
                }

                inputStream.close();
                out.flush();
                out.close();

            } catch (IOException e) {
                System.out.println(e.getMessage());
            }

        } catch (Exception e) {
        }
    }
}

Hab nur eine Progressbar drinnen.

 

[bERt0r]

Ich glaub du verstehst nicht ganz was Vererbung ist, und was dein super aufruf bewirkt. Ich nehme mal an deine "Main" Klasse ist ein JFrame. Wenn deine Encrypting die Main Klasse extended, dann ist die Encrypting Klasse auch ein JFrame, mit eigenem Progressbar.
Wenn du super aufrufst, greifst du nicht auf ein anderes Objekt zu. Du könntest in deinem Fall genausogut this statt super schreiben.
Deine Encrypting Klasse muss/soll/darf mMn gar nicht von Main erben. Was du benötigst, ist einfach auf die aufrufende Main Instanz zugreifen zu können. Dazu übergibst du dein Main Objekt per Konstruktor an deine Encryptingklasse und schreibst dir in der Mainklasse eine getProgressBar Methode.
[Edit]Das ganze geht übrigens auch noch weit eleganter mit EventListenern, das würde dich jetzt aber nur verwirren. Versuchs ruhig es so hinzubekommen.[/Edit]

 

[Michael...]

Hab nur eine Progressbar drinnen.

Das schon, aber - mal abgesehen davon, dass das sehr merkwürdig ist und hier evtl. sogar ein Vererbungsfehler vorliegt - wo ist den die

super.PBar

verbaut?

 

[VGLink]

@ bERt0r das hab ich mir gerade eben auch gedacht nur komm ich nicht darauf wie ich die main übergeben soll.

Ja da ich erst seit Semseteranfang immer wieder was mit Java beschäftige bin ich noch ein rechter Anfänger, das da nicht alles perfekt ist ist mir klar ;D Werd mich in den Ferien hinsetzten und nochmal die Verschlüsselung verbessern und ne schönere GUI usw versuchen zu zaubern ;D Jetzt möcht ich erstmal mit der Progressbar klar kommen ;D

@Michael kurz vor dem schluss Zeile 130 steht es ;D

 

[irgendjemand]

also das "extends" kommt RAUS ... somit auch alles was mit "super." anfängt ...

du erweiterst einfach deinen konstruktor um einen weiteren parameter und legst diese referenz dann in einer instanz-variable ab

 

[Michael...]

@Michael kurz vor dem schluss Zeile 130 steht es ;D

Das hab ich durchaus gesehen. Meine Frage war ja wo ist es (in der GUI) verbaut. Ich behaupte mal gar nicht. Da hier - wie ja auch schon von bERt0r geäußert - ein falschen Verständnis von Vererbung vorliegt.

 

[VGLink]

Also mein Ansatz ist jetzt so das ich die Progressbar an die Methode übergebe , doch kann ich diese ja dann nur Im Konstruktor verändern und nicht mehr nachträglich. Ja das mit extends ist mir vorhin auch in den Kopf gekommen aber da hatte ich schon gefragt XD


Hab jetzt die Main übergeben an den Konstruktor, das sieht nun so aus:

Thread encrypt = new Thread(new Encrypting(start, ziel, Frame));

Dann sieht das im Konstruktor so aus:

    private File file;
    private long length;
    private String Start = ("");
    private String Ziel = ("");
    private long status = 0;
    private int prozent = 0;
    public Main Frame2;
    public javax.swing.JProgressBar PBaro;

    public Encrypting(String start, String ziel, Main Frame) {
        file = new File(start);
        length = file.length();
        Start = start;
        Ziel = ziel;
        Fame2 = Frame;
        //PBaro = Frame.getPBar();
    }

Denke der Fehler liegt darin das Frame 2 zwar dann das gleich enthält wie meine vorherige main aber Änderungen darauf sich nicht auf die Main beziehen sondern eben nur darauf...

 

[bERt0r]

Das sollte so laufen, was funktioniert denn nicht? Startest du deinen Thread mit start oder run?

 

[jgh]

evtl. ist das auch nur ein AnzeigeFehler, der dadurch verursacht wird, weil du meinst Swing- und awt-Komponenten zusammen zu nutzen?!

 

[VGLink]

Wenn ich jetzt in der Rechnung Frame2.setProgBar(Prozent) ausführe wird die bar nicht geupdatet bleibt also bei 0. Starte den thread mit start(). Inwieweit können sich awt und swing behindern?

 

[Michael...]

Wenn ich jetzt in der Rechnung Frame2.setProgBar(Prozent) ausführe wird die bar nicht geupdatet bleibt also bei 0. Starte den thread mit start().

Poste doch mal einen kurzen ausführbaren Code, der das Problem nachstellt. z.B. einen JFrame mit nur einer JProgressBar und einem Thread, der einen beliebigen Wert auf der JProgressBar setzt.

Inwieweit können sich awt und swing behindern?

Denke zwar, dass Dein Problem an anderer Stelle liegt.
Aber hier zur Info:
Mixing Heavyweight and Lightweight Components
Wobei es bei höheren JDK Versionen angeblich möglich sein soll AWT und Swing Komponenten zu mischen - wobei ich keine Notwendigkeit sehe sowas zu tun.
Mixing heavy and light components

 

[jgh]

weil sich die Methoden -wann und wie bei swing, bzw bei awt neugezeichnet wird- grundsätzlich unterscheiden.

das wird -höchstwahrscheinlich- nicht das problem sein, sollte aber auf alle Fälle als Ursache ausgeschlossen werden!
also nimm lieber einen JFrame und die Komponenten aus dem swing-package, als die Sachen zu vermischen (Frame mit JProgressBar)

 

[VGLink]

Um eine Startbare Version zu kriegen musst du nur bei start und zielg jeweils pfade zu Dateien einfügen. Dabei ist es egal was für Dateitypen es sind. Kann aber auch di Jar und den Sourcecode hochladen wenn das weiterhilft.


EDIT: Danke an alle hab es hin gekriegt ;D Das Object das übergeben wurde war leer, hab den Fehler gerade im Debuger gefunden. Danke für die Hilfe und Ideen.