Problem mit Comparator!

[Tominator]

Hallo!
Ich hab ein kleines (test-)programm geschrieben, was ne Liste enthält, die nach nem Comparator geordnet wird, allerdings geht es nicht....
Hier erstmal der source (sind 3 klassen)
conTEST:

import java.util.Iterator;
import java.util.TreeSet;

public class conTEST {
    @SuppressWarnings("unchecked")
    public static void main(String[] args){
        conEntry e=new conEntry();

        TreeSet s = new TreeSet(new conSort("titel"));
        e.setAll("a", "a", "a", "a", "a", "a", 'n');
        s.add(e);
        e.setAll("b", "b", "b", "b", "b", "b", 'n');
        s.add(e);
        e.setAll("c", "c", "c", "c", "c", "c", 'n');
        s.add(e);
        e.setAll("d", "d", "d", "d", "d", "d", 'n');
        s.add(e);
        e.setAll("e", "e", "e", "e", "e", "e", 'n');
        s.add(e);
        e.setAll("f", "f", "f", "f", "f", "f", 'n');
        s.add(e);

        Iterator it = s.iterator();
        while (it.hasNext()) {
          System.out.println(((conEntry)it.next()).getTitel());
        }
    }
}

conEntry:

public class conEntry {
    int num = 0;

    String fpath = "";

    String titel = "";

    String dauer = "";

    String interpret = "";

    String album = "";

    String genre = "";

    char mark = 'n';

    conEntry(int unum, String ufpath, String utitel, String udauer,
            String uinterpret, String ualbum, String ugenre) {
        num = unum;
        fpath = ufpath;
        titel = utitel;
        dauer = utitel;
        interpret = uinterpret;
        album = ualbum;
        genre = ugenre;
        mark = 'n';
    }
    conEntry() {

    }

    public String getOrder(String order) {
        String temp="";
        if (order.equals("titel"))
            temp=titel;
        if (order.equals("dauer"))
            temp=dauer;
        if (order.equals("interpret"))
            temp=interpret;
        if (order.equals("album"))
            temp=album;
        if (order.equals("genre"))
            temp=genre;
        if (order.equals("fpath"))
            temp=fpath;
        return temp;
    }
    
    public String getTitel() {
        return titel;
    }

    public String getDauer() {
        return dauer;
    }

    public String getInterpret() {
        return interpret;
    }

    public String getAlbum() {
        return album;
    }

    public String getMusikrichtung() {
        return genre;
    }

    public String getFpath() {
        return fpath;
    }

    public boolean getMarking() {
        if (mark == 'm' || mark == 'b') {
            return true;
        } else {
            return false;
        }
    }

    public boolean getPlaying() {
        if (mark == 'p' || mark == 'b') {
            return true;
        } else {
            return false;
        }
    }

    public void setTitel(String temp) {
        titel = temp;
    }

    public void setDauer(String temp) {
        dauer = temp;
    }

    public void setInterpret(String temp) {
        interpret = temp;
    }

    public void setAlbum(String temp) {
        album = temp;
    }

    public void setMusikrichtung(String temp) {
        genre = temp;
    }

    public void setFilepath(String temp) {
        fpath = temp;
    }

    public void setMarking() {
        if (mark == 'p') {
            mark = 'b';
        }
        if (mark == 'n') {
            mark = 'm';
        }
    }

    public void setPlaying() {
        if (mark == 'n') {
            mark = 'p';
        }
        if (mark == 'm') {
            mark = 'b';
        }
    }

    public void delMarking() {
        if (mark == 'm') {
            mark = 'n';
        }
        if (mark == 'b') {
            mark = 'p';
        }
    }

    public void delPlaying() {
        if (mark == 'p') {
            mark = 'n';
        }
        if (mark == 'b') {
            mark = 'm';
        }
    }

    public void setAll(String ufpath, String utitel, String udauer,
            String uinterpret, String ualbum, String ugenre, char umark) {
        fpath = ufpath;
        titel = utitel;
        dauer = utitel;
        interpret = uinterpret;
        album = ualbum;
        genre = ugenre;
        mark = umark;
    }
}

conSort:

import java.util.Comparator;

class conSort implements Comparator {
    public String order="";
    conSort(String nw) {
        order=nw;
    }

    public int compare(Object o1, Object o2) {
        return ((conEntry) o1).getOrder(order).compareTo(((conEntry) o2).getOrder(order));
    }
}

