mehrere JComboBoxes

[Nali_karamba]

Hallo

folgende Ausgangs Situation, ich habe ein Object (im Beispiel heisst es "combos") was ein eigenes Model und eine eigene JComboBox besitzt.

in einem JPanel wird nun eine ArrayList von "combos" objecten angelegt diese bekommen alle das gleiche Model (setzen ihr eigenes Model, aber mit diesem gleich).

in dem JPanel wird zusätzlich gesagt das in jeder JComboBox der ArrayList aus Combos ein anderes Element gesetzt werden soll (per setSelectedElement()).

komischerweise wird aber bei allen das gleiche element gesetzt warum !?

hier mein Beispiel.

package justforme;

import java.util.ArrayList;
import javax.swing.*;

public class KB extends JFrame {

    public KB(){

        int[]array=new int[5];
        for(int i=0;i<array.length;i++){
            array[i]=i;
        }

        DefaultComboBoxModel test=new DefaultComboBoxModel();
        for(int i=0;i<5;i++){
            test.addElement(i);
        }
        panel panel=new panel(array,test);
        setContentPane(panel);
        pack();
        setVisible(true);



    }

    public class panel extends JPanel{

        public panel(int[]a,DefaultComboBoxModel b){
            ArrayList<Combos> Combos=new ArrayList();
            //erstellen der neuen Combos
            for(int i=0;i<a.length;i++){
                Combos.add(new Combos(a[i],b));
            }
            //hinzufügen der Combos und setzen des selected Elements
            for(int i=0;i<Combos.size();i++){
                this.add(Combos.get(i).Box);
                Combos.get(i).Box.setSelectedIndex(i);
            }
        }
    }

    public class Combos{
        //Jede Combo hat jetzt ihr eigenes Model
        DefaultComboBoxModel InternModel;
        JComboBox Box;
        
        public Combos(int i,DefaultComboBoxModel Model){
            InternModel=Model;
            Box=new JComboBox(InternModel);

        }
    }

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

 

[SlaterB]

die Selektion von ComboBoxen wird direkt im ComboBoxModel gespeichert, wenn es nur eins gibt dann auch nur genau ein selektiertes Element für alle Boxen,
wenn du gleiche Daten aber unterschiedliche Selektionen willst, dann musst du komplizierter mit verschiedenen Models arbeiten, die alle auf ein gemeinsames im Hintergrund zugreifen (für die Daten, nicht für die Selektion)

im Gegensatz dazu gibt es bei JList/ JTable von vornherein getrennt SelectionModel

 

[Nali_karamba]

Mhh ja ich weiss, nur in meinem Fall bietet sich ne ComboBox an bzw. wäre JList oder JTable eher unangebracht.

meine Idee (die auch funktioniert) ist die folgende. Also man übergibt keine DefaultComboBoxModel sondern nen Object[] und erstellt daraus dann jeweils die meine Models.

Geht das irgendwie noch besser !?

package justforme;

import java.util.ArrayList;
import javax.swing.*;

public class KB extends JFrame {

    public KB(){

        int[]array=new int[5];
        for(int i=0;i<array.length;i++){
            array[i]=i;
        }

        Object[]test =new Object[5];
        for(int i =0;i<test.length;i++){
            test[i]=i;
        }
        panel panel=new panel(array,test);
        setContentPane(panel);
        pack();
        setVisible(true);



    }

    public class panel extends JPanel{

        public panel(int[]a,Object[] b){
            ArrayList<Combos> Combos=new ArrayList();
            //erstellen der neuen Combos
            for(int i=0;i<a.length;i++){
                Combos.add(new Combos(a[i],b));
            }
            //hinzufügen der Combos und setzen des selected Elements
            for(int i=0;i<Combos.size();i++){
                this.add(Combos.get(i).Box);
                Combos.get(i).Box.setSelectedIndex(i);
            }
        }
    }

    public class Combos{
        //Jede Combo hat jetzt ihr eigenes Model
        DefaultComboBoxModel InternModel;
        JComboBox Box;

