本文介紹了10種最常見的編程錯誤，或是C#程序員要避免的陷阱。

常見錯誤1: 像使用值一樣使用參考或過來用
C++以及許多其他語言的程序員習慣于控制他們分配給變量的值是否為簡易的值或現(xiàn)有對象的引用。在C#中呢，這將由寫該對象的程序員決定，而不是由實例化該對象并對它進行變量賦值的程序員決定。這是新手C#程序員們的共同“問題”。
如果你不知道你正在使用的對象是否是值類型或引用類型，你可能會遇到一些驚喜。例如：

Point point1 = new Point(20, 30);
Point point2 = point1;
point2.X = 50;
Console.WriteLine(point1.X); // 20 (does this surprise you?)
Console.WriteLine(point2.X); // 50
 
Pen pen1 = new Pen(Color.Black);
Pen pen2 = pen1;
pen2.Color = Color.Blue;
Console.WriteLine(pen1.Color); // Blue (or does this surprise you?)
Console.WriteLine(pen2.Color); // Blue

如你所見,盡管Point和Pen對象的創(chuàng)建方式相同,但是當一個新的X的坐標值被分配到point2時, point1的值保持不變 。而當一個新的color值被分配到pen2,pen1也隨之改變。因此,我們可以推斷point1和point2每個都包含自己的Point對象的副本,而pen1和pen2引用了同一個Pen對象 。如果沒有這個測試,我們怎么能夠知道這個原理?
一種辦法是去看一下對象是如何定義的（在Visual Studio中，你可以把光標放在對象的名字上，并按下F12鍵）

public struct Point { … } // defines a “value” type
 public class Pen { … } // defines a “reference” type

　　如上所示，在C#中，struct關鍵字是用來定義一個值類型，而class關鍵字是用來定義引用類型的。 對于那些有C++編程背景人來說，如果被C++和C#之間某些類似的關鍵字搞混，可能會對以上這種行為感到很吃驚。

　　如果你想要依賴的行為會因值類型和引用類型而異，舉例來說，如果你想把一個對象作為參數(shù)傳給一個方法，并在這個方法中修改這個對象的狀態(tài)。你一定要確保你在處理正確的類型對象。

常見的錯誤2：誤會未初始化變量的默認值

　　在C#中，值得類型不能為空。根據(jù)定義，值的類型值，甚至初始化變量的值類型必須有一個值。這就是所謂的該類型的默認值。這通常會導致以下，意想不到的結果時，檢查一個變量是否未初始化：

 class Program {
 static Point point1; static Pen pen1; static void Main(string[] args) {
  Console.WriteLine(pen1 == null); // True
  Console.WriteLine(point1 == null); // False (huh?)
 }
 }

　　為什么不是【point 1】空？答案是，點是一個值類型，和默認值點（0,0）一樣，沒有空值。未能認識到這是一個非常簡單和常見的錯誤，在C#中

　　很多（但是不是全部）值類型有一個【IsEmpty】屬性，你可以看看它等于默認值：

　　常見錯誤3: 使用不恰當或未指定的方法比較字符串

　　在C#中有很多方法來比較字符串。

　　雖然有不少程序員使用==操作符來比較字符串，但是這種方法實際上是最不推薦使用的。主要原因是由于這種方法沒有在代碼中顯示的指定使用哪種類型去比較字符串。

　　相反，在C#中判斷字符串是否相等最好使用Equals方法：

　　使用帶comparisonType的Equals方法去比較字符串，不僅會使你的代碼更清晰，還會使你去考慮清楚要用哪種類型去比較字符串。這種方法非常值得你去使用，因為盡管在英語中，按順序進行的比較和按語言區(qū)域進行的比較之間并沒有太多的區(qū)別，但是在其他的一些語種可能會有很大的不同。如果你忽略了這種可能性，無疑是為你自己在未來的道路上挖了很多“坑”。舉例來說：

 string s = "strasse";
 
 // outputs False:
 Console.WriteLine(s == "straße");
 Console.WriteLine(s.Equals("straße"));
 Console.WriteLine(s.Equals("straße", StringComparison.Ordinal));
 Console.WriteLine(s.Equals("Straße", StringComparison.CurrentCulture)); 
 Console.WriteLine(s.Equals("straße", StringComparison.OrdinalIgnoreCase));
 
 // outputs True:
 Console.WriteLine(s.Equals("straße", StringComparison.CurrentCulture));
 Console.WriteLine(s.Equals("Straße", StringComparison.CurrentCultureIgnoreCase));

