1、拼接字符串

在C#編程中，字符串類(lèi)型的處理是比較容易出錯(cuò)的地方，在.NET Framework中，字符串是一個(gè)不可變的類(lèi)型，當(dāng)一個(gè)字符串被修改后，總是創(chuàng)建一個(gè)新的副本，不會(huì)改變?cè)醋址?，大多?shù)開(kāi)發(fā)人員總是喜歡使用下面這樣的方法格式化字符串：

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

這里它使用了多重串聯(lián)拼接，因此會(huì)在內(nèi)存中創(chuàng)建三個(gè)不必要的字符串垃圾副本，這種方式是最容易忽略的，最好的辦法是使用string.Format，因?yàn)樗鼉?nèi)部使用的是可變的StringBuilder，也為凈化代碼鋪平了道路，如下：

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

2、嵌套異常處理

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

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

如果相同的異常被處理多次，整個(gè)項(xiàng)目都這樣寫(xiě)?毫無(wú)疑問(wèn)，性能開(kāi)銷(xiāo)將會(huì)劇增。

解決辦法是讓異常處理方法獨(dú)立出來(lái)（一個(gè)大的方法只需要一個(gè)異常處理即可，特殊復(fù)雜場(chǎng)景可酌情多次使用），如：

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的選擇

大部分開(kāi)發(fā)人員更喜歡使用for循環(huán)，而無(wú)視foreach循環(huán)，因?yàn)閒or更容易使用，但操作大型數(shù)據(jù)集時(shí)，使用foreach無(wú)疑是最快的，

根據(jù)廣大網(wǎng)友實(shí)驗(yàn)證明（分別對(duì)記錄數(shù)為10000，100000，1000000條記錄的時(shí)候進(jìn)行采樣分析），

foreach的平均花費(fèi)時(shí)間只有for20%-30%左右。所以，我也要根據(jù)實(shí)際請(qǐng)求選擇使用而不是一直使用某一種。

C#中foreach在處理集合和數(shù)組相對(duì)于for存在以下幾個(gè)優(yōu)勢(shì)和劣勢(shì)：

一、foreach循環(huán)的優(yōu)勢(shì)

• foreach語(yǔ)句簡(jiǎn)潔
• 效率比f(wàn)or要高(C#是強(qiáng)類(lèi)型檢查，for循環(huán)對(duì)于數(shù)組訪問(wèn)的時(shí)候，要對(duì)索引的有效值進(jìn)行檢查)
• 不用關(guān)心數(shù)組的起始索引是幾(因?yàn)橛泻芏嚅_(kāi)發(fā)者是從其他語(yǔ)言轉(zhuǎn)到C#的，有些語(yǔ)言的起始索引可能是1或者是0)
• 處理多維數(shù)組(不包括鋸齒數(shù)組)更加的方便
• 在類(lèi)型轉(zhuǎn)換方面foreach不用顯示地進(jìn)行類(lèi)型轉(zhuǎn)換
• 當(dāng)集合元素如List<T>等在使用foreach進(jìn)行循環(huán)時(shí)，每循環(huán)完一個(gè)元素，就會(huì)釋放對(duì)應(yīng)的資源

二、foreach循環(huán)的劣勢(shì)C#中foreach在處理集合和數(shù)組相對(duì)于for存在以下幾個(gè)優(yōu)勢(shì)：

• 上面說(shuō)了foreach循環(huán)的時(shí)候會(huì)釋放使用完的資源，所以會(huì)造成額外的gc開(kāi)銷(xiāo)，所以使用的時(shí)候，請(qǐng)酌情考慮
• foreach也稱(chēng)為只讀循環(huán)，所以再循環(huán)數(shù)組/集合的時(shí)候，無(wú)法對(duì)數(shù)組/集合進(jìn)行修改
• 數(shù)組中的每一項(xiàng)必須與其他的項(xiàng)類(lèi)型相等

4、驗(yàn)證簡(jiǎn)單的原始數(shù)據(jù)類(lèi)型

很多人員都忽略?xún)?nèi)置的驗(yàn)證原始數(shù)據(jù)類(lèi)型的方法，如System.Int32（其他類(lèi)型亦然），因此都是自己實(shí)現(xiàn)的方法，下面就是一個(gè)自己實(shí)現(xiàn)的驗(yàn)證一個(gè)字符串是否是數(shù)值的代碼：

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

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

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

5、處理對(duì)象實(shí)現(xiàn)IDisposable接口

對(duì)象的處理和使用一樣重要，理想的辦法是在類(lèi)中實(shí)現(xiàn)IDisposable接口的dispose方法，在使用這個(gè)類(lèi)的對(duì)象后，可以通過(guò)調(diào)用dispose方法進(jìn)行處理。

下面的代碼顯示了一個(gè)SqlConnection對(duì)象的創(chuàng)建，使用和處理： 

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

在上面的方法中，連接處理在最后一個(gè)代碼塊中被明確調(diào)用，如果發(fā)生一個(gè)異常，catch代碼塊就會(huì)執(zhí)行，然后再執(zhí)行最后一個(gè)代碼塊處理連接，

因此在最后一個(gè)代碼塊執(zhí)行之前，連接將一直留在內(nèi)存中，.NET Framework的一個(gè)基本原則就是當(dāng)對(duì)象不被使用時(shí)就應(yīng)該釋放資源。

下面是調(diào)用dispose更好的辦法：

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

當(dāng)你使用using代碼塊時(shí)，對(duì)象上的dispose方法將在執(zhí)行退出代碼塊時(shí)調(diào)用，這樣可以保證SqlConnection的資源被處理和盡早釋放，

你也應(yīng)該注意到這個(gè)辦法也適用于實(shí)現(xiàn)IDisposable接口的類(lèi)。

6、聲明公共變量

聽(tīng)起來(lái)可能有點(diǎn)簡(jiǎn)單，但我們經(jīng)?？吹綖E用公共變量聲明的情況，先來(lái)看一個(gè)例子：

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類(lèi)中聲明了一個(gè)AccountNumber公共變量，理想情況下，AccountNumber應(yīng)該是只讀的，但MyAccount類(lèi)卻沒(méi)有對(duì)它實(shí)施任何控制。

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

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

這里MyAccount類(lèi)對(duì)AccountNumber公共變量實(shí)施了很好的控制，它變成只讀，不能由調(diào)用者類(lèi)修改。

7、利用System.Data.DataTable訪問(wèn)數(shù)據(jù)

人多人經(jīng)常使用列索引從數(shù)據(jù)庫(kù)訪問(wèn)數(shù)據(jù)，如：

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

按照這種寫(xiě)法，如果列順序在SQL查詢(xún)匹配數(shù)據(jù)時(shí)發(fā)生了變化，你的應(yīng)用程序?qū)?huì)受到影響，正確的做法應(yīng)該是使用列名訪問(wèn)數(shù)據(jù)。

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

這樣的代碼更加穩(wěn)固，列順序發(fā)生變化不會(huì)給應(yīng)用程序造成任何影響，

如果在一個(gè)地方使用局部變量保存列名更好，即使將來(lái)你的列名發(fā)生了變化，也不用修改應(yīng)用程序代碼。

到此這篇關(guān)于C#編程中最容易犯的7種編寫(xiě)錯(cuò)誤分享的文章就介紹到這了,更多相關(guān)C#編寫(xiě)錯(cuò)誤內(nèi)容請(qǐng)搜索腳本之家以前的文章或繼續(xù)瀏覽下面的相關(guān)文章希望大家以后多多支持腳本之家！

最新評(píng)論