コードをスリムにしたいです。

Question

以下のコードのもう少しスリムな書き方はありますか。

#include <stdio.h>
#pragma warning(disable: 4996)

int main(void) 
{
    int mon,day;
    printf("何月ですか?");
    scanf("%d",&mon);
    printf("何日ですか?");
    scanf("%d",&day);

    switch (mon) {
    case 1:
        if (day <= 19) {
            printf("あなたはやぎ座です");
        } else {
            printf("あなたはみずがめ座です");
        }
        break;
    case 2:
        if (day <= 18) {
            printf("あなたはみずがめ座です");
        } else {
            printf("あなたはうお座です");
        }
        break;
    case 3:
        if (day <= 20) {
            printf("あなたはみずがめ座です");
        } else {
            printf("あなたはおひつじ座です");
        }
        break;
　　　　・
　　　　・
　　　　・
}

Answer 1

ちなみに、コレ系のやつは条件をデータにしといてループでチェックするとラクです。

  struct seiza {
      char name[30];
      int st;
      int ed;
  };
  struct seiza data[13] = {
    {"おひつじ座",321,419},
    {"おうし座",420,520},
    {"ふたご座",521,621},
    {"かに座",622,722},
    {"しし座",723,822},
    {"おとめ座",823,922},
    {"てんびん座",923,1023},
    {"さそり座",1024,1122},
    {"いて座",1123,1121},
    {"やぎ座",1222,1231},
    {"やぎ座",101,119},
    {"みずがめ座",120,218},
    {"うお座",219,320}
  };
/* 以下略 */

--- 追記 ---
例えばこんな感じス

  int x = mon * 100 + day;
  int i;
  for (i=0; i<13; i++) {
      if (data[i].st <= x && data[i].ed >= x) {
          printf("あなたは%sです。",data[i].name);
          break;
      }
  }

Comments

[Red_Bull]

構造体の使い方がまだあまりわかっていないのでループで書く場合にはどのようにしたら良いのかもう少し詳しく教えていただきたいです

[Red_Bull]

ありがとうございます。
そのように使ったら良いんですね！

Answer 2

「月の100倍＋日」で1つの値にしてしまえば、年月を一気に判定できます。

int val = mon * 100 + day;
if(val <= 119) {
  printf("あなたはやぎ座です");
} else if (val <= 218) {
  printf("あなたはみずがめ座です");
}...

Comments

[Red_Bull]

ありがとうございます！