[毕业生的商业软件开发之路]尽早暴露错误原则 推荐

近期开始接触到在校学生、高校实习生和毕业生，在此说一下笔者对这些徘徊在职场门口的学生一些建议,希望能给这些初学者进入软件开发行业带来一些帮助,使得毕业生能更顺利的进入软件开发公司开始职场生涯，人生来一个完美的转弯。

-----------------------------------------------------------------------

开发者应当遵守“尽早暴露错误”的开发原则，这是因为程序错误是客观存在的事实，应当正视它并有效的处理它，而不是简单粗暴的和谐掉。而且在开发中应当尽早暴露出程序的错误，这有助于发现错误的本质，帮助改善程序质量；若一味的和谐掩盖错误，则错误越拖影响越大，最后不可收拾，程序崩溃。[袁永福版权所有]

下面举个这条原则的例子，现数据库中有一个名为Customers的数据表，其内容如下：

CustomerID	CompanyName	ContactName	ContactTitle	Address	City
1	少室山公司	方证	采购员	东园西甲 30 号	长平
2	擎天航空	雷震子	销售代表	常保阁东 80 号	莫斯科
3	华夏工程	李大禹	市场经理	广发北路 10 号	幽州
4	武当投资	宋青书	物主	临翠大街 80 号	巴伐利亚
5	擎天南京公司	大星星	物主	花园东街 90 号	许安

对此笔者可以写出以下的两种代码来读取并输出其中的数据：
第一种：

using (IDbConnection conn = new OleDbConnection())
{
// 连接数据库
conn.ConnectionString = "Provider=Microsoft.Jet.OLEDB.4.0;Data Source="
+ System.IO.Path.Combine(Application.StartupPath, "Customers.mdb");
conn.Open();
using (IDbCommand cmd = conn.CreateCommand())
{
// 设置SQL语句
cmd.CommandText = @"
Select
CustomerID ,
CompanyName ,
ContactName ,
ContactTitle ,
Address ,
City
From Customers";
// 读取并输出数据
IDataReader reader = cmd.ExecuteReader();
if (reader.Read())
{
Console.WriteLine("CustomerID  =" + reader["CustomerID"]);
Console.WriteLine("CompanyName =" + reader["CompanyName"]);
Console.WriteLine("ContactName =" + reader["ContactName"]);
Console.WriteLine("ContactTitle=" + reader["ContactTitle"]);
Console.WriteLine("Address     =" + reader["Address"]);
Console.WriteLine("City        =" + reader["City"]);
}//if
reader.Close();
}//using
}//using

第二种：

using (IDbConnection conn = new OleDbConnection())
{
// 连接数据库
conn.ConnectionString = "Provider=Microsoft.Jet.OLEDB.4.0;Data Source="
+ System.IO.Path.Combine(Application.StartupPath, "Customers.mdb");
conn.Open();
using (IDbCommand cmd = conn.CreateCommand())
{
// 设置SQL语句
cmd.CommandText = @"Select *  From Customers";
// 读取并输出数据
IDataReader reader = cmd.ExecuteReader();
if (reader.Read())
{
Console.WriteLine("CustomerID  =" + reader["CustomerID"]);
Console.WriteLine("CompanyName =" + reader["CompanyName"]);
Console.WriteLine("ContactName =" + reader["ContactName"]);
Console.WriteLine("ContactTitle=" + reader["ContactTitle"]);
Console.WriteLine("Address     =" + reader["Address"]);
Console.WriteLine("City        =" + reader["City"]);
}//if
reader.Close();
}//using
}//using

这两种代码的唯一的区别就在于其中使用了不同的SQL语句，第一种代码使用的SQL语句是

Select CustomerID , CompanyName , ContactName , ContactTitle , Address , City From Customers

而第二种代码使用的SQL语句是

Select * From Customers

第一种SQL语句字符比较多，开发者比较懒的话很容易写出第二种SQL语句。不过笔者推荐使用第一种SQL语句，因为第一种SQL语句符合尽早暴露程序错误的原则。[袁永福版权所有]
如果当数据库没有问题，则两种SQL语句都能执行，程序都能正确的读取数据，此时第二种写法反而貌似不错。
不过实际中应用程序开发和运行环境很复杂，比如发生了代码中字段名拼写错误、数据库字段结构发生改变，这些都是需要考虑到的问题。
例如数据表[袁永福版权所有]Customers中的字段Address由于某种原因删掉了，于是这两种代码都会发生错误。
对于第一种代码，程序会在执行代码“IDataReader reader = cmd.ExecuteReader();”时就会爆出异常“至少一个参数没有指定值 OleDbException”。此时开发者借助VS.NET可以很快的确定出发生错误的代码，并根据错误提示很容易猜测SQL语句写错了。于是开发者进行SQL语句与数据库结构的对比，很快就能发现错误的本质，那就是字段Address突然没了。
而对于第二种代码，程序顺利的执行了代码“IDataReader reader = cmd.ExecuteReader();”,但在执行代码“Console.WriteLine("Address =" + reader["Address"]);”时爆出异常“Address IndexOutOfRangeException”。对于这个错误，开发者首先会有些迷惑，不知为什么发生错误，因为错误提示信息和数据库联系不大，而后怀疑代码中的“reader["Address"]”出现字段名拼写错误，花上一段时间仔细校对确定无误后才会往前继续寻找错误的来源，会发现SQL语句没有拼写错误，最后才会想到去查数据库结构，绕了半天才发现数据库字段Address没了，这才是错误的本质。
在这个例子中，数据库字段Address删掉的那一刻起，系统就存在隐患，而第一种代码能在第一时间由于这个隐患而爆出错误，而开发者就能立即定位到离隐患最近的代码，得到跟隐患密切相关的错误提示信息，也就能非常快的发现问题的本质，从而解决问题。这是一种将错误扼杀在摇篮当中的做法，对开发者对程序都有好处。[袁永福版权所有]
而第二种代码很和谐，没能在第一时间让隐患爆出错误，让隐患养着更肥，一直默默的影响着程序的运行，最终隐患爆出更大的更让人摸不着头脑的错误，使得开发者需要花费更多的时间精力来处理这个错误，这是姑息隐患，将错误养大了出栏再杀的做法，对开发者对程序都有害处。
在VB中有一个写作“On Error Resume Next”的语句，号称能和谐掉程序中所有的错误，让程序不爆出任何异常，丫就彻彻底底的破坛子破摔、掩耳盗铃的做法，因此这种写法应用得比较少。
因此开发者要写出一个真正和谐的程序，其过程必然不能简单粗暴的和谐，要尽可能早的让各种隐患爆出程序错误；隐患要上访变成错误，程序不能打击压制，而且程序要疏通各种隐患上访的通道，让开发者更快的发现错误的本质，更快的解决错误。尊重科学规律，正视隐患和错误，正确处理错误而不是打压隐患，程序才能和谐健壮。
嘿嘿，人类社会又何尝不是这样啊。[袁永福版权所有]