Comparator und private Variablen

[Claudia92]

Hallo ihr Lieben!

Ich habe eine kleine Verständnisfrage zum Comparator. Angenommen ich habe eine Klasse mit einer privaten Instanzvariable:

class World {
	private int population;
}

Wenn ich nun eine Comparator-Klasse schreibe, kann ich auf diese Instanzvariable nicht mehr zugreifen:

class WorldComparator implements Comparator<World> {
	public int compare(World o1, World o2) {
		return o1.population - o2.population;
	}
}

Wie löse ich dieses Problem? Die Variable population sollte private bleiben und auch nicht über einen öffentlichen Getter erreichbar sein.

Meine Idee war, WorldComparator könnte eine innere Klasse von World sein, dann wäre das Sichtbarkeitsproblem gelöst:

class World {
	private int population;
	
	class WorldComparator implements Comparator<World> {
		public int compare(World o1, World o2) {
			return o1.population - o2.population;
		}
	}
}

Aber dann muss ich den Comparator so umständlich erstellen:

Collections.sort(worlds, new World().new WorldComparator());

Da gibt es bestimmt eine bessere Lösung, oder? :reflect:

 

[JCODA]

Du kannst einfach Comparable (Java 2 Platform SE v1.4.2) implementieren.

Lists (and arrays) of objects that implement this interface can be sorted automatically by Collections.sort

 

[Kevin94]

Mach die innere Klasse static.

 

[Claudia92]

@JCODA: Ich will es aber bewusst mit einem Comparator machen, um nach mehreren Kriterien sortieren zu können.

@Kevin94:

Collections.sort(worlds, new World.WorldComparator());

Ah, das gefällt mir, danke.

 

[bERt0r]

Nunja, wenn eine Klasse mit einer anderen verglichen werden soll, das Kriterium aber private sein soll hat man eben einen gewissen Interessenskonflikt. Um zwei Dinge zu vergleichen muss man eben auf beide Zugreifen können. Wieso muss "Population" denn unbedingt private sein bzw. kannst du keinen getter schreiben?

 

[hüteüberhüte]

Wieso ist einer statischen inneren Klasse denn der Zugriff auf Instanzvariablen erlaubt? :bahnhof:

 

[bygones]

Wieso ist einer statischen inneren Klasse denn der Zugriff auf Instanzvariablen erlaubt? :bahnhof:

geht auch nicht.... ist ein bisschen trickreich ;-)

    static class WorldComparator implements Comparator<Test> {
        public void foo() {
            System.out.println(population); // geht net
        }
        @Override
        public int compare(Test o1, Test o2) {
            return o1.population - o2.population;
        }
    }

 

[nillehammer]

Ich denke, hier dürfte man durchaus einen Getter für Population spendieren. Wenn der (warum auch immer) nicht public sein soll, dann mach ihn default-visible. Auf die Weise könntest Du Deine Comparatoren wenigstens als Top-Level Klassen programmieren (natürlich im selben Package). Compraratoren als innere statische Klassen kommen mir doch sehr komisch vor. Vor allem, weil Du ja geschrieben hast, dass es mehrere werden sollen. Dann müsstest Du ja bei jedem neuen die World-Klasse anfassen.