Vector in JList

[RW_NORD]

Hallo,

Ich bin neu hier im Forum und auch Änfänger im Programmieren mit Java.
Bin gerade dabei ein Projekt zu bearbeiten und komme nicht weiter.
Und zwar möchte ich gern eine JList mit einem Vector füllen, welcher ich mit verschiedenen Personen aus der DB gefüllt wird. Der Vector wird in der Klasse ClientInterface gefüllt und soll dann der JList in meiner GUI übergben werden. Zwar stehen die Personen in dem Vector drin, werden aber in der JList nicht untereinander angeordnet, sondern fortlaufend angezeigt. Komme leider nicht auf die Lösung wie ich das hinbekomme, oder ist vlt sogar die Auswahl einer Jlist für einen Vector unvorteilhaft. Ich psote mal den Code, in welchem mei Vector gefüllt wird.
Danke im voraus für eure Zeit und Antworten

    /**
     * Fülle eine JList mit allen Personen aus der Datenbank
     */
    public Vector<Vector> fuellJList_Person(Vector listData) {

        //Wenn bisher keine Verbindung hergstellt wurde, wird eine aufgebaut
        if (this.verwaltung == null) {
            this.initServerConnection();
        }

        try {

            Vector<Person> person = verwaltung.getAll_Person();
            Vector inhalte = new Vector();

                //Hier wird dem Vector 'inhalte' das Objekt Person mit den Elementen Titel, Vorname und Name
		//übergeben
		for ( Person c : person ) {
                    if (c != null ) {
                        inhalte.add(c.get_person_id()+ ": " + c.get_vorname() + " " + c.get_nachname());
                    }
                }
                
            listData.add(inhalte);

        } catch (RemoteException re) {
            System.out.println("RemoteException");
            System.out.println(re);
        }
        return listData;
    }

 

[freez]

An dem Beispielcode ist jetzt erst mal nix Aussergewöhnliches zu erkennen. Wie füllst du deine JList (Code)?

[EDIT]Nachtrag: du füllst in dein listData mit einem Vector namens Inhalt und gibst diesen zurück? Vielleicht postest du mal den Code, der diese Methode aufruft. Das sieht jetzt tatsächlich etwas merkwürdig aus. Vor allem, warum übergibst du den Vector listData der Methode und lieferst es wieder als return Wert zurück?[/EDIT]

 

[RW_NORD]

Wow das ging jetzt schnell...
Ich habe das in der Methode so gemacht weil ich ohne der Methode den Vector listData zu übergeben sonst bei dem return Wert ein Fehler kam. Hatte deshalb bisschen im Internet gestöbert und bin dann auf diese Schreibweise gekommen.
Der Code in dem ich die Methode aufruf und die JList füll ist dieser (kurzgefasst). Bitte nicht zu streng mit mir sein, wenn es nicht sauber progarmmiert ist.

Vector listData = new Vector();
DefaultListModel dlm = new DefaultListModel();
listTeilnehmer = new JList(dlm);
dlm.addElement(ci.fuellJList_Person(listData));

 

[Timothy Truckle]

listData.add(inhalte);

Du fügst hier den selben Vector

inhalte

in den anderen Vector

listData

ein.
Wenn Du dann den Vector

listData

an die JList übergibst ruft dies auf jedem Eintrag in

listData

toString()

auf. Die Einträge in

listData

sind aber viele Referenzen auf die selbe Instanz von

inhalte

.

Warum gibst Du eigentlich nicht direkt den Vector

person

zurück?
Die einzige Änderung, die man machen müßte wäre,

toString()

in

Person

zu implementieren.

bye
TT

 

[freez]

/**
     * Fülle eine JList mit allen Personen aus der Datenbank
     */
    public Vector<String> fuellJList_Person() {
             Vector inhalte = new Vector();
        //Wenn bisher keine Verbindung hergstellt wurde, wird eine aufgebaut
        if (this.verwaltung == null) {
            this.initServerConnection();
        }
 
        try {
 
            Vector<Person> person = verwaltung.getAll_Person();

 
                //Hier wird dem Vector 'inhalte' das Objekt Person mit den Elementen Titel, Vorname und Name
        //übergeben
        for ( Person c : person ) {
                    if (c != null ) {
                        inhalte.add(c.get_person_id()+ ": " + c.get_vorname() + " " + c.get_nachname());
                    }
                }
 
        } catch (RemoteException re) {
            System.out.println("RemoteException");
            System.out.println(re);
        }
        return inhalte;
    }

