カレンダーのプログラム

質問

指定した開始月と終了月を入力して、それを横に最大３つまで並べて
納まりきらないときは改行するというプログラムを作りたいです。

イメージとしては、2009年の9月から2009年の12月まで指定した場合は…

2009 / 9 　　　　　　　　　　　　　2009 / 10 　　　　　　　　　　　　2009 / 11
1 2 3 4 5 6 7 　　　　　　　　　1 2 3 4 5 6 7 　　　　　　　　　　1 2 3 4 5 6 7
8 9 10 11 12 13 14 　　　　　8 9 10 11 12 13 14 　　　　　　8 9 10 11 12 13 14
15 16 17 18 19 20 21 　　　15 16 17 18 19 20 21 　　　15 16 17 18 19 20 21
21 22 23 24 25 26 27 　　　21 22 23 24 25 26 27 　　　21 22 23 24 25 26 27
28 29 30 　　　　　　　　　　28 29 30 31 　　　　　　　　　　28 29 30

　　2009 / 12
1 2 3 4 5 6 7
8 9 10 11 12 13 14
15 16 17 18 19 20 21
21 22 23 24 25 26 27
28 29 30 31
（最初の日数などは適当ですし、ちょっとずれてしまっていますがご了承ください。あくまでもイメージなので…）

ところが下のプログラムを動作させようとしたところ全く
動きません。何がおかしいために動作しないのでしょうか？

よろしくお願いします。

#include<stdio.h>
#include<string.h>

int getyoubi(int year,int month,int day)
{

if(month==1 || month==2 )
{
year--;
month +=12;
}

return ((year+year/4-year/100+year/400+(13*month+8)/5 + day)%7);
}

int main(void)
{
int nen1,tuki1,nen2,tuki2,tukiho,nenho,tukih,w,i,j;
char sbuf[3][7][22];
int tuki[12]={31,28,31,30,31,30,31,31,30,31,30,31};
int k;
char tmp[4];

printf("カレンダーを表示します。\n");
printf("開始年月を入力せよ。\n");
printf("年：");
scanf("%d",&nen1);
printf("月：");
scanf("%d",&tuki1);
printf("終了年月を入力せよ。\n");
printf("年：");
scanf("%d",&nen2);
printf("月：");
scanf("%d",&tuki2);

do{

/*３つのカレンダー分の格納する*/
for(i=0;i<3;i++)
{

w=getyoubi(nen1,tuki1,1);

sprintf(sbuf[i][0],"%10d /%3d ",nen1,tuki1);

for(k=1;j<7;k++)
{
sbuf[i][k][0]='\0';
}

sprintf(sbuf[i][1],"%*s",3*w,"");
k=1;
for(j=1;j<tuki[tuki1];j++)
{
sprintf(tmp,"%3d",j);
strcat(sbuf[i][k],tmp);
if(j%7==6)
{
k++;
}
}
do{
if(j%7==6)
{
sprintf(sbuf[i][k],"%21s","");
}
else
{
do{
strcat(sbuf[i][k]," ");
j++;
}while(j%7!=6);
k++;
}
}while(k!=7);

tuki1++;
if(tuki1==13)
{
tuki1=1;
nen1++;
}

if(nen1>nen2 || nen1==nen2 && tuki1>tuki2)
{
break;
}

}

/*3つためたカレンダーを出力する。*/
for(i=0;i<3;i++)
{
printf("%s",sbuf[i][0]);
}

printf("\n");

for(i=0;i<3;i++)
{
printf(" 日 月 火 水 木 金 土 ");
}

printf("\n");

for(i=0;i<3;i++)
{
printf("----------------------");
}

printf("\n");

for(i=1;i<7;i++)
{
for(j=0;j<3;j++)
{
printf("%s",sbuf[j][i]);
}
}

/*指定した年の月になるまでループ*/
}while(nen1<nen2 || nen1==nen2 && tuki1<=tuki2);

return 0;
}

回答No.2

mtaka2 · ベストアンサー

動かない最大の要因は、62行目からのループ内で、if文のelse節でのみ、kが増えることですね。
そのため、else節に入らない場合には、いつまで経ってもkが変わらず、ループ終了条件を満たさないため、無限ループになってしまいます。
そのほか、
・閏年の2月の日数が正しく表示されない
・2009/9～2009/12 を指定した場合、2009/10・2009/11がダブって表示される(最後の3ヶ月分を全部表示しない場合、残りの2ヶ月は直前の月が表示される)
といった問題は残ってますが、そのあたりも修正してしまうと、元のコードの面影が無くなって、どこがおかしかったのか分かりにくいですから、体裁が整う必要最低限の修正だけをしてみました。
修正した部分にはコメントを入れてます。
---ここから---
#include<stdio.h>
#include<string.h>

