编码过程中,一点点儿需要注意的地方

记得之前写游戏玩家类的时候,对于玩家数据的修改,我们总是提供这样的接口。比如是对玩家金钱的修改。

public Player
{
  private int money = 0;
  public void addMoney(int money)
  {
      this.money = this.money + money;
  }
  public void deductMoney(int money)
  {
      this.money = this.money - money;
  }
  public int getMoney()
  {
     return this.money;
  }
}

而避免提供这样的接口

public Player
{
  private int money = 0;
  public void setMoney(int money)
  {
     this.money = money;
  }
  public int getMoney()
  {
     return this.money;
  }
}

后者,我们每次操作玩家金钱的时候,总是需要先getMoney,然后再添加或者删除,最后再调用setMoney方法。这样方式,有两个比较大的缺点。1:操作繁琐。 2:如果是多线程,将无法控制,在getMonet 和 setMoney,之间如果其他线程操作玩家Money,那么将要出现数据不一致的情况。 而前者避免了这两种情况。但是没有想到这个典型的错误,在应用代码中也是经常出现。于是就发现了下面的问题。

最近在工程中发现了N多这样的代码!

Member member = memberMapper.selectMemberById(id);
int money = xxx;
member.setMoney(member.getMoney() + money);
memberMapper.updateMemberSelective(member);

这个问题和上面的提供set get的方式一样。不过数据放到了数据库中而已。

问题除了那两个问题,还多了一个。
第一:操作繁琐,每次都是先从数据库select,然后再去计算新的值,最后再去数据库update。
第二:数据库不一致,如果在select和update之间,其他线程操作这个玩家的money,那么肯定会出现不一直的情况。
第三:两次数据库的操作,慢,卡!

所以正确的写法应该是:


int memberId = xxxx;
int money = xxxx;
memberMapper.addMemberMoney(memberId,money);

addMemberMoney实现:

update set membermoney = membermoney + money where id = memberid; 

这里虽说多提供了sql语句,但是于逻辑,于数据一致性方面,都有了很大的提升。

评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值