redis秒杀出现超卖现象的问题

问答 / 15 / 33 / 创建于 2年前 / 更新于 2年前

公司做一个秒杀的活动，十来个商品，商品的限购数量不一。因为秒杀的时间是固定的早上 10 点到晚上 8 点

所以我每天早上 9 点 45 都会加库存到 redis 中

redis秒杀出现超卖现象的问题

把秒杀的商品 redis 存储起来

redis秒杀出现超卖现象的问题

然后就是秒杀了
这里先判断是不是存在在秒杀商品中

redis秒杀出现超卖现象的问题

如果是秒杀商品的话，redis 判断库存是否为 0，不为 0 减一

redis秒杀出现超卖现象的问题

前几天人数多，但是今天尤其的多，今天出现了超卖现象，redis 打印值是 0，也没变成负值。想问下这个超卖，我的方法有哪些问题导致的呢

更新一下，用了 lpush 和 rpop，仍然出现了超卖现象

redis秒杀出现超卖现象的问题

准备试试 lua

用 lua 报错了，谁能看看呀
redis秒杀出现超卖现象的问题

redis秒杀出现超卖现象的问题

颠倒的玉石

课程读者 166 声望

技术小垃圾 @ 山西证券

暂无个人描述~

1 人点赞

从小程序个人账户申请开始，带你一步步进行开发一个微信小程序，直到提交微信控制台上线发布。

从零开始带你一步步开发一个 Go 博客项目，让你在最短的时间内学会使用 Go 进行编码。项目结构很大程度上参考了 Laravel。

推荐文章：

更多推荐...

博客

用 Laravel12 Startkit 做了一个 composer 私有包托管平台 😂 点赞超过 20 个开源，看看需要的人多不多 27 / 19 |

如何打造令后端面试官印象深刻的简历？ 14 / 10 |

博客

花了四个月打磨的 Laravel Plus 开源 30 / 98 |

博客

冯老师的困惑 —— 一个跑了两年的 BUG 19 / 16 |

博客

在laravel下实现全双工的websocket开发 21 / 8 |

分享创造

无需修改任何代码和扩展将你的Laravel项目性能提高20倍 101 / 98 |

第二天堂

48 声望

最佳答案

目测是这里出了问题

file

使用 redis string 类型，存库存的数量

Laravel

使用 redis list 类型

Laravel

redis list 类型初始化库存

Laravel

大概是这样，这是我之前测试的代码，仅供参考

2年前评论

第二天堂（作者）

@颠倒的玉石不用判断，就是你提前把库存放入数据，比如库存有5个：rpush 商品key 1 1 1 1 1，然后秒杀的时候直接 lpop 商品key这样，我上面有写我之前测试的例子，可以换参考一下

颠倒的玉石（楼主）

第二种方式就是比如5个库存，push5个数据。每次先判断list长度，然后看是否需要pop么

颠倒的玉石（楼主）

@chenrenhui 我理解你的意思了，这个lua说实话我不会，push可以试试。用事务可不可以呢

第二天堂（作者）

@颠倒的玉石事务之前测试了，使用的是watch, 但是高并发的时候好像会出现超卖，可能是我用法不对，所以一直没用

颠倒的玉石（楼主）

@chenrenhui 我还是先用用list这个吧，实在不行了再lua

颠倒的玉石（楼主）

@chenrenhui 你的测试工具用的什么呢

第二天堂（作者）

github.com/wg/wrk

看上隔壁小花了啦

@chenrenhui 加锁要在查询的前一步加才起效果

讨论数量: 33

Rache1

660 声望

这一步不原子

file

2年前评论

MArtian

if(Redis::decr($name) < 0){
    $errorCode = 10059;
    Redis::del($name);
    return false;
}

这样就行了

颠倒的玉石（楼主）

那这里不做判断的话，该怎么写呢

Rache1 （作者）

@颠倒的玉石可以用 redis 的事务；也可以用 redis 的 lua 脚本；也可以在 decr 后判断，不必先 get ，直接判断 decr 后的。decr 后再落库

颠倒的玉石（楼主）

@Rache1 我试试第三个

MArtian

@Rache1 不用那么麻烦，先 get 然后 decr 肯定不原子，直接 decr 就行了，根本用不到 lua

Rache1 （作者）

@MArtian 这就是我说的第三个解法了ヾ(•ω•`)o

MArtian

@Rache1 啊，哈哈 (◍•ᴗ•◍)

orange1994

@Rache1 直接decr

kis龍

Laravel 9.x 译者 498 声望

理论：

a 访问请求是 1，同时 b 请求也是 1

a 秒杀， b 也秒杀

最终会变成 -1 或 0，这个取决于怎么减，DECR 是 -1， set 设置会是 0

解决方法：

使用 decr 或 incr 去设置秒杀数据

INCR keyName , 不存在设置为 1，存在会 +1，并返回数据，如图

file

判断如果大于销量，秒杀结束，否则继续，有没有超限的问题你测试下（应该不会触发）

2年前评论

第二天堂

48 声望

目测是这里出了问题

file

使用 redis string 类型，存库存的数量

Laravel

使用 redis list 类型

Laravel

redis list 类型初始化库存

Laravel

大概是这样，这是我之前测试的代码，仅供参考

2年前评论

第二天堂（作者）

颠倒的玉石（楼主）

第二种方式就是比如5个库存，push5个数据。每次先判断list长度，然后看是否需要pop么

颠倒的玉石（楼主）

@chenrenhui 我理解你的意思了，这个lua说实话我不会，push可以试试。用事务可不可以呢

第二天堂（作者）

@颠倒的玉石事务之前测试了，使用的是watch, 但是高并发的时候好像会出现超卖，可能是我用法不对，所以一直没用

颠倒的玉石（楼主）

@chenrenhui 我还是先用用list这个吧，实在不行了再lua

颠倒的玉石（楼主）

@chenrenhui 你的测试工具用的什么呢

第二天堂（作者）

github.com/wg/wrk

看上隔壁小花了啦

@chenrenhui 加锁要在查询的前一步加才起效果

zjason

27 声望

你的 redis 操作不是原子性的，使用 lua 脚本，希望对你有帮助，自行调整

$str = <<<Lua
    local key   = KEYS[1];
    local redis_stock = redis.call('get', key);
    if (tonumber(redis_stock) > 0)
        then
            redis.call('decr', key);
            return true;
        else
            return false;
        end
Lua;

2年前评论