检查用户是否对帖子进行了投票

Question

我正在为类似博客的网站上的帖子创建投票系统。

每个帖子都有一个评分，存储在数据库中。用户可以点击支持或反对，数据库中的数字会增加或减少。简单的东西。

但是如何检查当前用户是否已经投票？

我最初的想法是在数据库中为每个帖子创建两个额外的列，“UsersUp”和“UsersDown”，存储对每个帖子投赞成票和投反对票的用户名。我会检查用户名是否存在，如果存在，投票将被删除，而不是添加另一个投票。 我什至可以完全删除“评分”列并通过计算 UsersUp 中的用户名数量并减去 UsersDown 中的用户名数量来实现相同的效果。

我是否以错误的方式处理这个问题？

我不是要任何代码，只是要我如何实现这一点的理论。

我希望用户不能两次投赞成票或反对票，并且他们需要能够通过再次点击来撤回投票。

Answer 1

您应该使用normalized database design。

您可能有一个User 表，根据问题，您有一个Post 表。

您可以使用第三个UserVote 表来表示用户、帖子和他们的投票之间的关系。这允许精确跟踪投票者、投票时间和投票方式（VoteTypeId 可以包含赞成或反对票的 ID）。

+--------+--------+------------+----------+
| UserId | PostId | VoteTypeId | VoteDate |
+--------+--------+------------+----------+
|    1   |    1   |      1     | 1/1/2017 |
+--------+--------+------------+----------+
|    2   |    1   |      1     | 2/1/2017 |
+--------+--------+------------+----------+
|    3   |    1   |      2     | 2/2/2017 |
+--------+--------+------------+----------+

这对数据进行建模。你应该如何从 c# 中调用它？

if( !HasVoted( userId, postId ) ){
    SaveVote( userId, postId, voteTypeId );
    UpdateVoteCount( postId, voteTypeId );
}

但是，这很容易出错。如果UpdateVoteCount() 失败了怎么办？

有多种方法可以解决这个问题，其中包括：

• 在需要时汇总投票计数（不要存储它们）。
• 将操作包装在事务中。

using( var scope = new TransactionScope() ){    
    if( !HasVoted( userId, postId ) ){
        SaveVote( userId, postId, voteTypeId );
        UpdateVoteCount( postId, voteTypeId );
    }

    scope.Complete();
}

阅读更多：TransactionScope

这有助于数据完整性，但代码中仍有race condition 允许系统被滥用。

我们需要锁定整个操作。实现此目的的一种方法是使用存储过程sp_getapplock，它允许您阻塞指定资源，直到锁的持有者释放它。

您可以使用单个命名锁，或者您可以锁定每个项目（如果对一个项目进行投票可能会产生更广泛的后果，例如声誉计算，这会很棘手）。

阅读更多：sp_getapplock

我们的序列变成：

1. 获取命名锁。
2. 开始交易。
3. 检查用户是否尚未投票。
4. 登记投票。
5. 更新其他值（例如总票数、用户声誉增益）。
6. 提交事务。
7. 释放锁。

虽然这听起来很复杂，而且锁听起来会成为瓶颈，但它实际上表现良好。几年前我对该设计进行了压力测试，并轻松实现了每秒 2500 多个投票操作的吞吐量。

调用代码应确保：

• 所取锁的超时时间很短
• 即使抛出异常，事务也会提交/回滚。
• 即使抛出异常，锁也会被释放。