happyhippy

这个世界的问题在于聪明人充满疑惑,而傻子们坚信不疑。--罗素


      没有软件会不存在bug,找到一个bug只能说明系统还存在Bug。MyGeneration也不例外,其中存在存在bug;但我们不能因为其有bug而不去用它,这是因噎废食。下面就是我在使用dOOdad时遇到的几个bug及我自己的解决办法,发出来与大家共享,避免大家与我做重复的调试工作。

 

1.  连接字符串初始化
        在配置文件web.config或app.config配置连接字符串。dOOdad项目中默认的key是"dbConnection",也可以修改BusinessEntity类中的_defaultConfig字段的值来修改key的默认值。
        另外,dOOdad项目有个问题就是连接字符串并没有初始化(可能是模版的bug),这时需要手动初始化。解决方法参看我的本系列第一篇文章使用MyGeneration生成sp和代码中的part5。


2.  向数据库中插入一行:

      下面是dOOdad给定一份示例代码,执行这段代码会有点小问题:

Employees emps = new Employees();
emps.AddNew();
emps.LastName 
= "Smith";
emps.HireDate 
= DataTime.Now;
emps.Save();
int empsID = emps.EmplyeeID;//插入后返回主键值。

         调用Save方法插入数据库没有问题,但是获取主键值会抛出异常。默认生成的代码没有将ID作为输出参数,所以Save后emps.EmplyeeID仍然为空,不能转换成int。更正方法是在dOOdad生成业务实体抽象类Employees的GetInsertCommand()方法中加一条语句:

CreateParameters(cmd);
cmd.Parameters[
"@ID"].Direction = ParameterDirection.Output;//要加入的语句
return cmd;

         现在执行上面的示例代码就不会出现异常了。

 


3.   BusinessEntity中Setstring方法也存在问题

protected void Setstring(string columnName, string data)
{
    
if(0 == data.Length) 
           _dataRow[columnName] 
= DBNull.Value;
    
else
           _dataRow[columnName] 
= data;
}

    显然,当为第二个参数data传入null时,会有问题。修正:if的判断条件改为:if ((null == data) || (0 == data.Length))但按照FxCop性能规则,这种判断字符串非空的方法不是很高效,推荐用String.IsNullOrEmpty(data)来判断是否为空。其他判读字符串非空的地方也存在同样的问题。

 


4. BusinessEntity类提供的ToXml()和FromXml()两个方法,实现了字符串到业务实体的转化。
         不过这种是由代价的:ToXml将业务实体换成string后,会丢失部分重要信息,如DataSet的Schema、DataRow的状态(Added/Modified/Deleted)、没有赋值的属性等,所以FromXml不能完全转换回原来的DataTable。
例如:

Employee emps = new Employee();
emps.LoadByPrimaryKey(id)
string s = emp.ToXml();
Employee temp 
= new Employee();
temp.LoadFromXml();

         执行上面的代码后,temp已经严重失真了。例如访问temp.ID就会抛出InvalidCastException异常,访问temp对象的其他非字符串属性也会抛出该异常。产生这个问题原因可能是经过ToXml和FromXml转换后,_dataRow[columnName]的数据类型信息丢失了,统一都成了string类型,强制类型转换会抛出异常。
         Getbool、Getint等一系列Get***方法(GetString除外)都存在这个问题,解决办法是用Convert类来转换,将强制类型转换换成Convert.To***()。

 

5.  更多的Bug,正等着大家去发现^_^



posted on 2006-08-23 05:17  Silent Void  阅读(1209)  评论(1编辑  收藏  举报