　　最安全的實踐是總是為Equals方法提供一個comparisonType的參數(shù)。

　　下面是一些基本的指導原則：

　　當比較用戶輸入的字符串或者將字符串比較結果展示給用戶時，使用本地化的比較(CurrentCulture 或者CurrentCultureIgnoreCase)。

　　當用于程序設計的比較字符串時，使用原始的比較(Ordinal 或者 OrdinalIgnoreCase)

　　InvariantCulture和InvariantCultureIgnoreCase一般并不使用，除非在受限的情境之下，因為原始的比較通常效率更高。如果與本地文化相關的比較是必不可少的，它應該被執(zhí)行成基于當前的文化或者另一種特殊文化的比較。

　　此外，對Equals 方法來說，字符串也通常提供了Compare方法，可以提供字符串的相對順序信息而不僅僅中測試是否相等。這個方法可以很好適用于<, <=, >和>= 運算符，對上述討論同樣適用。

　　常見誤區(qū)4: 使用迭代式 (而不是聲明式)的語句去操作集合

　　在C# 3.0中，LINQ的引入改變了我們以往對集合對象的查詢和修改操作。從這以后，你應該用LINQ去操作集合，而不是通過迭代的方式。

　　一些C#的程序員甚至都不知道LINQ的存在，好在不知道的人正在逐步減少。但是還有些人誤以為LINQ只用在數(shù)據(jù)庫查詢中，因為LINQ的關鍵字和SQL語句實在是太像了。

　　雖然數(shù)據(jù)庫的查詢操作是LINQ的一個非常典型的應用，但是它同樣可以應用于各種可枚舉的集合對象。（如：任何實現(xiàn)了IEnumerable接口的對象）。舉例來說，如果你有一個Account類型的數(shù)組，不要寫成下面這樣：

 decimal total = 0; foreach (Account account in myAccounts) { if (account.Status == "active") {
  total += account.Balance;
 }
 }

你只要這樣寫：

decimal total = (from account in myAccounts
    where account.Status == "active"
    select account.Balance).Sum();

　　雖然這是一個很簡單的例子，在有些情況下，一個單一的LINQ語句可以輕易地替換掉你代碼中一個迭代循環(huán)（或嵌套循環(huán)）里的幾十條語句。更少的代碼通常意味著產(chǎn)生Bug的機會也會更少地被引入。然而，記住，在性能方面可能要權衡一下。在性能很關鍵的場景，尤其是你的迭代代碼能夠對你的集合進行假設時，LINQ做不到，所以一定要在這兩種方法之間比較一下性能。

　　常見錯誤：在LINQ語句之中沒有考慮底層對象

　　對于處理抽象操縱集合任務，LINQ無疑是龐大的。無論他們是在內(nèi)存的對象，數(shù)據(jù)庫表，或者XML文檔。在如此一個完美世界之中，你不需要知道底層對象。然而在這兒的錯誤是假設我們生活在一個完美世界之中。事實上，相同的LINQ語句能返回不同的結果，當在精確的相同數(shù)據(jù)上執(zhí)行時，如果該數(shù)據(jù)碰巧在一個不同的格式之中。

　　例如，請考慮下面的語句：

decimal total=(from accout in myaccouts
where accout.status==‘a(chǎn)ctive"
     select accout .Balance).sum();

　　想象一下，該對象之一的賬號會發(fā)生什么。狀態(tài)等于“有效的”（注意大寫A)?

　　好吧，如果myaccout是Dbset的對象。（默認設置了不同區(qū)分大小寫的配置），where表達式仍會匹配該元素。然而，如果myaccout是在內(nèi)存陣列之中，那么它將不匹配，因此將產(chǎn)生不同的總的結果。

　　等一會，在我們之前討論過的字符串比較中, 我們看見 == 操作符扮演的角色就是簡單的比較. 所以，為什么在這個條件下， == 表現(xiàn)出的是另外的一個形式呢 ?

