Swing JList Elemente ausgeben fehlerhaft

[N1klaZ]

Hallo Community,

Ich habe eine List in die ich durch ein Button einen Link von einem Textfeld in eine Liste schreibe. Danach möchte ich durch einen weiteren Button das ausgewählte Element in ein Editor ausgeben. Und da liegt mein Problem wenn ich den Link anklicke und dann den Button klicke kommt keine Ausgabe. Da habe ich mir nun den Index ausgeben lassen und der ist bei jeder Ausgabe die ich gemacht habe -1. Das heißt ja das kein Element makiert wurde.

hier meine Liste

private JList getUrls() {
		if (urls == null) {
			urls = new JList(addurls);
			urls.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
		}
		return urls;
	}

mein Ausgabe Button

private JButton getConnect() {
		if (connect == null) {
			connect = new JButton();
			connect.setBounds(new Rectangle(146, 448, 89, 18));
			connect.setText("Connect");
			connect.addMouseListener(new java.awt.event.MouseAdapter() {
				public void mouseClicked(java.awt.event.MouseEvent e) {
				int index=urls.getSelectedIndex();
				System.out.print(index);				
					System.out.println();					
					System.out.println("mouseClicked()"); 
					
					
					// TODO Auto-generated Event stub mouseClicked()
				}
			});
		}
		return connect;
	}

Ich hoffe ihr könnt mir helfen. Vielen Dank im Vorraus.

//Edit: addurls ist ein DefaultListModel.

 

[SlaterB]

grummel, diese unnützen Codeausschnitte,
kaum baut man sie zu einem Programm zusammen (was du auch hättest tun können..)
schon funktioniert alles

public class TestGUI  extends JFrame {
    JList urls;
    JButton connect;

    public TestGUI()
    {
        add(getUrls(), BorderLayout.CENTER);
        add(getConnect(), BorderLayout.SOUTH);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(200, 300);
        setVisible(true);
    }

    private JList getUrls()
    {
        if (urls == null)
        {
            DefaultListModel addurls = new DefaultListModel();
            addurls.addElement("a");
            addurls.addElement("b");
            addurls.addElement("c");
            urls = new JList(addurls);
            urls.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
        }
        return urls;
    }

    private JButton getConnect()
    {
        if (connect == null)
        {
            connect = new JButton();
            connect.setBounds(new Rectangle(146, 448, 89, 18));
            connect.setText("Connect");
            connect.addMouseListener(new java.awt.event.MouseAdapter()
                {
                    public void mouseClicked(java.awt.event.MouseEvent e)
                    {
                        int index = urls.getSelectedIndex();
                        System.out.print(index);
                        System.out.println();
                        System.out.println("mouseClicked()");
                    }
                });
        }
        return connect;
    }

    public static void main(String[] args)
    {
        new TestGUI();
    }
}

dein Problem kann alle möglichen Ursachen haben, Button schaut sich anderes Model an als in der GUI angezeigt oder oder oder,
ohne vollständige (kurze) Programme geht rein gar nix

-----

oder es liegt an
> einen Link von einem Textfeld in eine Liste schreibe.
davon hast du gar keinen Code gepostet, allgemein kann das vielleicht die Selektion aufheben oder wer weiß was schlimmes

 

[Michael...]

Handelt es sich den bei den urls tatsächlich um die selbe Liste?

Anmerkung: für Button-Clicks verwendet man normalerweise einen ActionListener anstelle eines MouseListeners

 

[N1klaZ]

Ich wollte euch den gesammten Code eigentlich nur ersparen weil es extrem viel ist aber wenn das hilft den Fehler zufinden habe ich kein Problem damit den hier zu Posten.

import javax.swing.SwingUtilities;
import java.awt.BorderLayout;

import javax.swing.DefaultListModel;
import javax.swing.JMenu;
import javax.swing.JMenuItem;
import javax.swing.JPanel;
import javax.swing.JFrame;
import javax.swing.JScrollPane;
import javax.swing.KeyStroke;
import javax.swing.ListModel;

import java.awt.Dimension;
import javax.swing.JTextField;
import java.awt.Rectangle;
import java.awt.GridBagLayout;
import javax.swing.JButton;
import java.awt.GridBagConstraints;
import javax.swing.JList;
import javax.swing.JTextPane;
import java.awt.Color;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.ItemListener;
import java.awt.event.KeyEvent;