Allerdings gibt die Konsole mir nur "f" aus...
eigentlich sollte ja
"a
b
c
d
e
f"
kommen....
was ist falsch?!

ToMiNaToR

 

[mlange8801]

was ist falsch?!

Du überschreibts ja jedes mal deine conetrys...

      TreeSet s = new TreeSet(new conSort("titel")); 
        conEntry a=new conEntry(); 
        a.setAll("a", "a", "a", "a", "a", "a", 'n'); 
        s.add(a); 
        conEntry b=new conEntry();
        b.setAll("b", "b", "b", "b", "b", "b", 'n'); 
        s.add(b);
        conEntry c=new conEntry();
        c.setAll("c", "c", "c", "c", "c", "c", 'n'); 
        s.add(c);

 

[Tominator]

ist das nicht eigentlich egal?
Die werden doch eh in ne liste eingetragen!?
e ist ja quasi nur ne hilfsvariable?!

ToMiNaToR

Edit:

        s.add(new conEntry(0, "b", "b", "b", "b", "b", "b"));
        s.add(new conEntry(1, "f", "f", "b", "b", "b", "b"));
        s.add(new conEntry(2, "g", "g", "b", "b", "b", "b"));
        s.add(new conEntry(3, "z", "z", "b", "b", "b", "b"));

So gehts doch auch oder?
Ist günstiger

ToMiNaToR

 

[Leroy42]

ist das nicht eigentlich egal?
Die werden doch eh in ne liste eingetragen!?
e ist ja quasi nur ne hilfsvariable?!

Es ist nicht egal. e ist zwar nur eine Hilfsvariable aber immer dieselbe.
Und durch s.add(e) wird eben e selbst und nicht der aktuelle Inhalt von
e eingetragen. Nach deiner 1. Methode hast du am Schluß 5 identische Referenzen
von e in deiner Liste und diese Referenzen wiederum haben als Elemente nur
die, die du als letztet in e eingetragen hast.

Dein 2. Vorschlag ist nicht nur richtig, einfacher sondern sieht auch
hübscher aus :wink:

 

[Tominator]

Hmmm... noch ne andere Frage:
Wie kann ich aus nem TreeSet per angabe der Position einen Eintrag auslesen?
oder geht das garnicht?
Muss ich mit ner Schleife dann im Iterator solange next machen, bis es kommt?! (wär ja sehr umständlich)

ToMiNaToR

 

[Leroy42]

Nein das geht nicht.

Auch wenn es eine Methode get(int position) gäbe, könnte diese nicht
direkt auf das Element zugreifen sonder müßte entweder auf
iterator zurückgreifen oder sich selbst durch den Baum hangeln. :autsch:

Im übrigen sehe ich auch keine Anwendung dafür. Wie weiß der
Aufrufer was in seinem TreeSet an Position 42 steht? Der TreeSet
kann sich doch noch verändern. Und wenn sich sein Inhalt nicht
mehr ändern sollte, kann man aus ihm doch gleich ein Array machen lassen.

 

[SlaterB]

Du überschreibts ja jedes mal deine conetrys...

      TreeSet s = new TreeSet(new conSort("titel")); 
        conEntry a=new conEntry(); 
        a.setAll("a", "a", "a", "a", "a", "a", 'n'); 
        s.add(a); 
        conEntry b=new conEntry();
        b.setAll("b", "b", "b", "b", "b", "b", 'n'); 
        s.add(b);
        conEntry c=new conEntry();
        c.setAll("c", "c", "c", "c", "c", "c", 'n'); 
        s.add(c);

unterschiedliche Variablennamen sind übrigens nicht zwingend nötig,
geht auch wie ursprünglich nur mit e, wenn eben jedes mal ein neues Objekt erzeugt wird:

conEntry e=new conEntry();
TreeSet s = new TreeSet(new conSort("titel"));
e.setAll("a", "a", "a", "a", "a", "a", 'n');
s.add(e);

e=new conEntry();  // !!
e.setAll("b", "b", "b", "b", "b", "b", 'n');
s.add(e);

e=new conEntry(); // !!
e.setAll("c", "c", "c", "c", "c", "c", 'n');
s.add(e);

 

[Tominator]

Danke, aber ich regels jetz doch mit List
Hab da aber noch ein paar schwierigkeiten

ToMiNaToR