是否需为自定义方法添加@Transactional(readOnly=true)及相关疑问

给定代码示例 #

Repository 代码 #

public interface ApplicationRepository extends JpaRepository<Application, Integer> {

    public boolean existsByCode(String code);

    public Optional<Application> findByCode(String code);

}

Service 代码 #

@Service
@RequiredArgsConstructor
public class ApplicationService {

    private final ApplicationRepository appRepo;

    public Application findById(Integer id) throws RecordNotFoundException {
        return appRepo.findById(id)
                .orElseThrow(() -> new RecordNotFoundException("Application with id: " + id + " could not be found"));
    }

    public boolean existsByCode(String code) {
        return appRepo.existsByCode(code);
    }

    public Application findByCode(String code) throws RecordNotFoundException {
        return appRepo.findByCode(code).orElseThrow(
                () -> new RecordNotFoundException("Application with code: " + code + " could not be found"));
    }

}

问题解答 #

1. 默认Repository方法带@Transactional(readOnly=true)，自定义方法要不要加？加在Service还是Repository层？ #

Spring Data JPA的默认查询方法（比如findById、findAll）确实会自动带上@Transactional(readOnly=true)，但你自己定义的existsByCode、findByCode这类方法不会自动继承这个注解，所以需要手动添加。

关于添加位置：

• 优先加在Service层。业务逻辑的事务边界本来就该由Service层管控，这是分层架构的标准做法——Service层封装业务流程，统一控制事务范围；Repository层只负责数据访问，不该耦合事务逻辑。
• 如果加在Repository层，虽然能生效，但后续业务扩展时（比如一个Service方法调用多个Repository方法），没法统一控制事务，维护起来会很麻烦。

2. 第三个方法调用两个已标记@Transactional(readOnly=true)的方法，要不要也加这个注解？ #

分两种情况看：

• 如果这个方法只是单纯调用两个只读方法，没有任何写操作，建议添加@Transactional(readOnly=true)：
  1. 明确声明只读特性，让Spring做事务优化（比如关闭脏检查、减少事务日志），提升性能；
  2. 要是不加，默认的事务传播行为是REQUIRED——第三个方法会开启一个非只读事务，被调用的两个方法会加入这个事务，原本的只读优化就失效了。
• 如果这个方法包含写操作，那绝对不能加readOnly=true，得用默认事务配置或者按需调整事务属性。

内容的提问来源于stack exchange，提问作者Lorenzo Miscoli