今、C言語の勉強をしていて、「西暦と月をscanfで入力し、”西暦●●

質問

今、C言語の勉強をしていて、「西暦と月をscanfで入力し、”西暦●●年の●月は●日です”というプログラムを完成させなさい」という問題がありました。
また、条件が「4の倍数で、かつ100の倍数でない場合か、400の倍数の場合は閏年である」また
「西暦が０以下のとき、月が1以下12以上の場合はエラーメッセージを表示させなさい。」というものです。

そこで、コーディングした結果、下記のようになりました。

#include <stdio.h>
main()
{ int y,m,d;
printf("西暦＝");scanf("%d",&y);
printf("月＝");scanf("%d",&m);
if(y<0 || d>1 && d<12)
printf("エラーです。");
else if(m==1 || m==3 || m==5 || m==7 || m==8 || m==10 || m==12)
d=31;
printf("西暦%d年の%d月は%d日です。",y,m,d);
else if(m==4 || m==6 || m==9 || m==11 )
d=30;
printf("西暦%d年の%d月は%d日です。",y,m,d);
else if(m==2 && y%4==0 && y%100>0 || y%400==0)
d=29;
printf("西暦%d年の%d月は%d日です。",y,m,d);
else d=29;
printf("西暦%d年の%d月は%d日です。",y,m,d);
}

これを、学校の先生が開発したコンパイラで文法チェックを施したところ、
「予約語elseは使用できません」と文法エラーがでました。
「else」は、if文のthen/elseの意味で利用したのにそういう風にとらえてくれませんでした。
また、字下げは適当にやっているだけですので気にしないでください。

これ、どこかおかしいところありますか・・・？

回答No.2

silverbear · ベストアンサー

else if(m==2 && y%4==0 && y%100>0 || y%400==0)
d=29;
printf("西暦%d年の%d月は%d日です。",y,m,d);
else d=29;

ここですよね。
{}をつけない場合、if文は次の一行のみに効果を発揮します。
つまり、printfの一行はもうifの範囲を出ています。
ifの効果を発揮させたい範囲を{}で囲んでください。

また、こういった間違いを防ぐためにもif文を使う場合は極力{}を使うようにした方が良いと思います。

回答No.1

asuncion

＞「予約語elseは使用できません」と文法エラーがでました。

というエラーが出るのは、if文とelse句の勢力範囲を適切に書いていないからです。
if (何とか) {
// 何かの文
}
else {
// 何かの文
}
という風に、「何かの文」がたとえ1個だけの場合でも　{ と } で必ず囲む方がよいでしょう。

＞西暦が０以下のとき、月が1以下12以上の場合

西暦が０以下のとき
と
月が1以下12以上の場合
の間に入るのは「または」で間違いないのですね？

それにしたところで、

西暦が０以下のとき
と
y<0
とは食い違っています。
また、
月が1以下12以上の場合
と
d>1 && d<12
とも食い違っています。
そもそも、
月が1以下12以上の場合
がエラーであるとすると、1月や12月という正しい月までもが
エラーとみなされてしまいます。それでよいのでしょうか。

さらに、
d=29;
と
else d=29;
だと、平年の2月の日数を正しく求めていませんね。

さらにさらに、
printf("西暦%d年の%d月は%d日です。",y,m,d);
という同じ文が4回も登場する必要はありません。
最後の一度だけで十分なはずです。