Ganz sponatn würde ich sagen, dass es so richtiger wäre (ungetestet).

 

[RW_NORD]

Erstmal Danke Leute. Mit soviel Feedback habe ich gar nicht gerechnet, bin deswegen auch ein bisschen überfordert alles in mich aufzunhemen.
@ freez: Dein Code funktionert, nur leider bekomme ich die selbe Ausgabe in meiner JList wie vorher, also, [1: ......, 2: .......] in einer Zeile
@ Timothy Truckle: Auch Danke für deine Antwort. wenn ich direkt den Vector person zurückgeben will kommt leider ein Fehler. Wie genau meinst du das mit dem toString(), bin da gerade bisschen überfordert.

 

[freez]

Na klar :idea:

die Methode addElement() musst du pro Eintrag einmal aufrufen. Also so:

	DefaultListModel<String> dlm = new DefaultListModel<String>();
		for (String s : fuellJList_Person()) {
			dlm.addElement(s);
		}
		JList<String> list = new JList<String>(dlm);

Und hier mal meine vereinfachte Testversion der Methode fuellJList_Person:

public Vector<String> fuellJList_Person() {
		Vector<String> inhalte = new Vector<String>();

		String[] s = { "Paul", "Hans", "Emil", "Helmut", "John", "Emily", "Johann", "Blub" };
		for (int i = 0; i < s.length; i++)
			inhalte.add(i + ": " + s[i]);

		return inhalte;
	}

 

[freez]

Noch einfacher sollte es so gehen:

	...
	JList<String> list = createJList_Person();

         ...

	public JList<String> createJList_Person() {
		DefaultListModel<String> dlm = new DefaultListModel<String>();
		String[] s = { "Paul", "Hans", "Emil", "Helmut", "John", "Emily", "Johann", "Blub" };
		for (int i = 0; i < s.length; i++)
			dlm.addElement(i + ": " + s[i]);

		JList<String> list = new JList<String>(dlm);
		return list;

	}

 

[Timothy Truckle]

@ Timothy Truckle: Auch Danke für deine Antwort. wenn ich direkt den Vector person zurückgeben will kommt leider ein Fehler.

und welcher genau?

Wie genau meinst du das mit dem toString(), bin da gerade bisschen überfordert.

Let me google that for you

bye
TT

 

[RW_NORD]

Also mir zeigt es die "return person" Zeile rot an (Cannot find symbol person variable).
Sorry dumm von mir nicht einfach kurz zu googlen:autsch:

@Freez versuch gerade deinen Code zu implementieren, bekomme es leider irgendwie nicht hin. Sonst hätte ich mich schon längst mal bedankt bei dir, Sorry. Aber ich versuch es weiter.:toll:

 

[Timothy Truckle]

Also mir zeigt es die "return person" Zeile rot an (Cannot find symbol person variable).
Sorry dumm von mir nicht einfach kurz zu googlen:autsch:

Klar, die Variable

person

existiert außerhalb des try/catch-Blocks nicht.
Lösung:
das

return

vor das

catch

ziehen und im

catch

-Block als letztes eine RuntimeException werfen. Wenn Du ein

RemoteException

bekommst ist sicher sowieso kein vernünftiges Weiterarbeiten möglich...


bye
TT

 

[RW_NORD]

@ Timothy Truckle: Ja klingt logisch, Anfängerfehler:rtfm:

@freez: Also habe es geschafft deinen Code zu implementieren. Was soll ich sagen , es hat lang gedauert aber............................FUNKTIONIERT!!!!!!!! :toll::toll::toll::toll:

Danke an beide dass ihr euch Zeit genommen habt. Echt super von euch

Grüße NORD