Hi.
Habe ein ich glaub kleines Problem aber ich kann dieses Problem nicht beheben.
Ich hab nen Mp3Player mit ner Benutzeroberfläche gebastelt (javazoom layer oder so ähnlich). Nur leider klappt der Thread nicht ganz..
Kann mir mal jemand bitte helfen?

Also mein code sieht nun wie folgt aus:

Code:

public class PlayerThread implements Runnable { 
 
   private Statusbar bar; 
 
   /* 
    * (non-Javadoc) 
    * 
    * @see java.lang.Runnable#run() 
    */ 
   public void run() { 
      Mp3Player player = new Mp3Player(); 
         try { 
            player.play(); 
         } catch (FileNotFoundException e1) { 
         } catch (JavaLayerException e) {   
      } 
   } 
}
 

hier ein Ausschnitt der aktuellen GUI (mit main methode

Code:

if (e.getSource().equals(playBut)) { 
         tp = new PlayerThread(); 
         player = new Mp3Player(); 
         t = new Thread(tp); 
         try { 
            t.start(); 
            player.play(); 
            bar.setText(1,
                  + ((MP3File) list.get(jl.getSelectedIndex())) 
                        .getFilename()); 
 
         } catch (FileNotFoundException e1) { 
         } catch (JavaLayerException e1) {         
         } 
      }
 

Es gibt keinen Fehler, keine Exceptions, gar nix.... der Thread läßt einfach nur den Mp3-Player nicht nebenbei laufen... ich kann halt so bald ich auf play gedrückt habe nichts anderes mehr mit der GUI machen!

Generell funktioniert Dein Aufruf für Threads schon, wie folgendes Beispiel (ohne MP3-Player) zeigt:

class PlayerThread implements Runnable {
    public void run() {
        try {
            for (int i = 0; i < 10; i++) {
                System.out.println("Thread running...");
                Thread.currentThread().sleep(100);
            }
        } catch (final Exception e) {
            e.printStackTrace();
        }
    }
}
 
public class ThreadRunner {
 
    public static void main(String[] args) {
        PlayerThread tp = new PlayerThread();
        Thread t = new Thread(tp);
        try {
            t.start();
        } catch (RuntimeException e) {
            e.printStackTrace();
        }
    }
}
 

Ich sehe jedoch auf Anhieb ein Problem: Du hast zwei Mp3 Player gestartet: Einen im Hauptthread und einem im angelegten Thread. Du solltest den MP3 Player im Hauptthread auf jeden Fall entfernen.

P.S. Dieser Beitrag hat hier eigentlich nix verloren, sondern gehört z.B. in das "Anfänger Forum".

Dnake du bist der beste das hat geholfen. ich hab einfach nicht gemerkt, dass ich 2 instanzen vom mp3-player gemacht habe.

wirklich dummer fehler..

danke noch mal

Aber da tut sich noch ein Problem auf....
Wenn ich auf den play button drücke und danach auf den stop button drücke und dann noch mal auf den play button drücke gibt das programm
folgende Exception aus und macht nix:

Exception in thread "AWT-EventQueue-0" java.lang.IllegalThreadStateException
at java.lang.Thread.start(Unknown Source)
at View.actionPerformed(View.java:211)
at javax.swing.AbstractButton.fireActionPerformed(Unk nown Source)
at javax.swing.AbstractButton$Handler.actionPerformed (Unknown Source)
at javax.swing.DefaultButtonModel.fireActionPerformed (Unknown Source)
at javax.swing.DefaultButtonModel.setPressed(Unknown Source)
at javax.swing.plaf.basic.BasicButtonListener.mouseRe leased(Unknown Source)
at java.awt.Component.processMouseEvent(Unknown Source)
at javax.swing.JComponent.processMouseEvent(Unknown Source)
at java.awt.Component.processEvent(Unknown Source)
at java.awt.Container.processEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.LightweightDispatcher.retargetMouseEvent( Unknown Source)
at java.awt.LightweightDispatcher.processMouseEvent(U nknown Source)
at java.awt.LightweightDispatcher.dispatchEvent(Unkno wn Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Window.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForHierar chy(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarch y(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)

Jetzt bin ich extrem aufs Raten angewiesen.

Mein erster Tipp wäre, dass der Start-Button dazu führt, dass ein neuer Mp3Player installiert wird und zwar als eigener Thread während der Stop-Button den Player nicht wirklich löscht. Allerdings ist das mal ganz ins Blaue hinein geraten.

Alternativ könnte es sein, dass Du die Eingaben für die Oberfläche über zwei verschiedene Threads eingibst und das funktioniert praktisch nie. Man sollte die Oberfläche nur über einen Thread bedienen und die von anderen Threads generierten Eingaben auf diesen ersten Thread umlenken.

Ansonsten fehlen mir die Sourcen, um den Fehler besser einkreisen zu können.

Viel Erfolg beim Suchen!

Der zeigt mir immer den Fehler beim starten des Threads. also bei t.start()...
ich weiss aber immer noch was ich da falsch mach...
muss ein thread auch beendet werden oder kann der einfach so lange laufen bis das programm beendet wird?

Wie beende ich den thread...indem ich den zerstöre mit destroy() oder soll ich den nur unterbrechen mit interrupt() ??

So langsam weiss ich nicht mehr weiter

soll ich dir mal den quellcode bereitstellen ?

MfG
Eddi

Ein Thread wird automatisch beendet, wenn die Methode run() beendet wird, also liegt das Problem vermutlich nicht da dran.

Schick mir mal den kompletten Quellcode per Mail und zwar inkl. dem Mp3Player bzw. schick den Link, wo der Player herkommt. So verstopfen wir hier nicht das Forum Wenn ich den Fehler finde, gebe ich Dir im Forum die Antwort. Dann haben auch andere was davon.

... Fehler gefunden. Das Problem ist, das ein Thread nur einmal benutzt werden kann und dann neu angelegt werden muß. Im eigentlichen Code war das nicht passiert, sondern der Thread wurde nur einmal angelegt. Der Fehler tritt dann beim zweiten Start auf und löst die Exception aus.

Seit Java 5 gibt es die Klasse "java.util.concurrent.ThreadPoolExecutor", die mehrere Threads halten und "füttern" kann. Das spart Ressourcen, weil die Threads offen bleiben. In dem Fall werden aber nicht so viele Threads benutzt, so dass das nicht notwendig ist.