        public Combos(int i,Object[] Model){
            InternModel=new DefaultComboBoxModel(Model);
            Box=new JComboBox(InternModel);

        }
    }

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

 

[SlaterB]

grundsätzlich dringend wie die Feuerwehr zu empfehlen:
- Klassen groß schreiben,
- Variablen klein schreiben
- und ganz ganz ganz doll vermeiden, Variablen genau wie Klassen zu benennen..
(ArrayList<Combos> Combos=new ArrayList(); )

letzteres wird durch die Trennung Klassen groß/ Variablen klein fast schon vermieden,
dennoch auch weiter bedenklich,
>Person person = ..
mag noch gehen, aber
> ArrayList<Person> person = ..
wäre wieder relativ ärgerlich, besser personList nennen oder so

----

ansonsten ist das Object[] ein normaler Weg

 

[Nali_karamba]

Okay werde ich beherzigen,

noch eine Frage es gibt ja auch die möglichkeit das ganze mit hilfe eines Vectors zumachen (finde ich angenehmer) da sich das Model auch verändern wird.

allerdings wird mir von meinem NetBeans gesagt das die Collection "Vector" veraltet ist,
ist es deswegen ratsam keinen Vector zunehmen !? oder gibt es bereits ersatz dafür !?

bsp. mit Vector (hab die Tipps hier noch nicht berücksichtigt )

package justforme;

import java.util.ArrayList;
import java.util.Vector;
import javax.swing.*;

public class KB extends JFrame {

    public KB(){

        int[]array=new int[5];
        for(int i=0;i<array.length;i++){
            array[i]=i;
        }

        Vector test =new Vector();
        for(int i =0;i<5;i++){
            test.add(i);
        }
        panel panel=new panel(array,test);
        setContentPane(panel);
        pack();
        setVisible(true);



    }

    public class panel extends JPanel{

        public panel(int[]a,Vector b){
            ArrayList<Combos> Combos=new ArrayList();
            //erstellen der neuen Combos
            for(int i=0;i<a.length;i++){
                Combos.add(new Combos(a[i],b));
            }
            System.out.println(b);
            //hinzufügen der Combos und setzen des selected Elements
            for(int i=0;i<Combos.size();i++){
                this.add(Combos.get(i).Box);
                Combos.get(i).Box.setSelectedIndex(i);
            }
        }
    }

    public class Combos{
        //Jede Combo hat jetzt ihr eigenes Model
        DefaultComboBoxModel InternModel;
        JComboBox Box;

        public Combos(int i,Vector Model){
            InternModel=new DefaultComboBoxModel(Model);
            Box=new JComboBox(InternModel);

        }
    }

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

 

[SlaterB]

Vector an sich kann moderner durch ArrayList ersetzt werden welche du ja auch schon verwendest,
nur leider wurden DefaultComboBoxModel & Co. nicht auch angepasst, so dass diese ArrayList leider nicht verstehen,

mit der Warnung leben oder sie notfalls unterdrücken/ausschalten,
oder DefaultComboBoxModel kopieren

 

[Nali_karamba]

das mit dem Kopieren ist genau das was ich will

nur habe ich imo das Problem das das mit dem clonen nicht so klappt *hmpf*

schmeisst mir an den Kopf "clone() has protected access in Object"

package justforme;

import java.util.ArrayList;
import javax.swing.*;

public class KB extends JFrame {

    public KB(){

        int[]array=new int[5];
        for(int i=0;i<array.length;i++){
            array[i]=i;
        }

        DefaultComboBoxModel test=new DefaultComboBoxModel();
        for(int i=0;i<5;i++){
            test.addElement(i);
        }
        panel panel=new panel(array,test);
        setContentPane(panel);
        pack();
        setVisible(true);



    }

    public class panel extends JPanel{

        public panel(int[]a,DefaultComboBoxModel b){
            ArrayList<Combos> Combos=new ArrayList();
            //erstellen der neuen Combos
            for(int i=0;i<a.length;i++){
                Combos.add(new Combos(a[i],b));
            }
            //hinzufügen der Combos und setzen des selected Elements
            for(int i=0;i<Combos.size();i++){
                this.add(Combos.get(i).Box);
                Combos.get(i).Box.setSelectedIndex(i);
            }
        }
    }

    public class Combos implements Cloneable {
        //Jede Combo hat jetzt ihr eigenes Model
        DefaultComboBoxModel InternModel;
        JComboBox Box;

