这是一个创建于 2959 天前的主题，其中的信息可能已经有所发展或是发生改变。
写一个用于统计网站访问信息的功能时，遇到了个问题：
-->我的思路是这样的:当网站有人访问时，判断数据表里是否有记录今天访问的信息，如果没有着新建一条记录，然后就在访问量上+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