TreeSet Element löschen

[Enigma228]

Sinn des ganzen ist es in einer TreeSet alle Personen zu halten die in einer Tabelle dargestellt werden sollen.
Nun kommt aber der Punkt, wo ich Einträge aus der Tabelle löschen möchte.. Spalte 0 beinhaltet den Index..
das heisst ich wollte die entsprechende Person (Vergleich über den Index) aus der TreeSet löschen und die TreeSet an den eine Spezielle Funktion im Model übergeben, so das die Tabelle neu dargestellt wird.

Im Quellcode möchte ich eine Person aus dem TreeSet löschen..
der Abgleich mit den Index ist ok, er kommt auch zu dem Punkt an dem er die Person in der Treeset löschen sollte, aber er tuts nicht..
hat jemand einen Idee?

for(Person p:myPersons){
                                        // Vergleich Index Person mit Tabelleneintrag
					System.out.println(""+p.getIndex()+" = "+
							rtm_listeningresults.getValueAt(t_listeningresults.getSelectedRow(), 0).toString());
					if(p.getIndex().equalsIgnoreCase(rtm_listeningresults.getValueAt(t_listeningresults.getSelectedRow(), 0).toString())){
						dp.deleteEntry(rtm_listeningresults.getValueAt(t_listeningresults.getSelectedRow(), 0).toString());
						System.out.println("Size vor: "+myPersons.size());
						myPersons.remove(p);
						System.out.println("Size nach: "+myPersons.size());
						break;
					}
				}

Ergebnis
1247396414900 = 1247396414900
Size vor: 1
Size nach: 1

 

[Marco13]

Die Operationen im TreeSet verwenden die Vergleichsfunktionen (compareTo aus dem Comparable interface, und AFAIK equals von Object). Wenn die nicht passend zur gewünschten Sortierung implementiert sind, funktioniert das ganze nicht.

 

[Enigma228]

package doc;

import java.util.ArrayList;

public class Person implements Comparable<Person>{
	
	String index;
	String name, firstname, street, streetnumber, city, postcode;
	ArrayList<Msn> numbers= new ArrayList<Msn>();
	ArrayList<Mail> mails= new ArrayList<Mail>();
	
	public Person() {
		
	}
		
	public Person(String index, String name, String firstname, String street,
			String streetnumber, String city, String postcode,
			ArrayList<Msn> numbers, ArrayList<Mail> mails) {
		super();
		this.index = index;
		this.name = name;
		this.firstname = firstname;
		this.street = street;
		this.streetnumber = streetnumber;
		this.city = city;
		this.postcode = postcode;
		this.numbers = numbers;
		this.mails = mails;
	}
	public String getName() {
		return name;
	}
	public void setName(String name) {
		this.name = name;
	}
	public String getFirstname() {
		return firstname;
	}
	public void setFirstname(String firstnam) {
		this.firstname = firstnam;
	}
	public String getStreet() {
		return street;
	}
	public void setStreet(String street) {
		this.street = street;
	}
	public String getCity() {
		return city;
	}
	public void setCity(String city) {
		this.city = city;
	}
	public String getPostcode() {
		return postcode;
	}
	public void setPostcode(String postcode) {
		this.postcode = postcode;
	}
	public ArrayList<Msn> getNumbers() {
		return numbers;
	}
	public void setNumbers(ArrayList<Msn> arrayList) {
		this.numbers = arrayList;
	}
	public ArrayList<Mail> getMails() {
		return mails;
	}
	public void setMails(ArrayList<Mail> mails) {
		this.mails = mails;
	}
	
	public String getStreetnumber() {
		return streetnumber;
	}

	public void setStreetnumber(String streetnumber) {
		this.streetnumber = streetnumber;
	}


	public String getPerson(){
		String temp = new String();
		temp = getName()+", "+getFirstname()+"\n"+
			getStreet()+" "+getStreetnumber()+"\n"+
			getPostcode()+" "+getCity()+"\n";
		int n = numbers.size();
		for (int i=0;i<n;i++){
			temp = temp + numbers.get(i).getType()+" : "+numbers.get(i).getPrefix()+" "+
				numbers.get(i).getCallnumber()+" ("+numbers.get(i).getUsing()+")"+"\n";
		}
		int m = mails.size();
		for (int i=0;i<m;i++){
			temp = temp + mails.get(i).getMail()+"\n";
		}
		
		
		return temp;
	}

	public String getIndex() {
		return index;
	}

	public void setIndex(String index) {
		this.index = index;
	}

	@Override
	public int compareTo(Person p) {
		if (name.compareToIgnoreCase(p.getName())==0){
			if(firstname.compareToIgnoreCase(p.getFirstname())==0){
				if(street.compareToIgnoreCase(p.getStreet())==0){
					if(streetnumber.compareToIgnoreCase(p.getStreetnumber())==0){
						if(city.compareToIgnoreCase(p.getCity())==0){
							return 1;
						}else{return city.compareToIgnoreCase(p.getCity());}
					}else{return streetnumber.compareToIgnoreCase(p.getStreetnumber());}
				}else{return street.compareToIgnoreCase(p.getStreet());}
			}else{return firstname.compareToIgnoreCase(p.getFirstname());}
		}else{return name.compareToIgnoreCase(p.getName());}
	}
	
	

}

 

[Marco13]

Beim ersten Blick auf compareTo: "Wenn alles gleich ist (also alle unter-Vergleiche 0 zurückgeben) dann behaupte, dass die Personen NICHT gleich sind (also gib 1 zurück)".

Wenn das nicht hilft, muss ich wohl doch mal schauen, ob das compilierbar ist...

 

[Enigma228]

