如何实现不同包实体间关联并保持封装，将类设为包私有？

为避免分层包结构导致所有类被迫设为public，我采用了垂直包结构（截图示意）：

但目前遇到权限问题：

• User和DiagnosticResult因JPA的OneToMany/ManyToOne关联，必须设为public
• UserRepository因被DiagnosticResultService跨包调用，也得设为public

相关核心代码如下：

public class User extends BaseEntityAudit {
    // 基础字段
    @OneToMany(targetEntity = DiagnosticResult.class, mappedBy = "user", fetch = FetchType.LAZY)
    private List<DiagnosticResult> resultsList = new ArrayList<>();
}

public class DiagnosticResult extends BaseEntityAudit {
    // 基础字段
    @ManyToOne(fetch = FetchType.EAGER)
    @JoinColumn(name = "user_id")
    private User user;
}

class DiagnosticResultService {
    private final DiagnosticResultRepository resultRepository;
    private final UserRepository userRepository;

    DiagnosticResult saveDiagnosticResult(DiagnosticResult result, String login) {
        var retrievedUser = userRepository.findUserByLogin(login)
                .orElseThrow(() -> new EntityNotFoundException("User with login: " + login + " does not exist in database."));
        result.setUser(retrievedUser);
        retrievedUser.getResultsList().add(result);
        userRepository.save(retrievedUser);
        return resultRepository.save(result);
    }
}

我知道最佳实践是尽可能用包私有，但合并用户和诊断类到同一包会降低可读性，而且为了未来微服务拆分需要保持包分离。想请教：

1. 不合并包的前提下，能不能把User、UserRepository、DiagnosticResult设为包私有？
2. 我的包结构设计是不是有问题，应该把它们放在一起？
3. 计划添加外观类，听说包只需公开外观类和DTO，这是不是解决问题的关键？

1. 先明确：JPA实体能不能设为包私有？ #

可以，但有核心限制。JPA规范允许实体类是包私有，ORM框架（比如Hibernate）只要能访问实体字段/构造方法就行（字段访问模式下完全没问题）。但跨包的关联引用是硬矛盾：如果User是包私有，DiagnosticResult所在的包根本看不到它，直接设包私有走不通，得靠封装隔离解决。

2. 外观类+DTO是解决问题的核心 #

这正是你需要的方案，核心逻辑是把业务域内部的实体、Repository、Service全藏在包内，对外只暴露Facade（外观类）和DTO，具体操作：

• 每个垂直包只对外公开一个public的Facade类，比如UserFacade、DiagnosticResultFacade，作为跨包交互的唯一入口
• 实体（User/DiagnosticResult）、Repository、Service全部设为包私有，仅在各自业务包内可见
• 跨包交互完全用DTO：比如诊断包要关联用户时，不直接调用UserRepository，而是通过UserFacade的公开方法获取必要信息（比如用户ID），再在自己包内完成实体关联

调整后代码示例：

// user包下：public的UserFacade
public class UserFacade {
    private final UserService userService;

    // 对外只暴露必要能力，比如根据login拿用户ID
    public Long getUserIdByLogin(String login) {
        return userService.findUserByLogin(login)
                .orElseThrow(() -> new EntityNotFoundException("User not found"))
                .getId();
    }
}

// user包下：包私有User实体
class User extends BaseEntityAudit {
    // 字段和关联关系不变
}

// diagnostic包下：包私有DiagnosticResultService
class DiagnosticResultService {
    private final DiagnosticResultRepository resultRepository;
    private final UserRepository userRepository;
    private final UserFacade userFacade;

    DiagnosticResult saveDiagnosticResult(DiagnosticResult result, String login) {
        Long userId = userFacade.getUserIdByLogin(login);
        // 自己包内用UserRepository查询私有实体
        var retrievedUser = userRepository.findById(userId).orElseThrow(...);
        result.setUser(retrievedUser);
        retrievedUser.getResultsList().add(result);
        userRepository.save(retrievedUser);
        return resultRepository.save(result);
    }
}

// diagnostic包下：public的DiagnosticResultFacade
public class DiagnosticResultFacade {
    private final DiagnosticResultService resultService;

    public DiagnosticResultDTO saveDiagnosticResult(DiagnosticResultDTO resultDTO, String login) {
        // DTO转内部实体，调用Service处理
        DiagnosticResult result = convertToEntity(resultDTO);
        DiagnosticResult savedResult = resultService.saveDiagnosticResult(result, login);
        // 处理结果转DTO返回
        return convertToDTO(savedResult);
    }
}

这样所有核心内部类都能保持包私有，跨包调用完全被Facade和DTO隔离，既满足封装性，又保留了业务包的分离，还为未来微服务拆分做了铺垫——每个Facade刚好对应微服务的API边界。

3. 你的包结构没问题，不用合并 #

按业务域（用户、诊断）拆分的垂直包结构，比传统分层包（controller/service/dao）更适合微服务演进，完全没必要为了封装性合并包。只要注意包之间保持单向依赖（比如诊断包依赖用户包的Facade，不要反过来），就能维持结构的清晰性和可维护性。

额外补充：解决Repository跨包调用的根源 #

之前你在DiagnosticResultService直接注入UserRepository，是导致UserRepository必须公开的核心原因。现在通过Facade间接获取用户信息，UserRepository可以完全设为包私有，只在用户包内被UserService调用，彻底切断了跨包的Repository依赖。

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