Question

我正在使用Ado按ID检索单个记录。观察：

public async Task<Image> GetImage(int id)
{
    var image = new Image();

    using (SqlConnection conn = new SqlConnection(ConnectionString))
    {
        conn.Open();

        string sql = @" SELECT * FROM Images where id = @id";

        using (SqlCommand comm = new SqlCommand(sql, conn))
        {
            comm.Parameters.AddWithValue("@id", id);

            var reader = await comm.ExecuteReaderAsync();

            int ordId = reader.GetOrdinal("id");
            int ordName = reader.GetOrdinal("name");
            int ordPath = reader.GetOrdinal("path");

            while (reader.Read())
            {
                image.Id = reader.GetInt32(ordId);
                image.Name = reader.GetString(ordName);
                image.Path = reader.GetString(ordPath);
            }

            return image;
        }
    }
}

正如您所看到的，我正在使用While来遍历记录。因为while表示可能有多个记录需要迭代，我相信这可能是获取单个记录的错误方法。考虑到ADO对于一行一个字段具有ExecuteScalar，它们可能具有针对一行多个字段的指定方式。是否有指定的方法在ADO中获取单个记录？

Answer 1

我会选择你当前的方法，除了我消除了while循环。如果要确保只返回一条记录，请执行额外的Read以确保它返回false。这类似于LINQ Single运算符的语义。

if (!reader.Read())        
    throw new InvalidOperationException("No records were returned.");

image.Id = reader.GetInt32(ordId);
image.Name = reader.GetString(ordName);
image.Path = reader.GetString(ordPath);

if (reader.Read())
    throw new InvalidOperationException("Multiple records were returned.");

假设数据库中的id列是主键（唯一），则无需在SQL查询中指定TOP子句; SQL Server查询优化器会推断出由于WHERE子句，最多只返回一条记录。但是，如果您在id列上没有主键或唯一索引/约束，则应发出TOP (2)子句以限制返回的行数。您应该避免使用TOP (1)因为您无法检测（并引发错误）额外匹配。

string sql = @"SELECT TOP (2) * FROM Images WHERE id = @id"

Answer 2

如果您只阅读一次该怎么办：

using (SqlConnection conn = new SqlConnection(ConnectionString))
{
    conn.Open();

    string sql = @" SELECT id, name, path FROM Images where id = @id";

    using (SqlCommand comm = new SqlCommand(sql, conn))
    {
        comm.Parameters.AddWithValue("@id", id);           

        using (var reader = await comm.ExecuteReaderAsync())
        {
            if (!reader.Read())
                 throw new Exception("Something is very wrong");

            int ordId = reader.GetOrdinal("id");
            int ordName = reader.GetOrdinal("name");
            int ordPath = reader.GetOrdinal("path");

            image.Id = reader.GetInt32(ordId);
            image.Name = reader.GetString(ordName);
            image.Path = reader.GetString(ordPath);

            return image;
        }
    }
}

P.S。：我还更改了select语句，只在select语句中选择必填字段和包装阅读器。

Answer 3

您可以在查询中使用Top(1)来获取数据库中的单个记录：

SELECT Top(1) * FROM Images 
where id = @id
order by id desc -- will get the latest record

以正确的方式从SQL Server获取单个记录

3 个答案: