C#編程中最容易犯的編寫錯誤有哪些

這篇文章主要講解了“C#編程中最容易犯的編寫錯誤有哪些”，文中的講解內容簡單清晰，易于學習與理解，下面請大家跟著小編的思路慢慢深入，一起來研究和學習“C#編程中最容易犯的編寫錯誤有哪些”吧！

1、拼接字符串

在C#編程中，字符串類型的處理是比較容易出錯的地方，在.NET Framework中，字符串是一個不可變的類型，當一個字符串被修改后，總是創建一個新的副本，不會改變源字符串，大多數開發人員總是喜歡使用下面這樣的方法格式化字符串：

string updateSqlText = "UPDATE Table SET Name='" + name+ "' WHERE Id=" + id;

這里它使用了多重串聯拼接，因此會在內存中創建三個不必要的字符串垃圾副本，這種方式是最容易忽略的，最好的辦法是使用string.Format，因為它內部使用的是可變的StringBuilder，也為凈化代碼鋪平了道路，如下：

string updateSqlText = string.Format("UPDATE Table SET Name='{0}' WHERE Id={1}", name, id);

2、嵌套異常處理

在方法中添加異常處理模塊try-cathc是必然的，但是沒有必要在一個方法里面多次加上異常處理的嵌套方法，如下：

public class Class1
{
    public void MainMethod()
    {
        try
        {
            //some implementation
            ChildMethod1();
        }
        catch (Exception exception)
        {
            //Handle exception
        }
    }
    private void ChildMethod1()
    {
        try
        {
            //some implementation
            ChildMethod2();
        }
        catch (Exception exception)
        {
            //Handle exception
            throw;
        }
    }
    private void ChildMethod2()
    {
        try
        {
            //some implementation
        }
        catch (Exception exception)
        {
            //Handle exception
            throw;
        }
    }
}

如果相同的異常被處理多次，整個項目都這樣寫?毫無疑問，性能開銷將會劇增。

解決辦法是讓異常處理方法獨立出來（一個大的方法只需要一個異常處理即可，特殊復雜場景可酌情多次使用），如：

public void MainMethod()
{
    try
    {
        //some implementation
        ChildMethod1();
    }
    catch (Exception exception)
    {
        //Handle exception
    }
}
private void ChildMethod1()
{
    //some implementation
    ChildMethod2();
}

private void ChildMethod2()
{
    //some implementation
}

3、for和foreach的選擇

大部分開發人員更喜歡使用for循環，而無視foreach循環，因為for更容易使用，但操作大型數據集時，使用foreach無疑是最快的，

根據廣大網友實驗證明（分別對記錄數為10000，100000，1000000條記錄的時候進行采樣分析），

foreach的平均花費時間只有for20%-30%左右。所以，我也要根據實際請求選擇使用而不是一直使用某一種。

C#中foreach在處理集合和數組相對于for存在以下幾個優勢和劣勢：

一、foreach循環的優勢

• foreach語句簡潔

