请教大家一个很怪异的问题！ExecuteReader方法查询返回值问题？？？？此方法是petshop项目中的SqlHelper类下的。

[云中雁荡山]

具体描述：我使用petshop项目中的SqlHelper.cs类下面的ExecuteReader方法查询数据值的时候，得到的结构总是比预期的要少一条（比如用相同的sql语句在数据库中查询得到的结果是5条数据，而用此方法查询得出的结果只为4条。第一条数据不翼而飞了！）请大家帮忙看看。
相关方法如下：

///// <summary>
        ///// 获得数据列表
        ///// </summary>
        public DataSet GetList(string strWhere)
        {
            StringBuilder strSql = new StringBuilder();
            strSql.Append("select OpenGameNumID,OpenNumName,OpenNum,Remark ");
            strSql.Append(" FROM OpenGameNum ");

            if (strWhere.Trim() != "")
            {
                strSql.Append(" where " + strWhere);
            }

            using (SqlDataReader rdr = SqlHelper2.ExecuteReader(SqlHelper2.ConnectionStringLocalTransaction, CommandType.Text, strSql.ToString(), null))
            {
                while (rdr.Read())
                {
                    ds = ConvertDataReaderToDataSet(rdr);//这边的rdr的值已经少了一条了
                }
            }

            //DataSet ds = SqlHelper2.ExecuteDataSet(SqlHelper2.ConnectionStringLocalTransaction, CommandType.Text, strSql.ToString(), null);
            //用这种方法查询出来的结果是正确的

            return ds;
        }

以下为最终调用的方法：

/// <summary>
        /// Execute a SqlCommand that returns a resultset against the database specified in the connection string 
        /// using the provided parameters.
        /// </summary>
        /// <remarks>
        /// e.g.:  
        ///  SqlDataReader r = ExecuteReader(connString, CommandType.StoredProcedure, "PublishOrders", new SqlParameter("@prodid", 24));
        /// </remarks>
        /// <param name="connectionString">a valid connection string for a SqlConnection</param>
        /// <param name="commandType">the CommandType (stored procedure, text, etc.)</param>
        /// <param name="commandText">the stored procedure name or T-SQL command</param>
        /// <param name="commandParameters">an array of SqlParamters used to execute the command</param>
        /// <returns>A SqlDataReader containing the results</returns>
        public static SqlDataReader ExecuteReader(string connectionString, CommandType cmdType, string cmdText, params SqlParameter[] commandParameters)
        {
            SqlCommand cmd = new SqlCommand();
            SqlConnection conn = new SqlConnection(connectionString);
            SqlDataReader rdr = null;
            // we use a try/catch here because if the method throws an exception we want to 
            // close the connection throw code, because no datareader will exist, hence the 
            // commandBehaviour.CloseConnection will not work
            try
            {
                PrepareCommand(cmd, conn, null, cmdType, cmdText, commandParameters);
                rdr = cmd.ExecuteReader(CommandBehavior.CloseConnection);
                
                //cmd.Parameters.Clear();
                return rdr;
            }
            catch
            {
                conn.Close();
                throw;
            }
        }

相关的个方法：

 /// <summary>
        /// Prepare a command for execution
        /// </summary>
        /// <param name="cmd">SqlCommand object</param>
        /// <param name="conn">SqlConnection object</param>
        /// <param name="trans">SqlTransaction object</param>
        /// <param name="cmdType">Cmd type e.g. stored procedure or text</param>
        /// <param name="cmdText">Command text, e.g. Select * from Products</param>
        /// <param name="cmdParms">SqlParameters to use in the command</param>
        private static void PrepareCommand(SqlCommand cmd, SqlConnection conn, SqlTransaction trans, CommandType cmdType, string cmdText, SqlParameter[] cmdParms)
        {

            if (conn.State != ConnectionState.Open)
                conn.Open();

            cmd.Connection = conn;
            cmd.CommandText = cmdText;

            if (trans != null)
                cmd.Transaction = trans;

            cmd.CommandType = cmdType;

            if (cmdParms != null)
            {
                foreach (SqlParameter parm in cmdParms)
                    cmd.Parameters.Add(parm);
            }
        }


/// <summary>    
        /// 执行SQL语句并返回DataSet    
        /// </summary>    
        /// <param name="Sqlstr">SQL语句</param>    
        /// <returns></returns>    
        public static DataSet ExecuteDataSet(string connectionString, CommandType cmdType, string cmdText, params SqlParameter[] commandParameters)
        {
            DataSet ds = new DataSet();
            using (SqlConnection conn = new SqlConnection(connectionString))
            {
                SqlCommand cmd = new SqlCommand();
                PrepareCommand(cmd, conn, null, cmdType, cmdText, commandParameters);
                SqlDataAdapter da = new SqlDataAdapter(cmdText, conn);
                
                da.Fill(ds);
                conn.Close();
                return ds;
            }
        }

请高手指点！

[newdigitime]

因为你进行了rdr.read()操作,这样指针会向下移动一条.
所以应更改下面的:
while (rdr.Read())
{
ds = ConvertDataReaderToDataSet(rdr);//这边的rdr的值已经少了一条了
}
改为
ds = ConvertDataReaderToDataSet(rdr);
然后在ConvertDataReaderToDataSet方法中使用while (rdr.Read())

[来自故乡的风]

不是高手胡乱说几句：

感觉这几行挺怪异的。本来while (rdr.Read())是逐行操作的，但你循环里边的ConvertDataReaderToDataSet(rdr)是对整个rdr进行操作的，所以循环没必要，或者是ds总返回最后一条记录。

while (rdr.Read())
{
ds = ConvertDataReaderToDataSet(rdr);//这边的rdr的值已经少了一条了
}

建议你在循环里加个计数的变量，看看到底有多少行。

[skey123123]

[Quote=引用 5 楼 newdigitime 的回复:]
因为你进行了rdr.read()操作,这样指针会向下移动一条.
所以应更改下面的:
while (rdr.Read())
{
ds = ConvertDataReaderToDataSet(rdr);//这边的rdr的值已经少了一条了
}
改为
ds = ConvertDataReaderToDataSet(rdr);
然后在ConvertDataReade……
[/Quote]
我也感觉是你read（）之后，数据向下走了一条。

[wuyq11]

while (rdr.Read())
{
ds = ConvertDataReaderToDataSet(rdr);
}
不要转换，直接返回rdr
还有检查ConvertDataReaderToDataSet

[yangbo4235]

while (rdr.Read())------------如果我没看错事这个地方出问题了，执行Read()方法以后数据标记会向下移动一行，那么这时候执行ConvertDataReaderToDataSet(rdr);当然会少了第一行的数据，所以你把while里面条件改为rdr！=null这样应该就可以把数据全部放进去了
{
ds = ConvertDataReaderToDataSet(rdr);
}

[云中雁荡山]

首先谢谢大家的回复，但是大家看帖不仔细啊。我的帖子中已经提到了
ds = ConvertDataReaderToDataSet(rdr);//这边的rdr的值已经少了一条了
此处返回的rdr数据的条数已经少了一条了，我再怎么循环他也不可能是正确的了，请指点！