Patch方法无法更新文件中修改字段的问题排查

1. 核心错误：变量名混淆 #

你的updateFields私有方法里存在变量名拼写错误，导致根本没读取到传入的更新对象的字段值，这是字段不更新的直接原因：

private void updateFields(MyFile existingFile, MyFile updatedMyFile) {
    // 错误：误用了未定义的updatedFile变量，应该用参数updatedMyFile
    Field[] fields = updatedFile.getClass().getDeclaredFields();
    for (Field field : fields) {
        field.setAccessible(true);
        try {
            // 错误：同样误用了updatedFile，应该读取updatedMyFile的字段值
            Object updatedValue = field.get(updatedFile);
            if (updatedValue != null) {
                field.set(existingFile, updatedValue);
            }
        } catch (IllegalAccessException e) {
            e.printStackTrace();
        }
    }
}

修正后的代码：

private void updateFields(MyFile existingFile, MyFile updatedMyFile) {
    Field[] fields = updatedMyFile.getClass().getDeclaredFields();
    for (Field field : fields) {
        field.setAccessible(true);
        try {
            Object updatedValue = field.get(updatedMyFile);
            if (updatedValue != null) {
                field.set(existingFile, updatedValue);
            }
        } catch (IllegalAccessException e) {
            e.printStackTrace();
        }
    }
}

2. 额外优化建议 #

（1）修复创建时间覆盖问题 #

你的通用save方法每次都会重置creationDate，如果这个字段是记录实体创建时间的，更新操作时不应该修改它，建议区分创建和更新场景：

@Transactional
public ENTITY save(ENTITY entity) {
    entity.setEditedBy(getCurrentUserId());
    // 仅当是新实体（id为空）时设置创建时间
    if (entity.getId() == null) {
        entity.setCreationDate(new Date());
    }
    verifySave(entity);
    entity = persist(entity);
    afterSave(entity);
    return entity;
}

（2）简化JPA持久化操作 #

existingFile是通过getFile(id)从数据库加载的，处于JPA持久化上下文管理中，在@Transactional方法内修改字段后，事务提交时会自动同步到数据库，不需要手动调用save：

@Transactional
public MyFile updateFields(Long id, MyFile updatedMyFile) {
    MyFile existingFile = getFile(id);
    if (existingFile == null) {
        return null;
    }
    updateFields(existingFile, updatedMyFile);
    // 直接返回existingFile即可，无需调用super.save
    return existingFile;
}

（3）替换反射的更优雅方案 #

反射方式容易出现变量名、权限、继承字段等问题，推荐用ModelMapper或MapStruct这类工具实现非空字段拷贝，减少手动代码出错概率：

// ModelMapper示例
private void updateFields(MyFile existingFile, MyFile updatedMyFile) {
    ModelMapper modelMapper = new ModelMapper();
    modelMapper.getConfiguration()
        .setSkipNullEnabled(true)
        .setMatchingStrategy(MatchingStrategies.STRICT);
    modelMapper.map(updatedMyFile, existingFile);
}

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