如何通过Spring Data JPA实现实体新增与更新通用方法？

你当前的核心逻辑是符合Spring Data JPA的设计思路的，额外说明：你提到“没有使用Optional”其实是误解，findById方法固定返回Optional类型，你用到的orElse就是Optional提供的默认值取值API，这部分用法是正确的。

你现有代码有两个需要先修正的问题：

• 语法错误：findById(request.getId()); 末尾多了分号，会打断链式调用导致编译失败，删除该多余分号即可。
• 潜在异常：JpaRepository.findById()不允许入参为null，如果请求中没有携带ID会直接抛出IllegalArgumentException，需要提前处理空ID场景。

完善后的基础实现 #

@Transactional
public CommandDTO createOrUpdate(EmployeeRequest request) {
    Employee employee;
    // 空ID直接走新建逻辑，避免无效查询和空参数异常
    if (request.getId() == null) {
        employee = new Employee();
    } else {
        employee = employeeRepository
                .findById(request.getId())
                .orElse(new Employee());
    }
    
    employee.setName(request.getName());
    // 其他属性赋值逻辑
    
    // 仅瞬态的新建实体需要显式调用save，持久化上下文管理的托管态实体属性变更会在事务提交时自动落库
    if (employee.getId() == null) {
        employee = employeeRepository.save(employee);
    }
    return CommandDTO.builder().uuid(employee.getUuid()).build();
}

最佳实践优化方向 #

1. 简化属性赋值逻辑 #

如果实体字段较多，手动写每个字段的set方法冗余度高，还容易漏字段，可使用MapStruct等对象映射工具，配置忽略null值的映射规则，避免前端未传字段被错误覆盖为null，示例配置：

@Mapper(componentModel = "spring", nullValuePropertyMappingStrategy = NullValuePropertyMappingStrategy.IGNORE)
public interface EmployeeMapper {
    void updateEmployeeFromRequest(EmployeeRequest request, @MappingTarget Employee employee);
}

注入Mapper后直接调用employeeMapper.updateEmployeeFromRequest(request, employee)即可完成所有属性的赋值，无需手动处理每个字段。

2. 高并发场景添加乐观锁 #

如果存在并行更新同一条记录的场景，可在Employee实体中添加@Version字段实现乐观锁，避免更新覆盖丢失问题：

@Entity
public class Employee {
    // 原有字段省略
    @Version
    private Long version;
}

Spring Data JPA会自动处理版本校验，更新时如果版本不匹配会抛出ObjectOptimisticLockingFailureException，可按需捕获处理冲突。

3. 通用逻辑抽象复用 #

如果多个实体都需要用到createOrUpdate能力，可以封装抽象服务层实现通用逻辑，避免每个实体重复编写同类代码：

public abstract class BaseUpsertService<ID, E, R> {
    protected abstract JpaRepository<E, ID> getRepository();
    protected abstract E newEntityInstance();
    protected abstract void fillEntityFromRequest(R request, E entity);
    protected abstract ID getEntityId(E entity);
    protected abstract String getEntityUuid(E entity);

    @Transactional
    public CommandDTO createOrUpdate(ID id, R request) {
        E entity;
        if (id == null) {
            entity = newEntityInstance();
        } else {
            entity = getRepository().findById(id).orElseGet(this::newEntityInstance);
        }
        fillEntityFromRequest(request, entity);
        if (getEntityId(entity) == null) {
            entity = getRepository().save(entity);
        }
        return CommandDTO.builder().uuid(getEntityUuid(entity)).build();
    }
}

各个实体对应的服务层继承该抽象类，实现对应抽象方法即可直接复用upsert逻辑。

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