PlayButton mehrmals starten?

[ItzChris]

Hi Leute, ich hab ein Problem bei meinem "MP3Player".
Ich hab den Player jetzt soweit, das ich eine Datei auswählen kann und sie mit dem PlayButton starten kann, anschließen kann ich sie mit dem StopButton wieder stoppen.

Mein Problem ist das ich den PlayButton nur einmal verwenden kann.... Also wenn ich schon eine Datei abgespielt habe muss ich den Player wieder neu starten damit ich den Button wieder verwenden kann um ein neues Lied abzuspielen.

Wäre nett wenn mir jemand helfen kann, hier ist der Quellcode:

[highlight=Java]
import java.io.*;
import javax.swing.*;
import javax.swing.JFileChooser;
import javax.swing.event.*;
import java.awt.*;
import java.util.ArrayList;
import java.awt.event.*;
import javazoom.jl.player.advanced.*;
import javazoom.jl.decoder.*;

public class ProjektNeu extends JFrame{
JButton dir = new JButton("Durchsuchen");
JButton play = new JButton("Play");
JButton stop = new JButton("Stop");
JLabel statusbar = new JLabel("Auswahl anzeige");
String dateipfad;
startThread demoThread;
boolean running = true;
AdvancedPlayer p;

ProjektNeu(){
super("Projekt");
setSize(350,200);
setDefaultCloseOperation(EXIT_ON_CLOSE);

Lauscher l = new Lauscher();

demoThread = new startThread();

Container c = getContentPane();
c.setLayout(new FlowLayout());

dir.addActionListener(l);
play.addActionListener(l);
stop.addActionListener(l);

c.add(dir);
c.add(play);
c.add(stop);
c.add(statusbar);
}

class Lauscher implements ActionListener{
public void actionPerformed(ActionEvent ae){
String aktion = ae.getActionCommand();

if(aktion.equals("Durchsuchen")){
JFileChooser chooser = new JFileChooser();
chooser.setFileSelectionMode(JFileChooser.FILES_AND_DIRECTORIES);
int option = chooser.showOpenDialog(ProjektNeu.this);

if(option == JFileChooser.APPROVE_OPTION){
statusbar.setText("Sie haben dieser Verzeichnis geöffnet: " +((chooser.getSelectedFile() !=null)?chooser.getSelectedFile().getAbsolutePath():"nothing"));
File pfad = chooser.getSelectedFile();
dateipfad = pfad.getAbsolutePath();
}
else{
statusbar.setText("Sie haben abgebrochen");
}
}

if(aktion.equals("Play")){
try{

running = true;
while(running){
demoThread.start();
}
}
catch(Exception e){

}
}

if(aktion.equals("Stop")){
running = false;
if(!running){
p.close();
}
}
}
}

class startThread extends Thread{
public void run(){
try{
p = new AdvancedPlayer(new FileInputStream(dateipfad));
p.play();
}
catch(Exception e){

}
}
}

public static void main(String[] args) throws FileNotFoundException, JavaLayerException{
ProjektNeu pn = new ProjektNeu();
pn.setVisible(true);
pn.setLocation(300,300);
}
}
[/highlight]

Danke für eure Hilfe.

Mfg Chris!

 

[max40]

Ohne es mir genau angeguckt zu haben

[HIGHLIGHT="Java"] if(aktion.equals("Play")){
try{

running = true;
while(running){ // FALSCH oder warum willst im ms bereich starten?
demoThread.start();
} // FALSCH
}
catch(Exception e){

}
}

if(aktion.equals("Stop")){
running = false;
if(!running){ // FALSCH
p.close();
} // FALSCH
}[/HIGHLIGHT]

 

[ItzChris]

Also erstmal danke für die schnelle Antwort..
Aber wenn ich das so mache wie du es gesagt hast, hab ich immer noch das Problem das ich den PlayButton nur einmal verwenden kann.

 

[max40]

Ja jetzt sehe ich deinen Fehler, einen Thread kann man nur einmal starten!
und wenn du bei
catch(Exception e){

}

ein e.printStackTrace gemacht hättest, wärst vielleicht selber drauf gekommen!
Aber Fehler kann man ja ignorieren

 

[ItzChris]

Also ich hab jetzt den Thread in der Play- Funktion erstellt, damit müsste der Fehler ja eigentlich behoben sein.
Anschließend den Thread in der Stop- Funktion auf null gesetzt.

Nur hat mir das bis jetzt auch noch nicht viel weitergeholfen.

[highlight=Java]
if(aktion.equals("Play")){
try{
demoThread = new startThread();
running = true;
demoThread.start();
}
catch(Exception e){

}
}

if(aktion.equals("Stop")){
if(running){
demoThread = null;
}
running = false;
p.close();
}
}
}

class startThread extends Thread{
public void run(){
try{

p.play();
}
catch(Exception e){

}
}
}
[/highlight]

Oder brauch ich dafür den playback listener?

 

[max40]

nochmal:
Mach bei einer Exception ein e.printStackTrace

dein p.close() schließt bestimmt auch dein FileInputStream, wenn ja, wo wird er wieder erzeugt/geöffnet?

und wenn dann immer noch nicht weiter kommst, dann zeigst nochmal deine Code!

 

[ItzChris]

Ok jetzt hab ich verstanden was du meintest. Funktioniert jetzt einwandfrei danke nochmal.