c# - 表中的 session 过多导致无法生成新 session

Question

我编写了一个简单的函数，它在登录时生成一个 session 变量并将其存储在 sql 数据库的 session 表中。它在 99% 的时间里都工作得很好，但偶尔，似乎 session 表已满或什么的，因为登录页面将重定向到主页，就像正确登录一样，但是不会创建 session ，也没有用户数据将出现在网络应用程序的任何页面上。

直接通过 Sql Server Mgmt Studio 访问 session 表并手动删除该用户名下的所有现有 session 似乎可以在短期内解决问题，但我想知道导致问题的原因，以及如何更改我的代码以防止将来发生这种情况。

也欢迎任何关于如何改进我的方法的建设性建议

谢谢!

这是我的 C#:

protected void CreateSession(object sender, EventArgs e)
        {
        //generate SessionId
        var AvailableCharacters = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyx1234567890";
        var random = new Random();
        var SessionId = new string(Enumerable.Repeat(AvailableCharacters, 20).Select(s => s[random.Next(s.Length)]).ToArray());

        String UserIdOut = null;

        Session["New"] = SessionId.ToString();

        string connstring = ConfigurationManager.ConnectionStrings["ConnectionString"].ConnectionString;
        using (SqlConnection conn = new SqlConnection(connstring))
        {
            //retreive user id
            try
            {
                SqlCommand cmd = new SqlCommand("spInsertSessionId", conn);
                cmd.CommandType = System.Data.CommandType.StoredProcedure;

                cmd.Parameters.Add("@username", SqlDbType.VarChar).Value = tb_username.Text;
                cmd.Parameters.Add("@createdate", SqlDbType.VarChar).Value = DateTime.Now.ToString();
                cmd.Parameters.Add("@SessionId", SqlDbType.VarChar).Value = SessionId.ToString();

                conn.Open();
                cmd.ExecuteNonQuery();
            }
            catch(Exception ex)
            {
                message = "error inserting session: " + ex.Message;
                ClientScript.RegisterStartupScript(this.GetType(), "Popup", "ShowPopup('" + message + ex.Message + "');", true);
            }
        }
    }

这是插入 session ID 的存储过程:

CREATE PROCEDURE [dbo].[spInsertSessionId]
    @username       VARCHAR(200)    =   NULL    ,
    @UserId         VARCHAR(200)    =   NULL    ,
    @createdate     VARCHAR(200)    =   NULL    ,
    @SessionId      VARCHAR(200)    =   NULL

AS
BEGIN
SELECT 
    @UserId     =   user_id
FROM dbo.users
    WHERE
        username    =   @username

INSERT INTO dbo.session
(
    UserId          ,
    CreateDate      ,
    SessionId
)
VALUES 
(
    @userid         ,
    @createdate     ,
    @SessionId
)

DELETE FROM dbo.session 
        WHERE UserId = @UserId
            AND SessionId NOT IN 
            (
            SELECT TOP(5) SessionId FROM dbo.session 
                WHERE UserId    =   @UserId
                ORDER BY CreateDate DESC
            )
END

Answer 1

根据提供的信息，我大胆猜测问题出在 spInsertSessionId 过程中。插入 session 后，删除“旧” session ，仅保留按 CreateDate DESC 排序的前 5 个 session 。但是，您对 CreateDate 列使用 varchar，如果您不使用具有此属性的日期时间格式，则不保证字符串的排序方式与日期相同。在您的情况下，您只需执行 DateTime.Now.ToString()，这甚至取决于您的文化。所以在某些情况下，您刚刚插入的 session 可能不在您的标准排序的前 5 名中，因此在创建后立即删除。即使不是这种情况 - 不要使用 varchar 来存储日期。

示例:

 declare @d1 varchar(max) = '10/21/2016 00:00:00';
 declare @d2 varchar(max) = '10/22/2015 00:00:00';
 select (case when @d1 > @d2 then 1 else 0 end)

这里d1明显晚于d2，但是查询返回“0”。这些日期来自具有不变文化的 DateTime.ToString()。