2

运行以下代码时，它会遗漏一行。当我执行 files.Count 时，它说有 4 行，但没有为第 4 行存储数据。当我从 SQL 管理器中运行存储过程时，它会返回所有 4 行和所有数据。有什么帮助吗？

            List<File> files = new List<File>();
            SqlConnection active_connection = new SqlConnection(m_connection_string);
            SqlCommand cmd = new SqlCommand();
            SqlDataReader dr = null;

            try
            {
                active_connection.Open();
                cmd.Connection = active_connection;
                cmd.CommandType = CommandType.StoredProcedure;
                cmd.CommandText = "dalsp_Select_Organization_Files";

                SqlParameter param;

                param = cmd.Parameters.Add("@p_organization_guid", SqlDbType.UniqueIdentifier);
                param.Value = new Guid(organization_guid);

                param = cmd.Parameters.Add("@p_file_type", SqlDbType.NVarChar, 50);
                param.Value = file_type;

                dr = cmd.ExecuteReader(CommandBehavior.CloseConnection);

                if (dr.HasRows)                    
                {                    
                    while (dr.Read())
                    {
                        File file = new File();
                        file.OrganizationGuid = dr["OrganizationGuid"].ToString();
                        file.FileGuid = dr["FileGuid"].ToString();
                        file.FileLocation = dr["FileLocation"].ToString();
                        file.FileName = dr["FileName"].ToString();
                        file.FileType = (FileTypeEnum)Enum.Parse(typeof(FileTypeEnum), dr["FileType"].ToString());
                        file.FileExtension = dr["FileExtension"].ToString();
                        file.FileDescription = dr["FileDescription"].ToString();
                        file.ThumbnailPath = dr["ThumbnailPath"].ToString();
                        files.Add(file);
                    }       
                }
                dr.Close();
                dr = null;

                active_connection.Close();
                cmd = null;

            }
            catch (Exception)
            {
                throw;
            }
            finally
            {
                if (active_connection.State != ConnectionState.Closed)
                {
                    active_connection.Close();
                    active_connection.Dispose();
                }
            }

            return files;
4

4 回答 4

5

如果您说您的文件集合有 4 个项目，但 4 个项目不包含任何值，那是什么意思？是 null，对象没有数据，还是抛出索引超出范围异常？

你在做一个文件[4]或类似以下的东西吗？

for(int x = 1; x < files.length; x++)
{
     files[x]
}

那是行不通的。记住 C# 中基于 0 的索引。

作为旁注，您可以通过执行以下操作来取消您的 try catch 语句：

using (SqlConnection connection = new SqlConnection(conn_string))
{
    connection.Open();
    using (SqlCommand cmd = new SqlCommand("SELECT * FROM MyTable", connection))
    {
         using (SqlDataReader dr = cmd.ExecuteReader())
         {
             return result;
         }
    }
}

using 语句将保证读取器和连接的处置（并因此关闭）。

于 2008-11-20T06:24:32.400 回答
2

您应该放弃“if (dr.HasRows)”，它所做的只是在 while 循环中重复检查。

您不应该在连接上调用 Close，也不应该将其设置为 null。相反，您应该将连接包装在“使用”块中，如下所示：

using (SqlCommand cmd = new SqlCommand()) {
   //use the connection
}

数据阅读器和 SQL 命令也是如此。

下一点什么都不做，删除它。

  catch (Exception) { throw; }
于 2008-11-20T08:35:08.643 回答
1

您是否尝试过在调试器中逐步执行此操作，并在执行阅读器之前检查您的命令参数？您在结果集中获得的值是否与直接在 sql 上运行 sproc 时相同？

我可能是错的，因为有几种方法可以做到这一点，但是在您将参数添加到命令的方式中看起来有些古怪。

我通常使用的模式更像：

SqlParameter param = new SqlParameter();  
// set param stuff - here or in ctor   
cmd.Parameters.Add(param);

主要是关于什么对你有用....

大多数情况下，我遇到存储过程问题的时候是我对参数进行处理的时候。

于 2008-11-20T06:08:49.140 回答
1

代码对我来说看起来是正确的。我认为您肯定希望通过调试器来查看从 ExecuteReader 返回了多少行。我的一条评论是“if (dr.HasRows)”有点多余，因为“while (dr.Read())”会给你同样的效果。

我想问的另一个问题是你知道你是错过了第一条记录还是最后一条记录？

布赖恩

于 2008-11-20T06:26:18.620 回答