用于更新数据库的按钮不起作用

Question

我有带表格的数据库

• dbo.auctions : [id] [productid] [lastbider] [bidvalue] [lastbid]
• dbo.products : [id] [name]
• dbo.users : [id] [name] [password] [bids]

我需要编写一个程序来检查[bids] 是否大于0。

如果是，则更新[lastbid] 并将[bidvalue] 添加到其中，例如lastbid = lastbid + bidvalue 和[bids] 降低1，bids = bids - 1。我需要将其连接到Visual Studio 中的Button。

我写了这个程序：

ALTER PROCEDURE [dbo].[usp_makebid] (
    @userid int, @auctionid int
)


 AS
DECLARE @bids int, @lastbid int
SELECT @bids = COUNT (*)
FROM users
WHERE users.id = @userid

IF @bids > 0
BEGIN
    UPDATE auctions
    SET lastbid = @lastbid + bidvalue
    WHERE id = @auctionid
        AND lastbid = @lastbid
    UPDATE users
    SET bids = @bids - 1
    WHERE bids = @bids
    END

并将其连接到Button：

protected void Button1_Click(object sender, EventArgs e)
{

        using (var con = new SqlConnection(@"data source=JOVAN-PC;database=aukcija_jovan_gajic;integrated security=true"))
        {
            con.Open();
            var userid = 1;
            var auctionid = 2;


            var cmd = new SqlCommand("usp_makebid", con);
            cmd.Parameters.AddWithValue("@userid", userid);
            cmd.Parameters.AddWithValue("@auctionid", auctionid);
            cmd.CommandType = CommandType.StoredProcedure;
            cmd.ExecuteNonQuery();
            con.Close();
    }
}

但是当我点击按钮时，什么也没有发生。

主页看起来是这样的，也许这可以给你一个大概的想法：

Answer 1

传递用户 ID 和拍卖 ID 的值，然后更新您的表格。

示例：

int userid = 456789;
int auctionid = 12345;

cmd.Parameters.AddWithValue("@userid", SqlDbType.Int).Value = userid;
cmd.Parameters.AddWithValue("@auctionid", SqlDbType.Int).Value = auctionid;

Answer 2

第一个问题是您将 SqlDbType.Int 枚举作为 @userid 和 @auctionid 参数的值传递，这不是您想要做的，而是应该提供一个值。

第二个问题在您的存储过程中。

SELECT bids 
FROM users 
WHERE users.id = @userid

完全没有效果，IF @bids > 0 条件将始终为假。

猜你想得到接受users.id = @userid条件的用户数，那么你应该用

替换上面的选择

SELECT @bids = COUNT(*) 
FROM users 
WHERE users.id = @userid

您应该最终得到以下存储过程：

ALTER PROCEDURE [dbo].[usp_makebid]
(
  @userid INT ,
  @auctionid INT
)
AS 
    DECLARE @bids INT, @lastbid INT

    SELECT @bids = COUNT(*)
    FROM users
    WHERE users.id = @userid

    IF @bids > 0 
    BEGIN
        UPDATE auctions
        SET lastbid = @lastbid + bidvalue
        WHERE id = @auctionid
          AND lastbid = @lastbid

        UPDATE users
        SET bids = @bids - 1
        WHERE bids = @bids
    END

还要注意cmd.Connection = con; 行是多余的，因为Connection 已经通过SqlCommand 构造函数初始化。

试试这个方法：

protected void Button1_Click(object sender, EventArgs e)
{
    using (var con = new SqlConnection(@"data source=JOVAN-PC;database=aukcija_jovan_gajic;integrated security=true"))
    {
        con.Open();
        var userid = 1;
        var auctionid = 2;

        var cmd = new SqlCommand("usp_makebid", con);
        cmd.Parameters.AddWithValue("@userid", userid);
        cmd.Parameters.AddWithValue("@auctionid", auctionid);
        cmd.CommandType = CommandType.StoredProcedure;

        cmd.ExecuteNonQuery();
        con.Close();
    }
}