是否有更简洁或优雅的方式使用实体框架将多个参数传递给 SQL 存储过程？

Question

现在我正在使用 Entity Framework 将多个参数传递给一个存储过程，它看起来像这样..

public long AddDealer(Dealer dealer)
    {
        return Convert.ToInt32(AWJE.Database.SqlQuery<Dealer>(
            "usp_InsertDealer @Name, @Description",
            new SqlParameter("@DealerID", dealer.DealerID).Value,
            new SqlParameter("@Name", dealer.Name),
            new SqlParameter("@Description", dealer.Description)
            ));
    }

是否有更优雅或更简洁的方式来传递多个参数，而不是显示的方式？如果我遇到其他需要传递更多参数的存储过程，那么这种方式看起来会很快变得混乱。

Answer 1

您的代码示例显然不是一个工作示例：

1. 我不明白如何将Dealer 类型的对象转换为int。
2. 如果您的方法预期返回long，那么为什么要将存储过程的结果转换为int？
3. 您正在传递 @DealerID 参数，但它不是 SP 调用的一部分。
4. 不知道为什么其中一个SqlParameters 调用.Value 添加到它。

所以，让我调整一下，假设你的出发点是这样的：

public long AddDealer(Dealer dealer)
{
    return AWJE.Database.SqlQuery<Dealer>(
        "usp_InsertDealer @Name, @Description",
        new SqlParameter("@Name", dealer.Name),
        new SqlParameter("@Description", dealer.Description)
        ).DealerID;
}

或者您可能决定 SP 直接返回 long，如下所示：

public long AddDealer(Dealer dealer)
{
    return AWJE.Database.SqlQuery<long>(
        "usp_InsertDealer @Name, @Description",
        new SqlParameter("@Name", dealer.Name),
        new SqlParameter("@Description", dealer.Description)
        );
}

无论哪种方式，我看到的唯一简化是您可以将调用更改为SqlQuery，以便使用位置参数而不是命名参数。进行该更改允许您放弃创建显式 SqlParameter 实例。然后可以将调用简化为：

public long AddDealer(Dealer dealer)
{
    return AWJE.Database.SqlQuery<long>(
        "usp_InsertDealer @Name, @Description",
        dealer.Name,
        dealer.Description
        );
}

...您如何命名@Name 或@Description 不再重要，但您必须确保以正确的顺序传入参数值。

除此之外，我不知道你能让这个更干净或更优雅。