OOP Compile Fehler

[Widdsen]

Hallo miteinander, ich bekomme bei folgender Methode immer einen Compilefehler und zwar : "missing return statement" dabei habe ich doch schon 2 returns drin !
Die Methode soll in der Menge an Keys meiner HashMap suchen ob der paramter schon vorhanden ist! wenn ja soll sie den alten value returnen und den neuen in die Map einfügen! wenn nicht nur einfügen und null ausgeben!
Edit: Wenn ich nach der for-schleife

return null;

mache gehts aber die Methode funktioniert dann nicht richtig^^

 public String putTaxKeyOfProductGroup(String productGroup, String taxKey)
    {
        String oldtaxkey;
       for( String str : mwstschlüssel.keySet()){
           if(  mwstschlüssel.keySet().contains(productGroup)){
               oldtaxkey = mwstschlüssel.get(productGroup) ;
               mwstschlüssel.put(productGroup, taxKey);
               return oldtaxkey;
               }
          else{
              mwstschlüssel.put(productGroup, taxKey);
              return null;
            }
        }
      
    }

 

[ARadauer]

was ist wenn die liste leer ist und du gar nicht in die schleife kommst? was gibst du dann zurück?

 

[Widdsen]

achso ja dann "null" ! hmmm dann scheint die methode nicht richtig zu funtionieren....
siehst du nen fehler?
weil eig. sollte der alte value ja zurückgegeben werden! aber es kommt immer null!

 

[Andi_CH]

Nein! Stell dir vor du hast eine leere Liste - du kommst zum for - da wird aber nichts ausgeführt
Du gehst zu Zeile 15 und ....
Ja genau -- es gibt kein return Statement in diesem Fall.

Da musst du etwas zurückgeben.

Was du mit "altem Wert" meinst ist mir unklar. Wenn du "oldTaxkey meinst - der ist an dieser Stelle wirklich null, also kannst du auch return null; hinschreiben

 

[Widdsen]

ok ich hab die for-schleife jetzt mal rausgenommen! meine mehtode sieht nun folgendermaßen aus:

public String putTaxKeyOfProductGroup(String productGroup, String taxKey)
    {
        String oldtaxkey;
        
            if(  mwstschlüssel.keySet().contains(productGroup)){
                oldtaxkey = mwstschlüssel.get(productGroup) ;
                mwstschlüssel.put(productGroup, taxKey);
                return oldtaxkey;
            }
            else{
                mwstschlüssel.put(productGroup, taxKey);
                return null;
            }
        }

was nocht nicht funktioniert ist , dass der alte value von dem key productgroup den ich ja in "oldtaxkey" gespeichert habe, nicht ausgegeben wird!

Edit: ah doch geht! haha^^ Danke für die Hilfe!