JList Objekte richtig anzeigen in JDK 6

[gardiac2002]

Hallo,

ich versuche gerade eine JList zu bauen. Mein Problem, wir arbeiten mit JDK 6 und die Person-Objekte werden in Objects umgewandelt. JList ruft Object#toString auf statt die toString-Methode von den Person-Objekten.

Hier mein Code:

import javax.swing.JList;
import javax.swing.DefaultListModel;
import javax.swing.JPanel;

...

Vector<Person> personList;
JPanel west;
DefaultListModel listModel;
JList swingPersonList;

...

west = new JPanel();

personList = new Vector<Person>(); 
personList.add(new Person("Anna"));
personList.add(new Person("Franz"));
...

listModel = new DefaultListModel();

for(Person p:personList){
  listModel.add(p);
}
		
swingPersonList = new JList(listModel);
west.add(swingPersonList);

...

Die toString()-Methode habe ich in Person überschrieben. Bringt mir aber nichts. Was kann ich da machen?

Lg,
philipp

 

[Gast2]

JList ruft Object#toString auf statt die toString-Methode von den Person-Objekten.

Das möchte ich mal anzweifeln

Zeig doch bitte mal die Klasse Person.

 

[gardiac2002]

package jrunner;

import java.io.File;
import java.io.FileNotFoundException;
import java.io.IOException;
import java.io.PrintWriter;
import java.io.Serializable;
import java.util.ArrayList;
import java.util.Observable;
import java.util.Observer;


public class Person extends JRunnerObject implements Observer, Cloneable, Serializable {
	
	private static final long serialVersionUID = 1L;
	private String name;
	private int age;
	private int height;
	private double weight;
	private boolean female;
	private boolean male;
	
	private ExportPersonData export;
	
	{
		name = "";
		age = 0;
		height = 0;
		weight = 0.0;
		female = false;
		male = false;
	}
	//Constructor is private. It is not possible
	//to create Person instances. Only through the 
	//factory method.
	private Person(ExportPersonData e){
		export = new ExportPersonObject();
	}
	
	 private void writeObject(java.io.ObjectOutputStream out)throws IOException{
		 
	 }
	 private void readObject(java.io.ObjectInputStream in)throws IOException, ClassNotFoundException{
		 
	 }
	
	//Setters and Getters for all fields.
	public String getName(){
		return this.name;
	}
	public int getAge(){
		return this.age;
	}
	public int getHeight(){
		return this.height;
	}
	public double getWeight(){
		return this.weight;
	}
	public boolean getFemale(){
		return this.female;
	}
	public boolean getMale(){
		return this.male;
	}
	public void setName(String n){
		this.name = n;
	}
	public void setAge(int a){
		this.age = a;
	}
	public void setHeight(int h){
		this.height = h;
	}
	public void setWeight(double w){
		this.weight = w;
	}
	public void setFemaleTrue(){
		this.male = false;
		this.female = true;
	}
	public void setMaleTrue(){
		this.female = false;
		this.male = true;
	}
	
	public static Person getPersonInstance(ExportPersonData export){
		return new Person(export);
	}
	@Override
	public String toString(){
		return name;
	}
	
	//The update Method is implemented through java.util.Observer
	//It listens to an Observable class. When the Observer is finished
	//with an action it activates the update method.
	public void update(Observable arg, Object obj) {
		
		PersonGUI in = (PersonGUI) arg;
		
//		Import all person variables from PersonGUI
//		to the Person object.
		this.setName(in.getName());
		this.setAge(in.getAge());
		this.setHeight(in.getHeight());
		this.setWeight(in.getWeight());
		
		if(in.isFemale()){this.setFemaleTrue();}
		else{
			if(in.isMale()){
				this.setMaleTrue();}
			}
		
		Person personData;
		
		try {
			personData = (Person) (this.clone());
			export.exportPersonData(personData);	
		} 
		catch (CloneNotSupportedException e) {e.printStackTrace();}
		

	}
	public static void main(String[] args){
		
		Person p = Person.getPersonInstance(new ExportPersonObject());
		p.setName("Philipp");
		
		System.out.println(p.toString());
		
	}


}

 

[gardiac2002]

Du findest den überschriebenen Code auf Zeile 90 bis 93.

 

[Gast2]

Folgender Code funktionier bei mir wie erwartet:

public class JListDemo {
	public static void main(String[] args) {
		Person p1 = new Person("Hans", "im Glück");
		Person p2 = new Person("Klaus", "Müller");
		
		DefaultListModel dlm = new DefaultListModel();
		dlm.addElement(p1);
		dlm.addElement(p2);
		
		JFrame frame = new JFrame("JList Demo");
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		
		JList list = new JList(dlm);
		list.setPreferredSize(new Dimension(200,300));
		
		frame.add(list);
		frame.pack();
		frame.setVisible(true);
	}
	
	private static class Person {
		private String first;
		private String last;
		
		public Person(String first, String last) {
			this.first = first;
			this.last = last;
		}
		
		@Override
		public String toString() {
			return first + " " + last;
		}
	}
}

Was mir auffällt ist:

for(Person p:personList){
  listModel.add(p);
}

Die Methode add() gibts im DefaultListModel doch eigentlich gar nicht so? Ist das ne eigene Klasse von dir?

 

[gardiac2002]

Mein Fehler beim Übertragen: im Code steht eh listModel.addElement(p);

Das ListDemo funktioniert problemlos!
Ich habe nun versucht in der toString Methode einfach irgendeinen String zurückzugeben. Da wird die Liste angezeigt.
Das Problem liegt anscheinend bei Person oder eigentlich dem Import von Person.

Die Objekte werden erkannt - die dazugehörigen Felder leider nicht. Der Fehler muss wohl in den Methoden

         private void writeObject(java.io.ObjectOutputStream out)throws IOException{
		 
	 }
	 private void readObject(java.io.ObjectInputStream in)throws IOException, ClassNotFoundException{
		 
	 }

sein.