关于代码冗余的疑问,该如何处理

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


------解决方案--------------------------------------------------------
老大厉害
------解决方案--------------------------------------------------------
精简代码
------解决方案--------------------------------------------------------
觉得代码２就可以了。