这是一个创建于 2614 天前的主题，其中的信息可能已经有所发展或是发生改变。
写一个用于统计网站访问信息的功能时，遇到了个问题：
-->我的思路是这样的:当网站有人访问时，判断数据表里是否有记录今天访问的信息，如果没有着新建一条记录，然后就在访问量上+1 ；如果已经有记录了，着直接在访问量上+1 ；代码如下：
//统计访问次数
protected function ttVisit($aid){
if(is_numeric($aid)){
$model = M('monitor');
$row = $model->where(['aid'=>$aid, 'xdate'=>date('Y-m-d')])->field('moid')->find();
if(!$row){
$row['moid'] = $model->data(['xdate'=>date('Y-m-d'), 'aid'=>$aid])->add();
if(!$row['moid']){
return false;
}
}
$model->where(['moid'=>$row['moid']])->setInc('top');
return $row['moid'];
}
return false;
}
其中$aid 是对应某个网站的 ID,top 是数据库字段，表示访问量。xdate 是要统计访问量的日期。
好了，这样的逻辑一点问题都没有，但是真正运行起来，理想状态数据表应该是这样的:
id aid xdate top
1 1 2017-11-01 20
2 2 2017-11-01 100
3 5 2017-11-01 96
然而现实运行起来是这样的
id aid xdate top
1 1 2017-11-01 1
2 1 2017-11-01 19
3 2 2017-11-01 100
4 5 2017-11-01 94
5 5 2017-11-01 1
6 5 2017-11-01 1
为什么理想和现实差距这么大？
是我的逻辑有问题了吗?
-->我的思路是这样的:当网站有人访问时，判断数据表里是否有记录今天访问的信息，如果没有着新建一条记录，然后就在访问量上+1 ；如果已经有记录了，着直接在访问量上+1 ；代码如下：
//统计访问次数
protected function ttVisit($aid){
if(is_numeric($aid)){
$model = M('monitor');
$row = $model->where(['aid'=>$aid, 'xdate'=>date('Y-m-d')])->field('moid')->find();
if(!$row){
$row['moid'] = $model->data(['xdate'=>date('Y-m-d'), 'aid'=>$aid])->add();
if(!$row['moid']){
return false;
}
}
$model->where(['moid'=>$row['moid']])->setInc('top');
return $row['moid'];
}
return false;
}
其中$aid 是对应某个网站的 ID,top 是数据库字段，表示访问量。xdate 是要统计访问量的日期。
好了，这样的逻辑一点问题都没有，但是真正运行起来，理想状态数据表应该是这样的:
id aid xdate top
1 1 2017-11-01 20
2 2 2017-11-01 100
3 5 2017-11-01 96
然而现实运行起来是这样的
id aid xdate top
1 1 2017-11-01 1
2 1 2017-11-01 19
3 2 2017-11-01 100
4 5 2017-11-01 94
5 5 2017-11-01 1
6 5 2017-11-01 1
为什么理想和现实差距这么大？
是我的逻辑有问题了吗?
13 条回复 • 2017-11-01 17:29:26 +08:00
 |
1
b821025551b 2017-11-01 16:02:19 +08:00
并发问题，要加锁。
|
 |
2
zjsxwc 2017-11-01 16:08:56 +08:00 via Android
用队列处理更方便
|
 |
3
frozenway OP @b821025551b 数据库是 MyISAM,加不了锁
|
|
5
b821025551b 2017-11-01 16:14:15 +08:00
两个解决方案：
1：上队列； 2：做个定时任务，比如今天凌晨 3 点把明天的数据顺序的 add 一遍，保证明天的数据在明天只有 update 操作。 |
 |
6
sagaxu 2017-11-01 16:24:28 +08:00
加唯一索引(aid, xdate)，然后用 upsert(insert-on-duplicate)
顺便问一句，楼主你是来黑 PHP 的吗? |
 |
7
vescape920 2017-11-01 16:24:33 +08:00
不知道我理解的对不对 就是每个用户每天第一次访问时 insert 记录 之后是 update 该条记录
你可以把 user_id 和 date 建一个 unique 索引 在写 sql 的时候使用 ON DUPLICATE KEY UPDATE |
 |
8
akira 2017-11-01 16:28:35 +08:00
aid 和 xdate 做个复合主键 ，代码什么的完全不用动了。
缺点是会丢部分数据 |
|
9
frozenway OP |
 |
10
fcten 2017-11-01 17:01:28 +08:00
mysql 是 php 不可分割的一部分！（滑稽.jpg
|
|
11
fcten 2017-11-01 17:04:23 +08:00
顺便，这样统计访问量性能非常差，建议做合理缓存
|
 |
12
yxn1910 2017-11-01 17:16:10 +08:00
访问次数这样的非敏感数据建议先缓存，定时入库，在这里使用事务性价比太低。
|
 |
13
linpf 2017-11-01 17:29:26 +08:00
并发量太大，在第一次请求时，数据库判断没有记录，然后再插入新纪录的时候，第二次请求也通过了有无记录的判断，导致重复插入。
最好的办法是加锁。但是加锁也不必依赖数据库。使用 TP 框架的缓存做一个简单的锁也可以。不妨试一下。 |
Select Language