JComboBox bug ?!

[weljo]

irgendwie komm ich aus der forschleife nicht mehr raus wenn ich die methode addItem verwende. aber wiso ???

[JAVA=42]
Vector<TestObject> content = new Vector<TestObject>();
content.add(new TestObject("a"));
content.add(new TestObject("b"));
content.add(new TestObject("c"));
content.add(new TestObject("d"));
content.add(new TestObject("e"));
content.add(new TestObject("f"));
JComboBox list = new JComboBox();

System.out.println(content.size());
for(int i=0; i <= content.size(); i++)
{
System.out.println("wtf "+i);
list.addItem(content.elementAt(i));
}
[/code]

 

[eRaaaa]

Was heißt du kommst nicht mehr raus? Bekommst du nicht eher eine ArrayIndexOutOfBoundsException ?

for(int i=0; i < content.size(); i++)

 

[Foermchen82]

An der for-schleife kann ich erstmal nichts verkehrtes finden, aber du kannst den Vektor doch direkt im Konstuktor der ComboBox mitgeben. Wieso machst du das nicht?

 

[weljo]

1.ich weis dass ich direkt mitgeben kann
2. keine ArrrayIndexOutOfBounds exception

ich hab nicht wirklich ein problem damit bin nur zufellig drauf gestoßen und i finds sehr sehr komisch dass i gegen unendlich geht und die schleife nicht verlässt

 

[eRaaaa]

Kann ich nicht nachvollziehen...dein KSKB(mal abgesehen vom TestObject welche sich mal durch String ersetzt habe) endet bei mir wie schon angesprochen in einer Exception....???:L

(wird vllt bei TestObject irgendwie der Vector verändert? So dass sich content.size() halt verändert ? /eidt. obwohl ist ja auch quatsch...)

 

[weljo]

habs jetzt einmal mit anderer syntax probiert, welche eigentlich das selbe machen soll und bekomm eine exception

		for(String s:content)
		{
			list.addItem(s);		
		}

Exception in thread "main" java.util.ConcurrentModificationException

@eRaaaa ist mit String genau das selbe ... lauf einfach ins unendliche

 

[Gast2]

Mach mal ein Kskb...

 

[Meru]

Exception in thread "main" java.util.ConcurrentModificationException

Das lässt ja schon darauf schließen, dass du einerseits die Liste durchlaufen willst, aber andererseits die Liste zur GLEICHEN Zeit bearbeitet wird.

Wie bereits angesprochen, ein Kskb

 

[Ebenius]

Hängt ein Listener an der ComboBox oder dem Modell, dass den Inhalt des selektierten Elements der Collection hinzufügt. Sowas kann schnell passieren, wenn man solche History-ComboBoxes baut (die sich merken, was der Benutzer zu letzt eingegeben hat).

Ansonsten: StackTrace ausgeben und gucken wo die Veränderung her kommt.

Ebenius

 

[weljo]

das Kskb ist zwar bereits oben jedoch werd ich es erneut posten.
[JAVA=42]
Vector<String> content = new Vector<String>();
content.add(new String("a"));
content.add(new String("b"));
content.add(new String("c"));
content.add(new String("d"));
content.add(new String("e"));
content.add(new String("f"));
JComboBox list = new JComboBox(new DefaultComboBoxModel(content));

System.out.println(content.size());
for(int i=0; i < content.size(); i++)
{
System.out.println("hallo "+i);
list.addItem(content.elementAt(i));
}

[/code]

 

[eRaaaa]

Na das ist jetzt wieder WAS GANZ ANDERES, denn hier setzt du ja den Vector schon als Model der Combobox!
D.h. deine Schleife ist unnötig ! Und was passiert ist, du addest dem Vector bei jedme Durchgang ein Element hinzu - size vergrößert sich, daher Endlosschleife !

 

[Ebenius]

Nebenbei als Hinweis: [c][noparse][java=42]WAS AUCH IMMER[/code][/noparse][/c] solltest Du benutzen, wenn Du Deinen Quelltext ab Zeile 42 numerieren möchtest. Normaler Weise will man das nicht und schreibt nur [c][noparse]

WAS AUCH IMMER

[/noparse][/c]

Ein KSKB ist -- wie der Name sagt -- klein, selbständig und kompilierbar. Dein Beispiel oben ist aber leider nicht selbständig. So könnte Dein KSKB aussehen:

package com.ebenius;

import java.awt.BorderLayout;
import java.util.Vector;

import javax.swing.*;

public class ComboBoxTestGui {

  private static void createAndShowGui() {
    Vector<String> content = new Vector<String>();
    content.add(new String("a"));
    content.add(new String("b"));
    content.add(new String("c"));
    content.add(new String("d"));
    content.add(new String("e"));
    content.add(new String("f"));
    JComboBox list = new JComboBox(new DefaultComboBoxModel(content));

    System.out.println(content.size());
    for (int i = 0; i < content.size(); i++) {
      System.out.println("hallo " + i);
      list.addItem(content.elementAt(i));
    }

    final JPanel contentPane = new JPanel(new BorderLayout(6, 6));
    contentPane.add(list);

    final JFrame f = new JFrame("Test Frame: ComboBoxTestGui");
    f.setContentPane(contentPane);
    f.pack();
    f.setLocationRelativeTo(null);
    f.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
    f.setVisible(true);
  }

  public static void main(String[] args) {
    SwingUtilities.invokeLater(new Runnable() {

      public void run() {
        createAndShowGui();
      }
    });
  }
}

Das eigentliche Problem: Du erstellst ein DefaultComboBoxModel, das seine Daten bereits aus dem Vector beziehst, den Du übergeben hast. Die Schleife ab Zeile 21 führt dazu, dass immer ein Element dem Modell hinzugefügt wird, welches es in den Vector hineinschreibt, damit wird [c]size()[/c] des Vectors natürlich größer und die Geschichte endet niemals. Die ganze Schleife weglassen, dann sind alle Elemente schon enthalten. Oder das Argument aus dem [c]new DefaultComboBoxModel()[/c] weglassen und die Einträge mit der Schleife hinzufügen.

Ebenius

 

[weljo]

ok thx für den tipp hab jedoch im bsp auch noch das command list.removeAll(); vergessen, welches mir die sache eigentlich unerklährlich darstellen ließ. jetzt ist es mir klar wiso. müsste also einfach content.size() zuvor in einem integer schreiben der während des durchlaufs nicht verändert wird

 

[eRaaaa]

Also ich kann dir nicht mehr folgen
Was hast du dann erreicht? Du bekommst dann zwar keine Endlosschleife mehr, hast aber die Werte doppelt im Modell!? Bzw. was soll jetzt auf einmal list.removeAll?

Vllt erklärst du einfach mal was du überhaupt machen willst ^^

 

[Gast2]

Also ich kann dir nicht mehr folgen
Was hast du dann erreicht? Du bekommst dann zwar keine Endlosschleife mehr, hast aber die Werte doppelt im Modell!? Bzw. was soll jetzt auf einmal list.removeAll?

Vllt erklärst du einfach mal was du überhaupt machen willst ^^

nee du ich blicks auch net so ganz was er vor hat ist irgendwie realitätsfremd ^^...