Java Schaltjahr berechnen Code

[Aemulit]

Hallo,
Ich habe die Aufgabe, ein Programm zu entwickeln, welches nach einer Eingabe einer Zahl berechnen, ob diese ein Schaltjahr ist, oder nicht.
Ich programmiere noch nicht sehr lange und würde mich freuen, wenn ihr mir helfen würdet, ohne Begriffe wie "return" o.ä. klarzukommen.

Folgendes Problem:
Unabhängig von der Jahreszahl, die ich eingebe, wird mir die Nachricht für die positive boolean Variable des Schaltjahres ausgegeben.
Wenn ich ein Jahr eingebe, wird geschaut, ob es sich um ein Schaltjahr handelt oder nicht. Der boolean-Wert wird dementsprechend auf true oder false gesetzt. Nun schaffe ich es aber nicht, das auch auszugeben, vielmehr kommt bei dem Fenster, was die Nachricht anzeigt, dass jedes beliebige Jahr ein Schaltjahr ist.

Ich habe 2 mal dieses Programm geschrieben. Einmal ohne und einmal mit boolean Variable. Das ohne boolean Variable läuft so, wie es soll, aber das mit der Variable gibt immer das Gleiche aus. Ich denke ja, dass das nen enormer Anfängerfehler ist, aber ich komme hier leider nicht mehr weiter.

Die Codes hänge ich euch unten an:


[CODE lang="java" title="Mit Boolean Variable -- gibt immer das selbe aus:"]import javax.swing.JOptionPane;

public class Schaltjahr {
public static void main(String[] args) {
String eingabe;
int jahr;
boolean istSchaltjahr;
eingabe = JOptionPane.showInputDialog("Geben Sie eine Jahreszahl ein: ");
jahr = Integer.parseInt(eingabe);

if (jahr%4 == 0 && jahr%100 != 0 || jahr%400 == 0) {
istSchaltjahr = true;
} else {
istSchaltjahr = false;
}

if (istSchaltjahr = true) {
JOptionPane.showMessageDialog(null, jahr + "ist ein Schaltjahr!");
}
if (istSchaltjahr = false) {
JOptionPane.showMessageDialog(null, jahr + "ist kein Schaltjahr!");
}



}
}[/CODE]

[CODE lang="java" title="funktionierender Code ohne boolean Variable"]import javax.swing.JOptionPane;

public class test {
public static void main(String[] args) {
String eingabe;
int jahr;
eingabe = JOptionPane.showInputDialog("Geben Sie eine Jahreszahl ein: ");
jahr = Integer.parseInt(eingabe);

if (jahr%4 == 0 && jahr%100 != 0 || jahr%400 == 0) {
JOptionPane.showMessageDialog(null, jahr + "ist ein Schaltjahr!");
} else {
JOptionPane.showMessageDialog(null, jahr + "ist kein Schaltjahr!");
}



}
}[/CODE]

 

[Turing0001]

Hallo,
Ich habe die Aufgabe, ein Programm zu entwickeln, welches nach einer Eingabe einer Zahl berechnen, ob diese ein Schaltjahr ist, oder nicht.
Ich programmiere noch nicht sehr lange und würde mich freuen, wenn ihr mir helfen würdet, ohne Begriffe wie "return" o.ä. klarzukommen.

Folgendes Problem:
Unabhängig von der Jahreszahl, die ich eingebe, wird mir die Nachricht für die positive boolean Variable des Schaltjahres ausgegeben.
Wenn ich ein Jahr eingebe, wird geschaut, ob es sich um ein Schaltjahr handelt oder nicht. Der boolean-Wert wird dementsprechend auf true oder false gesetzt. Nun schaffe ich es aber nicht, das auch auszugeben, vielmehr kommt bei dem Fenster, was die Nachricht anzeigt, dass jedes beliebige Jahr ein Schaltjahr ist.

Ich habe 2 mal dieses Programm geschrieben. Einmal ohne und einmal mit boolean Variable. Das ohne boolean Variable läuft so, wie es soll, aber das mit der Variable gibt immer das Gleiche aus. Ich denke ja, dass das nen enormer Anfängerfehler ist, aber ich komme hier leider nicht mehr weiter.

Die Codes hänge ich euch unten an:


[CODE lang="java" title="Mit Boolean Variable -- gibt immer das selbe aus:"]import javax.swing.JOptionPane;

