JPanel, JButtons, Borderlayout

[MarsUM]

Hallo,
habe an meinem Audioplayer noch ein kleines Problem mit dem Adden der Buttons:


Mein Code im Konstruktor:

    public Player(PlayList pl){
    	
    	setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    	try {
    		UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
    	}
    	catch (Exception e) {
    		e.printStackTrace();
    	}
    	
    	
    JButton play = new JButton(new ImageIcon("play.png"));
    JButton pause = new JButton(new ImageIcon("pause.png"));
    JButton next = new JButton(new ImageIcon("next.png"));
    JButton stop = new JButton(new ImageIcon("stop.png"));
    JButton pl_editor = new JButton(new ImageIcon("pl_editor.png"));
    
    JPanel oben = new JPanel(new BorderLayout());
    JPanel unten = new JPanel(new BorderLayout());
    

    unten.add(play, BorderLayout.SOUTH);
    unten.add(pause, BorderLayout.SOUTH);
    unten.add(stop, BorderLayout.SOUTH);
    unten.add(next, BorderLayout.SOUTH);
    unten.add(pl_editor, BorderLayout.SOUTH);

    songDescription = new JLabel();
    playTime = new JLabel();
    
    oben.add(songDescription, BorderLayout.NORTH);  
    oben.add(playTime, BorderLayout.NORTH);  
    
    editorVisible = false;
    stopped = true;
    playList = pl;
    playListEditor = new PlayListEditor(this, playList);
	
   
    // (k)
    play.addActionListener(this);
    pause.addActionListener(this);
    stop.addActionListener(this);
    next.addActionListener(this);
    pl_editor.addActionListener(this);
    
    play.setActionCommand("AC_PLAY");
    stop.setActionCommand("AC_STOP");
    pause.setActionCommand("AC_PAUSE");
    next.setActionCommand("AC_NEXT");
    pl_editor.setActionCommand("AC_EDITOR");
	
   
    next.setEnabled(true);
    pl_editor.setEnabled(true);
    play.setEnabled(true);
    pause.setEnabled(false);
    stop.setEnabled(false);

Die Buttons erscheinen allerdings nicht, nur ein kleines leeres fenster öffnet sich beim starten.

Die 5 Buttons möchte ich mit der add-methode einem JPanel hinzufügen.
Anschließend für die zwei JLabel-Attribute Instanzen erzeugen und anschliessend das JPanel (mit den Buttons) und die JLabels wieder mit der add-methode einem JFrame hinzufügen. Das Ganze mit Borderlayout.

Wo habe ich was falsch gemacht?

 

[SlaterB]

die Reihenfolge der Fehler
1.
kein vollständiges Programm gepostet
2.
wenn überhaupt immer nur erstmal ein Button testen,
generell Programme für 1. so klein wie möglich, am besten < 30 Zeilen, unbekannte Klasse wie PlayListEditor raus
3.
keine Icons, die vielleicht nicht gefunden werden, ein Text-Button reicht doch erstmal
(wenn der geht, aber dann mit Icon nicht mehr, dann kann man das natürlich auch fragen, aber dann MIT dieser Information)

jetzt wirds interessanter:
4.
BorderLayout erlaubt nur eine Komponente in Gebieten wie SOUTH, wenn dort mehrere rein sollen
-> neues JPanel
5.
setVisible(true); ist nicht zu sehen, wenn das vor den anderen Befehlen steht, dann wird eh alles schiefgehen (Folge aus Punkt 1.)
(edit: ok, ist ein JFrame-Konstruktor, dann kann es vorher kaum stehen)

 

[MarsUM]

die Reihenfolge der Fehler
1.
kein vollständiges Programm gepostet
2.
wenn überhaupt immer nur erstmal ein Button testen,
generell Programme für 1. so klein wie möglich, am besten < 30 Zeilen, unbekannte Klasse wie PlayListEditor raus
3.
keine Icons, die vielleicht nicht gefunden werden, ein Text-Button reicht doch erstmal
(wenn der geht, aber dann mit Icon nicht mehr, dann kann man das natürlich auch fragen, aber dann MIT dieser Information)

jetzt wirds interessanter:
4.
BorderLayout erlaubt nur eine Komponente in Gebieten wie SOUTH, wenn dort mehrere rein sollen
-> neues JPanel
5.
setVisible(true); ist nicht zu sehen, wenn das vor den anderen Befehlen steht, dann wird eh alles schiefgehen (Folge aus Punkt 1.)
(edit: ok, ist ein JFrame-Konstruktor, dann kann es vorher kaum stehen)

Dann hier mal die gesamte Player.java:

package studiplayer.ui;

import javax.swing.*;
import java.io.File;
import java.awt.BorderLayout;
import java.awt.Container;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;

import studiplayer.audio.*;


public class Player extends JFrame implements ActionListener{
	
	//Konstruktor
	
	
    public Player(PlayList pl){
    	
    	setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    	try {
    		UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
    	}
    	catch (Exception e) {
    		e.printStackTrace();
    	}
    	
    	
    JButton play = new JButton(new ImageIcon("play.png"));
    JButton pause = new JButton(new ImageIcon("pause.png"));
    JButton next = new JButton(new ImageIcon("next.png"));
    JButton stop = new JButton(new ImageIcon("stop.png"));
    JButton pl_editor = new JButton(new ImageIcon("pl_editor.png"));
    
    JPanel oben = new JPanel(new BorderLayout());
    JPanel oben2 = new JPanel(new BorderLayout());
    JPanel unten = new JPanel(new BorderLayout());
    JPanel unten2 = new JPanel(new BorderLayout());
    JPanel unten3 = new JPanel(new BorderLayout());
    JPanel unten4 = new JPanel(new BorderLayout());
    JPanel unten5 = new JPanel(new BorderLayout());
    

    unten.add(play, BorderLayout.SOUTH);
    unten2.add(pause, BorderLayout.SOUTH);
    unten3.add(stop, BorderLayout.SOUTH);
    unten4.add(next, BorderLayout.SOUTH);
    unten5.add(pl_editor, BorderLayout.SOUTH);

    songDescription = new JLabel();
    playTime = new JLabel();
    
    oben.add(songDescription, BorderLayout.NORTH);  
    oben2.add(playTime, BorderLayout.NORTH);  
    
    editorVisible = false;
    stopped = true;
    playList = pl;
    playListEditor = new PlayListEditor(this, playList);
	
   
    // (k)
    play.addActionListener(this);
    pause.addActionListener(this);
    stop.addActionListener(this);
    next.addActionListener(this);
    pl_editor.addActionListener(this);
    
    play.setActionCommand("AC_PLAY");
    stop.setActionCommand("AC_STOP");
    pause.setActionCommand("AC_PAUSE");
    next.setActionCommand("AC_NEXT");
    pl_editor.setActionCommand("AC_EDITOR");
	
   
    next.setEnabled(true);
    pl_editor.setEnabled(true);
    play.setEnabled(true);
    pause.setEnabled(false);
    stop.setEnabled(false);
    
    // (f) letzter pkt

    if(playList.isEmpty())
    {
        songDescription.setText("");
        setTitle("");
    } else
    {
        songDescription.setText(playList.getCurrentAudioFile().toString());
        setTitle(playList.getCurrentAudioFile().toString());
    }										
	
    
    
	this.pack();
	this.setVisible(true);
	
}

	
// (d)

    public static void main(String args[])
    {
        PlayList list;
        if(args.length > 0)
            list = new PlayList(args[0]);
        if((new File(DEFAULT_PLAYLIST)).exists())
            list = new PlayList(DEFAULT_PLAYLIST);
        else
            list = new PlayList();
        Player test = new Player(list);
        test.setVisible(true);
    }
	
	
// (g)
    
    public void stopCurrentSong()
    {
        setStopped(true);
        try {
			playList.getCurrentAudioFile().stop();
		} catch (NotPlayableException e) {
			e.printStackTrace();
		}
        playTime.setText("00:00");
    }
	
    
// (h)
    
    private class PlayerThread extends Thread
    {

        public void run()
        {
            while(!stopped) 
            {
                try
                {
                    playList.getCurrentAudioFile().play();
                }
                catch(NotPlayableException e)
                {
                    e.printStackTrace();
                }
                songDescription.setText(playList.getCurrentAudioFile().toString());
                setTitle(playList.getCurrentAudioFile().toString());
            }
        }

        final Player this$0;

        private PlayerThread()
        {
            this$0 = Player.this;
        }

        PlayerThread(PlayerThread playerthread)
        {
            this();
        }
    }

    private class TimerThread extends Thread
    {

        public void run()
        {
            while(!stopped) 
            {
                try
                {
                    playTime.setText(playList.getCurrentAudioFile().getFormattedPosition());
                }
                catch(NotPlayableException e1)
                {
                    e1.printStackTrace();
                }
                try
                {
                    Thread.sleep(1000L);
                }
                catch(InterruptedException e)
                {
                    e.printStackTrace();
                }
            }
        }

        final Player this$0;

        private TimerThread()
        {
            this$0 = Player.this;
        }

        TimerThread(TimerThread timerthread)
        {
            this();
        }
    }
    
    
// (j)
    
    public void playCurrentSong()
    {
        setStopped(false);
        (new PlayerThread(null)).start();
        (new TimerThread(null)).start();
    }


// (L)    
    
public void actionPerformed(ActionEvent e) {
    
	String command = e.getActionCommand();
    if(command.equals("AC_PLAY"))
    {
        playCurrentSong();
        play.setEnabled(false);
        stop.setEnabled(true);
        pause.setEnabled(true);
    } 
   
    else if(command.equals("AC_PAUSE"))
    {
        try {
			playList.getCurrentAudioFile().togglePause();
		} catch (NotPlayableException e1) {
			e1.printStackTrace();
		}
        play.setEnabled(false);
        pause.setEnabled(true);
        stop.setEnabled(true);
    }
    
    else if(command.equals("AC_STOP"))
    {
        stopCurrentSong();
        play.setEnabled(true);
        pause.setEnabled(false);
        stop.setEnabled(false);
    }
   
    else if(command.equals("AC_NEXT"))
    {
        playList.changeCurrent();
        play.setEnabled(false);
        pause.setEnabled(true);
        stop.setEnabled(true);
    	
    	{if(!stopped)
    	{
            stopCurrentSong();
            playCurrentSong();
    	}
    	else
    {
            songDescription.setText(playList.getCurrentAudioFile().toString());
            setTitle(playList.getCurrentAudioFile().toString());
    }	}
    }
    
    else if(command.equals("AC_EDITOR"))
        if(!editorVisible)
        {
            playListEditor.setVisible(true);
            setVisible(true);
        } else
        {
            playListEditor.setVisible(false);
            setVisible(false);
        }
	
}    
    

public void setStopped(boolean stopped)
{
    this.stopped = stopped;
}

public void setEditorStatus(boolean editorVisible)
{
    this.editorVisible = editorVisible;
}

JButton play;
JButton pause;
JButton next;
JButton stop;
JButton pl_editor;
    
private JLabel songDescription;
private JLabel playTime;
private boolean stopped;
private boolean editorVisible;
private PlayList playList;
private PlayListEditor playListEditor;
public static String DEFAULT_PLAYLIST = "playList.m3u";




}

Habe jetz auch mal mehrere JPanels geschaffen. (Falls du das so meintest)

setVisible is wohl auch nicht vor den anderen Befehlen.

 

[SlaterB]

hmm, gut, das entscheidene habe ich wohl vergessen

die ganzen Panel oben, unten usw. müssen auch in die GUI eingefügt werden

this.add(oben, BorderLayou.NORTH);
usw.


viele EinzelPanel zu erzeugen bringt wenig, du musst doch ein Konzept haben bevor du da was machst,
wie soll es überhaupt aussehen, male ein Bild auf Papier,
für den Anfang lieber nur ein JPanel ohne ein Layout zu setzen (also Standard-FlowLayout)

 

[MarsUM]

Ein grossartiges Layout brauche ich gar nicht.

Habe mir das schon vorher überlegt, es gibt einfach zwei zeilen:

Die obere beinhaltet Titelinfos, rechts daneben Gesamt-Laufzeit d. Titels.

Untere Zeile (von links nach rechts):
Aktuelle Abspielposition, Start, Pause, Stop, Next, Playlisteditor.


So, hab das Panel nun auch in die GUI eingefügt und die die Buttons mit einfachen Text belegt. das unten2, unten3, etc wieder raus

FYI:

    JPanel oben = new JPanel(new BorderLayout());
    this.add(oben, BorderLayout.NORTH);
    JPanel unten = new JPanel(new BorderLayout());
    this.add(unten, BorderLayout.SOUTH);

    unten.add(play, BorderLayout.SOUTH);
    unten.add(pause, BorderLayout.SOUTH);
    unten.add(stop, BorderLayout.SOUTH);
    unten.add(next, BorderLayout.SOUTH);
    unten.add(pl_editor, BorderLayout.SOUTH);

    songDescription = new JLabel();
    playTime = new JLabel();
    
    oben.add(songDescription, BorderLayout.NORTH);  
    oben.add(playTime, BorderLayout.NORTH);

Wenn ichs ablaufen lasse, kommt allerdings nur der Button mit PlaylistEditor. Ich schätze dass es eben dann daran liegt, dass nur eine Komponente in South erlaubt ist und er dann eben die letzte nimmt...

Wie kriege ich die dann alle in die zweite untere Zeile?

 

[SlaterB]

aus
unten = new JPanel(new BorderLayout());
mache
unten = new JPanel();

und aus
unten.add(play, BorderLayout.SOUTH);
wird
unten.add(play);
usw.

 

[MarsUM]

So der Player startet nun schon entsprechend, sogar mit den passenden image-dateien als Buttons. Musik lässt sich auch schon drüber abspielen.

Allerdings spinnt der Actionlistener/Actionperformer/ActionCommander (die Funktion halt)... Die Buttons sollten laut code deaktiviert bzw aktiviert werden. je nach Zustand. Bin den Code einige mal durchgegegangen und find den fehler mal wieder nicht.

Der Code:

package studiplayer.ui;

import javax.swing.*;
import java.io.File;
import java.awt.BorderLayout;
import java.awt.Container;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;

import studiplayer.audio.*;


public class Player extends JFrame implements ActionListener{
	
    private class PlayerThread extends Thread
    {

        public void run()
        {
            while(!stopped) 
            {
                try
                {
                    playList.getCurrentAudioFile().play();
                }
                catch(NotPlayableException e)
                {
                    e.printStackTrace();
                }
                songDescription.setText(playList.getCurrentAudioFile().toString());
                setTitle(playList.getCurrentAudioFile().toString());
            }
        }

        final Player this$0;

        private PlayerThread()
        {
            this$0 = Player.this;
        }

        PlayerThread(PlayerThread playerthread)
        {
            this();
        }
    }

    private class TimerThread extends Thread
    {

        public void run()
        {
            while(!stopped) 
            {
                try
                {
                    playTime.setText(playList.getCurrentAudioFile().getFormattedPosition());
                }
                catch(NotPlayableException e1)
                {
                    e1.printStackTrace();
                }
                try
                {
                    Thread.sleep(1000L);
                }
                catch(InterruptedException e)
                {
                    e.printStackTrace();
                }
            }
        }

        final Player this$0;

        private TimerThread()
        {
            this$0 = Player.this;
        }

        TimerThread(TimerThread timerthread)
        {
            this();
        }
    }
    
    	
	
    public void setStopped(boolean stopped)
    {
        this.stopped = stopped;
    }

    public void setEditorStatus(boolean editorVisible)
    {
        this.editorVisible = editorVisible;
    }

	
	//Konstruktor

    public Player(PlayList pl){
    	
    	setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    	try {
    		UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
    	}
    	catch (Exception e) {
    		e.printStackTrace();
    	}
    	
    	
    JButton play = new JButton(new ImageIcon("...play.png"));
    JButton pause = new JButton(new ImageIcon("...\pause.png"));
    JButton next = new JButton(new ImageIcon("...next.png"));
    JButton stop = new JButton(new ImageIcon("...stop.png"));
    JButton pl_editor = new JButton(new ImageIcon("...pl_editor.png"));
    
    JPanel oben = new JPanel();
    this.add(oben, BorderLayout.NORTH);
    JPanel unten = new JPanel();
    this.add(unten, BorderLayout.SOUTH);

    songDescription = new JLabel();
    playTime = new JLabel();
    
    unten.add(playTime);
    
    unten.add(play);
    play.setActionCommand("AC_PLAY");
    play.addActionListener(this);
    
    unten.add(pause);
    pause.setActionCommand("AC_PAUSE");
    pause.addActionListener(this);
    
    unten.add(stop);
    stop.setActionCommand("AC_STOP");
    stop.addActionListener(this);
    
    unten.add(next);
    next.setActionCommand("AC_NEXT");
    next.addActionListener(this);
    
    unten.add(pl_editor);
    pl_editor.setActionCommand("AC_EDITOR");
    pl_editor.addActionListener(this);
    
    oben.add(songDescription);  
  
    
    editorVisible = false;
    stopped = true;
    playList = pl;
    playListEditor = new PlayListEditor(this, playList);
	
  
    
    // (f) letzter pkt

    if(playList.isEmpty())
    {
        songDescription.setText("");
        setTitle("");
    } else
    {
        songDescription.setText(playList.getCurrentAudioFile().toString());
        setTitle(playList.getCurrentAudioFile().toString());
    }										
	
    
    
    // (k)
    
    next.setEnabled(true);
    pl_editor.setEnabled(true);
    play.setEnabled(true);
    pause.setEnabled(false);
    stop.setEnabled(false);
    
	this.pack();
	this.setVisible(true);
	
}

// (L)    
    
public void actionPerformed(ActionEvent e) {
    
	String command = e.getActionCommand();
    if(command.equals("AC_PLAY"))
    {
        playCurrentSong();
        play.setEnabled(false);
        stop.setEnabled(true);
        pause.setEnabled(true);
    } 
   
    else if(command.equals("AC_PAUSE"))
    {
        try {
			playList.getCurrentAudioFile().togglePause();
		} catch (NotPlayableException e1) {
			e1.printStackTrace();
		}
        play.setEnabled(false);
        pause.setEnabled(true);
        stop.setEnabled(true);
    }
    
    else if(command.equals("AC_STOP"))
    {
        stopCurrentSong();
        play.setEnabled(true);
        pause.setEnabled(false);
        stop.setEnabled(false);
    }
   
    else if(command.equals("AC_NEXT"))
    {
        playList.changeCurrent();
        play.setEnabled(false);
        pause.setEnabled(true);
        stop.setEnabled(true);
    	
    	{if(!stopped)
    	{
            stopCurrentSong();
            playCurrentSong();
    	}
    	else
    {
            songDescription.setText(playList.getCurrentAudioFile().toString());
            setTitle(playList.getCurrentAudioFile().toString());
    }	}
    }
    
    else if(command.equals("AC_EDITOR"))
        if(!editorVisible)
        {
            playListEditor.setVisible(true);
            setVisible(true);
        } else
        {
            playListEditor.setVisible(false);
            setVisible(false);
        }
	
}    
    
//(g)

public void stopCurrentSong()
{
    setStopped(true);
    try {
		playList.getCurrentAudioFile().stop();
	} catch (NotPlayableException e) {
		e.printStackTrace();
	}
    playTime.setText("00:00");
}

//(j)

public void playCurrentSong()
{
    setStopped(false);
    (new PlayerThread(null)).start();
    (new TimerThread(null)).start();
}

//(d)

public static void main(String args[])
{
    PlayList list;
    if(args.length > 0)
        list = new PlayList(args[0]);
    if((new File(DEFAULT_PLAYLIST)).exists())
        list = new PlayList(DEFAULT_PLAYLIST);
    else
        list = new PlayList();
    Player test = new Player(list);
    test.setVisible(true);
}

JButton play;
JButton pause;
JButton next;
JButton stop;
JButton pl_editor;
    
private JLabel songDescription;
private JLabel playTime;
private boolean stopped;
private boolean editorVisible;
private PlayList playList;
private PlayListEditor playListEditor;
public static String DEFAULT_PLAYLIST = "playList.m3u";




}

Dadurch dass die Buttons deaktiviert sind, kann ich auch kein lied weiter machen und/oder das Lied stoppen, pausieren, fortsetzen etc...

 

[sambalmueslie]

mal so paar kleinigkeiten am Rande, die es dir und anderen etwas einfacher machen sollten.

Für die ActionCommands nutze doch statische private Strings.

Das hat den Vorteil, dass du nur an einer Stelle den wirklichen String stehen hast und nicht auf Schreibfehler achten musst.

Beispiel

private static final String AC_PLAY = "AC_PLAY";

...


play.setActionCommand(AC_PLAY)

....


if(command.equals(AC_PLAY))
...

das richtige Schreiben der Konstanten überprüft dir dein Compiler

dann noch was anderes.
Deine Playerklasse mutiert gerade zur eierlegenden-woll-milch-Kuh.

Versuch das doch etwas aufzuteilen. (MVC ist dein Freund

Beispielsweise so:

eine Klasse für die Anzeige (View)
eine Klasse für die Funktion (Controller)

damit hast du das ganze etwas getrennt.

Dann noch eine persönliche Vorliebe ist das verwenden von Gettern für die Gui-Elemente
die müssen nicht privat sein und erstellen den Button wenn er nicht existiert.

Dadurch wird dein Konstruktor etwas übersichtlicher

Beispiel

public Player(){
 setLayout(new BorderLayout(5,5));
 add(getTopPanel(), BorderLayout.NORTH);
....
}

private void JPanel getTopPanel(){
 if(topPanel == null){
  topPanel = new JPanel();
 }
 return topPanel;
}

private JPanel topPanel;

ok war viel und etwas OffTopic

Aber vielleicht hilft es dir etwas

Gruß Oli

 

[SlaterB]

im Konstruktor

play = new JButton(new ImageIcon("...play.png"));
statt
JButton play = new JButton(new ImageIcon("...play.png"));
usw.

sonst erzeugst du neue lokale Variablen, die wie die Klassenattribute heißen