Question

asp.net高手看看

public static List<AboutUsField> GetAll()
{
string sequel ="select * from AboutUs order by Ids asc";
IDataReader dataReader = SqlHelper.ExecuteReader(SqlHelper.conndata, CommandType.Text, sequel);
return LoadListFromDataReader(dataReader);
dataReader.Close();

}

private static List<AboutUsField> LoadListFromDataReader(IDataReader dataReader)
{
List<AboutUsField> list = new List<AboutUsField>();
while (dataReader.Read())
{
list.Add(GetModel(dataReader));
}
return list;
}

private static AboutUsField GetModel(IDataReader dataReader)
{
AboutUsField aboutUsInfo = new AboutUsField();
if (!dataReader["Ids"].Equals(DBNull.Value)) aboutUsInfo.Ids = Convert.ToInt32(dataReader["Ids"]);
if (!dataReader["Title"].Equals(DBNull.Value)) aboutUsInfo.Title = Convert.ToString(dataReader["Title"]);
if (!dataReader["Body"].Equals(DBNull.Value)) aboutUsInfo.Body = Convert.ToString(dataReader["Body"]);
return aboutUsInfo;
}

IDataReader （也就是SqldataReader）用了后要及时的关闭，我的代码关闭IDataReader.close(); 放在上面三段程序什么地方最佳。

Answer 1

public static List<AboutUsField> GetAll()
{
string sequel ="select * from AboutUs order by Ids asc";
IDataReader dataReader = SqlHelper.ExecuteReader(SqlHelper.conndata, CommandType.Text, sequel);
List<AboutUsField> temp=LoadListFromDataReader(dataReader);
dataReader.Close();
return temp;//在return之前得关闭reader

}

Answer 2

我建议你把对数据库的直接操作写成一个类，类里也包含close方法，每次使用只需调用那个类里的增删改查方法后，在finally里面去调用close方法，你这段代码有点复杂了和冗余了

Answer 3

public static List<AboutUsField> GetAll()
{
string sequel ="select * from AboutUs order by Ids asc";
IDataReader dataReader = SqlHelper.ExecuteReader(SqlHelper.conndata, CommandType.Text, sequel);
List<AboutUsField> list=new List<AboutUsField>();
list=LoadListFromDataReader(dataReader);
dataReader.Close();
return list;

}

private static List<AboutUsField> LoadListFromDataReader(IDataReader dataReader)
{
List<AboutUsField> list = new List<AboutUsField>();
while (dataReader.Read())
{
list.Add(GetModel(dataReader));
}
dataReader.Close();
return list;
}

private static AboutUsField GetModel(IDataReader dataReader)
{
AboutUsField aboutUsInfo = new AboutUsField();
if (!dataReader["Ids"].Equals(DBNull.Value)) aboutUsInfo.Ids = Convert.ToInt32(dataReader["Ids"]);
if (!dataReader["Title"].Equals(DBNull.Value)) aboutUsInfo.Title = Convert.ToString(dataReader["Title"]);
if (!dataReader["Body"].Equals(DBNull.Value)) aboutUsInfo.Body = Convert.ToString(dataReader["Body"]);
dataReader.Close();
return aboutUsInfo;
}