大家看看这段抽奖代码有没有安全隐患

[TypeErrorNone]

自增的方式是 redis 的 incr 方法。 服务是集群，redis 是单机。

[qq316107934]

第一次获取当前兑奖次数是不是多余了

[bk201]

这是什么语言的语法

[712e1959]

@bk201 #2 go

[TypeErrorNone]

@qq316107934 是多余，只为求个心安

[tyrantZhao]

没啥问题啊

[mengzhuo]

大哥，这压根不是抽奖吧，只是检查库存而已？

[scukmh]

不要用 this，人家都提示了、

[mamahaha]

这个:=只有 go 语言在用吧？

[KaynW]

@mamahaha python 也用

[KaynW]

@scukmh 同, 黄线都飙出来了

[picone]

你这个是实现原子+1 吧，为啥不用 atomic

[zjsxwc]

@picone 楼主说了多机集群

[andyhuzhill]

@mamahaha #8 Pascal VHDL 都有用

[Vegetable]

这个 this 看的我浑身难受...
整体上这个思路我觉得不太对
你这是一个检查方法?canDosth，但是你在内部通过 incr 操作了 redis，意思库存在这就给扣掉了，所以你的检查操作不是幂等的，这样的设计有些反直觉了。

基于 incr 设计的方法，#1 楼说的对，你第一步检查是多余的，假如实现一个幂等的检查方法，这个方法在不做事务处理时必然是不安全的，而做了事务处理的话，检查方法又失去了意义。所以去掉第一步检查，这方法改为扣库存，通过扣库存是否成功来判断是否执行下一步就行了。

so，方法名改一下，去掉第一个检查库存，如果有必要为后边的判断不通过的情况加上回滚

[trustbutverify]

@mamahaha py3.8 开始用