Gui components disappearing

[Stef Van Gils]

Hello, I made a gui, this gui containts a JTree, a JList, encapsulated in a JScrollPane and a couple of JButtons.

I thought it worked, but today I double clicked accidentally on a JTree entry and all things disappearded.
If i hover over the components, they become visible again, but it's not the intention. Is there any solution to let the other components be visible when i double click?

Here's my code:

package gui;

import java.awt.datatransfer.StringSelection;
import java.awt.dnd.DnDConstants;
import java.awt.dnd.DragGestureEvent;
import java.awt.dnd.DragGestureListener;
import java.awt.dnd.DragGestureRecognizer;
import java.awt.dnd.DragSource;
import java.awt.dnd.DragSourceDragEvent;
import java.awt.dnd.DragSourceDropEvent;
import java.awt.dnd.DragSourceEvent;
import java.awt.dnd.DragSourceListener;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;

import javax.swing.DefaultListModel;
import javax.swing.DropMode;
import javax.swing.ImageIcon;
import javax.swing.JComponent;
import javax.swing.JFileChooser;
import javax.swing.JFrame;
import javax.swing.JMenuBar;
import javax.swing.JMenu;
import javax.swing.JMenuItem;
import javax.swing.JScrollPane;
import javax.swing.ListSelectionModel;
import javax.swing.ScrollPaneConstants;
import javax.swing.SwingUtilities;
import javax.swing.TransferHandler;
import javax.swing.TransferHandler.DropLocation;

import backbone.JEB;
import backbone.Music;
import backbone.MusicLocal;
import backbone.Parser;
import backbone.PlayList;
import backbone.Settings;

import javax.swing.JButton;
import java.awt.BorderLayout;
import java.awt.Point;
import java.io.File;
import java.util.ArrayList;
import java.util.Enumeration;
import java.util.Vector;

import javax.swing.JSlider;
import javax.swing.Icon;
import javax.swing.JList;
import javax.swing.JTree;
import javax.swing.event.TreeSelectionEvent;
import javax.swing.event.TreeSelectionListener;
import javax.swing.tree.DefaultMutableTreeNode;
import javax.swing.tree.DefaultTreeModel;
import javax.swing.tree.TreeModel;
import javax.swing.tree.TreeNode;
import javax.swing.tree.TreePath;
import javax.swing.tree.TreeSelectionModel;

import enums.Style;

public class Gui extends JFrame implements TreeSelectionListener, DragGestureListener, DragSourceListener{
	private JFrame frame;
	private JEB jeb;
	private JButton btnPlay;
	private JButton btnStop;
	private JButton btnPrev;
	private JButton btnNext;
	private ImageIcon play;
	private ImageIcon pause;
	private JButton btnRepeat;
	private JButton btnShuffle;
	private ImageIcon repeat;
	private ImageIcon repeat1;
	private ImageIcon repeatenabled;
	private ImageIcon shuffle;
	private ImageIcon shuffleOn;
	private DefaultListModel<Music> model;
	private final JList<Music> list = new JList<Music>();
	private ArrayList<DefaultMutableTreeNode> nodes;
	private JTree tree;
	private JScrollPane scroll;

	public JEB getjeb() {
		return jeb;
	}



