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

作者：佚名 2015-06-01 15:11:37运维 数据库运维 数据库update时切忌这样干，样干这样干是悲剧什么?请随我来看看。

数据库update时切忌这样干，数据这样干是样干什么?请随我来看看。

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

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

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

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

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

 

[[135474]]

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

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语句中加入，而不是类字段中做处理。

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

博文出处：http://blog.csdn.net/qing_gee/article/details/46233301
 

责任编辑：Ophira 来源： CSDN博客 数据库update

(责任编辑：焦点)