Eclipse 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;
	    }

 

[JohannisderKaeufer]

[JAVA=53]if (unternehmenNachPreis.containsKey((int) unX.getProduktpreis())) {
final Set<Unternehmen> unternehmenMitBestimmtemProduktpreis = unternehmenNachPreis.get((int) unX.getProduktpreis() * 100);[/code]

Diese zwei Zeilen sind inkonsistent.

Einmal wird geschaut ob mit dem Schlüssel ein Eintrag in der Map ist um dann nach dem Eintrag mal 100 zu Suchen.

Allgemein finde ich es persönlich besser das ganze so zu machen

Set<Unternehmen> unternehmenSet = unternehmenNachPreis.get(...unX.get...);
if(unternehmenSet == null) {
  unternehmenSet = new HashSet<Unternehmen>();
  unternehmenNachPreis.put(...unX.get..., unternehmenSet());
}
unternehmenSet.add(unX);

 

[Counter3103]

[JAVA=53]if (unternehmenNachPreis.containsKey((int) unX.getProduktpreis())) {
final Set<Unternehmen> unternehmenMitBestimmtemProduktpreis = unternehmenNachPreis.get((int) unX.getProduktpreis() * 100);[/code]

Diese zwei Zeilen sind inkonsistent.

Einmal wird geschaut ob mit dem Schlüssel ein Eintrag in der Map ist um dann nach dem Eintrag mal 100 zu Suchen.

Den Fehler hatte ich heute Nacht bereits ausgebessert nur den Code hier nicht^^..

Allgemein finde ich es persönlich besser das ganze so zu machen

Set<Unternehmen> unternehmenSet = unternehmenNachPreis.get(...unX.get...);
if(unternehmenSet == null) {
  unternehmenSet = new HashSet<Unternehmen>();
  unternehmenNachPreis.put(...unX.get..., unternehmenSet());
}
unternehmenSet.add(unX);

Ich verstehe nicht ganz welchen Hintergedanken du dabei hast, denn es wird auf jeden Fall ein Set übergeben mit mehreren Objekten. Nur in dem Fall, dass von beispielsweise 2 Objekten der Wert gleich ist, fällt eines raus. Der Gedanke war für den Preis dann eine Art anzahl anzugeben, wie oft er vorkommt und zu welchen Objekten er gehört.