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 人点赞

从零开发一个电商项目，功能包括电商后台、商品 & SKU 管理、购物车、订单管理、支付宝支付、微信支付、订单退款流程、优惠券等

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

推荐文章：

更多推荐...

博客

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

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

博客

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

博客

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

博客

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

分享创造

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

第二天堂

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

663 声望

这一步不原子

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 译者 500 声望

理论：

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年前评论