您的位置:首页 > 数据库

数据库update时这样干,你就悲剧了

2015-06-04 08:46 274 查看
数据库update时切忌这样干,这样干是什么?请随我来看看。

早些时候,客户发来这样一段文字“XX的,XXXXXXXX有个10万没到账,帮加上去。”我马上动手查看数据库,寻找原因,随后回复了这样一句话“开玩笑吧”。虽然之前发生过类似这样的问题,但都是小额资金,就手动在数据库里加了,遇到这么大的资金还是头一次,于是我加紧翻看数据库记录的节奏。

我发现用户的入金记录的确已经是审核通过状态,而用户的可用资金却没有加上,这已经让人意外了。

接着我查看代码,并没有发现逻辑上的问题,这又让我感到迷茫。

再翻看日志,发现没有任何异常,好吧,感觉被设了局一样。

回头想想,这种情况并不多见,程序在多数情况下并没有这种情况,一切正常。这糟了,没有错误日志就是最大的错误。





静下心来,我想有可能是事务出现的问题,因为对于资金表,同一个用户的资金在同一个时间点是可能同时被更新的,我猜想,是入金记录先插入了出入金记录表,等到更新资金表的时候被锁住,事务超时后插入的记录没有回滚,于是我通过这样的方式来证明一下。

START TRANSACTION;
INSERT INTO record VALUES(1,123);
UPDATE money SET money = money + 10;
COMMIT;

START TRANSACTION;
UPDATE money SET money = money + 100;
COMMIT;

通过手动控制事务锁,发现不是我想的那样。然后我又试了试下面这种方式:

START TRANSACTION;
UPDATE money SET money = money + 10;
INSERT INTO record VALUES(1,123);
COMMIT;

START TRANSACTION;
UPDATE money SET money = money + 100;
COMMIT;

发现两者的效果是一样的,事务都会回滚。

这个时候,我的思路被禁锢了,我和同事讨论起来,看看他有什么想法,经过一番思想斗争后,果然同事还是给出了他的想法,我深感赞同。

MoneyUser moneyUser = moneyUserMapper.selectByPrimaryKey(members.getUid());
// 冻结资金-出金金额
moneyUser.setFrozenl(moneyUser.getFrozen().subtract(moneyTransfer.getount()));
// 可用资金+出金金额
moneyUser.setTotaaymoney(moneyUser.getTotalpayey().add(moneyTransfer.getount()));
this.moneyUserMapper.updateByPrimaryKey(moneyUser);

这样的代码在出现并发的时候,获取moneyUser对象的时候,其中的冻结资金和可用资金加入都是0,那么两个并发在update的时候,两个都执行了,但是起初的冻结资金和可用资金相同,就会导致有一笔资金没有加上。

调试的时候更容易发现这个问题。OH,my 噶,真是个大问题,还出在资金的问题时,我真庆幸这个问题发生的频率不够高。不过我到底是应该庆幸还是不庆幸呢,也许次数发生的多起来的时候,我就更容易来反思来找出问题的症结所在。

那么问题怎么解决呢?

UPDATE moneser
SET
frozapital = frozpital - #{amount,jdbcType=DECIMAL},
totaloney = totaloney + #{amount,jdbcType=DECIMAL}
WHERE uid = #{uid,jdbcType=INTEGER}

通过在sql语句中进行字段的更新操作,而不是在Java类中,因为mysql本身是会有这方面的处理方式的。

这个问题,之前领导有建议我在update数据的时候尽量在sql语句中加入,而不是类字段中做处理。

总结:这个问题,如果你不经意的话,很容易发生这个问题,我当初就没有这样的意识,真是可悲可叹!

博文出处:/article/1462967.html
内容来自用户分享和网络整理,不保证内容的准确性,如有侵权内容,可联系管理员处理 点击这里给我发消息
标签: