zoukankan      html  css  js  c++  java
  • 看似简单!解读C#程序员最易犯的7大错误 转载

    编程时犯错是必然的,即使是一个很小的错误也可能会导致昂贵的代价,聪明的人善于从错误中汲取教训,尽量不再重复犯错,在这篇文章中,我将重点介绍C#开发人员最容易犯的7个错误。

      格式化字符串

      在C#编程中,字符串类型是最容易处理出错的地方,其代价往往也很昂贵,在.NET Framework中,字符串是一个不可变的类型,当一个字符串被修改后,总是创建一个新的副本,不会改变源字符串,大多数开发人员总是喜欢使用下面这样的方法格式化字符串:
    C# code

    string updateQueryText = "UPDATE EmployeeTable SET Name='" + name
    + "' WHERE EmpId=" + id;


      上面的代码太乱了,由于字符串是不可变的,这里它又使用了多重串联,因此会在内存中创建三个不必要的字符串垃圾副本。

      最好的办法是使用string.Format,因为它内部使用的是可变的StringBuilder,也为净化代码铺平了道路。 
    C# code

    string updateQueryText = string.Format("UPDATE EmployeeTable SET Name='{0}'
    WHERE EmpId={1}", name, id);


      • 嵌套异常处理

      开发人员喜欢在方法的末尾加上异常处理的嵌套方法,如:
    C# code

    public class NestedExceptionHandling
    {
    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;
    }
    }
    }


      如果相同的异常被处理多次,上面的代码会发生什么?毫无疑问,性能开销将会剧增。

      解决办法是让异常处理方法独立出来,如:
    C# code

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

    private void ChildMethod1()
    {
    //some implementation
    ChildMethod2();
    }

    private void ChildMethod2()
    {
    //some implementation
    }
    }


      • 在大型数据集上使用foreach

      大部分开发人员更喜欢使用foreach循 环,而无视for循环,因为foreach更容易使用,但操作大型数据集时,使用foreach已经被证明是代价高昂的,在下面的代码中,我同时使用 for和foreach遍历相同的数据库,在图1中显示了两种循环方法消耗的时间。
    C# code

    static void Main(string[] args)
    {
    DataTable dt
    = PopulateData();
    Stopwatch watch
    = new Stopwatch();

    //For loop
    watch.Start();
    for (int count = 0; count < dt.Rows.Count; count++)
    {
    dt.Rows[count][
    "Name"] = "Modified in For";
    }
    watch.Stop();
    Console.WriteLine(
    "Time taken in For loop: {0}",watch.Elapsed.TotalSeconds);

    //Foreach loop
    watch.Start();
    foreach (DataRow row in dt.Rows)
    {
    row[
    "Name"] = "Modified in ForEach";
    }
    watch.Stop();
    Console.WriteLine(
    "Time taken in For Each loop: {0}",watch.Elapsed.TotalSeconds);

    Console.ReadKey();
    }


    1

      图 1 for和foreach循环遍历相同数据库消耗的时间对比
      从上图可以看出,foreach循环明显要慢一些,它消耗的时间几乎是for循环的两倍,这是因为foreach循环中的dt.Rows要访问数据库中的所有行。因此需要遍历大型数据集时最好使用for循环。

      验证简单的原始数据类型

      大多数开发人员都不知道内置的验证原始数据类型的方法,如System.Int32,因此很多人都是自己实现的,下面就是一个自己实现的验证一个字符串是否是数值的代码:
    C# code

    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: 
    C# code

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

      根据我的经验,
    int.TryParse是更快,更简洁的方法。  

      处理对象实现IDisposable接口

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

      下面的代码显示了一个SqlConnection对象的创建,使用和处理:
    [code
    =C#]
    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更好的办法:
    C# code

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


      当你使用using代码块时,对象上的dispose方法将在执行退出代码块时调用,这样可以保证SqlConnection的资源被处理和尽早释放,你也应该注意到这个办法也适用于实现IDisposable接口的类。

      声明公共变量

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

    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类却没有对它实施任何控制。

      声明公共变量正确的做法应该是使用属性,如: 
    C# code

    public class MyAccount
    {
    private string _accountNumber;
    public string AccountNumber
    {
    get { return _accountNumber; }
    }

    public MyAccount()
    {
    _accountNumber
    = "XXXXXXXXXXXXX";
    }
    }


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

      利用System.Data.DataTable访问数据

      我常常看到开发人员使用列索引从数据库访问数据,如:
    C# code

    public class MyClass
    {
    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查询匹配数据时发生了变化,你的应用程序将会受到影响,正确的做法应该是使用列名访问数据。 
    C# code

    public class MyClass
    {
    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类错误持有不同的看法,欢迎发表你的意见和想法。

     转载地址:http://topic.csdn.net/u/20101110/14/a9c32275-5da2-4230-93eb-83a37285496d.html?37002

  • 相关阅读:
    分析Ajax并爬取微博列表
    链表01
    Install ping command from ubuntu docker
    In container,I can't use man
    centos install glances
    centos 服务器命令下安装GUI
    firefox浏览器配置
    xshell替代产品
    Linux下将一个文件压缩分包成多个小文件
    Linux生成大文件
  • 原文地址:https://www.cnblogs.com/liye/p/1875442.html
Copyright © 2011-2022 走看看