　　答案是，當在LINQ語句中的基礎對象都引用到SQL表中的數(shù)據(jù)（如與在這個例子中，在實體框架為DbSet的對象的情況下），該語句被轉換成一個T-SQL語句。然后遵循的T-SQL的規(guī)則，而不是C＃的規(guī)則，所以在上述情況下的比較結束是不區(qū)分大小寫的。

　　一般情況下，即使LINQ是一個有益的和一致的方式來查詢對象的集合，在現(xiàn)實中你還需要知道你的語句是否會被翻譯成什么比C＃的引擎或者是其他表達，來確保您的代碼的行為將如預期在運行時。

　　常見錯誤6：對擴展方法感到困惑或者被它的形式欺騙

　　如同先前提到的，LINQ狀態(tài)依賴于IEnumerable接口的實現(xiàn)對象，比如，下面的簡單函數(shù)會合計帳戶集合中的帳戶余額：

 public decimal SumAccounts(IEnumerable<Account> myAccounts) {  return myAccounts.Sum(a => a.Balance);
 }

　　在上面的代碼中，myAccounts參數(shù)的類型被聲明為IEnumerable<Account>，myAccounts引用了一個Sum 方法 (C# 使用類似的 “dot notation” 引用方法或者接口中的類),我們期望在IEnumerable<T>接口中定義一個Sum()方法。但是，IEnumerable<T>沒有為Sum方法提供任何引用并且只有如下所示的簡潔定義：

 public interface IEnumerable<out T> : IEnumerable {
  IEnumerator<T> GetEnumerator();
 }

　　但是Sum方法應該定義到何處？C#是強類型的語言，因此如果Sum方法的引用是無效的，C#編譯器會對其報錯。我們知道它必須存在，但是應該在哪里呢？此外，LINQ提供的供查詢和聚集結果所有方法在哪里定義呢？

　　答案是Sum并不在IEnumerable接口內(nèi)定義，而是一個定義在System.Linq.Enumerable類中的static方法（叫做“extension method”）

 namespace System.Linq {
 public static class Enumerable {  ...
  // the reference here to “this IEnumerable<TSource> source” is
  // the magic sauce that provides access to the extension method Sum
  public static decimal Sum<TSource>(this IEnumerable<TSource> source,
           Func<TSource, decimal> selector);  ...
 }
 }

　　可是擴展方法和其它靜態(tài)方法有什么不同之處，是什么確保我們可以在其它類訪問它？

　　擴展方法的顯著特點是第一個形參前的this修飾符。這就是編譯器知道它是一個擴展方法的“奧妙”。它所修飾的參數(shù)的類型（這個例子中的IEnumerable<TSource>）說明這個類或者接口將顯得實現(xiàn)了這個方法。

　?。硗庑枰赋龅氖?，定義擴展方法的IEnumerable接口和Enumerable類的名字間的相似性沒什么奇怪的。這種相似性只是隨意的風格選擇。）

　　理解了這一點，我們可以看到上面介紹的sumAccounts方法能以下面的方式實現(xiàn)：

 public decimal SumAccounts(IEnumerable<Account> myAccounts) {
  return Enumerable.Sum(myAccounts, a => a.Balance);
 }

　　事實上我們可能已經(jīng)這樣實現(xiàn)了這個方法，而不是問什么要有擴展方法。擴展方法本身只是C#的一個方便你無需繼承、重新編譯或者修改原始代碼就可以給已存的在類型“添加”方法的方式。

　　擴展方法通過在文件開頭添加using [namespace];引入到作用域。你需要知道你要找的擴展方法所在的名字空間。如果你知道你要找的是什么，這點很容易。

　　當C#編譯器碰到一個對象的實例調(diào)用了一個方法，并且它在這個對象的類中找不到那個方法，它就會嘗試在作用域中所有的擴展方法里找一個匹配所要求的類和方法簽名的。如果找到了，它就把實例的引用當做第一個參數(shù)傳給那個擴展方法，然后如果有其它參數(shù)的話，再把它們依次傳入擴展方法。（如果C#編譯器沒有在作用域中找到相應的擴展方法，它會拋措。）

　　對C#編譯器來說，擴展方法是個“語法糖”，使我們能把代碼寫得更清晰，更易于維護（多數(shù)情況下）。顯然，前提是你知道它的用法，否則，它會比較容易讓人迷惑，尤其是一開始。

　　應用擴展方法確實有優(yōu)勢，但也會讓那些對它不了解或者認識不正確的開發(fā)者頭疼，浪費時間。尤其是在看在線示例代碼，或者其它已經(jīng)寫好的代碼的時候。當這些代碼產(chǎn)生編譯錯誤（因為它調(diào)用了那些顯然沒在被調(diào)用類型中定義的方法），一般的傾向是考慮代碼是否應用于所引用類庫的其它版本，甚至是不同的類庫。很多時間會被花在找新版本，或者被認為“丟失”的類庫上。

　　在擴展方法的名字和類中定義的方法的名字一樣，只是在方法簽名上有微小差異的時候，甚至那些熟悉擴展方法的開發(fā)者也偶爾犯上面的錯誤。很多時間會被花在尋找“不存在”的拼寫錯誤上。

　　在C#中，用擴展方法變得越來越流行。除了LINQ，在另外兩個出自微軟現(xiàn)在被廣泛使用的類庫Unity Application Block和Web API framework中，也應用了擴展方法，而且還有很多其它的?？蚣茉叫拢脭U展方法的可能性越大。

　　當然，你也可以寫你自己的擴展方法。但是必須意識到雖然擴展方法看上去和其它實例方法一樣被調(diào)用，但這實際只是幻。事實上，擴展方法不能訪問所擴展類的私有和保護成員，所以它不能被當做傳統(tǒng)繼承的替代品。

常見錯誤7: 對手頭上的任務使用錯誤的集合類型

C#提供了大量的集合類型的對象，下面只列出了其中的一部分：　　

Array,ArrayList,BitArray,BitVector32,Dictionary<K,V>,HashTable,HybridDictionary,List<T>,NameValueCollection,OrderedDictionary,Queue, Queue<T>,SortedList,Stack, Stack<T>,StringCollection,StringDictionary.

　　但是在有些情況下，有太多的選擇和沒有足夠的選擇一樣糟糕，集合類型也是這樣。數(shù)量眾多的選擇余地肯定可以保證是你的工作正常運轉。但是你最好還是花一些時間提前搜索并了解一下集合類型，以便選擇一個最適合你需要的集合類型。這最終會使你的程序性能更好，減少出錯的可能。

　　如果有一個集合指定的元素類型（如string或bit）和你正在操作的一樣，你最好優(yōu)先選擇使用它。當指定對應的元素類型時，這種集合的效率更高。

　　為了利用好C#中的類型安全，你最好選擇使用一個泛型接口，而不是使用非泛型的借口。泛型接口中的元素類型是你在在聲明對象時指定的類型，而非泛型中的元素是object類型。當使用一個非泛型的接口時，C#的編譯器不能對你的代碼進行類型檢查。同樣，當你在操作原生類型的集合時，使用非泛型的接口會導致C#對這些類型進行頻繁的裝箱(boxing)和拆箱(unboxing)操作。和使用指定了合適類型的泛型集合相比，這會帶來很明顯的性能影響。

　　另一個常見的陷阱是自己去實現(xiàn)一個集合類型。這并不是說永遠不要這樣做，你可以通過使用或擴展.NET提供的一些被廣泛使用的集合類型來節(jié)省大量的時間，而不是去重復造輪子。 特別是，C#的C5 Generic Collection Library 和CLI提供了很多額外的集合類型，像持久化樹形數(shù)據(jù)結構，基于堆的優(yōu)先級隊列，哈希索引的數(shù)組列表，鏈表等以及更多。

　　常見錯誤8：遺漏資源釋放

　　CLR 托管環(huán)境扮演了垃圾回收器的角色，所以你不需要顯式釋放已創(chuàng)建對象所占用的內(nèi)存。事實上，你也不能顯式釋放。C#中沒有與C++ delete對應的運算符或者與C語言中free()函數(shù)對應的方法。但這并不意味著你可以忽略所有的使用過的對象。許多對象類型封裝了許多其它類型的系統(tǒng)資源（例如，磁盤文件，數(shù)據(jù)連接，網(wǎng)絡端口等等）。保持這些資源使用狀態(tài)會急劇耗盡系統(tǒng)的資源，削弱性能并且最終導致程序出錯。

　　盡管所有C#的類中都定義了析構方法，但是銷毀對象（C#中也叫做終結器）可能存在的問題是你不確定它們時候會被調(diào)用。他們在未來一個不確定的時間被垃圾回收器調(diào)用（一個異步的線程，此舉可能引發(fā)額外的并發(fā)）。試圖避免這種由垃圾回收器中GC.Collect()方法所施加的強制限制并非一種好的編程實踐，因為可能在垃圾回收線程試圖回收適宜回收的對象時，在不可預知的時間內(nèi)致使線程阻塞。

　　這并意味著最好不要用終結器，顯式釋放資源并不會導致其中的任何一個后果。當你打開一個文件、網(wǎng)絡端口或者數(shù)據(jù)連接時，當你不再使用這些資源時，你應該盡快的顯式釋放這些資源。

　　資源泄露幾乎在所有的環(huán)境中都會引發(fā)關注。但是，C#提供了一種健壯的機制使資源的使用變得簡單。如果合理利用，可以大增減少泄露出現(xiàn)的機率。NET framework定義了一個IDisposable接口，僅由一個Dispose()構成。任何實現(xiàn)IDisposable的接口的對象都會在對象生命周期結束調(diào)用Dispose()方法。調(diào)用結果明確而且決定性的釋放占用的資源。

　　如果在一個代碼段中創(chuàng)建并釋放一個對象，卻忘記調(diào)用Dispose()方法，這是不可原諒的，因為C#提供了using語句以確保無論代碼以什么樣的方式退出，Dispose()方法都會被調(diào)用（不管是異常，return語句，或者簡單的代碼段結束)。這個using和之前提到的在文件開頭用來引入名字空間的一樣。它有另外一個很多C#開發(fā)者都沒有察覺的，完全不相關的目的，也就是確保代碼退出時，對象的Dispose()方法被調(diào)用：

 using (FileStream myFile = File.OpenRead("foo.txt")) {
 myFile.Read(buffer, 0, 100);
 }

　　在上面示例中使用using語句，你就可以確定myFile.Dispose()方法會在文件使用完之后被立即調(diào)用，不管Read()方法有沒有拋異常。

　　常見錯誤9: 回避異常

　　C#在運行時也會強制進行類型檢查。相對于像C++這樣會給錯誤的類型轉換賦一個隨機值的語言來說，C#這可以使你更快的找到出錯的位置。然而，程序員再一次無視了C#的這一特性。由于C#提供了兩種類型檢查的方式，一種會拋出異常，而另一種則不會，這很可能會使他們掉進這個“坑”里。有些程序員傾向于回避異常，并且認為不寫 try/catch 語句可以節(jié)省一些代碼。

　　例如，下面演示了C#中進行顯示類型轉換的兩種不同的方式：

 // 方法 1:
 // 如果 account 不能轉換成 SavingAccount 會拋出異常
 SavingsAccount savingsAccount = (SavingsAccount)account;
 
 // 方法 2:
 // 如果不能轉換，則不會拋出異常，相反，它會返回 null
 SavingsAccount savingsAccount = account as SavingsAccount;

　　很明顯，如果不對方法2返回的結果進行判斷的話，最終很可能會產(chǎn)生一個 NullReferenceException 的異常，這可能會出現(xiàn)在稍晚些的時候，這使得問題更難追蹤。對比來說，方法1會立即拋出一個 InvalidCastExceptionmaking，這樣，問題的根源就很明顯了。

　　此外，即使你知道要對方法2的返回值進行判斷，如果你發(fā)現(xiàn)值為空，接下來你會怎么做？在這個方法中報告錯誤合適嗎？如果類型轉換失敗了你還有其他的方法去嘗試嗎？如果沒有的話，那么拋出這個異常是唯一正確的選擇，并且異常的拋出點離其發(fā)生點越近越好。

　　下面的例子演示了其他一組常見的方法，一種會拋出異常，而另一種則不會：

 int.Parse();  // 如果參數(shù)無法解析會拋出異常
 int.TryParse(); // 返回bool值表示解析是否成功
 
 IEnumerable.First();   // 如果序列為空，則拋出異常
 IEnumerable.FirstOrDefault(); // 如果序列為空則返回 null 或默認值

　　有些程序員認為“異常有害”，所以他們自然而然的認為不拋出異常的程序顯得更加“高大上”。雖然在某些情況下，這種觀點是正確的，但是這種觀點并不適用于所有的情況。

　　舉個具體的例子，某些情況下當異常產(chǎn)生時，你有另一個可選的措施（如，默認值），那么，選用不拋出異常的方法是一個比較好的選擇。在這種情況下，你最好像下面這樣寫：

 if (int.TryParse(myString, out myInt)) { // use myInt
 } else { // use default value
 }
　　而不是這樣：
 try {
 myInt = int.Parse(myString); // use myInt
 } catch (FormatException) { // use default value
 }

　　但是，這并不說明 TryParse 方法更好。某些情況下適合，某些情況下則不適合。這就是為什么有兩種方法供我們選擇了。根據(jù)你的具體情況選擇合適的方法，并記住，作為一個開發(fā)者，異常是完全可以成為你的朋友的。

常見錯誤10: 累積編譯器警告而不處理

　　這個錯誤并不是C#所特有的，但是在C#中這種情況卻比較多，尤其是從C#編譯器棄用了嚴格的類型檢查之后。

　　警告的出現(xiàn)是有原因的。所有C#的編譯錯誤都表明你的代碼有缺陷，同樣，一些警告也是這樣。這兩者之間的區(qū)別在于，對于警告來說，編譯器可以按照你代碼的指示工作，但是，編譯器發(fā)現(xiàn)你的代碼有一點小問題，很有可能會使你的代碼不能按照你的預期運行。

　　一個常見的例子是，你修改了你的代碼，并移除了對某些變量的使用，但是，你忘了移除該變量的聲明。程序可以很好的運行，但是編譯器會提示有未使用的變量。程序可以很好的運行使得一些程序員不去修復警告。更有甚者，有些程序員很好的利用了Visual Studio中“錯誤列表”窗口的隱藏警告的功能，很容易的就把警告過濾了，以便專注于錯誤。不用多長時間，就會積累一堆警告，這些警告都被“愜意”的忽略了（更糟的是，隱藏掉了）。

　　但是，如果你忽略掉這一類的警告，類似于下面這個例子遲早會出現(xiàn)在你的代碼中。

 class Account {
 
  int myId;  int Id; // 編譯器已經(jīng)警告過了，但是你不聽
 
  // Constructor
  Account(int id) {   this.myId = Id;  // OOPS!
  }
 
 }

　　再加上使用了編輯器的智能感知的功能，這種錯誤就很有可能發(fā)生。

　　現(xiàn)在，你的代碼中有了一個嚴重的錯誤（但是編譯器只是輸出了一個警告，其原因已經(jīng)解釋過），這會浪費你大量的時間去查找這錯誤，具體情況由你的程序復雜程度決定。如果你一開始就注意到了這個警告，你只需要5秒鐘就可以修改掉，從而避免這個問題。

　　記住，如果你仔細看的話，你會發(fā)現(xiàn)，C#編譯器給了你很多關于你程序健壯性的有用的信息。不要忽略警告。你只需花幾秒鐘的時間就可以修復它們，當出現(xiàn)的時候就去修復它，這可以為你節(jié)省很多時間。試著為自己培養(yǎng)一種“潔癖”，讓Visual Studio 的“錯誤窗口”一直顯示“0錯誤， 0警告”，一旦出現(xiàn)警告就感覺不舒服，然后即刻把警告修復掉。

　　當然了，任何規(guī)則都有例外。所以，有些時候，雖然你的代碼在編譯器看來是有點問題的，但是這正是你想要的。在這種很少見的情況下，你最好使用 #pragma warning disable [warning id] 把引發(fā)警告的代碼包裹起來，而且只包裹警告ID對應的代碼。這會且只會壓制對應的警告，所以當有新的警告產(chǎn)生的時候，你還是會知道的。.

　　C#是一門強大的并且很靈活的語言，它有很多機制和語言規(guī)范來顯著的提高你的生產(chǎn)力。和其他語言一樣，如果對它能力的了解有限，這很可能會給你帶來阻礙，而不是好處。正如一句諺語所說的那樣“knowing enough to be dangerous”（譯者注：意思是自以為已經(jīng)了解足夠了，可以做某事了，但其實不是）。

　　熟悉C#的一些關鍵的細微之處，像本文中所提到的那些（但不限于這些），可以幫助我們更好的去使用語言，從而避免一些常見的陷阱。

最新評論