int getyoubi(int year,int month,int day)
{

if(month==1 || month==2 )
{
year--;
month +=12;
}

return ((year+year/4-year/100+year/400+(13*month+8)/5 + day)%7);
}

int main(void)
{
int nen1,tuki1,nen2,tuki2,tukiho,nenho,tukih,w,i,j;
char sbuf[3][7][22];
int tuki[12]={31,28,31,30,31,30,31,31,30,31,30,31};
int k;
char tmp[4];

printf("カレンダーを表示します。\n");
printf("開始年月を入力せよ。\n");
printf("年：");
scanf("%d",&nen1);
printf("月：");
scanf("%d",&tuki1);
printf("終了年月を入力せよ。\n");
printf("年：");
scanf("%d",&nen2);
printf("月：");
scanf("%d",&tuki2);

do{

/*３つのカレンダー分の格納する*/
for(i=0;i<3;i++)
{

w=getyoubi(nen1,tuki1,1);

sprintf(sbuf[i][0],"%10d /%3d ",nen1,tuki1);

for(k=1;k<7;k++) /* j<7 ではなく k<7 に */
{
sbuf[i][k][0]='\0';
}

sprintf(sbuf[i][1],"%*s",3*w,"");
k=1;
for(j=1;j<=tuki[tuki1-1];j++)/* 「j<tuki[…」 だと、最終日が表示されない。配列tukiは添字0から始まるので、「tuki[tuki1]」ではなく「tuki[tuki1-1]」に */
{
sprintf(tmp,"%3d",j);
strcat(sbuf[i][k],tmp);
if((j+w-1)%7==6) /* 「j%7==6」だと、月の開始曜日までスペースで埋めた分、最初の行の日数がずれる */
{
k++;
}
}
do{
if((j+w-1)%7==0) /* 「j%7==6」だと、開始曜日の分ずれる。変数jは最終日の次の日になってるので、6との比較ではなく0と比較*/
{
sprintf(sbuf[i][k],"%21s","");
}
else
{
do{
strcat(sbuf[i][k]," ");
j++;
}while((j+w-1)%7!=0);/* 「j%7==6」だと、開始曜日の分ずれる。変数jは最終日の次の日になってるので、6との比較ではなく0と比較*/
}
k++; /* ループ中では、必ずkに1を足さなければならない。元のコードでは、上記ifのelse時のみkに1を足していたため、無限ループになっていた */
}while(k!=7);

tuki1++;
if(tuki1==13)
{
tuki1=1;
nen1++;
}

if(nen1>nen2 || nen1==nen2 && tuki1>tuki2)
{
break;
}

}

/*3つためたカレンダーを出力する。*/
for(i=0;i<3;i++)
{
printf("%s",sbuf[i][0]);
}

printf("\n");

for(i=0;i<3;i++)
{
printf(" 日 月 火 水 木 金 土 ");
}

printf("\n");

for(i=0;i<3;i++)
{
printf("----------------------");
}

printf("\n");

for(i=1;i<7;i++)
{
for(j=0;j<3;j++)
{
printf("%s ",sbuf[j][i]);
}
printf("\n");/* ここで改行が必要 */
}

/*指定した年の月になるまでループ*/
}while(nen1<nen2 || nen1==nen2 && tuki1<=tuki2);

return 0;
}
---ここまで---

お礼

回答ありがとうございました。
ご指摘の通り修正したところ自分のプログラムの至らなさを痛感しました。
抜本的に修正していきたいと思っております。

回答No.1

asuncion

＞printf("月：");
＞scanf("%d",&tuki1);

ここで入力する月は１～１２を想定していますか？
だとすると、

＞int tuki[12]={31,28,31,30,31,30,31,31,30,31,30,31};
＞for(j=1;j<tuki[tuki1];j++)

ゼロ始まりの配列と１つずれてませんか？

他にもおかしいところがあるかもしれませんが、
ソースを全部見たわけではありません。あしからず。

お礼

回答ありがとうございました。
確かに、配列は0から始まるのでずれてしまいますね。
修正したいと思います。