快捷導(dǎo)航

小菜編程成長記（一面試受挫——代碼無錯就是好？）第1/3頁

更新時間：2006年10月17日 00:00:00 作者：

小菜今年計算機(jī)專業(yè)大四了，學(xué)了不少軟件開發(fā)方面的東西，也學(xué)著編了些小程序，躊躇滿志，一心要找一個好單位。當(dāng)投遞了無數(shù)份簡歷后，終于收到了一個單位的面試通知，小菜欣喜若狂。
        到了人家單位，前臺小姐給了他一份題目，上面寫著，“請用C++、Java、C#或VB.NET任意一種面向?qū)ο笳Z言實現(xiàn)一個計算器控制臺程序，要求輸入兩個數(shù)和運算符號，得到結(jié)果。”
        小菜一看，這個還不簡單，三下五除二，10分鐘不到，小菜寫完了，感覺也沒錯誤。交卷后，單位說一周內(nèi)等通知吧。于是小菜只得耐心等待。可是半個月過去了，什么消息也沒有，小菜很納悶，我的代碼實現(xiàn)了呀，為什么不給我機(jī)會呢。
        小菜找到工作三年的師哥大鳥，請教原因，大鳥問了題目和了解了小菜代碼的細(xì)節(jié)以后，哈哈大笑，說道：“小菜呀小菜，你上當(dāng)了，人家單位出題的意思，你完全都沒明白，當(dāng)然不會再聯(lián)系你了”。
        小菜說：“我的代碼有錯嗎？單位題目不就是要我實現(xiàn)一個計算器的代碼嗎，我這樣寫有什么問題?！?BR>

復(fù)制代碼代碼如下:

class Program 
{ 
    static void Main(string[] args) 
    { 
        Console.Write("請輸入數(shù)字A："); 
        string A = Console.ReadLine(); 
        Console.Write("請選擇運算符號(+、-、*、/)："); 
        string B = Console.ReadLine(); 
        Console.Write("請輸入數(shù)字B："); 
        string C = Console.ReadLine(); 
        string D = ""; 

        if (B == "+") 
            D = Convert.ToString(Convert.ToDouble(A) + Convert.ToDouble(C)); 
        if (B == "-") 
            D = Convert.ToString(Convert.ToDouble(A) - Convert.ToDouble(C)); 
        if (B == "*") 
            D = Convert.ToString(Convert.ToDouble(A) * Convert.ToDouble(C)); 
        if (O == "/") 
            D = Convert.ToString(Convert.ToDouble(A) / Convert.ToDouble(C)); 

        Console.WriteLine("結(jié)果是：" + D); 
    }      
}

小菜的代碼有什么問題呢？
二代碼規(guī)范、重構(gòu)

大鳥說：“且先不說出題人的意思，單就你現(xiàn)在的代碼，就有很多不足的地方需要改進(jìn)。比如變量命名，你的命名就是ABCD，變量不帶有任何具體含義，這是非常不規(guī)范的；判斷分支，你這樣的寫法，意味著每個條件都要做判斷，等于計算機(jī)做了三次無用功；數(shù)據(jù)輸入有效性判斷等，如果用戶輸入的是字符符號而不是數(shù)字怎么辦？如果除數(shù)時，客戶輸入了0怎么辦？這些都是可以改進(jìn)的地方?！?

“哦，說得沒錯，這個我以前聽老師說過，可是從來沒有在意過，我馬上改，改完再給你看看?！?BR>

復(fù)制代碼代碼如下:

class Program 
{ 
static void Main(string[] args) 
{ 
try 
{ 
Console.Write("請輸入數(shù)字A："); 
string strNumberA = Console.ReadLine(); 
Console.Write("請選擇運算符號(+、-、*、/)："); 
string strOperate = Console.ReadLine(); 
Console.Write("請輸入數(shù)字B："); 
string strNumberB = Console.ReadLine(); 
string strResult = ""; 

switch (strOperate) 
{ 
case "+": 
strResult = Convert.ToString(Convert.ToDouble(strNumberA) + Convert.ToDouble(strNumberB)); 
break; 
case "-": 
strResult = Convert.ToString(Convert.ToDouble(strNumberA) - Convert.ToDouble(strNumberB)); 
break; 
case "*": 
strResult = Convert.ToString(Convert.ToDouble(strNumberA) * Convert.ToDouble(strNumberB)); 
break; 
case "/": 
if (strNumberB != "0") 
strResult = Convert.ToString(Convert.ToDouble(strNumberA) / Convert.ToDouble(strNumberB)); 
else 
strResult = "除數(shù)不能為0"; 
break; 
} 

Console.WriteLine("結(jié)果是：" + strResult); 

Console.ReadLine(); 


} 
catch (Exception ex) 
{ 
Console.WriteLine("您的輸入有錯：" + ex.Message); 
} 
} 
} 