Schaltjahre, Klassenmethode, Parametertyp

[andrade]

Hallo zusammen,
ich lern gerade für eine Prüfung und habe dabei die folgende Aufgabe gelöst.
Könnt ihr vielleicht einmal drüber schauen, ob das so wohl richtig ist bzw. ob ich etwas vergessen habe? Danke!

/* Aufgabe: a)Schreiben Sie eine Methode, die für ein beliebiges Jahr entscheidet, ob es sich um ein
* Schaltjahr handelt, und das Ergebnis als boolschen Wert zurückliefert.
* b) Schreiben Sie die Methode so, dass sowohl der Parameter- als auch Rückgabetyp echte Java-Klassen sind.
* c) Realisieren Sie diese Berechnung, falls möglich, als eine Klassenmethode. Begründen Sie kurz, warum
* das hier möglich ist bzw. nicht möglich ist.
*/
public class Schaltjahre
{
   public static Boolean bestimmeSchaltjahr(Integer jahr) {
       if ((jahr % 4 ==0 && jahr % 100 != 0) || (jahr % 400 == 0))
            return Boolean.TRUE;
       else
            return Boolean.FALSE;
    }
}

/* zu c) Ja, ist möglich! Weil das Ergebnis der Berechnung nur von den Paramatern
* und nicht vom Objektzustand abhängt. */

 

[Dukel]

Kommt denn das richtige Ergebnis heraus? Teste es doch mit unterschiedlichen Schalt- und Nicht-Schalt-Jahren. (evtl. als Unit Test)

 

[andrade]

Also das Ergebnis der Berechnung stimmt. Mir gehts eher darum, ob ich die Aufgabenteile b und c auch richtig gelöst habe.

 

[kneitzel]

Also das sieht erst einmal recht gut aus. Ein paar Hinweise hätte ich noch:
- Dein Parameter ist eine Referenz (Der Typ ist eine Klasse und kein simpler Datentyp). Daher kann dieser null sein. Dies solltest Du prüfen, ehe Du diesen nutzt.
- Du gibst direkt das Ergebnis der Bedingung zurück. Daher könntest Du das if weglassen und direkt die Bedingung zurück geben. Durch den Rückgabetyp Boolean wird dein boolean automatisch umgewandelt ("boxing", der einfache Wert wird in eine Instanz einer Klasse, also sozusagen in eine Box, gesteckt). Wenn Du dies verdeutlichen willst, könntest Du noch ein (Boolean) davor setzen, aber das ist unnötig und wir von einigen IDEs dann auch entsprechend angezeigt:

public static Boolean istSchaltjahr(Integer jahr) {
    if (jahr == null)
        throw new IllegalArgumentException("Argument jahr cannot be null!");
    
    return (Boolean)(jahr % 4 ==0 && jahr % 100 != 0) || (jahr % 400 == 0);
}

Wie gesagt: das (Boolean) beim return kann (und in meinen Augen sollte) weg.

Konrad

 

[andrade]

das sieht doch super aus! Danke dir!

 

[kneitzel]

Deine Lösung war aber auch schon super! Ich denke bei einer Aufgabe hättest Du damit schon volle Punktzahl gehabt. Die zwei Kleinigkeiten gehen halt mehr Richtung Clean Code. Und beim zweiten Punkt hätte ich in der Uni fast die Angst, dass derjenige, der die Aufgabe kontrolliert, das Autoboxing nicht kennt und dann bemängelt, dass ich da evtl. ein boolean zurück gebe.

 

[andrade]

Danke Autoboxing haben wir in der Vorlesung behandelt, also sollte der Prof das auch draufhaben.