Probleme bei selbst erstelltem Algorithmus

[Counter3103]

Hallo Leute,

im Rahmen eines Projektes habe ich versucht eine Algorithmus zu schreiben.
Sinn des Algorithmus ist es Punkte an mehrere Objekte zu verteilen, ohne diese im Objekt selber zu speichern. Die Verteilung findet anhand eines bestimmen Attributwertes statt, um im Nachhinein ein anderes jeweiliges Attribut neu zu berechnen.
Mein Problem an der Sache ist, dass bei der Verteilung Objekte leer ausgehen, welche den gleichen Attributwert besitzen.

Ich sitze nun schon mehrer Wochen an dem Problem, bin verschiedentlich weiter gekommen aber immer hackt es irgendwo.
Ich hoffe ihr könnt mir helfen, ich bin langsam echt depirimiert -.-...

Hier der Code:


public static int[] priceRating(Unternehmen un1, Unternehmen un2, Unternehmen un3, Unternehmen un4){

//Kann nicht gleich als Set übergeben werden, aus diversen Gründen
//Verbesserung ist in arbeit

Set<Unternehmen> alleUnternehmen = new HashSet<Unternehmen>();

alleUnternehmen.add(un1);
alleUnternehmen.add(un2);
alleUnternehmen.add(un3);
alleUnternehmen.add(un4);

final Map<Integer, Set<Unternehmen>> unternehmenNachPreis = gruppiereUnternehmenNachPreis(alleUnternehmen);
final List<Integer> sortedPrice = holeSortiertePreisliste(unternehmenNachPreis);

Map<Unternehmen, Integer> indexMap = new HashMap<Unternehmen, Integer>();

indexMap.put(un1, 0);
indexMap.put(un2, 1);
indexMap.put(un3, 2);
indexMap.put(un4, 3);

int[] creditPoints = new int[4];
int i = 0;

for( Integer price : sortedPrice ){

Set<Unternehmen> unternehmenProPreis = unternehmenNachPreis.get( price );

for ( Unternehmen unY : unternehmenProPreis){
Integer index = indexMap.get(unY);
System.out.println("Dies ist der Index " + index);
//unterschiedliche CP-Verteilung je nach Kriterium
creditPoints[index] = 40 - i * 10;

i++;

}

}

return creditPoints;
// Unternehmen 1 hat den Platzhalter creditPoints[0],
// Unternehmen 2 hat den Platzhalter creditPoints[1],
// usw.
// -->gespeichert sind die tatsächlichen CreditPoints für das Unternehmen!

}

private static Map<Integer, Set<Unternehmen>> gruppiereUnternehmenNachPreis( final Set<Unternehmen> alleUnternehmen) {
final Map<Integer, Set<Unternehmen>> unternehmenNachPreis = new HashMap<Integer, Set<Unternehmen>>();
for (Unternehmen unX : alleUnternehmen) {
if (unternehmenNachPreis.containsKey((int) unX.getProduktpreis())) {
final Set<Unternehmen> unternehmenMitBestimmtemProduktpreis = unternehmenNachPreis.get((int) unX.getProduktpreis() * 100);
unternehmenMitBestimmtemProduktpreis.add(unX);
} else {
final Set<Unternehmen> unternehmenMitProduktpreis = new HashSet<Unternehmen>();
unternehmenMitProduktpreis.add(unX);
unternehmenNachPreis.put((int) unX.getProduktpreis() * 100, unternehmenMitProduktpreis);
}
}
return unternehmenNachPreis;
}

private static List<Integer> holeSortiertePreisliste(
final Map<Integer, Set<Unternehmen>> unternehmenNachPreis) {
final List<Integer> sortedPrice = new ArrayList<Integer>();
for (Integer i : unternehmenNachPreis.keySet()) {
sortedPrice.add(i);
}

// sortieren
Collections.sort(sortedPrice, new SortCriterion());
return sortedPrice;
}

[Christian Marquardt]

a) ist das kein Probelm von Java und Eclipse -> verschoben

b) gibt es eine Codeformatierung. Ohne die schaue ich mit das nicht weiter an (Zumal du das woanders prima kannst)

c) Muss es möglich sein, dein Problem verbal zu beschreiben ohne, dass man den Code durchgehen muss.

d) ist Java eine Sprache die OOP unterstützt. Dein ganzer Aufbau (static, final), zeigt, dass du das in keinster Weise nutzt

e) Crosspost -> geschlossen