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

  格式化字符串

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

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

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

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

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

  •   嵌套异常处理

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

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

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

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

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中显示了两种循环方法消耗的时间。

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,因此很多人都是自己实现的,下面就是一个自己实现的验证一个字符串是否是数值的代码:

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);   根据我的经验,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更好的办法:

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

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

  声明公共变量

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

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公共变量实施了很好的控制,它变成只读,不能由调用者类修改。

  利用System.Data.DataTable访问数据

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

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

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

看似简单!解读C#程序员最易犯的7大错误的更多相关文章

  1. C# 程序员最常犯的 10 个错误

    关于C# C#是达成微软公共语言运行库(CLR)的少数语言中的一种.达成CLR的语言可以受益于其带来的特性,如跨语言集成.异常处理.安全性增强.部件组合的简易模型以及调试和分析服务.作为现代的CLR语 ...

  2. C# 程序员最常犯的 10 个错误(转)

    关于C#关于本文常见错误 #1:把引用当做值来用,或者反过来常见错误 #2:误会未初始化变量的默认值常见错误 #3:使用不恰当或未指定的方法比较字符串常见错误 #4:使用迭代式 (而不是声明式)的语句 ...

  3. C# 程序员最常犯的 10 个错误http://www.oschina.net/translate/top-10-mistakes-that-c-sharp-programmers-make

    来源:http://www.oschina.net/translate/top-10-mistakes-that-c-sharp-programmers-make 关于C# C#是达成微软公共语言运行 ...

  4. 惊呆了!Java程序员最常犯的错竟然是这10个

    和绝大多数的程序员一样,我也非常的宅.周末最奢侈的享受就是逛一逛技术型网站,比如说 programcreek,这个小网站上有一些非常有意思的主题.比如说:Java 程序员最常犯的错竟然是这 10 个, ...

  5. C/C++程序员面试易错题

    c部分::::::::::::::::::::::::::::::::::: . 关键字volatile有什么含意? 并给出三个不同的例 子. [参考答案]一个定义为volatile的变量是说这变量可 ...

  6. Java程序员最常犯的错误盘点之Top 10

    1. 数组转ArrayList 为了实现把一个数组转换成一个ArrayList,很多Java程序员会使用如下的代码: Arrays.asList确实会返回一个ArrayList对象,但是该类是Arra ...

  7. 谦先生的程序员日志之我的hadoop大数据生涯一

    从一个初级程序员到高级程序员的经历 你好!我是谦先生,我是茫茫程序猿中的一猿,平凡又执着. 刚入行的时候说实话,啥都不懂,就懂点皮毛的java,各种被虐狗的感觉.又写js又写css又写后台...慢慢被 ...

  8. 细数那些Java程序员最容易犯那些错

    java作为最受欢迎程度榜榜首语言,自然是广大开发者使用最多的语言.正因为有如此广泛的使用性,java开发中发生异常也比比皆是,接下来我们就来看看那些java开发中最容易出现的那些错误. 1.重复造轮 ...

  9. 一道看似简单的go程序的深入分析

    先上代码: func main() { var a [10]int for i := 0; i < 10; i++ { go func(i int) { for { a[i]++ } }(i) ...

随机推荐

  1. vue-阻止事件冒泡-开启右键-键盘类事件

    一: 阻止事件冒泡 布局: 当点击按钮时,会触发button的click 也会触发父级的方法 <div id="box"> <div @click="p ...

  2. 获取json的节点名称

    好几次想取json的节点名称,今天搞定了. procedure GetJsonNames(o: ISuperObject; Strs: TStrings); var ite: TSuperAvlIte ...

  3. 超酷消息警告框插件(SweetAlert)

    今天给大家推荐一款不错的超酷消息警告框–SweetAlert:SweetAlert是一款不需要jQuery支持的原生js提示框,风格类似bootstrap.它的提示框不仅美丽动人,并且允许自定义,支持 ...

  4. centos 6.10 永久修改主机名

    1> 修改配置文件 vim /etc/sysconfig/network #HOSTNAME=localhost.localdomain HOSTNAME=tomcat   2> 修改ho ...

  5. redis过期key删除

    LZ一开始配置到启动类里面,结果出现了主线程阻塞的情况. 如下是流程: 首先修改配置文件redis.conf中的:notify-keyspace-events Ex,默认为notify-keyspac ...

  6. python笔记4----字典

    1.哈希: 输入任意长度,输出固定长度. 即判断是否哈希,即判断可不可变. 2.创建字典 (1)直接创建:dic={1:'a',2:'b',3:'c'} (2)dict函数创建: list=[(1,' ...

  7. JAVA中各个包的主要作用

    00:48:0800:48:1022013013-06-282013-06-2800:48:182013-06-2800:48:20  java.util是JAVA的utility工具包 java.l ...

  8. Linux 文件压缩

    压缩工具   compress/uncompress:对应 .Z 结尾的压缩格式文件  压缩格式:gz.bz2.xz.zip.Z gzip  压缩文件并删除源文件(生成.gz的文件) gunzip 解 ...

  9. C#实现简单的串口通信

    前言 本着学习研究的态度,用c#语言实现简单的串口通信工具. 一.串口通信原理 串口通信 串口通信(Serial Communications)的概念非常简单,串口按位(bit)发送和接收字节.尽管比 ...

  10. Hibernate 一对多

    表与表之间关系回顾(重点) 1 一对多 (1)分类和商品关系,一个分类里面有多个商品,一个商品只能属于一个分类 (2)客户和联系人是一对多关系 - 客户:与公司有业务往来,百度.新浪.360 - 联系 ...