import javax.swing.JMenuBar;
import javax.swing.JEditorPane;
import javax.swing.ListSelectionModel;
import javax.swing.event.ListSelectionListener;

public class Main extends JFrame  {									// Klassen anfang

	private static final long serialVersionUID = 1L;
	private JPanel jContentPane = null;
	private JPanel jPanel = null;
	private JButton connect = null;
	private JButton Add = null;
	private JTextField urleingabe = null;
	private JList urls = null;
	private JTextPane test = null;
	private JMenuBar jJMenuBar = null;
	JMenu menu;														// menu Punkt wird Iniziallisiert
	JMenuItem öffnen, speichern;
	private JEditorPane Editor = null;
	private JScrollPane jScrollPane = null;
	private DefaultListModel addurls = new DefaultListModel();
	private JScrollPane scroll = null;
	
	/**
	 * This method initializes jPanel	
	 * 	
	 * @return javax.swing.JPanel	
	 */
	private JPanel getJPanel() {
		if (jPanel == null) {
			jPanel = new JPanel();
			jPanel.setLayout(null);
			jPanel.setBounds(new Rectangle(472, 0, 235, 515));
			jPanel.add(getConnect(), null);
			jPanel.add(getAdd(), null);
			jPanel.add(getUrleingabe(), null);
			jPanel.add(getTest(), null);
			jPanel.add(getScroll(), null);
		}
		return jPanel;
	}

	/**
	 * This method initializes connect	
	 * 	
	 * @return javax.swing.JButton	
	 */
	private JButton getConnect() {
		if (connect == null) {
			connect = new JButton();
			connect.setBounds(new Rectangle(146, 448, 89, 18));
			connect.setText("Connect");
			connect.addMouseListener(new java.awt.event.MouseAdapter() {
				public void mouseClicked(java.awt.event.MouseEvent e) {
				int index=urls.getSelectedIndex();
				System.out.print(index);				
					System.out.println();					
					System.out.println("mouseClicked()"); 
					
					
					// TODO Auto-generated Event stub mouseClicked()
				}
			});
		}
		return connect;
	}

	/**
	 * This method initializes Add	
	 * 	
	 * @return javax.swing.JButton	
	 */
	private JButton getAdd() {
		if (Add == null) {
			Add = new JButton();
			Add.setBounds(new Rectangle(175, 495, 62, 18));
			Add.setText("Add");
			Add.addActionListener(new java.awt.event.ActionListener() {
				public void actionPerformed(java.awt.event.ActionEvent e) {
					addurls.addElement(urleingabe.getText());										
					urls = new JList(addurls);
					urleingabe.setText("");
					// TODO Auto-generated Event stub actionPerformed()
				
				}
			});
		}
		return Add;
	}

	/**
	 * This method initializes urleingabe	
	 * 	
	 * @return javax.swing.JTextField	
	 */
	private JTextField getUrleingabe() {
		if (urleingabe == null) {
			urleingabe = new JTextField();
			urleingabe.setBounds(new Rectangle(0, 472, 236, 25));
			urleingabe.setText("");
		}
		return urleingabe;
	}

	/**
	 * This method initializes urls	
	 * 	
	 * @return javax.swing.JList	
	 */
	private JList getUrls() {
		if (urls == null) {
			urls = new JList(addurls);
			urls.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
		}
		return urls;
	}

	/**
	 * This method initializes test	
	 * 	
	 * @return javax.swing.JTextPane	
	 */
	private JTextPane getTest() {
		if (test == null) {
			test = new JTextPane();
			test.setBounds(new Rectangle(1, 450, 131, 20));
			test.setEnabled(true);
			test.setBackground(new Color(238, 238, 238));
			test.setForeground(Color.black);
			test.setEditable(false);
			test.setText("URL eingeben:");
		}
		return test;
	}

