关于代码冗余的疑问
现在有段这样的代码:
//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;
}


[解决办法]
老大厉害
[解决办法]
精简代码
[解决办法]
觉得代码２就可以了。