关于代码冗余的疑问
现在有段这样的代码:
//code style 1
//--------------code begin
private DataSet GetDateSet()
{
DataSet dsData = new DataSet();//些行代码
XmlDataDocument doc = new XmlDataDocument();
doc.DataSet.ReadXml(Server.MapPath( "x.xml "));
dsData = doc.DataSet;
return dsData;
}
//------------------code end
如果改为下面的代码会不会好一些:
// code style 2
//-------------code begin
private DataSet GetDateSet()
{
//DataSet dsData = new DataSet();这行不要
XmlDataDocument doc = new XmlDataDocument();
doc.DataSet.ReadXml(Server.MapPath( "x.xml "));
//dsData = doc.DataSet;这行不要
//return dsData; 这行改为下面的代码
return doc.DataSet;
}
//------------------code end
我认为新建的一个 DataSet 对象是多余, 新建一个 DataSet 对象不仅消耗时间和资源, 而且在后面的 dsData = doc.DataSet 复制也要消耗时间和资源, 大家以为然否?!
或者说 code style 1 有什么好处, code style 2 有什么不足.
------解决方案--------------------为什么不直接使用DataSet的ReadXml方法呢?
private DataSet GetDateSet()
{
DataSet dsData = new DataSet();
dsData.ReadXml(Server.MapPath( "x.xml "));
return dsData;
}
------解决方案--------------------老大厉害
------解决方案--------------------精简代码
------解决方案--------------------觉得代码2就可以了。