Aufgabe Arithmetische Operatoren

[MacMurdock]

Ich muss folgende Aufgabe lösen:

"Erstellen Sie eine Methode groessere(), der zwei Zahlen übergeben werden.
Die Methode soll die groessere der beiden Zahlen zurückgeben."

Das hab ich bis jetzt gemacht:

 public int grössere(int zahl1, int zahl2)
    {
        if (zahl1 > zahl2)
        {
            return (zahl1);
        }
        else if (zahl2 > zahl1)
        {
            return (zahl2);
        }    
    }

Wenn ich den code compile kommt die Meldung "missing return statement"
Muss da zum Schluss nochmal ne return Anweisung kommen?
Danke im Vorraus!

 

[bone2]

der compiler wartet aufs else{}
ersetz einfach das zweite else if dadurch. wenn a nicht größer b ist, muss b eh automatisch größer oder gleich sein, das muss man nicht nochmal prüfen


edit: oder anders: es fehlt ein return für den fall zahl1 = zahl2

 

[Michael...]

Was ist wenn beide Bedinungen nicht erfüllt sind? Dann muss ja auch irgendwas zurückgegeben werden.

 

[Andi_CH]

der compiler wartet aufs else{}
ersetz einfach das zweite else if dadurch. wenn a nicht größer b ist, muss b eh automatisch größer oder gleich sein, das muss man nicht nochmal prüfen


edit: oder anders: es fehlt ein return für den fall zahl1 = zahl2

nein er wartet nicht auf ein else

er wartet nur auf ein drittes return

das else braucht es nicht

 

[bone2]

ach ja stimmt natürlich^^ ich dachte dabei auch eher an das, was ist, wenn beides nicht zutrifft. wie michael schon schreibt. es ist ja möglich das er bei keinem der returns ankommt

 

[MacMurdock]

Und wieso genau wartet der compiler auf ein drittes return?
Wie müsste das im code aussehen?

 

[MacMurdock]

kann mir niemand sagen wie das dritte return im code auszusehen hätte?

 

[MacMurdock]

Was ich leider vergessen hab zu erwähnen, die Methode soll mit den testdaten 1,2 und 2,1 funktionieren.
Das heisst eine return Anweisung für den fall zahl1 = zahl2 ist überflüssig.
Ich wär echt dankbar wenn mir jemand erklären könnte wie man die Methode jetzt genau abschliesst das muss nämlich bis morgen funktionieren^^

 

[jre]

einfach als letzte anweisung in deiner methode "return zahl1" - da die beiden zahlen gleich groß sind spielt es auch keine rolle welche du zurück gibst.
wichtig ist nur dass jeder pfad überhaupt etwas zurück gibt.

 

[jdk2]

wobei man noch anmerken könnte, dass man sich den 2. teil auch gleich sparen kann, denn wenn zahl 1 nicht größer ist, ist zahl 2 auf jeden fall richtig

public int grössere(int zahl1, int zahl2)
    {
        if (zahl1 > zahl2)
        {
            return (zahl1);
        }
        return (zahl2);    
    }

 

[MacMurdock]

Danke nochmal für die Hilfe!
Ich stand en bisschen aufm Schlauch ich habs jetzt einfach so gelöst:

public int grössere(int zahl1, int zahl2)
    {
        if (zahl1 > zahl2)
        {
            return (zahl1);
        }
        else
        {
            return (zahl2);
        }    
    }

 

[Terep]

Für diese Aufgabenstellung gibt es den Bedingungsoperator ?:
[JAVA=42]
int gross = ( zahl1 > zahl2) ? zahl1 : zahl2;
[/code]

Terep

 

[jre]

Für diese Aufgabenstellung gibt es den Bedingungsoperator ?:
[JAVA=42]
int gross = ( zahl1 > zahl2) ? zahl1 : zahl2;
[/code]

Terep

genau dafür gibts dann auch Math.max(long,long) =D

beantwortet aber nicht die aufgabe die op gestellt und fast selbstständig gelöst wurde

"Erstellen Sie eine Methode groessere(), der zwei Zahlen übergeben werden.
Die Methode soll die groessere der beiden Zahlen zurückgeben."