HashMap+Vererbung

[El Hadji]

Servus Community,
Ich habe eine Frage. Ich habe 2 Klassen, Kino und eine Unterklasse KinoMitErmaessigung. Die Kino Klasse hat eine Methode die den Preis für alle Reihen in einer Map zurück gibt, Reihen = Key, Preise = value. Kino MitErmaessigung soll natürlich nur die Preise der Map verändern und zurück geben.

Hier mal die Oberklasse Kino:

import java.util.*;

public class Kino
{
    // instance variables - replace the example below with your own
    private int n;
    private int m;
    private Map<Integer, Double> diePreise;

  
    public Kino(int n, int m, Map<Integer, Double> diePreise)
    {
        this.n = n;
        this.m = m;
        this.diePreise = diePreise;
    }

  
    public Map<Integer, Double> getDiePreise()
    {
        return diePreise;
    }
   
   
}

Und meine Unterklasse:

import java.util.*;

public class KinoMitErmaessigung extends Kino
{
    // instance variables - replace the example below with your own
  
    public KinoMitErmaessigung(int n, int m, Map<Integer, Double> diePreise)
    {
       super(n,m,diePreise);
    }

   
    public Map<Integer, Double> getErmaessigtePreise(double ermaessigung)
    {
       Map <Integer, Double> neu = super.getDiePreise();
       Set<Integer> liste = super.getDiePreise().keySet();
      
       liste.toArray();
      
       
        for(int i = 0; i < liste.length; i++)
        {
            neu.replace(liste[i], neu.get(liste[i]), neu.get(liste[i]*((100-ermaessigung)/100)));
        }
       
        return neu;
      
    }
}

Natürlich funktioniert das umschreiben des Sets in eine Array nicht. Aber ansonsten müsste der Ansatz richtig sein oder?
mfg El Hadji

 

[El Hadji]

Ok das mit dem umschreiben zum Array bin ich umgangen aber wieso funktioniert das replacen meiner Mappe nicht?

import java.util.*;

public class KinoMitErmaessigung extends Kino
{
    // instance variables - replace the example below with your own
  
    public KinoMitErmaessigung(int n, int m, Map<Integer, Double> diePreise)
    {
       super(n,m,diePreise);
    }

   
    public Map<Integer, Double> getErmaessigtePreise(double ermaessigung)
    {
       Map <Integer, Double> neu = super.getDiePreise();
      
      
      
       
        for(Integer t : super.getDiePreise().keySet())
        {
            neu.replace(t, neu.get(t), neu.get(t*((100-ermaessigung)/100)));
        }
       
        return neu;
      
    }
}

 

[mihe7]

Ein einfaches put reicht.

 

[El Hadji]

Sorry die Preußen schießen zu schnell. Bin selbst auf den dummen Fehler gestoßen!

So müsste es natürlich stimmen!
Was meint ihr?

import java.util.*;

public class KinoMitErmaessigung extends Kino
{
    // instance variables - replace the example below with your own
  
    public KinoMitErmaessigung(int n, int m, Map<Integer, Double> diePreise)
    {
       super(n,m,diePreise);
    }

   
    public Map<Integer, Double> getErmaessigtePreise(double ermaessigung)
    {
       Map <Integer, Double> neu = super.getDiePreise();
      
      
      
       
        for(Integer t : super.getDiePreise().keySet())
        {
            neu.replace(t, neu.get(t), neu.get(t)*(100-ermaessigung)/100);
        }
       
        return neu;
      
    }
}

 

[El Hadji]

Ein einfaches put reicht.

Ok aber replace ist ja auch nicht falsch oder?
put weil wenn ein Key mit einem neuen Value belegt wird, wird er sowieso überschrieben oder?

 

[mihe7]

Replace funktioniert, wenn - wie in Deinem Fall - bereits ein Eintrag für den Schlüssel existiert. Und put funktioniert genau aus dem Grund, den Du geschrieben hast

 

[Robat]

Weil ja eine Java-8 Lösung heute nicht mehr fehlen darf

int x = 45;
final BiFunction<Double, Double, Double> mapFunction = ( a, b ) -> a * ( ( 100 - x ) / 100d );
map.forEach(( key, value ) -> map.merge(key, value, mapFunction));

 

[El Hadji]

Ich versteh ja noch nichtmal java-7^^

 

[Robat]

Das war auch eher als kleiner Scherz gedacht

 

[mrBrown]

Weil ja eine Java-8 Lösung heute nicht mehr fehlen darf

int x = 45;
final BiFunction<Double, Double, Double> mapFunction = ( a, b ) -> a * ( ( 100 - x ) / 100d );
map.forEach(( key, value ) -> map.merge(key, value, mapFunction));

map.replaceAll((reihe, preis) -> preis * (100 - ermaessigung) / 100);

 

[Robat]

@mrBrown viel zu leserlich / einfach

 

[mihe7]

viel zu leserlich / einfach

Genau, einfach kann jeder