	/**
	 * This method initializes jJMenuBar	
	 * 	
	 * @return javax.swing.JMenuBar	
	 */
	private JMenuBar getJJMenuBar() {
		if (jJMenuBar == null) {
			jJMenuBar = new JMenuBar();
		}
		menu = new JMenu("Datei");									// Menu Punkt Datei wird erstellt
		menu.setMnemonic(KeyEvent.VK_A);							// Tastenkürzel Alt+a wird gesetzt
		jJMenuBar.add(menu);										// Datei wird der Menubar zugeordnet
		
		öffnen = new JMenuItem("öffnen",KeyEvent.VK_T);				// MenuItem öffnen wird erstellt
		öffnen.setAccelerator(KeyStroke.getKeyStroke(
		        KeyEvent.VK_1, ActionEvent.ALT_MASK));				// Tasten Kürzel wird gesetzt
		menu.add(öffnen);											// MenuItem wird dem Menu Datei hinzugefügt
		
		speichern = new JMenuItem("speichern",KeyEvent.VK_T);			// MenuItem speichern wird erstellt
		speichern.setAccelerator(KeyStroke.getKeyStroke(
		        KeyEvent.VK_2, ActionEvent.ALT_MASK));				// Tasten Kürzel wird gesetzt
		menu.add(speichern);	
		
		return jJMenuBar;
	}
	/**
	 * This method initializes Editor	
	 * 	
	 * @return javax.swing.JEditorPane	
	 */
	private JEditorPane getEditor() {
		if (Editor == null) {
			Editor = new JEditorPane();
		}
		return Editor;
	}

	
	/**
	 * This method initializes jScrollPane	
	 * 	
	 * @return javax.swing.JScrollPane	
	 */
	private JScrollPane getJScrollPane2() {
		if (jScrollPane == null) {
			jScrollPane = new JScrollPane();
			jScrollPane.setBounds(new Rectangle(1, 0, 472, 517));
			jScrollPane.setViewportView(getEditor());
			jScrollPane.setVerticalScrollBarPolicy(jScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED);
			
			jScrollPane.setHorizontalScrollBarPolicy(jScrollPane.HORIZONTAL_SCROLLBAR_AS_NEEDED);
								
		}
		return jScrollPane;
	}

	/**
	 * This method initializes scroll	
	 * 	
	 * @return javax.swing.JScrollPane	
	 */
	private JScrollPane getScroll() {
		if (scroll == null) {
			scroll = new JScrollPane();
			scroll.setBounds(new Rectangle(-1, 0, 238, 450));
			scroll.setViewportView(getUrls());
			scroll.setVerticalScrollBarPolicy(jScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED);
			scroll.setHorizontalScrollBarPolicy(jScrollPane.HORIZONTAL_SCROLLBAR_AS_NEEDED);
		}
		return scroll;
	}

	/**
	 * @param args
	 */
	public static void main(String[] args) {						// Hauptprogramm anfang
		// TODO Auto-generated method stub
		SwingUtilities.invokeLater(new Runnable() {
			public void run() {
				Main thisClass = new Main();
				thisClass.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
				thisClass.setVisible(true);
			}
		});
	}

	/**
	 * This is the default constructor
	 */
	public Main() {													// Methode
		super();
		initialize();
	}

	/**
	 * This method initializes this
	 * 
	 * @return void
	 */
	private void initialize() {										// Methode
		this.setSize(717, 573);
		this.setJMenuBar(getJJMenuBar());
		this.setContentPane(getJContentPane());
		this.setTitle("JFrame");
	}

	/**
	 * This method initializes jContentPane
	 * 
	 * @return javax.swing.JPanel
	 */
	private JPanel getJContentPane() {								// Methode des JContentPane
		if (jContentPane == null) {
			jContentPane = new JPanel();
			jContentPane.setLayout(null);
			jContentPane.add(getJPanel(), null);
			jContentPane.add(getJScrollPane2(), null);
		}
		return jContentPane;
	}

}  //  @jve:decl-index=0:visual-constraint="10,10"

//Edit: Ich glaube ich habe den Fehler gefunden es liegt daran das ich die Listen 2 mal erstelle einmmal wenn ich den Button Add betätige und einmal wo ich die Listeinizialisieren oder ?

//Edit: Ok jetzt funktioniert es dran lag es...

 

[Michael...]

Ich wollte euch den gesammten Code eigentlich nur ersparen weil es extrem viel ist aber wenn das hilft den Fehler zufinden habe ich kein Problem damit den hier zu Posten.

Das hast Du falsch verstanden ;-) den Code soweit reduzieren, dass er das Problem darstellt und am besten noch direkt kompliert und ausgeführt werden kann. Dabei stopert man meist selbst über den Fehler.

Dein Fehler ist, dass bei jedem hinzufügen eine neue Liste erstellst.