【C言語の質問です】西暦年を引数とし、閏年かどうか判定する関数について

Question

＊たくさんの方よりご指摘をいただけてとてもありたいです。
引き続き、よろしくお願い致します。

C言語を勉強している初心者のものです。

今課題をいただき、その作成をしております。
先日提出(全18問）したのですが、大部分が間違っているので、
再度問題を見なおした上で提出してくれと言われてしまいました。

今回は課題なので、間違っている点を教えていただけませんでした。
個人的には、どの辺りが間違っているのかわからないため、
客観的な視点が欲しく、今回ご相談させていただきました。

18問の中の1つですが、
ソースが問題文との趣旨と外れているようでしたら
ご指摘いただけますと幸いです。

【問題】
西暦年を引数とし、閏年かどうか判定する関数を作成せよ。

【閏年の条件】
・年が400の倍数なら閏年である
・年が400の倍数でない、100の倍数でなければ閏年ではない
・年が100の倍数でない、4の倍数であれば閏年である
・年が4の倍数でなければ閏年ではない

【回答】

#include <stdio.h>

int uruu(int year);

int main(void)
{
	int year,hold;
	
	printf("西暦を入力してください\n");
	scanf("%d",&year);
	hold = uruu(year);
	
	switch (hold)
	{
		case 0:
			printf("閏年ではありません。\n");
			break;
		case 1:
			printf("閏年です。\n");
			break;
		case 2:
			printf("数値がマイナスです。再度入力しなおしてください。\n");
			break;
	};

	return 0;
}

int uruu(int year)
{
	if(year <= 0)
	{
		return 2;
	}
	
	if (year % 400 == 0) 
	{
		return 1;
	} 
	 if (year % 4 == 0) 
	{
		return 1;
	} 
	
	if (year % 100 != 0)
	{
		return 0;
	}
	if (year % 4 != 0)
	{
		return 0;
	}
	return 0;
}

皆様いつもご回答ありがとうございます。

Comments

[WoodenHamlet]

そもそも西暦元年の扱い（西暦0年を許すか）、紀元前の入力を許すか、の問題がありますね。

[退会済みユーザー]

ご回答ありがとうございます。
今、0年と打つと、
>printf("数値がマイナスです。再度入力しなおしてください。\n");
こちらの回答が返ってくるようになってはいます・・
紀元前400年などは、条件面に沿っていれば閏年と出ますが、
ここの判断は難しいですね。。。
入力を許さない場合は、do-whileを使う形でしょうか。

[WoodenHamlet]

ん？紀元前だとたとえば-400が入力になるからエラーでないの？

[退会済みユーザー]

すいません、間違えました。。。
マイナス入力は
>数値がマイナスです。再度入力しなおしてください。\n");
このように再度入力をお願いする形してしています。

[WoodenHamlet]

まあ紀元前をエラーとするなら今のままでいいと思うけど、入力していいなら紀元年は存在しない（数直線上の０がなくて―1の次が１になってるよね、年表じゃ）のを留意する必要があるね

[退会済みユーザー]

なるほど。。。
たしかにそのとおりですね！
ちょっと考えてみます！
ありがとうございました！！

Answer 1

enum は習われましたか？関数の返り値が数通りで、その返り値で分岐を行う場合、enum を用いると可読性が上がります

#include <stdio.h>

typedef enum {NOT_URUU, IS_URUU, ERROR_URUU} urRet;
urRet uruu(int year);

int main(void)
{
    int year;

    printf("西暦を入力してください\n");
    scanf("%d",&year);

    switch (uruu(year))
    {
        case NOT_URUU:
            printf("閏年ではありません。\n");
            break;
        case IS_URUU:
            printf("閏年です。\n");
            break;
        case ERROR_URUU:
            printf("数値がマイナスです。再度入力しなおしてください。\n");
            break;
        default:		// たとえ絶対に入らないような場合でも default はつけましょう
        	break;
    };

    return 0;
}

urRet uruu(int year)
{
    if(year <= 0)
    {
        return ERROR_URUU;
    }

	if ( (year % 400 == 0) 
	   ||(year % 4   == 0 && year % 100 != 0) )
	{
		return IS_URUU;
	} else {
		return NOT_URUU;
	}
}

Comments

[退会済みユーザー]

ご回答ありがとうございます。

すいません、まだenumは習っておりませんが、
記載頂いた文書を元に勉強してみたいと思います。
ありがとうございました！！

Answer 2

こんにちは。

問題文自体がなんか変です。

・年が400の倍数なら閏年である

・年が400の倍数でない、100の倍数でもなければ閏年ではない
・年が100の倍数でない、4の倍数であれば閏年である
・年が4の倍数でなければ閏年ではない

400の倍数でない100の倍数の時を定義してないですね。

誤> ・年が400の倍数でない、100の倍数でもなければ閏年ではない
正> ・年が400の倍数でない、100の倍数であれば閏年ではない

ですね？

次にもう少し厳密に表現すると下記と思います。

・年が400の倍数なら、閏年である

・年が400の倍数でない、かつ、100の倍数であれば、閏年ではない
・年が100の倍数でない、かつ、4の倍数であれば、閏年である
・年が4の倍数でなければ、閏年ではない

この通りに条件判断を作ってみてはどうでしょうか？

そして、試しに-1年くらいから3000年くらいまでfor文で回して、上記のどの条件にも当てはまらないケースがあるのかないのか見てみる必要があるかも。なんか抜けてそうな気がしません？
もし、抜けてこなかったら、必要のない条件判断をしていることになります。それはどれでしょうか？
そして、更に何か不要な条件判断をしているかも知れません。

Comments

[退会済みユーザー]

ご回答ありがとうございます。
すいません、修正させていただいたのですが、
私に記載に一部誤りがありました。
誤> ・年が400の倍数でない、100の倍数でもなければ閏年ではない
正> ・年が400の倍数でない、100の倍数でなければ閏年ではない

とはいえ、
>正> ・年が400の倍数でない、100の倍数であれば閏年ではない
こちらの認識にて間違いないです。
ご指摘ありがとうございました。
早速そちらの条件面にて作ってみます。

そして、試しに-1年くらいから3000年くらいまでfor文で回して、上記のどの条件にも当てはまらないケースがあるのかないのか見てみる必要があるかも。なんか抜けてそうな気がしません？
もし、抜けてこなかったら、必要のない条件判断をしていることになります。それはどれでしょうか？
そして、更に何か不要な条件判断をしているかも知れません。
上記もありがとうございます。
一旦for文を作ってみたいと思いますが、
for文を作ってエラーがなければ問題ないという認識でよろしかったでしょうか。

[Chironian]

プログラムの動作としては問題ないと思います。
プログラムの作りとしては、あまり褒められたプログラムにはなってないと思います。

バグの多くは条件判断で発生します。つまり、無用な条件判断が残っているとバグの元ですし、仕様変更する際、バグを避けるため無駄に苦労しますので。

[退会済みユーザー]

なるほど！
条件判断は難しいですね・・・
文章を整理してからプログラミングしていきたいと思います。
いつもありがとうございます！

Answer 3

400で割りきれない、かつ100で割り切れるのはうるう年ではなくなるので、判定順序はこんな感じ。

if ( year <= 0 ) { return 2; }
if ( year % 400 == 0 ) { return 1; }
if ( year % 100 == 0 ) { return 0; }
if ( year % 4 == 0 ) { return 1; }
return 0;

Answer 4

このままだと、1900年のような「100で割り切れるけど400で割り切れない年」を誤判定してしまいます。「400で割り切れるかチェック」→「100で割り切れるかチェック」→「4で割り切れるかチェック」という順番にする必要があります。

Comments

[退会済みユーザー]

ご回答ありがとうございます。
すいません、私の認識が間違っていたらぜひ教えていただきたいのですが、
条件の2つ目なのですが、400で割り切れなかった数値でも
100で割り切れれば閏年になりますでしょうか。
100の倍数でなければ閏年ではないでしたので、
100の倍数であれば閏年なのかと思いました。
いかがでしょうか。

[退会済みユーザー]

すいませんでした、私の認識が間違っていたようです。。
400で割り切れなくても100で割り切れたらそれは閏年でないということですね。
ご指摘どおりでした！
ありがとうございます。