Javaでうるう年かどうかの判断のコードを書きましたが、すべて普通の年と判断されてしまいます。

Question

前提・実現したいこと

Javaでうるう年かどうか判断するコードを作っています。

ここに質問の内容を詳しく書いてください。
（例）PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

エラーメッセージは出ないのですが、どんな年をいれても全て普通の年だと判断されてしまいます。

エラーメッセージ

該当のソースコード

public class LeapYearCalculator{
public static boolean isLeapYear(int year){
if(year<0 || year>9999){
return false;
} else if(year%4==0){
return true;
}
return false;

}

}

public class Main {
public static void main(String[] args) {

    boolean A=LeapYearCalculator.isLeapYear(2017);
    if(A==true){
        System.out.println("leap year");
    } else {
        System.out.println("usual year");
    }
    
}

}

ソースコード

別のやり方で書いているコードを見つけました。
私と同じことをやっているように見えます。なぜ、こちらは動くのに私のは正常に動かないのかわかりません。

public class LeapYear {
public static boolean isLeapYear(int year){
if( year>=1 && year<=9999){
if( (year %400==0)||(year%100 !=0 && year%4==0))
return true;
}
else
return false;
return false;
}
}

試したこと int yearの因数に - 1600, 1600, 2017 などをいれてみました。

全て普通の年だと判断されてしまいました。
ここに問題に対して試したことを記載してください。
別のやり方で書いているコードを見つけました。これは正常に動くようです。

私と書き方は違うますが、やっていることは同じだと思うのでやはり私のコードの不良の原因がわかりません、、すみません、ヒントだけでも教えていただけると助かります。

補足情報（FW/ツールのバージョンなど）

Udemyの教材で学習しています。Java独学を初めて一ヶ月くらいです。
ここにより詳細な情報を記載してください。

Comments

[Zuishin]

> boolean A=LeapYearCalculator.isLeapYear(2017);

引数(ひきすう)を変えたつもりで 2017 で固定されているのでは？

[ozwk]

> 全て普通の年だと判断されてしまいました。

こちらで1600年を入れたところ"leap year"が表示されました

[talabagani]

ほんとうですね。お騒がせいたしました。本当にありがとうございました。

Answer 1

2017でなく1600にするとleap yearとなるはずですが、ならないとしたら、実行しているプログラムと質問文に書いているプログラムが違うと言うことでしょう。

なお、「4で割れたら閏年」というのはヨーロッパ中世以前では正しかった判定です。

Answer 2

こういうものを見ると勉強になります。

• https://github.com/openjdk/jdk/blob/master/src/java.base/share/classes/java/time/Year.java#L317

# 西暦1000004年がどうなのかは怪しいかもしれませんが。

Comments

[dodox86]

リンク先より：
> public static boolean isLeap(long year) {
> return ((year & 3) == 0) && ((year % 100) != 0 || (year % 400) == 0);
> }

(year & 3) == 0 はちょっと面白いですね。自分で書くときはやらない、JDKならでは感があります。

[xebme]

2のべきの剰余はべきの数だけビットオンして&をとる。0なら割り切れる。ということですか。

[shiketa]

> (year & 3) == 0 はちょっと面白いですね。
一瞬むむっとおもい、なるほどと。わたしもそう思いましたので、引いてみました。
> 自分で書くときはやらない、
そうですね。ソースレビュー担当なら指摘したくなります。
> JDKならでは感があります。
JDKだからこそ模範になってほしい感もありますけどね。

[xebme]

式を左から評価すると高速な篩になっている。うるう年ではないもののほうが多いから。理解しました。

[dodox86]

ビット演算によるこの手の操作はC言語やC++などでも少しでも高速化、効率化を目指すようなコードで見かけますが、式と値の評価順も併せて考えると、妥当っぽいですね。

[shiketa]

業務システムのヒトなので、「四で割った余りを」なら`value % 4`と書いてほしい。「二で割って」を`value >> 1`と書かれたらダメを出したい...。効率も大事だけど、読みやすさも。

Answer 3

うるう年の判定

うるう年を判定するための集合とデシジョンテーブルを示します。最も内側の集合から判定を行うと、ロジックは簡単になります。elseを使わないで、１，２，３の順に判定して、false/trueを返してください。

Answer 4

回答ではありません・・・

やっていることは同じだと思う

そうでしょうか？

if(year<0 || year>9999){
    return false;
} else if(year%4==0){
    return true;
} 
return false;

if( year>=1 && year<=9999){
    if( (year %400==0)||(year%100 !=0 && year%4==0))
        return true;
}
else
    return false;

→うるう年は4年に1度とは限らない