Thread geht jetzt - aber - Fehler "Null-Pointer"

[Rex]

Ehm... Liebe Freunde,

hier zunächste der Code:

class pbar implements Runnable{
  private boolean keeprunning=true;
  JProgressBar jbar;
  public void run(){
      jbar = new JProgressBar(0, 100);
    while(keeprunning){
      Thread.yield(); //100% Auslastung vermeiden
   }
  }
    public void setValue(int percentage){
        jbar.setValue(percentage);
    }
   public void stop(){
     keeprunning=false;
     jbar = null;
   }
}

Und der Part für den Aufruf:

 private static void pipe(InputStream in, OutputStream out) throws IOException {
        byte[] buf = new byte[500000];
        int nread;
        int navailable;
        int total = 0; //Menge an Bytes bisher gesendet
        long percentage; //Percent done...
        pbar pbar = new pbar();
        Thread barthread =   new Thread(pbar );
 
        synchronized (in) {
            while((nread = in.read(buf, 0, buf.length)) >= 0) {
                out.write(buf, 0, nread);
                total += nread; //Wieviel bereits gesendet?
                percentage = (filesize/total)*100;
                pbar.setValue((int) percentage);              
            }
        }
        out.flush();
        buf = null;
        pbar.stop();
    }

Aber folgender Fehler tritt auf:

java.lang.NullPointerException
at pbar.setValue(ClientHttpRequest.java:513)
at ClientHttpRequest.pipe(ClientHttpRequest.java:183)
at ClientHttpRequest.setParameter(ClientHttpRequest.java:210)
at ClientHttpRequest.setParameter(ClientHttpRequest.java:222)
at ClientHttpRequest.setParameter(ClientHttpRequest.java:233)
at ClientHttpRequest.setParameters(ClientHttpRequest.java:260)
at ClientHttpRequest.post(ClientHttpRequest.java:296)
at ClientHttpRequest.post(ClientHttpRequest.java:413)
at dfwebasyouwishuploader.processFile(DFApplet.java:54)
at dfwebasyouwishuploader$1.drop(DFApplet.java:42)
at java.awt.dnd.DropTarget.drop(DropTarget.java:430)
at sun.awt.dnd.SunDropTargetContextPeer.processDropMessage(SunDropTargetContextPeer.java:500)
at sun.awt.dnd.SunDropTargetContextPeer.access$800(SunDropTargetContextPeer.java:53)
at sun.awt.dnd.SunDropTargetContextPeer$EventDispatcher.dispatchDropEvent(SunDropTargetContextPeer.java:812)
at sun.awt.dnd.SunDropTargetContextPeer$EventDispatcher.dispatchEvent(SunDropTargetContextPeer.java:736)
at sun.awt.dnd.SunDropTargetEvent.dispatch(SunDropTargetEvent.java:29)
at java.awt.Component.dispatchEventImpl(Component.java:3826)
at java.awt.Container.dispatchEventImpl(Container.java:2024)
at java.awt.Component.dispatchEvent(Component.java:3803)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:463)
at java.awt.EventDispatchThread.pumpOneEventForHierarchy(EventDispatchThread.java:242)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:163)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:157)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:149)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:110)

Vielen Dank!
MFG
Rex

 

[EOB]

poste mal den passenden code....klassen werden mit einem grossen anfangsbuchstaben geschrieben!

gruesse

 

[Roar]

schau dir nochmal an wie man Threads benutzt, das hast du anscheinend noch nicht so wirklich verstanden :bae:

 

[hupfdule]

Und schreib dazu an welcher Zeile der Fehler aufgetreten ist.

 

[Ceos]

wenn du doch bitte in den themen antworten würdest wo das problem stand und nicht für jeden neuen Post ein neues Thema öffnest, würden dich die leute auch sicher nicht so harsch anflamen

zu dem Problem, du musst auch mal

barthread.start();

aufrufen, damit die Run()-methode des Thread "barthread" auch ausgeführt und damit die "pbar" initialisiert wird

vesuch es mal mit

barthread.start();

in Zeile 9 vom 2ten CodeBlock eingefügt

 

[SlaterB]