• 效率比for要高(C#是強類型檢查，for循環對于數組訪問的時候，要對索引的有效值進行檢查)

• 不用關心數組的起始索引是幾(因為有很多開發者是從其他語言轉到C#的，有些語言的起始索引可能是1或者是0)

• 處理多維數組(不包括鋸齒數組)更加的方便

• 在類型轉換方面foreach不用顯示地進行類型轉換

• 當集合元素如List<T>等在使用foreach進行循環時，每循環完一個元素，就會釋放對應的資源

二、foreach循環的劣勢C#中foreach在處理集合和數組相對于for存在以下幾個優勢：

• 上面說了foreach循環的時候會釋放使用完的資源，所以會造成額外的gc開銷，所以使用的時候，請酌情考慮

• foreach也稱為只讀循環，所以再循環數組/集合的時候，無法對數組/集合進行修改

• 數組中的每一項必須與其他的項類型相等

4、驗證簡單的原始數據類型

很多人員都忽略內置的驗證原始數據類型的方法，如System.Int32（其他類型亦然），因此都是自己實現的方法，下面就是一個自己實現的驗證一個字符串是否是數值的代碼：

public bool CheckIfNumeric(string value)
{
    bool isNumeric = true;
    try
    {
        int i = Convert.ToInt32(value);
    }
    catch (FormatException exception)
    {
        isNumeric = false;
    }
    return isNumeric;
}

它使用了try catch語句進行捕捉判斷，因此不是最佳的做法，更好的辦法是象下面這樣使用int.TryParse：

int output = 0;
bool isNumeric = int.TryParse(value, out output);

5、處理對象實現IDisposable接口

對象的處理和使用一樣重要，理想的辦法是在類中實現IDisposable接口的dispose方法，在使用這個類的對象后，可以通過調用dispose方法進行處理。

下面的代碼顯示了一個SqlConnection對象的創建，使用和處理： 

public void DALMethod()
{
    SqlConnection connection = null;
    try
    {
        connection = new SqlConnection("XXXXXXXXXX");
        connection.Open();
        //implement the data access
    }
    catch (Exception exception)
    {
        //handle exception
    }
    finally
    {
        connection.Close();
        connection.Dispose();
    }
}

在上面的方法中，連接處理在最后一個代碼塊中被明確調用，如果發生一個異常，catch代碼塊就會執行，然后再執行最后一個代碼塊處理連接，

因此在最后一個代碼塊執行之前，連接將一直留在內存中，.NET Framework的一個基本原則就是當對象不被使用時就應該釋放資源。

下面是調用dispose更好的辦法：

public void DALMethod()
{
    using (SqlConnection connection = new SqlConnection("XXXXXXXXXX"))
    {
        connection.Open();
        //implement the data access
    }
}

當你使用using代碼塊時，對象上的dispose方法將在執行退出代碼塊時調用，這樣可以保證SqlConnection的資源被處理和盡早釋放，

你也應該注意到這個辦法也適用于實現IDisposable接口的類。

6、聲明公共變量

聽起來可能有點簡單，但我們經常看到濫用公共變量聲明的情況，先來看一個例子：

static void Main(string[] args)
{
    MyAccount account = new MyAccount();
    //The caller is able to set the value which is unexpected
    account.AccountNumber = "YYYYYYYYYYYYYY";
    Console.ReadKey();
}
public class MyAccount
{
    public string AccountNumber;
    public MyAccount()
    {
        AccountNumber = "XXXXXXXXXXXXX";
    }
}

在上面的MyAccount類中聲明了一個AccountNumber公共變量，理想情況下，AccountNumber應該是只讀的，但MyAccount類卻沒有對它實施任何控制。

聲明公共變量正確的做法應該是使用屬性，如： 

public class MyAccount
{
    private string _accountNumber;
    public string AccountNumber
    {
        get { return _accountNumber; }
    }
    public MyAccount()
    {
        _accountNumber = "XXXXXXXXXXXXX";
    }
}

這里MyAccount類對AccountNumber公共變量實施了很好的控制，它變成只讀，不能由調用者類修改。

7、利用System.Data.DataTable訪問數據

人多人經常使用列索引從數據庫訪問數據，如：

public void MyMethod()
{
    //GetData fetches data from the database using a SQL query
    DataTable dt = DataAccess.GetData();
    foreach (DataRow row in dt.Rows)
    {
        //Accessing data through column index
        int empId = Convert.ToInt32(row[0]);
    }
}

按照這種寫法，如果列順序在SQL查詢匹配數據時發生了變化，你的應用程序將會受到影響，正確的做法應該是使用列名訪問數據。

private const string COL_EMP_ID = "EmpId";
public void MyMethod()
{
    //GetData fetches data from the database using a SQL query
    DataTable dt = DataAccess.GetData();
    foreach (DataRow row in dt.Rows)
    {
        //Accessing data through column name
        int empId = Convert.ToInt32(row[COL_EMP_ID]);
    }
}

這樣的代碼更加穩固，列順序發生變化不會給應用程序造成任何影響，

如果在一個地方使用局部變量保存列名更好，即使將來你的列名發生了變化，也不用修改應用程序代碼。

感謝各位的閱讀，以上就是“C#編程中最容易犯的編寫錯誤有哪些”的內容了，經過本文的學習后，相信大家對C#編程中最容易犯的編寫錯誤有哪些這一問題有了更深刻的體會，具體使用情況還需要大家實踐驗證。這里是億速云，小編將為大家推送更多相關知識點的文章，歡迎關注！