java applet mit swing -> hundert prozent cpu auslastung

[Guest]

hallo!
mein problem besteht darin, dass mein applet zwar einwandfrei (aber langsam) läuft, die cpu auslastung aber dauerhaft auf hundert prozent ansteigt. ich starte 4 threads, die aber alle nach priorität gestaffelt sind und im run mindestens 200 ms schlafen bei jeder ausführung der while(true) schleife. woran kann das liegen? :roll:

 

[Marco13]

Ziemlich wenig Input, um die Frage "mit Sicherheit richtig" zu beantworten, aber.... solange IMMER mindestens EIN Thread was zu tun hat, ist die Auslastung 100% - und neben den 4 Threads, die du erstellst, gibt es noch ETLICHE andere, die Standardmäßig laufen - der prominenteste ist der Event-Dispatch-Thread, der u.U. schon eine ganze Menge zu tun haben kann (aber so wenig wie möglich zu tun haben sollte)

 

[anti43]

hm ja das problem ist, das ich gleichzeitig ständig ein bild aktualisiere und auf der anderen seite einen datainputstream überwache und dann auch noch was sende auf knopfdruck. leider hab ich die verwendung von thread.interrupt bzw interrupted nicht wirklich verstanden.. könntest du mir das ein wenig näher bringen?
wenn ein thread mit interrupt angehalten ist, stoppt er dann die ausführung seines run() ? und führt ihn dann nach aufruf mit thread.interrupted wieder fort oder so?

 

[Marco13]

Was nun "interrupt" damit zu tunhat, ist mir nicht ganz klar. (Ehrlich gesagt: Ich habe "interrupt" noch nie verwendet - wofür brauchst du das denn (oder wofür glaubst du, es zu brauchen :wink: :?: ). Jedenfalls bewirkt es afaik NICHT, dass der Thread low-level-mäßig gekillt wird, sondern nur, dass eventuelles Warten beendet wird.

Um einen Thread anzuhalten und weiterlaufen zu lassen, solltest du dir mal die Anmerkungen in der JavaDoku durchlesen
http://java.sun.com/j2se/1.4.2/docs/guide/misc/threadPrimitiveDeprecation.html
allerdings weiß ich - wie gesagt - nicht genau genug, was du vorhast...

 

[Gast]

ach ich dachte das wäre dafür :-D

also danke für den link, ich machs nun mit

public synchronized void mousePressed(MouseEvent e) {
e.consume();

threadSuspended = !threadSuspended;

if (!threadSuspended)
notify();
}

and adding the following code to the "run loop":
synchronized(this) {
while (threadSuspended)
wait();
}

und halte damit mal einen thread an, wenn nichts passiert... vielen dank!

 

[anti43-]

hm leider kommt

Exception in thread "AWT-EventQueue-1" java.lang.IllegalMonitorStateException
at java.lang.Object.notify(Native Method)
javascript:go()
Quick Reply absenden
beim aufruf der notify() function. leider verstehe ich die sache mit diesen monitoren nicht ganz. wie werde ich denn besitzer wenn nicht (wie laut javadoc) mit dem aufruf der

synchronized(this) {
while (threadSuspended)
wait();
}

-function??

 

[anti43]

das javascript blah ist natürlich nicht in der fehlermeldung sondern hier von der seite ;-)

 

[Marco13]

Das notify muss in einem Block aufgerufen werden, der auf dasselbe Objekt synchronisiert ist, wie das "wait", auf das man sich bezieht.

Wenn du glaubst, das mit
public synchronized void mousePressed(MouseEvent e) {
zu erreichen, musst du aufpassen: Das KÖNNTE sich erstmal nur auf einen (anonymen!?) MouseListener beziehen. (Und das notify könnte auch (fälschlicherweise) auf diesem MouseListener aufgerufen werden.

Poste ggf. mal die relevanten code-Stellen der Klasse, in der die wait/notify Sachen liegen.

 

[anti43]

hm ich poste mal, denn ich komm einfach nicht weiter:

   ActionListener al = new ActionListener() {
                public void actionPerformed( ActionEvent e ) {
                    str = e.getActionCommand();
                    
                    synchronized(functionX){
                        functionX.threadstopped=false;
                        functionX.thread.notify();
                       
                        functionX.steAction(str);
                    }
                }
            };

thread ist der einzige Thread in functionX.
und das steht im run() von functionX^:

    public void run() {
        
        while(true) {
            
            synchronized(this) {
                while (threadstopped)
                    
                    try {
                        wait();
                    } catch (InterruptedException ex) {
                        ex.printStackTrace();
                    }
            }

 

[Marco13]

Code 1, Zeile 7: functionX.thread.notify(); // synchronized auf functionX
Code 2, Zeile 9: wait(); // synchronized auf functionX

Das wait() wird auf dem functionX-Objekt aufgerufen. Das notify() wird auf functionX.thread aufgerufen. Auf Basis des (etwas unzureichenden) Codes könnte es funktionieren, wenn du nicht
functionX.thread.notify();
sondern
functionX.notify();
aufrufst.

 

[anti43]

danke, das wars!! vielen dank!

 

[anti43]

erledigt