        public Combos(int i,DefaultComboBoxModel Model) throws CloneNotSupportedException{
            
            InternModel=(DefaultComboBoxModel)Model.clone();
            Box=new JComboBox(InternModel);
             

        }
    }

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

Warum !?

 

[SlaterB]

clone() funktioniert nicht ohne weiteres, ich glaube das muss jede Klasse erst neu implementieren, dann auch gleich als public überschreiben,
wieso willst du aber das Model clonen()? der Konstruktor mit den Daten geht doch genauso, da wird schon jedesmal das gleiche Model rauskommen,

mit 'DefaultComboBoxModel kopieren' meinte ich, den Quelltext der Klasse in eine eigene Klasse zu schreiben und dort überall Vector gegen ArrayList auszutauschen,
dann könntest du ArrayList statt Vector verwenden

 

[Nali_karamba]

mhh ich glaube ich verstehe gerade nicht genau wie du es meinst

so geht es ja nicht ....

package justforme;

import java.util.ArrayList;
import java.util.Vector;
import javax.swing.*;

public class KB extends JFrame {

    public KB(){

        int[]array=new int[5];
        for(int i=0;i<array.length;i++){
            array[i]=i;
        }

        ArrayList test =new ArrayList();
        for(int i =0;i<5;i++){
            test.add(i);
        }
        panel panel=new panel(array,test);
        setContentPane(panel);
        pack();
        setVisible(true);
    }

    public class panel extends JPanel{

        public panel(int[]a,ArrayList b){
            ArrayList<combos> Combos=new ArrayList();
            //erstellen der neuen Combos
            for(int i=0;i<a.length;i++){
                Combos.add(new combos(a[i],b));
            }
            System.out.println(b);
            //hinzufügen der Combos und setzen des selected Elements
            for(int i=0;i<Combos.size();i++){
                this.add(Combos.get(i).Box);
                Combos.get(i).Box.setSelectedIndex(i);
            }
        }
    }



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

und

package justforme;

import java.util.ArrayList;
import javax.swing.DefaultComboBoxModel;
import javax.swing.JComboBox;

public class combos{
        //Jede Combo hat jetzt ihr eigenes Model
        DefaultComboBoxModel InternModel;
        JComboBox Box;

        public combos(int i,ArrayList Model){
            InternModel=new DefaultComboBoxModel(Model);
            Box=new JComboBox(InternModel);

        }
    }

 

[SlaterB]

und ich verstehe nicht was du an meinen Erklärungen nicht verstehst,
DefaultComboBoxModel kennt nur Vector, nicht ArrayList

also 1) bleibe bei Vector oder Object[], clone() hat damit nix zu tun

oder 2) verwende nicht DefaultComboBoxModel sondern eine andere Klasse die ArrayList akzeptiert
(z.B. Quellcode von DefaultComboBoxModel kopieren, oder ganz selber schreiben)

 

[Nali_karamba]

Okay jetzt weiss ich was du meinst.
Hab mich trotzdem für was anderes entschieden, ich überliefere einfach immer ne ArrayList und baue erst im Model alles in eine Model um und zwar so--->

public Combos(int i,ArrayList<String> Model){
            InternModel=new DefaultComboBoxModel();
            for(String s:Model){
                InternModel.addElement(s);
            }
            Box=new JComboBox(InternModel);

        }

hier das ganze DIng --->

package justforme;

import java.util.ArrayList;
import java.util.Vector;
import javax.swing.*;

public class KB extends JFrame {

    public KB(){

        int[]array=new int[5];
        for(int i=0;i<array.length;i++){
            array[i]=i;
        }

        ArrayList<String> test =new ArrayList();
        for(int i =0;i<5;i++){
            test.add(i+"");
        }
        panel panel=new panel(array,test);
        setContentPane(panel);
        pack();
        setVisible(true);



    }

    public class panel extends JPanel{

        public panel(int[]a,ArrayList b){
            ArrayList<Combos> Combos=new ArrayList();
            //erstellen der neuen Combos
            for(int i=0;i<a.length;i++){
                Combos.add(new Combos(a[i],b));
            }
            System.out.println(b);
            //hinzufügen der Combos und setzen des selected Elements
            for(int i=0;i<Combos.size();i++){
                this.add(Combos.get(i).Box);
                Combos.get(i).Box.setSelectedIndex(i);
            }
        }
    }

    public class Combos{
        //Jede Combo hat jetzt ihr eigenes Model
        DefaultComboBoxModel InternModel;
        JComboBox Box;

        public Combos(int i,ArrayList<String> Model){
            InternModel=new DefaultComboBoxModel();
            for(String s:Model){
                InternModel.addElement(s);
            }
            Box=new JComboBox(InternModel);

        }
    }

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

Vielen Dank für deine geduldige Hilfe auch wenn ich nicht der einfachste Fragensteller bin.

LG