	public Gui(final JEB jeb){
		this.jeb = jeb;
		frame = new JFrame();
		frame.setSize(800, 400);
		JMenuBar menuBar = new JMenuBar();
		frame.setJMenuBar(menuBar);
		nodes = new ArrayList<DefaultMutableTreeNode>();
		JMenu mnNewMenu = new JMenu("Bestand");
		menuBar.add(mnNewMenu);

		JMenuItem mntmNewMenuItem = new JMenuItem("Openen");
		mnNewMenu.add(mntmNewMenuItem);

		final Gui g = this;
		mntmNewMenuItem.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				JFileChooser c = new JFileChooser();
				c.setMultiSelectionEnabled(true);
				int rVal = c.showOpenDialog(g);
				if (rVal == JFileChooser.APPROVE_OPTION) {
					File[] files = c.getSelectedFiles();
					for(File f: files)
						jeb.getAllMusic().getMusic().add((new Parser(f.getAbsolutePath()).parse()));
					jeb.setCurrentMusic(jeb.getAllMusic().getMusic());
					jeb.setCurrentPlayList(jeb.getAllMusic());
					jeb.setCurrentShowingPlaylist(jeb.getAllMusic());
					update();
				}
			}
		});

		JMenuItem mntmAdd = new JMenuItem("Afspeellijst Toevoegen");
		mnNewMenu.add(mntmAdd);
		final Gui gui = this;
		mntmAdd.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				new PlaylistGui(jeb, gui);
			}
		});

		JSlider slider = new JSlider();
		slider.setBounds(90, 314, 200, 26);
		frame.getContentPane().add(slider);

		play = new ImageIcon("play.bmp");
		pause = new ImageIcon("pause.bmp");
		ImageIcon stop = new ImageIcon("stop.bmp");
		ImageIcon prev = new ImageIcon("prev.bmp");
		ImageIcon next = new ImageIcon("next.bmp");
		repeat = new ImageIcon("repeat.bmp");
		repeat1 = new ImageIcon("repeat1.bmp");
		repeatenabled = new ImageIcon("repeatenabled.bmp");
		shuffle = new ImageIcon("shuffle.bmp");
		shuffleOn = new ImageIcon("shuffleOn.bmp");

		btnPlay = new JButton(play);
		btnPlay.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				actionPerformedPlay(e);
			}
		});
		btnPlay.setBounds(61, 314, 20, 20);
		frame.getContentPane().add(btnPlay);

		btnPrev = new JButton(prev);
		btnPrev.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				actionPerformedPrev(e);
			}
		});
		btnPrev.setBounds(42, 314, 20, 20);
		btnPrev.addMouseListener(new MouseListener() {

			@Override
			public void mouseReleased(MouseEvent arg0) {
				// TODO Auto-generated method stub

			}

			@Override
			public void mousePressed(MouseEvent arg0) {
				// TODO Auto-generated method stub

			}

			@Override
			public void mouseExited(MouseEvent arg0) {
				// TODO Auto-generated method stub

			}

			@Override
			public void mouseEntered(MouseEvent arg0) {
				// TODO Auto-generated method stub

			}

			@Override
			public void mouseClicked(MouseEvent e) {
				if (e.getClickCount() == 2 && e.getSource().equals(btnPrev)) {
					jeb.setIndexCurrentSong(jeb.getIndexCurrentSong()-1);
				}

			}
		});
		frame.getContentPane().add(btnPrev);

		btnStop = new JButton(stop);
		btnStop.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				actionPerformedStop(e);
			}
		});
		btnStop.setBounds(300, 314, 20, 20);
		frame.getContentPane().add(btnStop);

		btnNext = new JButton(next);
		btnNext.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				actionPerformedNext(e);
			}
		});
		btnNext.setBounds(319, 314, 20, 20);
		frame.getContentPane().add(btnNext);

		btnRepeat = new JButton(repeat);
		btnRepeat.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				actionPerformedRepeat(e);
			}


		});
		btnRepeat.setBounds(0, 314, 40, 20);
		frame.getContentPane().add(btnRepeat);

		btnShuffle = new JButton(shuffle);
		btnShuffle.setBounds(344, 314, 40, 20);
		frame.getContentPane().add(btnShuffle);

		btnShuffle.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				actionPerformedShuffle(e);
			}
		});

		list.addMouseListener(new MouseListener() {

			@Override
			public void mouseReleased(MouseEvent e) {
				// TODO Auto-generated method stub

			}

			@Override
			public void mousePressed(MouseEvent e) {
				// TODO Auto-generated method stub

			}

			@Override
			public void mouseExited(MouseEvent e) {
				// TODO Auto-generated method stub

			}

			@Override
			public void mouseEntered(MouseEvent e) {
				// TODO Auto-generated method stub

			}

			@Override
			public void mouseClicked(MouseEvent e) {
				if(e.getSource().equals(list)){
					JList<Music> list = (JList<Music>)e.getSource();
					if (e.getClickCount() == 2) {
						int index = list.locationToIndex(e.getPoint());
						doubleClickedJList(index);
					}
				}
			}
		});
		list.setBounds(200, 0, 600, 303);
		list.setSelectionMode(ListSelectionModel.MULTIPLE_INTERVAL_SELECTION);
		list.getSelectionModel().setSelectionInterval(0, 3);
		scroll = new JScrollPane(ScrollPaneConstants.VERTICAL_SCROLLBAR_ALWAYS, 
                                                               ScrollPaneConstants.HORIZONTAL_SCROLLBAR_ALWAYS);
		scroll.setBounds(200,0,585,303);
		scroll.setViewportView(list);
		frame.getContentPane().add(scroll);

		addPlaylists();


		addDragDropFunctionality();
		frame.paint(getGraphics());
		frame.paintAll(getGraphics());
		frame.setVisible(true);
		

	}




	private DefaultMutableTreeNode allMusic;
	private DefaultMutableTreeNode nodePlayList;
	private void addPlaylists() {
		DefaultMutableTreeNode root = new DefaultMutableTreeNode();
		allMusic = new DefaultMutableTreeNode(jeb.getAllMusic());
		nodePlayList = new DefaultMutableTreeNode("Afspeellijsten");
		nodePlayList.setAllowsChildren(true);
		allMusic.setAllowsChildren(false);
		for(int i=1; i<jeb.getPlayLists().size(); i++){
			DefaultMutableTreeNode m = new DefaultMutableTreeNode(jeb.getPlayLists().get(i));
			nodes.add(m);
			nodePlayList.add(m);
		}
		root.add(allMusic);
		root.add(nodePlayList);
		tree = new JTree(root);
		tree.setRootVisible(false);
		tree.setBounds(0, 0, 200, 303);
		tree.getSelectionModel().setSelectionMode
		(TreeSelectionModel.SINGLE_TREE_SELECTION);
		tree.addTreeSelectionListener((TreeSelectionListener) this);
		tree.setDropMode(DropMode.INSERT);
		expandAll();
		frame.getContentPane().add(tree);
	}

	public void updatePlaylists(){

		for(int i=1; i<jeb.getPlayLists().size(); i++){
			PlayList play = jeb.getPlayLists().get(i);
			boolean isChild = false;

			for(int j =0; j< nodePlayList.getChildCount(); j++){
				DefaultMutableTreeNode m =(DefaultMutableTreeNode) nodePlayList.getChildAt(j);
				if(m.equals(play)){
					isChild = true;
					break;
				}
			}
			if(!isChild){
				DefaultMutableTreeNode m = new DefaultMutableTreeNode(jeb.getPlayLists().get(i));
				nodes.add(m);
				nodePlayList.add(m);
			}
		}
		expandAll();
	}

	private void expandAll() {
		// TODO Auto-generated method stub
		for (int i = 0; i < tree.getRowCount(); i++) {
			tree.expandRow(i);
		}
	}



	public void initialize(){
		frame.setSize(500, 500);
	}

	public JFrame getFrame() {
		return frame;
	}
	public void update(){
		ArrayList<Music> music = jeb.getCurrentMusic();
		model = new DefaultListModel<Music>();
		for(Music m: music){
			model.addElement(m);
		}
		list.setModel(model);
		//scroll.setViewportView(list);
		updatePlaylists();
		System.out.println("update");

	}



	private boolean playEnabled = false;
	public void actionPerformedPlay(ActionEvent e) {
		if(playEnabled){
			btnPlay.setIcon(play);
			jeb.pause();
			playEnabled = false;
		}else {
			jeb.play();
			if(jeb.isPlaying()){
				btnPlay.setIcon(pause);
				playEnabled = true;
			}
		}



	}
	public void actionPerformedStop(ActionEvent e) {
		jeb.stop();
		btnPlay.setIcon(play);
		playEnabled = false;
		jeb.setPlaying(false);
	}
	public void actionPerformedPrev(ActionEvent e) {
		jeb.playPrev();

	}
	public void actionPerformedNext(ActionEvent e) {
		jeb.playNext();

	}

	public void actionPerformedShuffle(ActionEvent e) {
		if(Settings.shuffle){
			btnShuffle.setIcon(shuffle);
			jeb.undoShuffle();
			Settings.shuffle = false;
		}
		else{
			btnShuffle.setIcon(shuffleOn);
			jeb.shuffle();
			Settings.shuffle = true;
		}

	}
	public void actionPerformedRepeat(ActionEvent e) {
		if(Settings.repeat == false && Settings.repeatOne ==false){
			btnRepeat.setIcon(repeatenabled);
			Settings.repeat = true;
			Settings.repeatOne = false;
		}else if(Settings.repeat==true && Settings.repeatOne == false){
			btnRepeat.setIcon(repeat1);
			Settings.repeat = false;
			Settings.repeatOne = true;
		}else {
			btnRepeat.setIcon(repeat);
			Settings.repeat = false;
			Settings.repeatOne = false;
		}
	}
	public void doubleClickedJList(int position){
		actionPerformedStop(new ActionEvent(new Object(), 0, "", 0));
		jeb.setIndexCurrentSong(position);
		jeb.setCurrentPlayList(jeb.getCurrentShowingPlaylist());
		actionPerformedPlay(new ActionEvent(new Object(), 0, "", 0));
	}

	@Override
	public void valueChanged(TreeSelectionEvent e) {
		DefaultMutableTreeNode node = (DefaultMutableTreeNode)
				tree.getLastSelectedPathComponent();
		System.out.println("haha");
		if (node == null)
			//Nothing is selected.  
			return;

		Object nodeInfo = node.getUserObject();
		if (node.isLeaf() && !(node.equals(nodePlayList))) {

			PlayList play = (PlayList) nodeInfo;
			if(play != jeb.getCurrentShowingPlaylist()){
				jeb.setCurrentMusic(play.getMusic());
				jeb.setCurrentShowingPlaylist(play);
				update();
			}

		}
	}

	private DragSource ds;
	private StringSelection transferable;
	private void addDragDropFunctionality() {
		ds = new DragSource();
		DragGestureRecognizer dgr = ds.createDefaultDragGestureRecognizer(list,
				DnDConstants.ACTION_COPY, (DragGestureListener) this);
	}
	public void dragGestureRecognized(DragGestureEvent dge) {
		if(list.getSelectedValue()!=null){
			transferable = new StringSelection(list.getSelectedValue().toString());
			ds.startDrag(dge, DragSource.DefaultCopyDrop, transferable, (DragSourceListener) this);
		}
	}

	public void dragEnter(DragSourceDragEvent dsde) {

	}

	public void dragExit(DragSourceEvent dse) {
	}

	public void dragOver(DragSourceDragEvent dsde) {
	}

	public void dragDropEnd(DragSourceDropEvent dsde) {
		Point p = dsde.getLocation();
		SwingUtilities.convertPointFromScreen(p, tree);
		TreePath th = tree.getPathForLocation((int)p.getX(), (int)p.getY());
		if(th!=null){
			DefaultMutableTreeNode node = (DefaultMutableTreeNode) th.getLastPathComponent();
			if(node!=nodePlayList && node != allMusic){
				PlayList play = (PlayList) node.getUserObject();
				for(Music music: list.getSelectedValuesList())
					play.addMusic((MusicLocal) music);
				update();
			}
		}

	}
	public void dropActionChanged(DragSourceDragEvent dsde) {
	}



}

[Tony Docherty]

That's too much uncommented code for me to look through right now so the following is a total guess but when GUI components disappear as you have described it can be down to structurally changing the GUI in some way (eg add/removing components) and not revalidating it.

[Paul Clapham]

I haven't looked at the code much either, but I can say that I have an application which uses JTrees extensively, and it used to be that my tree model didn't report changes correctly and as a result I used to get blank spaces between the rows and other undesirable mis-features.

[Stef Van Gils]

And what was the solution to it?

I add the components when I create the JFrame, I only change the entries of a JList and the nodes of the JTree