关于service层加synchronized关键字的坑

@Override
@Transactional(rollbackFor = Exception.class)
public synchronized SyTaskChildDO initTaskChild(SyTaskChildQueryParam param) {
    try {
        List<SyTaskChildDO> taskCidList = syTaskChildDAO.listByParam(param);
        if(taskCidList.isEmpty()){
            return null;
        }
        SyTaskChildDO childDO = taskCidList.get(0);
        childDO.setState(TaskStateEnum.IN_EXECUTE.getCode());
        //修改任务状态为执行中
        Long childNum = syTaskChildDAO.update(childDO);
        if(childNum<1){
            return null;
        }
        return childDO;
    } catch (Exception e) {
        LOGGER.error("SyTaskChildManageServiceImpl initTaskChild Exception",e);
        throw e;
    }
}

在同步方法上面添加了事务，这样会导致同步方法有时候会失效。

原因在于事务的范围比synchronized范围大，当锁释放了才会提交事务，在这期间就有可能别的线程读取到任务去执行

解决方案：

1、新加一个方法，加上同步锁，然后再调用你那个加了事物注解的方法

2、取消同步synchronized，使用分布式锁方案解决这个问题

原文：https://blog.csdn.net/dzy_001/article/details/90049428