PHP 遇到了一个匪夷所思的问题

[frozenway]

写一个用于统计网站访问信息的功能时，遇到了个问题：
-->我的思路是这样的:当网站有人访问时，判断数据表里是否有记录今天访问的信息，如果没有着新建一条记录，然后就在访问量上+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

为什么理想和现实差距这么大？
是我的逻辑有问题了吗?

[b821025551b]

并发问题，要加锁。

[zjsxwc]

用队列处理更方便

[frozenway]

@b821025551b 数据库是 MyISAM,加不了锁

[frozenway]

@zjsxwc 没接触过队列，能说清楚点吗？

[b821025551b]

两个解决方案：
1：上队列；
2：做个定时任务，比如今天凌晨 3 点把明天的数据顺序的 add 一遍，保证明天的数据在明天只有 update 操作。

[sagaxu]

加唯一索引(aid, xdate)，然后用 upsert(insert-on-duplicate)

顺便问一句，楼主你是来黑 PHP 的吗?

[vescape920]

不知道我理解的对不对 就是每个用户每天第一次访问时 insert 记录 之后是 update 该条记录
你可以把 user_id 和 date 建一个 unique 索引 在写 sql 的时候使用 ON DUPLICATE KEY UPDATE

[akira]

aid 和 xdate 做个复合主键 ，代码什么的完全不用动了。
缺点是会丢部分数据

[frozenway]

@akira
@vescape920
@sagaxu @b821025551b
多谢大神指点

[fcten]

mysql 是 php 不可分割的一部分！（滑稽.jpg

[fcten]

顺便，这样统计访问量性能非常差，建议做合理缓存

[yxn1910]

访问次数这样的非敏感数据建议先缓存，定时入库，在这里使用事务性价比太低。

[linpf]

并发量太大，在第一次请求时，数据库判断没有记录，然后再插入新纪录的时候，第二次请求也通过了有无记录的判断，导致重复插入。

最好的办法是加锁。但是加锁也不必依赖数据库。使用 TP 框架的缓存做一个简单的锁也可以。不妨试一下。