die JProgressBar muss nicht in einem Thread, die ist ganz normal in einem JFrame oder Applet eingebungen,
genau wie eine JTextField oder irgendwas anderes,

die aufwendige Kopier-Arbeit, die mehrere Sekunden dauert, die gehört in einem Thread,
die muss unabhängig von der GUI sein, also insbesondere nicht in irgendeinem ActionListener (ODER DROPLISTENER!!),
sondern eben z.B. aus einem Listener in einem neuen Thread starten,

denn stände der Code in einem Listener, würde sich die GUI nicht aktualisieren, bis der Durchlauf beendet worden wäre,

wenn aber der Arbeits-Code in einem eigenen Thread läuft,
dann wird er ab und zu auch mal pausiert,
und die GUI bekommt Luft, sich neu zu zeichnen

evtl. muss dafür noch repaint() oder so gerufen werden,
vielleicht reicht aber auch die Änderung des ProgressBar-Wertes

------

also:
in Pipe einen neuen Thread mit dem Code dort starten,
vom Thread aus die ganz normale JProgressbar aktualisieren,
die natürlich irgendwo in einem sichtbaren JPanel eingefügt sein muss, nicht in einem Thread!

 

[Ceos]

gut darauf hab ich jetzt nicht geachtet, aber fakt ist das er vergessen hat den thread zu starten ;P

 

[Leroy42]

gut darauf hab ich jetzt nicht geachtet...

Und auf die run-Methode des Threads habt ihr auch nicht geachtet? :shock:

public void run() { 
  jbar = new JProgressBar(0, 100); 
  while(keeprunning) { 
    Thread.yield(); //100% Auslastung vermeiden 
  } 
}

Dabei ist dieser Code doch mit das Verwegenste und zugleich Schönste was mir bisher untergekommen ist.
:meld:

Vor allem die Eleganz mit der Rex (der Autor dieses Kleinods)
den doch enormen Berechnungsaufwand der in der Schleife erfolgt, durch ein so,
auf den ersten Blick banal wirkendes, jedoch zugleich eloquentes, Thread.yield() entlastet,
wird erst durch ein nochmaliges Hinschauen dem Betrachter offenbar.

Ich wage es zu behaupten, das wir alle einer Meinung sind, daß dieses Kunststück
Eingang in die Hall of Fame finden und noch Generationen
nach uns zur Inspiration dienen wird!

:applaus:

 

[Ceos]

das Thread.yield() war meine kreation *schäm* aber das er es SOOO verwendet war mir nicht klar q_q ich dachte er wollte diverse sachen vollbringen und nur kurz warten bis der andere prozess beendet q_q .... so iss das REX wenn man 20 topics zu einem thema öffnet, man weis gar net woran man iss .... da hier irgendwas mit dateitransfer war nochmal korrektur von mir:

der dateitransfer bitte in einen eigenen thread und den ladebalken als JDialog, der bleibt aktiv solange er nicht geschlossen wird.

zu dem while(warteBedinnung) Thread.yield();
verrat mir doch mal bitte wer wie man ein Programm an einer bestimmten stelle "anhalten" kann bis irgendeine Bedingung erfüllt ist, mich würde eine elegantere Lösung als dieses Thread.yield(); interessieren
danke im voraus

 

[Bert Brenner]

Mit Object#wait und Object#notify

 

[Rex]

wie wende ich dass an?

 

[Leroy42]

Diese Schleife

while((nread = in.read(buf, 0, buf.length)) >= 0) { 
                out.write(buf, 0, nread); 
                total += nread; //Wieviel bereits gesendet? 
                percentage = (filesize/total)*100; 
                pbar.setValue((int) percentage);              
            }

gehört in die run-Methode eines eigenen Threads, etwa so:


Statt direkt pipe aufzurufen, folgendes Konstrukt:

new Thread(new Runnable() {
  public void run() {
    pipe(deinInStream, deinOutStream);
  }}).start();