public class Schaltjahr {
public static void main(String[] args) {
String eingabe;
int jahr;
boolean istSchaltjahr;
eingabe = JOptionPane.showInputDialog("Geben Sie eine Jahreszahl ein: ");
jahr = Integer.parseInt(eingabe);

if (jahr%4 == 0 && jahr%100 != 0 || jahr%400 == 0) {
istSchaltjahr = true;
} else {
istSchaltjahr = false;
}

if (istSchaltjahr = true) {
JOptionPane.showMessageDialog(null, jahr + "ist ein Schaltjahr!");
}
if (istSchaltjahr = false) {
JOptionPane.showMessageDialog(null, jahr + "ist kein Schaltjahr!");
}



}
}[/CODE]

[CODE lang="java" title="funktionierender Code ohne boolean Variable"]import javax.swing.JOptionPane;

public class test {
public static void main(String[] args) {
String eingabe;
int jahr;
eingabe = JOptionPane.showInputDialog("Geben Sie eine Jahreszahl ein: ");
jahr = Integer.parseInt(eingabe);

if (jahr%4 == 0 && jahr%100 != 0 || jahr%400 == 0) {
JOptionPane.showMessageDialog(null, jahr + "ist ein Schaltjahr!");
} else {
JOptionPane.showMessageDialog(null, jahr + "ist kein Schaltjahr!");
}



}
}[/CODE]

Hallo Aemulit,

das liegt an einer kleinen Ungenauigkeit in Deiner if-Anweisung. Nötig ist hier ein ==, denn Du willst ja einen logischen Vergleich durchführen. Was Du aber machst ist eine Zuweisung . Also if(istSchaltjahr == true) ... dann funktioniert es wie es soll. Passiert mir auch immer noch zwischendurch...

 

[Aemulit]

Vielen Dank! ^^ ich hab mir schon den Kopf zerbrochen...

 

[temi]

Anstatt von if (istSchaltjahr = true) { kannst du auch einfach if (istSchaltjahr) { schreiben. In der Klammer der if-Anweisung wird ein Ausdruck erwartet, der vom Typ boolean ist. Da die Variable "istSchaltjahr" bereits ein boolean ist, kannst du den expliziten Vergleich auf true weglassen.

 

[Blender3D]

Ich habe 2 mal dieses Programm geschrieben. Einmal ohne und einmal mit boolean Variable.

Hier eine dritte Variante mit Funktionsaufruf.
[CODE lang="java" title="Schaltjahr" highlight="3-4"]public class Schaltjahr {
public static void main(String[] args) {
int jahr = eingabeJahr("Geben Sie eine Jahreszahl ein: ", 1900, 2999);
String message = jahr + " ist " + (istSchaltJahr(jahr) ? "" : "k") + "ein Schaltjahr!";
JOptionPane.showMessageDialog(null, message);
}

public static int eingabeJahr(String text, int min, int max) {
int jahr = min;
boolean ok = false;
do {
String eingabe = JOptionPane.showInputDialog(text, min + "");
try {
jahr = Integer.parseInt(eingabe);
if (jahr < min || jahr > max)
JOptionPane.showMessageDialog(null, "Der Bereich für gültige Jahre ist [" + min + "-" + max + "]");
else
ok = true;
} catch (NumberFormatException e) {
JOptionPane.showMessageDialog(null, "Bitte nur Zahlen eingeben!");
}
} while (!ok);
return jahr;
}

public static boolean istSchaltJahr(int jahr) {
if (jahr % 4 == 0 && jahr % 100 != 0 || jahr % 400 == 0)
return true;
return false;
}
}[/CODE]

 

[temi]

Ok, dann noch eine kleine Abwandlung deiner Variante 😉

boolean istSchaltjahr;
   
if (jahr%4 == 0 && jahr%100 != 0 || jahr%400 == 0) {
    istSchaltjahr = true;
} else {
    istSchaltjahr = false;
}

// das geht auch einfacher

boolean istSchaltjahr = (jahr % 4 == 0 && jahr % 100 != 0) || jahr % 400 == 0;

Grund ist der gleiche wie schon oben. Du weist einer boolean Variablen den Wert true oder false zu, aber bereits die Bedingung in der if-Anweisung entspricht dem gesuchten boolean-Wert.

Gleiches gilt auch für die Methode istSchaltjahr() von @Blender3D. Da könnte man auch direkt das Ergebnis der Verknüpfung als Rückgabewert zurückgeben und auf die if-Anweisung verzichten.

 

[Blender3D]

Gleiches gilt auch für die Methode istSchaltjahr() von @Blender3D.

Stimmt 😉

 

[temi]

Stimmt 😉

Wobei du ja sicher zeigen wolltest, wie das "return early" Muster funktioniert. Was auch völlig legitim ist.