die Treeset sortiert ja auch richtig, wenn ich einen neuen Eintrag mache. Also spielt das Comparable keine Rolle oder..
das Interface sorgt doch nur dafür das Objekte vergleichbar sind zum sortieren..

und er hat ja auch die richtige Person zum löschen gefunden.. bloss remove funkt. nicht..

 

[Marco13]

Zwei gleiche Personen und zwei identische Pesonen in die TreeSet eingefügt, ergibt eine TreeSet mit Größe 3. Sollte aber Größe 1 sein.

class PersonTest
{
    public static void main(String args[])
    {
        Person p0 = new Person("0", "Name", "FirstName", "Street", "No. 123", "City", 
            "12345", new ArrayList<Msn>(), new ArrayList<Mail>());
        Person p1 = new Person("0", "Name", "FirstName", "Street", "No. 123", "City",
            "12345", new ArrayList<Msn>(), new ArrayList<Mail>());
        Person p2 = p0;

        Set<Person> set = new TreeSet<Person>();

        set.add(p0);
        set.add(p1);
        set.add(p2);

        System.out.println(set);
    }

}

Die Methode "compareTo" stellt SEHR strenge Anforderungen an die Art, wie sie implementiert ist: Comparable (Java Platform SE 6) - und diese Anforderungen sind bei deiner Implementierung nicht erfüllt. (<-Punkt).

Wenn sie erfüllt wären, würde die TreeSet auch funktionieren (wobei ich glaube, dass man "equals" noch richtig überschreiben müßte (und damit auch hashCode, auch wenn das hier im Moment nicht benutzt wird)).

 

[Enigma228]

hast du ne Idee wie ich Equals überschreiben müsste in der PersonKlasse?

 

[Enigma228]

Trotzdem wird der Eintrag in der TreeSet nicht gelöscht

package doc;

import java.util.ArrayList;

public class Person implements Comparable<Person>{
	
	String index;
	String name, firstname, street, streetnumber, city, postcode;
	ArrayList<Msn> numbers= new ArrayList<Msn>();
	ArrayList<Mail> mails= new ArrayList<Mail>();
	
	public Person() {
		
	}
		
	public Person(String index, String name, String firstname, String street,
			String streetnumber, String city, String postcode,
			ArrayList<Msn> numbers, ArrayList<Mail> mails) {
		super();
		this.index = index;
		this.name = name;
		this.firstname = firstname;
		this.street = street;
		this.streetnumber = streetnumber;
		this.city = city;
		this.postcode = postcode;
		this.numbers = numbers;
		this.mails = mails;
	}
	public String getName() {
		return name;
	}
	public void setName(String name) {
		this.name = name;
	}
	public String getFirstname() {
		return firstname;
	}
	public void setFirstname(String firstnam) {
		this.firstname = firstnam;
	}
	public String getStreet() {
		return street;
	}
	public void setStreet(String street) {
		this.street = street;
	}
	public String getCity() {
		return city;
	}
	public void setCity(String city) {
		this.city = city;
	}
	public String getPostcode() {
		return postcode;
	}
	public void setPostcode(String postcode) {
		this.postcode = postcode;
	}
	public ArrayList<Msn> getNumbers() {
		return numbers;
	}
	public void setNumbers(ArrayList<Msn> arrayList) {
		this.numbers = arrayList;
	}
	public ArrayList<Mail> getMails() {
		return mails;
	}
	public void setMails(ArrayList<Mail> mails) {
		this.mails = mails;
	}
	
	public String getStreetnumber() {
		return streetnumber;
	}

	public void setStreetnumber(String streetnumber) {
		this.streetnumber = streetnumber;
	}


	public String getPerson(){
		String temp = new String();
		temp = getName()+", "+getFirstname()+"\n"+
			getStreet()+" "+getStreetnumber()+"\n"+
			getPostcode()+" "+getCity()+"\n";
		int n = numbers.size();
		for (int i=0;i<n;i++){
			temp = temp + numbers.get(i).getType()+" : "+numbers.get(i).getPrefix()+" "+
				numbers.get(i).getCallnumber()+" ("+numbers.get(i).getUsing()+")"+"\n";
		}
		int m = mails.size();
		for (int i=0;i<m;i++){
			temp = temp + mails.get(i).getMail()+"\n";
		}
		
		
		return temp;
	}

	public String getIndex() {
		return index;
	}

	public void setIndex(String index) {
		this.index = index;
	}

	@Override
	public int compareTo(Person p) {
		if (name.compareToIgnoreCase(p.getName())==0){
			if(firstname.compareToIgnoreCase(p.getFirstname())==0){
				if(street.compareToIgnoreCase(p.getStreet())==0){
					if(streetnumber.compareToIgnoreCase(p.getStreetnumber())==0){
						if(city.compareToIgnoreCase(p.getCity())==0){
							return 1;
						}else{return city.compareToIgnoreCase(p.getCity());}
					}else{return streetnumber.compareToIgnoreCase(p.getStreetnumber());}
				}else{return street.compareToIgnoreCase(p.getStreet());}
			}else{return firstname.compareToIgnoreCase(p.getFirstname());}
		}else{return name.compareToIgnoreCase(p.getName());}
	}

	@Override
	public boolean equals(Object obj) {
		Person p = (Person)obj;
		return getIndex().equals(p.getIndex());
	}

	@Override
	public int hashCode() {
		return index.hashCode();
	}
}

 

[Marco13]

Compareto ist noch genauso wie vorher. Gib' da mal nicht 1 sondern 0 zurück, vielleicht geht es dann schon, wenn nicht schau' ich noch mal.

 

[Enigma228]

Das wars..