und die pipe-Methode leicht erweitern

 private static void pipe(InputStream in, OutputStream out) throws IOException { 
        byte[] buf = new byte[500000]; 
        int nread; 
        int navailable; 
        int total = 0; //Menge an Bytes bisher gesendet 
        long percentage; //Percent done... 
        pbar pbar = new pbar(); 
        Thread barthread =   new Thread(pbar ); 

        synchronized (in) { 
            while((nread = in.read(buf, 0, buf.length)) >= 0) { 
                out.write(buf, 0, nread); 
                total += nread; //Wieviel bereits gesendet? 
                percentage = (filesize/total)*100; 
                pbar.setValue((int) percentage); 

                // Jetzt dem EDT die Zeit geben, die Progressbar neu zu zeichnen
                try {Thread.sleep(20);} catch(InterruptedException e) {}             

            } 
        } 
        out.flush(); 
        buf = null; 
        pbar.stop(); 
    }

 

[Ceos]

kewl danke ^^ das mit sleep(int) geht aber nur wenn einem das sleep zur verfügung steht, das war bei einigen meiner probleme aber nicht so ...

 

[Rex]

 public void setParameter(String name, String filename, InputStream is) throws IOException {
        boundary();
        writeName(name);
        write("; filename=\"");
        write(filename);
        write('"');
        newline();
        write("Content-Type: ");
        String type = connection.guessContentTypeFromName(filename);
        if (type == null) type = "application/octet-stream";
        writeln(type);
        newline();
        new Thread(new Runnable() {
  public void run() {
    pipe(is, os);
  }}).start(); 

        newline();
    }

local Variable is is accessed from within inner class; needs to be declared final


OMG!

MFG
Rex

 

[Leroy42]

Das ist richtig: Alle Variablen die in einer inneren anonymen Klasse benutzt werden
und nicht lokal zu inneren Methode sind (bei dir die run-Methode) dürfen nur
Methoden- und Parameter-Variablen die als final deklariert sind:

Bei dir also is und os.

final InputStream is = ...

Wobei ich mich jetzt aber Frage, wo du is und os überhaupt deklarierst? :shock:

 

[Leroy42]

OMG!

Wofür steht denn diese Abkürzung?
Ohne Mit Grüssen? :shock:

 

[Ceos]

OMG
OH MEIN GOOOTT

 

[Rex]

back to topic:

    private static void pipe(InputStream in, OutputStream out) throws IOException {
        byte[] buf = new byte[500000];
        int nread;
        int navailable;
        int total = 0; //Menge an Bytes bisher gesendet
        long percentage; //Percent done...

        synchronized (in) {
            while((nread = in.read(buf, 0, buf.length)) >= 0) {
                out.write(buf, 0, nread);
                total += nread; //Wieviel bereits gesendet?
                percentage = (filesize/total)*100;          
            }
        }
        out.flush();
        buf = null;
    }

Hier ist die alte Auführung, weil ich dazu noch eine Frage habe:
Jemand hat gemeint, dass 100 % schlagartig erreicht werden würde, - und zwar am Schluss - wenn alles Fertig ist.
Die Zwischenstufen würden aber einfach fehlen!
FRAGE: Stimmt dass?

MFG
Rex

 

[SlaterB]

testen??
das geht doch selbst mit simplen
System.out.prinltn("nun "+total+" bytes gelesen");
in der Schleife


sieht für mich so aus als würden in einer Schleife immer 500.000 gelesen,
bei kleineren Dateien wäre das natürlich in einem Schlag erreicht,
bei größeren hat man mehrere Durchläufe

es muss natürlich total/filesize heißen,
bei int kann es sein dass da auf 0 oder 1 gerundet wird,
insofern vielleicht wirklich so, dass bis kurz vor Schluss (0.99) immer bei 0 bleibt und erst mit dem letzten byte auf 1 = 100% umspringt

wieso ist eigentlich percentage ein long?


das sind aber nur Spielereien bei der Rechnung, denen man wiederum mit einfach System.out.prinltn auf die Schliche kommt,
erst mit 100 malnehmen und dann teilen,

50/150 = 0 -> 50/150 * 100 = 0%

50 * 100 = 5000 -> (50 * 100)/150 = 33%

 

[Rex]

Das heißt, dass was mit der Prozentrechnung net stimmt?


MFG
Rex

 

[Rex]

CLOSED

Thread recylcer.