016Java问题记录003循环里的二次判断与状态更新

1 问题重现

在使用循环结构时，如果使用了定时任务，或者代码会多次调用循环结构，可能会导致有些对象会被循环多次。

举例来说，如果有一个结账的代码，每五分钟会执行一次：

@Scheduled(cron = "0 0/5 * * * ?")
@Async
public void handle(){
    List<Customer> list = customerDao.listCustomerByStatus(0);// ["zhangsan", "lisi"]
    for (Customer customer : list) {
        customerDao.updateCustomerStatus(customer.getName(), 1);// 表示正在结账
        // 结账代码
        customerDao.updateCustomerStatus(customer.getName(), 2);// 表示结账完成
    }
}

假设结账代码的执行时间太长，超过了五分钟。

第一次触发定时任务时，查询到状态为0的顾客有ZhangSan和LiSi，进入循环代码块，将ZhangSan的状态改成1并执行结账代码。

由于结账代码执行时间超过了五分钟，所以当下一次定时任务触发的时候，ZhangSan还在执行结账的代码，这时LiSi的状态还是0。

第二次触发定时任务时，查询到状态为0的顾客有LiSi，进入循环代码块，将LiSi的状态改成1并执行结账代码。

当第一次触发的定时任务执行完ZhangSan的结账代码时，循环执行下一个顾客LiSi的结账代码，而此时第二次触发的定时任务已经在执行LiSi的结账代码了，这就会导致顾客LiSi的结账代码被重复执行了两次。

2 解决办法

第一种办法是先循环一遍查到的数据，将状态改为1，然后执行循环对每一个顾客结账。

第二种办法是重新判断状态，在每次循环执行结账代码时，都重新查询一下顾客的状态，不过这种方式需要每次都查询数据库。

3 问题说明

在使用循环代码的时候要注意是否有状态的改变，如果有的话就需要考虑是否存在重复处理的隐患。