避免一个用户多次登录修改版

原来的代码参见这里
http://www.qiuhao.com//dispbbs.asp?boardID=2&ID=6228&page=1

今天仔细看了这段代码,发现这段代码大有优化的余地

因为maxSessions 这个值可能会很大,我这里测试环境有3万多个,启动时要浪费半分钟左右

另外sessionId是个随机数,用1到maxSessions其实是很少有命中的(sessionId大家可以在标题栏上看到)

本来想顺着管理员的思路优化(但没弄通,把思路写在这里供大家参考)

1. xSession::numSession返回当前已经注册的会话 ,所以当这个数字是1是就不用查了

2. 用户的查询是否可以放在一开始进行呢,重复执行没有必要

3. 仅仅遍历1-xSession::numSession的用户,但这个没弄通因为sessionId不是和1-xSession::numSession的顺序数对应的

所以我就找能一次获得所有会话列表的途径,其实在管理-联机用户中就有,就是SysClientSessions表

所以,这个任务就很简单,变成查询这个表的事情

 

 

void  startupPost()
{

    SysClientSessions sessions;
    userId currentUserId;
    int loginNum;
    ;

//只有一个会话,当然不需要检查
      if (xSession::numSession()==1)
        return;
      currentUserId   = curUserId();

      while select userId  from sessions  where sessions.userId==currentUserId{

            loginNum++;
            if (loginNum>1){
                Box::stop("@GCN996"); //提示文本,不能重复登录之类的,自己替换
                infoLog.shutDown(true);
                return;


             }

      }

      return;

}

其实大家可以发现,这个查询本来可以更简单的,像

 

 

select   count (sessionId)  from  sessions  group   by  userId  where  sessions.userId == currentUserId;

不知道怎么回事,这代码在我这里就是编译不了,所以改成上面这样的了