解读C#编程中最容易忽略7种编写习惯！

超、凢脫俗 2024-03-16 14:43 43阅读 0赞

> 编程时犯错是必然的，我们来解读一下编程中最容出现的错误

[回到顶部][Link 1]

## 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);

[回到顶部][Link 1]

## 2、嵌套异常处理 ##

在方法中添加异常处理模块try-cathc是必然的，但是没有必要在一个方法里面多次加上异常处理的嵌套方法，如下：

![51c1d08d226fabe1af8a2ec399dfbe5d.gif][]

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

![893435a7d8436c928dd398d62eee7f59.gif][]

如果相同的异常被处理多次，整个项目都这样写?毫无疑问，性能开销将会剧增。

解决办法是让异常处理方法独立出来（一个大的方法只需要一个异常处理即可，特殊复杂场景可酌情多次使用），如：

![a9f429029d59a8f5c13ae8f9d1ee1d36.gif][]

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

![a7f96cfce6b66d052d4e297b48f45f71.gif][]

[回到顶部][Link 1]

## 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.  数组中的每一项必须与其他的项类型相等

[回到顶部][Link 1]

## 4、验证简单的原始数据类型 ##

很多人员都忽略内置的验证原始数据类型的方法，如System.Int32（其他类型亦然），因此都是自己实现的方法，下面就是一个自己实现的验证一个字符串是否是数值的代码：

![7428a1e1066d17e48bf581e21f3b126e.gif][]

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

![417f33fd8b357b71273adaf933341bd6.gif][]

它使用了try catch语句进行捕捉判断，因此不是最佳的做法，更好的办法是象下面这样使用int.TryParse：

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

[回到顶部][Link 1]

## 5、处理对象实现IDisposable接口 ##

对象的处理和使用一样重要，理想的办法是在类中实现IDisposable接口的dispose方法，在使用这个类的对象后，可以通过调用dispose方法进行处理。

下面的代码显示了一个SqlConnection对象的创建，使用和处理：

![e048d7c580b6ae071be5cd2b93a4d2ca.gif][]

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

![0c905e275eb03f1dc95194a7ec3db9e2.gif][]

在上面的方法中，连接处理在最后一个代码块中被明确调用，如果发生一个异常，catch代码块就会执行，然后再执行最后一个代码块处理连接，

因此在最后一个代码块执行之前，连接将一直留在内存中，.NET Framework的一个基本原则就是当对象不被使用时就应该释放资源。

下面是调用dispose更好的办法：

![ad4b40ef85e3762d2ad6e3f9d89defc1.gif][]

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

![ad4b40ef85e3762d2ad6e3f9d89defc1.gif][]

当你使用using代码块时，对象上的dispose方法将在执行退出代码块时调用，这样可以保证SqlConnection的资源被处理和尽早释放，

你也应该注意到这个办法也适用于实现IDisposable接口的类。

[回到顶部][Link 1]

## 6、声明公共变量 ##

听起来可能有点简单，但我们经常看到滥用公共变量声明的情况，先来看一个例子：

![7c2f25b5a677c61859a13424f4883c82.gif][]

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

![7428a1e1066d17e48bf581e21f3b126e.gif][]

在上面的MyAccount类中声明了一个AccountNumber公共变量，理想情况下，AccountNumber应该是只读的，但MyAccount类却没有对它实施任何控制。

声明公共变量正确的做法应该是使用属性，如：

![5c06f1099c37e5a65878462c0479e73c.gif][]

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

![7453157b70674e49ad89c3fad8384952.gif][]

这里MyAccount类对AccountNumber公共变量实施了很好的控制，它变成只读，不能由调用者类修改。

[回到顶部][Link 1]

## 7、利用System.Data.DataTable访问数据 ##

人多人经常使用列索引从数据库访问数据，如：

![309c85399cdb542e6dad50815d8bb4fa.gif][]

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

![0c905e275eb03f1dc95194a7ec3db9e2.gif][]

按照这种写法，如果列顺序在SQL查询匹配数据时发生了变化，你的应用程序将会受到影响，正确的做法应该是使用列名访问数据。

![8fa7e03e315dd069c519e54c21fe1208.gif][]

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

![2763fc95ed11131bcc4f9e3d075089a4.gif][]

这样的代码更加稳固，列顺序发生变化不会给应用程序造成任何影响，

如果在一个地方使用局部变量保存列名更好，即使将来你的列名发生了变化，也不用修改应用程序代码。

[Link 1]: https://www.cnblogs.com/xiongze520/p/17164309.html#_labelTop
[51c1d08d226fabe1af8a2ec399dfbe5d.gif]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/16/50b631d9e26a4176b72d9362a67261a3.png
[893435a7d8436c928dd398d62eee7f59.gif]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/16/1f0d880690f24d989942962edb5a1811.png
[a9f429029d59a8f5c13ae8f9d1ee1d36.gif]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/16/6a565831528841b18816cdb961d2c7f8.png
[a7f96cfce6b66d052d4e297b48f45f71.gif]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/16/fb22d6d86ab74056aed920c4813dc9eb.png
[7428a1e1066d17e48bf581e21f3b126e.gif]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/16/2e33e7e3dae94202b919327361cf9581.png
[417f33fd8b357b71273adaf933341bd6.gif]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/16/51d2cff871d94eef87aeb51e0960ad0b.png
[e048d7c580b6ae071be5cd2b93a4d2ca.gif]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/16/65cfef4a86fc4c9f9650030299bb0e3e.png
[0c905e275eb03f1dc95194a7ec3db9e2.gif]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/16/99fbb74728c9419789caa7befae72f02.png
[ad4b40ef85e3762d2ad6e3f9d89defc1.gif]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/16/a8fcc3d109f44cc580d65eecaab24e01.png
[7c2f25b5a677c61859a13424f4883c82.gif]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/16/db088a40e8bb4ec5ab038d3e58e16332.png
[5c06f1099c37e5a65878462c0479e73c.gif]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/16/d7d5c449dbe640209652fb891dffb1ee.png
[7453157b70674e49ad89c3fad8384952.gif]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/16/69944a6b88ea4803940471f5b6607b1c.png
[309c85399cdb542e6dad50815d8bb4fa.gif]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/16/1b4a13a990f847b0a5cadabcbe80425a.png
[8fa7e03e315dd069c519e54c21fe1208.gif]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/16/3ef0e7a668fd4fe097e886064ca4839b.png
[2763fc95ed11131bcc4f9e3d075089a4.gif]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/16/c0d1888fe2cf422e9da6b0142601dd76.png