JList Cellrenderer fügt mehrfach ein

[dreadnought]

Hi

Ich habe ein Problem mit meiner JList: Seit ich einen eigene Cellrenderer habe und versuche ein JPanel mit JLabels einzufügen werden alle Listenelemente mehrfach eingefügt.
Das aussehen der einzelnen Elemente hab ich unter Kontrolle. Nur die Position...

Beim ersten Element sieht alles super aus.
aber wenn ich das 2 Element eingefügt wird dann sieht die Liste folgendermaßen aus:

2.Element 1.Element
1.Element 2.Element

beim 3. Element siehts so aus:

2.Element 3.Element 1.Element
3.Element 1.Element 2.Element
1.Element 2.Element 3.Element
usw.:lol:

dabei ist die Liste artikelEingescanntListe.setLayoutOrientation(JList.VERTICAL); ohne WRAP ausgerichtet.

die Klasse die in die Liste eingefügt wird sieht so aus:

	public class ListenPosten
	{
		private JLabel rechnungspostenArtikelOderWarenGruppeLabel;
		private JLabel rechnungspostenGebindeinhaltLabel;
		private JLabel rechnungspostenGebindeLabel;
		private JLabel rechnungspostenAnzahlLabel;
		private JLabel rechnungspostenPreisLabel;
		private JLabel rechnungspostenGesamtPreisLabel;
		private JLabel rechnungspostenRabattLabel;
		
		public ListenPosten(String rechnungspostenArtikelOderWarenGruppeString,
				String rechnungspostenGebindeinhaltString, String rechnungspostenGebindeString,
				String rechnungspostenAnzahlString,	String rechnungspostenPreisString,
				String rechnungspostenGesamtPreisString, String rechnungspostenRabattPreisString, int Stelle)
		{
			rechnungspostenArtikelOderWarenGruppeLabel = new JLabel(rechnungspostenArtikelOderWarenGruppeString);
			rechnungspostenGebindeinhaltLabel = new JLabel(rechnungspostenGebindeinhaltString);
			rechnungspostenGebindeLabel = new JLabel(rechnungspostenGebindeString);
			rechnungspostenAnzahlLabel = new JLabel(rechnungspostenAnzahlString);
			rechnungspostenPreisLabel = new JLabel(rechnungspostenPreisString);
			rechnungspostenGesamtPreisLabel = new JLabel(rechnungspostenGesamtPreisString);
			rechnungspostenRabattLabel = new JLabel(rechnungspostenRabattPreisString);
			
		}
...

und ein paar "Getter"

und der CellRenderer sieht im Moment so aus:

	public class MyCellRenderer extends JPanel implements ListCellRenderer {
		
		private static final long serialVersionUID = 8400636002225624699L;
		
		
		public MyCellRenderer()
		{
			// Konstruktor des JPanels mit FlowLayout aufrufen
			super(new FlowLayout(FlowLayout.LEFT));
			
			// JPanel undurchsichtig machen
			this.setOpaque(true);
			
			rechnungspostenpane = new JPanel();
			rechnungspostenpane.setOpaque(false);
			this.add(rechnungspostenpane);        
		}
		
		public Component getListCellRendererComponent(JList artikelEingescanntlistModel, // JList Objekt
				Object value, // anzuzeigende Komponente
				int index,    // Zellenindex
				boolean istSelektiert,  // Ist selektiert?
				boolean hatFokus)  // Hat den Fokus?
		{
			Dimension Dimie = new Dimension(80, feldhoehe);
			Dimension Dimieklein = new Dimension(70, feldhoehe);
			Dimension Dimiemittel = new Dimension(90, feldhoehe);
			Dimension Dimiegroß = new Dimension(280, feldhoehe);
			
			((ListenPosten)value).getRechnungspostenArtikelOderWarenGruppeLabel().setPreferredSize(Dimiegroß);			
			((ListenPosten)value).getRechnungspostenGebindeinhaltLabel().setPreferredSize(Dimieklein);
			((ListenPosten)value).getRechnungspostenGebindeLabel().setPreferredSize(Dimie);
			((ListenPosten)value).getRechnungspostenAnzahlLabel().setPreferredSize(Dimie);
			((ListenPosten)value).getRechnungspostenPreisLabel().setPreferredSize(Dimie);
			((ListenPosten)value).getRechnungspostenGesamtPreisLabel().setPreferredSize(Dimiemittel);
			((ListenPosten)value).getRechnungspostenRabattLabel().setPreferredSize(Dimiemittel);
			
			((ListenPosten)value).getRechnungspostenArtikelOderWarenGruppeLabel().setFont(allesandere);
			((ListenPosten)value).getRechnungspostenGebindeinhaltLabel().setFont(allesandere);
			((ListenPosten)value).getRechnungspostenGebindeLabel().setFont(allesandere);
			((ListenPosten)value).getRechnungspostenAnzahlLabel().setFont(allesandere);
			((ListenPosten)value).getRechnungspostenPreisLabel().setFont(allesandere);
			((ListenPosten)value).getRechnungspostenGesamtPreisLabel().setFont(allesandere);
			((ListenPosten)value).getRechnungspostenRabattLabel().setFont(allesandere);
//			dem Rechnungsosten nacheinander Artikelbezeichnung oder Warengruppe
			rechnungspostenpane.add(((ListenPosten)value).getRechnungspostenArtikelOderWarenGruppeLabel());
			
//			GebindeInhalt
			rechnungspostenpane.add(((ListenPosten)value).getRechnungspostenGebindeinhaltLabel());
			
//			Gebinde
			rechnungspostenpane.add(((ListenPosten)value).getRechnungspostenGebindeLabel());
			
//			Anzahl
			rechnungspostenpane.add(((ListenPosten)value).getRechnungspostenAnzahlLabel());
			
//			Preis
			rechnungspostenpane.add(((ListenPosten)value).getRechnungspostenPreisLabel());
			
//			Gesamtpreis
			rechnungspostenpane.add(((ListenPosten)value).getRechnungspostenGesamtPreisLabel());
			
//			und vielleicht später noch Rabatt hinzufügen
			rechnungspostenpane.add(((ListenPosten)value).getRechnungspostenRabattLabel());
			
//			Hintergrundfarbe des JPanels bei Fokuswechseln definieren
			if(istSelektiert) rechnungspostenpane.setBackground(Color.lightGray); // Hat den Fokus
			else setBackground(artikelEingescanntlistModel.getBackground()); // Hat den Fokus nicht
			
			return this;
		}
	}

Was mache ich falsch? :cry: ???:L
Bitte um Hilfe Ich glaub ich hab nen Brett vorm Kopf. :?:

 

[André Uhres]

..Was mache ich falsch?..

Du darfst in getListCellRendererComponent keine neuen Labels hinzufügen.
Das Hinzufügen der Labels geschieht im Konstruktor:

package list;
/*
 * ListenDemo.java
 */
import java.awt.*;
import javax.swing.*;
public class ListenDemo extends JFrame {
    public ListenDemo() {
        super("ListenDemo");
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(600,300);
        setLocationRelativeTo(null);
        JList list = new JList();
        list.setCellRenderer(new MyCellRenderer());
        DefaultListModel model = new DefaultListModel();
        model.addElement(new ListenPosten("a1", "a1", "a1", "a1", "a1", "a1", "a1", 0));
        model.addElement(new ListenPosten("b1", "b1", "b1", "b1", "b1", "b1", "b1", 0));
        model.addElement(new ListenPosten("c1", "c1", "c1", "c1", "c1", "c1", "c1", 0));
        list.setModel(model);
        add(new JScrollPane(list));
    }
    public static void main(final String args[]) {new ListenDemo().setVisible(true);}
}
class ListenPosten {
    final private JLabel rechnungspostenArtikelOderWarenGruppeLabel;
    final private JLabel rechnungspostenGebindeinhaltLabel;
    final private JLabel rechnungspostenGebindeLabel;
    final private JLabel rechnungspostenAnzahlLabel;
    final private JLabel rechnungspostenPreisLabel;
    final private JLabel rechnungspostenGesamtPreisLabel;
    final private JLabel rechnungspostenRabattLabel;
    public ListenPosten(final String rechnungspostenArtikelOderWarenGruppeString,
            final String rechnungspostenGebindeinhaltString, 
            final String rechnungspostenGebindeString,
            final String rechnungspostenAnzahlString,   
            final String rechnungspostenPreisString,
            final String rechnungspostenGesamtPreisString, 
            final String rechnungspostenRabattPreisString, 
            final int stelle) {
        rechnungspostenArtikelOderWarenGruppeLabel = new JLabel(rechnungspostenArtikelOderWarenGruppeString);
        rechnungspostenGebindeinhaltLabel = new JLabel(rechnungspostenGebindeinhaltString);
        rechnungspostenGebindeLabel = new JLabel(rechnungspostenGebindeString);
        rechnungspostenAnzahlLabel = new JLabel(rechnungspostenAnzahlString);
        rechnungspostenPreisLabel = new JLabel(rechnungspostenPreisString);
        rechnungspostenGesamtPreisLabel = new JLabel(rechnungspostenGesamtPreisString);
        rechnungspostenRabattLabel = new JLabel(rechnungspostenRabattPreisString);
    }
    public  JLabel getArtikelOderWarenGruppeLabel() {
        return rechnungspostenArtikelOderWarenGruppeLabel;
    }
    public  JLabel getGebindeinhaltLabel() {
        return rechnungspostenGebindeinhaltLabel;
    }
    public  JLabel getGebindeLabel() {
        return rechnungspostenGebindeLabel;
    }
    public  JLabel getAnzahlLabel() {
        return rechnungspostenAnzahlLabel;
    }
    public  JLabel getPreisLabel() {
        return rechnungspostenPreisLabel;
    }
    public  JLabel getGesamtPreisLabel() {
        return rechnungspostenGesamtPreisLabel;
    }
    public  JLabel getRabattLabel() {
        return rechnungspostenRabattLabel;
    }
}
class MyCellRenderer extends JPanel implements ListCellRenderer {
    final private static long serialVersionUID = 8400636002225624699L;
    final private int feldhoehe = 8;
    final private Font allesandere = Font.decode("Arial-BOLD-11");
    final private JLabel artikel,gebindeinhalt,gebinde,anzahl,preis,gesamtPreis,rabatt;
    public MyCellRenderer() {
        // Konstruktor des JPanels mit FlowLayout aufrufen
        super(new FlowLayout(FlowLayout.LEFT));
        // JPanel undurchsichtig machen
        setOpaque(true);
        artikel = new JLabel();
        gebindeinhalt = new JLabel();
        gebinde = new JLabel();
        anzahl = new JLabel();
        preis = new JLabel();
        gesamtPreis = new JLabel();
        rabatt = new JLabel();
        Dimension dimie = new Dimension(80, feldhoehe);
        Dimension dimieklein = new Dimension(70, feldhoehe);
        Dimension dimiemittel = new Dimension(90, feldhoehe);
        Dimension dimiegroß = new Dimension(280, feldhoehe);
        artikel.setPreferredSize(dimiegroß);
        gebindeinhalt.setPreferredSize(dimieklein);
        gebinde.setPreferredSize(dimie);
        anzahl.setPreferredSize(dimie);
        preis.setPreferredSize(dimie);
        gesamtPreis.setPreferredSize(dimiemittel);
        rabatt.setPreferredSize(dimiemittel);
        artikel.setFont(allesandere);
        gebindeinhalt.setFont(allesandere);
        gebinde.setFont(allesandere);
        anzahl.setFont(allesandere);
        preis.setFont(allesandere);
        gesamtPreis.setFont(allesandere);
        rabatt.setFont(allesandere);
        add(artikel);
        add(gebindeinhalt);
        add(gebinde);
        add(anzahl);
        add(preis);
        add(gesamtPreis);
        add(rabatt);
    }
    public Component getListCellRendererComponent(final JList artikelEingescanntlistModel, // JList Objekt
            final Object value, // anzuzeigende Komponente
            final int index,    // Zellenindex
            final boolean istSelektiert,  // Ist selektiert?
            final boolean hatFokus)  // Hat den Fokus?
    {
        artikel.setText(((ListenPosten)value).getArtikelOderWarenGruppeLabel().getText());
        gebindeinhalt.setText(((ListenPosten)value).getGebindeinhaltLabel().getText());
        gebinde.setText(((ListenPosten)value).getGebindeLabel().getText());
        anzahl.setText(((ListenPosten)value).getAnzahlLabel().getText());
        preis.setText(((ListenPosten)value).getPreisLabel().getText());
        gesamtPreis.setText(((ListenPosten)value).getGesamtPreisLabel().getText());
        rabatt.setText(((ListenPosten)value).getRabattLabel().getText());
        if(istSelektiert) //         Hintergrundfarbe des JPanels bei Fokuswechseln definieren
            setBackground(Color.lightGray); // Hat den Fokus
        else setBackground(artikelEingescanntlistModel.getBackground()); // Hat den Fokus nicht
        return this;
    }
}

 

[dreadnought]

Danke!
Es geht.
Super Hey das Forum kann ich weiterempfehlen.
So schnelle Hilfe.