C言語について プログラミング

Question

前提・実現したいこと

C言語でキーボードから入力した日付の、次の日付を表示させたいです
閏年は、考えていません

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

エラーは表示されなかったのですが、キーボード入力した日付がそのまま明日の日付として表示されてしまいます

### 該当のソースコード
#include<stdio.h>

void tomorrow(int *y, int *m, int *d)
{
  int day[12]={31,28,31,30,31,30,31,31,30,31,30,31};　//各月の最終日のテーブル

  if(*d==day[*m]){
    *d=1;　//月末だった場合、1日に戻して、月を１増やす
    *m++;
    if(*m==12){
      *m=1;　//年末の場合
      *y++;
    }else{
      *d++;　//それ以外
    }
  }
}


int main(void)
{
  int year1, year2;
  int month1, month2;
  int day1, day2;
  
  printf("本日の日付を入力してくだい\n");
  printf("年 月 日\n");
  scanf("%d %d %d", &year1, &month1, &day1);
  
  year2=year1;
  month2=month1;
  day2=day1;
  
  tomorrow(&year2, &month2, &day2);
  
  printf("明日の日付 : %d 年 %d 月 %d 日\n", year2, month2, day2);

  return 0;
}
```C　C言語

試したこと

月末処理や関数の中身など、色々と変更して試したのですが、正しく出力できませんでした。

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

Answer 1

[低評価は覚悟の上] こんな回答は求められていない(よね?)

#define _CRT_SECURE_NO_WARNINGS /* desable C4996 (for VC++) */
#include <stdio.h>
#include <time.h>  // mktime, gmtime
#include <stdlib.h>
#include <string.h> // memset

void tomorrow(int* y, int* m, int* d) {
  struct tm today;
  memset(&today, 0, sizeof(struct tm));
  today.tm_hour = 9; /* ちょっとズルした^^; */
  today.tm_year = *y - 1900;
  today.tm_mon  = *m - 1;
  today.tm_mday = *d;
  time_t t = mktime(&today) + 3600*24 ; // 一日後(3600*24秒後)
  struct tm* tomorrow = gmtime(&t);
  *y = tomorrow->tm_year + 1900;
  *m = tomorrow->tm_mon + 1;
  *d = tomorrow->tm_mday;
}


int main(void) {
    int year = 2020;
    int month = 2;
    int day = 29;

    printf("今日の日付 : %d 年 %d 月 %d 日\n", year, month, day);
    tomorrow(&year, &month, &day);
    printf("明日の日付 : %d 年 %d 月 %d 日\n", year, month, day);

    return 0;
}

Answer 2

if(*d==day[*m]){

まずこの条件に一致しない限り何も処理が行われないのが問題。

あとday配列は0オリジンなので、月で参照するときはday[*m-1]とする必要がある。
もしくはday[13]として、day[0]はダミーにするか。

Answer 3

if(*d==day[*m]){

この条件が成立してない場合、なにもしないでリターンしてしまいますね

Answer 4

表検索(table lookup) の配列には static を付けましょう。
そうでないと、int day[12]; a[0] = 31; a[1] = 28; ... a[11] = 31; という
コードに展開されてしまいます。

&& を使って書くと、

void tomorrow(int *y, int *m, int *d)
{
	static int day[] = { 31,28,31,30,31,30,31,31,30,31,30,31 };
	++*d > day[*m - 1] && (*d = 1, ++*m > 12) && (*m = 1, ++*y);
}

&& の代わりに if文を使えば、

	if (++*d > day[*m - 1]) {
		*d = 1;
		if (++*m > 12) {
			*m = 1;
			++*y;
		}
	}