[發問] 請幫我看看哪裡有問題

jhwish

請高手幫忙看看，

藍色的字體是我自己加的,黑色字體是老師寫的,

不知道哪寫錯了... >"< 我是初學者


using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace hw6
{
    class Program
    {
        static void Main(string[] args)
        {
            int momth, day;
            string star;
            Console.WriteLine("請輸入出生月份");
            momth = int.Parse(Console.ReadLine());
            Console.WriteLine("請輸入出生日");
            day = int.Parse(Console.ReadLine());

            if (momth == 1) { if (day >= 20) { star = "水瓶座"; } else { star = "摩羯座"; } }
            if (momth == 2) { if (day >= 19) { star = "雙魚座"; } else { star = "水瓶座"; } }
            if (momth == 3) { if (day >= 21) { star = "牡羊座"; } else { star = "雙魚座"; } }
            if (momth == 4) { if (day >= 20) { star = "金牛座"; } else { star = "牡羊座"; } }
            if (momth == 5) { if (day >= 21) { star = "雙子座"; } else { star = "金牛座"; } }
            if (momth == 6) { if (day >= 22) { star = "巨蟹座"; } else { star = "雙子座"; } }
            if (momth == 7) { if (day >= 23) { star = "獅子座"; } else { star = "巨蟹座"; } }
            if (momth == 8) { if (day >= 23) { star = "處女座"; } else { star = "獅子座"; } }
            if (momth == 9) { if (day >= 23) { star = "天秤座"; } else { star = "處女座"; } }
            if (momth == 10) { if (day >= 24) { star = "天蠍座"; } else { star = "天秤座"; } }
            if (momth == 11) { if (day >= 23) { star = "射手座"; } else { star = "天蠍座"; } }
            if (momth == 12) { if (day >= 22) { star = "魔羯座"; } else { star = "射手座"; } }
  


            else { star = "非人類";}
            Console.WriteLine("你的星座是" + star);
            Console.Read();
        }
    }
}

c_c_lai

回復 1# jhwish
妳的本意是不是如下：

1.     if (momth == 1) { if (day >= 20) { star = "水瓶座"; } else { star = "摩羯座"; } }
   
2.     else if (momth == 2) { if (day >= 19) { star = "雙魚座"; } else { star = "水瓶座"; } }
   
3.     else if (momth == 3) { if (day >= 21) { star = "牡羊座"; } else { star = "雙魚座"; } }
   
4.     else if (momth == 4) { if (day >= 20) { star = "金牛座"; } else { star = "牡羊座"; } }
   
5.     else if (momth == 5) { if (day >= 21) { star = "雙子座"; } else { star = "金牛座"; } }
   
6.     else if (momth == 6) { if (day >= 22) { star = "巨蟹座"; } else { star = "雙子座"; } }
   
7.     else if (momth == 7) { if (day >= 23) { star = "獅子座"; } else { star = "巨蟹座"; } }
   
8.     else if (momth == 8) { if (day >= 23) { star = "處女座"; } else { star = "獅子座"; } }
   
9.     else if (momth == 9) { if (day >= 23) { star = "天秤座"; } else { star = "處女座"; } }
   
10.     else if (momth == 10) { if (day >= 24) { star = "天蠍座"; } else { star = "天秤座"; } }
    
11.     else if (momth == 11) { if (day >= 23) { star = "射手座"; } else { star = "天蠍座"; } }
    
12.     else if (momth == 12) { if (day >= 22) { star = "魔羯座"; } else { star = "射手座"; } }
    
13.     else { star = "非人類";}

複製代碼

倘若是，建議妳使用 Switch 語法較能表達。

c_c_lai

回復 1# jhwish
原本之內容為：

1.     '  假設 day = 23
   
2.     if (momth == 1) {
   
3.         if (day >= 20) {
   
4.             star = "水瓶座";
   
5.         } else {
   
6.             star = "摩羯座";
   
7.         }
   
8.     }
   
9.     else {
   
10.         star = "非人類";
    
11.     }
    
12.     '  只要是一月份且 day >= 20 的都是 "水瓶座"，否則為 "摩羯座"；其他月份的都是 "非人類"。

複製代碼

經妳修正後變為：

1.     '  假設 day = 23
   
2.     if (momth == 1) {
   
3.         if (day >= 20) {
   
4.             star = "水瓶座";
   
5.         } else {
   
6.             star = "摩羯座";
   
7.         }
   
8.     }
   
9.     '  只要是一月份且 day >= 20 的都是 "水瓶座"，否則為 "摩羯座"。
   
10. 
    
11.     if (momth == 2) { if (day >= 19) { star = "雙魚座"; } else { star = "水瓶座"; } }
    
12.     if (momth == 3) { if (day >= 21) { star = "牡羊座"; } else { star = "雙魚座"; } }
    
13.     if (momth == 4) { if (day >= 20) { star = "金牛座"; } else { star = "牡羊座"; } }
    
14.     if (momth == 5) { if (day >= 21) { star = "雙子座"; } else { star = "金牛座"; } }
    
15.     if (momth == 6) { if (day >= 22) { star = "巨蟹座"; } else { star = "雙子座"; } }
    
16.     if (momth == 7) { if (day >= 23) { star = "獅子座"; } else { star = "巨蟹座"; } }
    
17.     if (momth == 8) { if (day >= 23) { star = "處女座"; } else { star = "獅子座"; } }
    
18.     if (momth == 9) { if (day >= 23) { star = "天秤座"; } else { star = "處女座"; } }
    
19.     if (momth == 10) { if (day >= 24) { star = "天蠍座"; } else { star = "天秤座"; } }
    
20.     if (momth == 11) { if (day >= 23) { star = "射手座"; } else { star = "天蠍座"; } }
    
21.      if (momth == 12) {
    
22.         if (day >= 22) {
    
23.             star = "魔羯座";
    
24.         } else {
    
25.             star = "射手座";
    
26.         }
    
27.     }
    
28.     else {
    
29.         star = "非人類";
    
30.     }
    
31.     '  最後結果只要是十二月份且 day >= 22 的都是 "魔羯座"，否則為 "射手座"；
    
32.     '  其他不符條件的都是 "非人類"。

複製代碼