Sortering af JLabels i JScollPane, samt noget meget mærkeligt

Det kan have noget at gøre med den layoutmanager du bruger. Men det er svært at sige, når vi ikke umiddelbart kan se hvordan den skal fungere.

Eller du bruger den måske slet ikke? Fik bare compilerfejl.

Beklager..Jeg har fjernet den layoutmanager. Jeg bruger den ikke.

Jeg har ikke lige andet end notepad at teste i lige nu, men jeg kigger lige på det senere, når jeg kommer hjem.

Så fik jeg endelig tid til at kigge på det. Jeg har kigget lidt på din kode, og ændret lidt til. Jeg har blandt andet fjernet noget af det kode der ikke blev brugt.

package Test;

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;

public class NewJFrame extends javax.swing.JFrame
{
    private JScrollPane jScrollPane1;
    private JButton jButton1;
    private JPanel panel;
    private int count = 0;

    /**
    * Auto-generated main method to display this JFrame
    */
    public static void main(String[] args)
    {
        NewJFrame inst = new NewJFrame();
        inst.setVisible(true);
    }
   
    public NewJFrame()
    {
        super();
        initGUI();
    }
   
    public void addLabel()
    {
        AddDevice device = new AddDevice(this, "Device: " + count++);
        panel.add(device);
        this.validate();
        panel.updateUI();
    }
   
    public void removeLabel(AddDevice device)
    {
        panel.remove(device);
        this.validate();
        panel.updateUI();
    }
   
    private void initGUI()
    {
        try
        {
            setDefaultCloseOperation(WindowConstants.DISPOSE_ON_CLOSE);
           
            panel = new JPanel();
            panel.setLayout(new BoxLayout(panel, BoxLayout.Y_AXIS));
            jScrollPane1 = new JScrollPane(panel);
           
            this.getContentPane().add(jScrollPane1, BorderLayout.CENTER);

           
            jButton1 = new JButton();
            this.getContentPane().add(jButton1, BorderLayout.SOUTH);
            jButton1.setText("Add");
            jButton1.addActionListener(new ActionListener()
            {
                public void actionPerformed(ActionEvent evt)
                {
                    addLabel();
                }
            });
           
            pack();
            setSize(400, 300);
        }
        catch (Exception e)
        {
            e.printStackTrace();
        }
    }
}


package Test;

import java.awt.Component;
import java.awt.FlowLayout;
import java.awt.Label;
import java.awt.MenuItem;
import java.awt.PopupMenu;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;

import javax.swing.ImageIcon;
import javax.swing.JLabel;
import javax.swing.BorderFactory;
import javax.swing.border.BevelBorder;
import javax.swing.JMenuItem;
import javax.swing.JPopupMenu;
import javax.swing.JTextArea;



public class AddDevice extends JLabel implements ActionListener, ItemListener
{
    public JPopupMenu popup;
    JMenuItem menuItem;
    NewJFrame owner;
   
    public AddDevice(NewJFrame owner, String text)
    {
        this.setText(text);
        this.setLayout(new FlowLayout());
       
        popup = new JPopupMenu();
        menuItem = new JMenuItem("Remove");
        popup.add(menuItem);
       
        menuItem.addActionListener(this);
        popup.add(menuItem);
       
        MouseListener listener = new PopupListener();
        this.addMouseListener(listener);
       
        this.owner = owner;
    }
   
    public void actionPerformed(ActionEvent e)
    {
        owner.removeLabel(this);   
    }
   
    class PopupListener extends MouseAdapter
    {
        public void mousePressed(final MouseEvent e)
        {
            maybeShowPopup(e);
        }

        public void mouseReleased(final MouseEvent e)
        {
            maybeShowPopup(e);
        }

        private void maybeShowPopup(final MouseEvent e)
        {
            if (e.isPopupTrigger())
            {
                popup.show(e.getComponent(), e.getX(), e.getY());
            }
        }   
    }
    public void itemStateChanged(ItemEvent arg0) {}
}

Læg mærke til at jeg bruger BoxLayout:

http://java.sun.com/docs/books/tutorial/uiswing/layout/box.html
Og generelt:
http://java.sun.com/docs/books/tutorial/uiswing/layout/visual.html


En anden ting jeg har ændret, er at du havde en statisk instans af din JFrame inde i din JFrame - Jeg håber det kun var for at teste, for det er ikke måden at gøre det på ;)

Send i stedet instansen med i constructoren som jeg gør.

Og et svar hvis det var brugbart.

Brugbart?

Ja det var helt perfekt, mikkel. Tusind tak for hjælpen