C#编程中最容易犯的7种编写错误分享

更新时间：2023年03月01日 08:25:38 作者：熊泽-学习中的苦与乐

编程时犯错是必然的，这篇文章主要和大家来分享7个C#编程中最容易犯的7种编写错误，文中的示例代码讲解详细，感兴趣的小伙伴可以了解一下

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#编程中最容易犯的7种编写错误分享的文章就介绍到这了,更多相关C#编写错误内容请搜索脚本之家以前的文章或继续浏览下面的相关文章希望大家以后多多支持脚本之家！

您可能感兴趣的文章:

unity实现按住鼠标选取区域截图
这篇文章主要为大家详细介绍了unity实现按住鼠标选取区域截图，文中示例代码介绍的非常详细，具有一定的参考价值，感兴趣的小伙伴们可以参考一下
2020-04-04
C#中abstract的用法详解
abstract可以用来修饰类,方法,属性,索引器和时间,这里不包括字段. 使用abstrac修饰的类,该类只能作为其他类的基类,不能实例化,而且abstract修饰的成员在派生类中必须全部实现,不允许部分实现,否则编译异常
2016-12-12
解决WPF绘制矢量图形模糊的问题
这篇文章介绍了WPF绘制矢量图形模糊问题的解决方法，文中通过示例代码介绍的非常详细。对大家的学习或工作具有一定的参考借鉴价值，需要的朋友可以参考下
2022-06-06
C#中的协变与逆变深入讲解
这篇文章主要给大家介绍了关于C#中协变与逆变的相关资料，文中通过示例代码介绍的非常详细，对大家的学习或者工作具有一定参考学习价值，需要的朋友们下面随着小编来一起学习学习吧
2018-12-12
详解C#中多态性学习/虚方法/抽象方法和接口的用法
这篇文章主要为大家详细介绍了C#中多态性学习、虚方法、抽象方法和接口的用法的相关知识，文中的示例代码讲解详细，感兴趣的小伙伴可以了解一下
2023-03-03
C#影院售票系统毕业设计（1）
这篇文章主要介绍了C#影院售票系统毕业设计，献上了9个类的设计，需要的朋友可以参考下
2015-11-11
详解C#如何实现一个安全的事件订阅器
事件订阅器是一个对象,它订阅（或监听）某个事件,并在事件发生时执行相应的操作,本文主要介绍了C#实现一个安全的事件订阅器的相关知识,感兴趣的可以了解下
2024-01-01
Visual C#.Net 网络程序开发-Socket篇
Visual C#.Net 网络程序开发-Socket篇...
2007-03-03
详谈C# 图片与byte[]之间以及byte[]与string之间的转换
下面小编就为大家带来一篇详谈C# 图片与byte[]之间以及byte[]与string之间的转换。小编觉得挺不错的，现在就分享给大家，也给大家做个参考。一起跟随小编过来看看吧
2017-02-02
Unity屏幕雪花另类实现方式示例
这篇文章主要介绍了Unity屏幕雪花另类实现方式示例，文中通过示例代码介绍的非常详细，对大家的学习或者工作具有一定的参考学习价值，需要的朋友们下面随着小编来一起学习学习吧
2019-10-10