pythonで閏年でない場合の結果が表示されない

Question

現在、pythonで簡易的な閏年判定プログラムを作成しています。
以下プログラムで、結果が閏年の場合は、print文が実行されるのですが、
閏年ではない場合にprint文が実行されません。
表示されないのは、何か条件が足りていないからでしょうか？
ご教授のほどよろしくお願いします。

#閏年かどうかを判定する
def leap_year_judgment(target_year):
    if target_year % 400 == 0:
        print(target_year,"は閏年です")
    elif target_year % 4 == 0:
        print(target_year,"は閏年です")
    elif target_year % 100 == 0 and target_year % 400 == 0:
        print(target_year,"は閏年ではありません")

leap_year_judgment(2000)

Answer 1

if 条件1:
   処理1
elif 条件2:
   処理2

処理X

この場合. まず 条件1をチェックして成立すれば処理1 をおこなってから処理X をおこないます。
条件1 が成立しないかったら、条件２をチェックします。
成立したら、処理2 をおこなってから処理X を行います。
成立しなければ、処理X だけを行います。

if .. elsf ... の場合、チェックする条件の順番は重量です。
また最後に else を書くか書かないかも重要です。

この点に注意して質問文のコードを見直す、多の回答のコードを研究すると良いです。

Answer 2

閏年になる条件をまとめると、

• 400で割り切れる
• 4で割り切れ、かつ100で割り切れない
  のいずれかになります。

これを条件式として書けば、以下のようになります。

def leap_year_judgment(target_year):
    if target_year % 400 == 0 or (target_year % 4 == 0 and target_year % 100 != 0):
        print(target_year, "は閏年です")
    else:
        print(target_year, "は閏年ではありません")

Answer 3

elif target_year % 100 == 0:
        print(target_year,"は閏年ではありません")

上記の部分のprintが作用するための条件を見ると、

elif target_year % 100 == 0 and target_year % 400 == 0:

となっています。
これは、100で割り切れて、さらに400でも割り切れると言うことです。
このような条件の数字が引数に代入されたとしても、それ以前の

if target_year % 400 == 0:

や

elif target_year % 4 == 0:

で、動作してしまうので、閏年ではない判定の出力が出ないのだと思います。

おそらく100でも400でも割り切れないと言う条件を実装しようとしてらっしゃるのだと推測しますので、「イコールでない」を表す!=を用いて

elif target_year % 100 != 0 and target_year % 400 != 0:
  print(target_year,"は閏年ではありません")

と実装されるか、もしくは単に、それまでのいずれの条件にも一致しないものと言う条件のelseを用いて、

else:
  print(target_year,"は閏年ではありません")

と実装されるといいと思います。

Answer 4

たとえば、2001年の場合を考えてみましょう

1
target_year % 400 == 0
この条件にはあてはまりません

2
target_year % 4 == 0
この条件にもあてはまりません

3
target_year % 100 == 0 and target_year % 400 == 0
この条件は、まず前半だけ考えましょう

target_year % 100 == 0
この条件にあてはまりません

ということは、and なので、後半の条件を考えるまでもなく、あてはまりません

1, 2, 3 より、すべての条件にあてはまらないので
なにも出力されません

閏年の判定のしかた

閏年の条件が示されていないので、
ここでの解説は次の Wikipedia の定義に従います

西暦年が4で割り切れる年は(原則として)閏年。
ただし、西暦年が100で割り切れる年は(原則として)平年。
ただし、西暦年が400で割り切れる年は必ず閏年。

この「ただし」を if として実装します
そして、条件に示されていない else を忘れないようにします

def leap_year_judgment(target_year):
    """
    閏年かどうかを判定する
        西暦年が4で割り切れる年は(原則として)閏年。
        ただし、西暦年が100で割り切れる年は(原則として)平年。
        ただし、西暦年が400で割り切れる年は必ず閏年。
    https://ja.wikipedia.org/wiki/%E9%96%8F%E5%B9%B4
    """
    if target_year % 400 == 0:
        print(target_year,"は閏年です")
    elif target_year % 100 == 0:
        print(target_year,"は閏年ではありません")
    elif target_year % 4 == 0:
        print(target_year,"は閏年です")
    else:
        print(target_year,"は閏年ではありません")


leap_year_judgment(2000)
leap_year_judgment(2001)
leap_year_judgment(2004)
leap_year_judgment(2100)

実行結果:

$ python test.py
2000 は閏年です
2001 は閏年ではありません
2004 は閏年です
2100 は閏年ではありません

また、論理演算はベン図を書くと理解が深まるかもしれません

Answer 5

閏年の条件を知っていますか？
・4で割れれば閏年
・しかし100で割れれば閏年で無い
・でも400で割れればやっぱり閏年
・4で割れない場合は閏年じゃ無い
の4パターンです。3パターンではありません。

あと、条件判断の順番もおかしいです。どの条件から先にチェックすべきかよく考えましょう。

あるいは、4条件を並列じゃなくて、上記の通りにコーディングするのも手です。

    if target_year % 4 == 0:
        if arget_year % 100 == 0:
            if arget_year % 400 == 0:
                print(target_year,"は閏年です")
            else:
                print(target_year,"は閏年ではありません")
        else:
            print(target_year,"は閏年です")
    else:
        print(target_year,"は閏年ではありません")