MyGeneration学习笔记(3) : dOOdads及生成代码的一些bug

      没有软件会不存在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，正等着大家去发现^_^