解讀C#程式設計中最容易忽略7種編寫習慣!

學習中的苦與樂發表於2023-02-28
程式設計時犯錯是必然的,我們來解讀一下程式設計中最容出現的錯誤

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迴圈的優勢
  1. foreach語句簡潔
  2. 效率比for要高(C#是強型別檢查,for迴圈對於陣列訪問的時候,要對索引的有效值進行檢查)
  3. 不用關心陣列的起始索引是幾(因為有很多開發者是從其他語言轉到C#的,有些語言的起始索引可能是1或者是0)
  4. 處理多維陣列(不包括鋸齒陣列)更加的方便
  5. 在型別轉換方面foreach不用顯示地進行型別轉換
  6. 當集合元素如List<T>等在使用foreach進行迴圈時,每迴圈完一個元素,就會釋放對應的資源
二、foreach迴圈的劣勢C#中foreach在處理集合和陣列相對於for存在以下幾個優勢:
  1. 上面說了foreach迴圈的時候會釋放使用完的資源,所以會造成額外的gc開銷,所以使用的時候,請酌情考慮
  2. foreach也稱為只讀迴圈,所以再迴圈陣列/集合的時候,無法對陣列/集合進行修改
  3. 陣列中的每一項必須與其他的項型別相等

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]);
    }
}

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

如果在一個地方使用區域性變數儲存列名更好,即使將來你的列名發生了變化,也不用修改應用程式程式碼。

 

喜歡就點贊加關注。

歡迎關注訂閱微信公眾號【熊澤有話說】,更多好玩易學知識等你來取
作者:熊澤-學習中的苦與樂
公眾號:熊澤有話說

QQ群:711838388
出處:https://www.cnblogs.com/xiongze520/p/17164309.html
您可以隨意轉載、摘錄,但請在文章內註明作者和原文連結。  

 

 

相關文章