Hibernate JPA迁移JDBC时OR条件查询无返回结果问题

你之前用findById()的实现思路完全不符合这个查询场景的逻辑，查不到数据是必然的，核心问题有3个：

• findById() 是基于主键的精确等值查询，生成的SQL会把复合主键包含的所有字段拼成 字段名 = ? 的AND匹配条件，根本无法实现你需要的「字段匹配入参 OR 字段值为'ALL'」的OR逻辑
• 你构造主键对象时，把空入参手动替换成'ALL'，本质是要查「字段值完全等于你传入的参数（包括你手动设置的'ALL'）」的记录，和原SQL要匹配「表中字段存了'ALL'的兜底规则」的逻辑完全不一致
• 你的复合主键类DeviceAccessCredKey包含password字段，你构造主键时没有给这个字段赋值，查询时框架会默认按password = null做匹配，自然不可能查到有效数据

推荐两种无额外依赖、和Spring Data JPA生态完全适配的实现方式，按需选择即可。

方案1：自定义JPQL查询（逻辑直观，易维护） #

直接在Repository接口中写和原SQL逻辑对齐的JPQL，是这个场景下最推荐的实现方式，参数传递和原JDBC逻辑完全一致，后续调整条件也很方便：

1. 改造Repository接口，添加自定义查询方法：

public interface DeviceAccessCredentialsRepository extends PagingAndSortingRepository<DeviceAccessCreditals, DeviceAccessCredKey> {
    @Query("SELECT d FROM DeviceAccessCreditals d WHERE " +
            "(d.deviceAccessCredKey.deviceName = :deviceName OR d.deviceAccessCredKey.deviceName = 'ALL') " +
            "AND (d.deviceAccessCredKey.service = :service OR d.deviceAccessCredKey.service = 'ALL') " +
            "AND (d.deviceAccessCredKey.vendor = :vendor OR d.deviceAccessCredKey.vendor = 'ALL') " +
            "AND d.deviceAccessCredKey.credentialUse = :credentialUse " +
            "AND d.deviceAccessCredKey.passwordType = :passwordType")
    List<DeviceAccessCreditals> queryMatchedCredentials(
            @Param("deviceName") String deviceName,
            @Param("service") String service,
            @Param("vendor") String vendor,
            @Param("credentialUse") String credentialUse,
            @Param("passwordType") String passwordType
    );
}

2. 业务代码中直接调用，传参逻辑和原JDBC完全保持一致即可：

// 注意保留原逻辑中vendor转大写的处理，和历史行为对齐
List<DeviceAccessCreditals> result = deviceAccessCredRepository.queryMatchedCredentials(
        deviceDetails.getName(),
        deviceDetails.getService(),
        deviceDetails.getVendor().toUpperCase(),
        use,
        passwordType
);

方案2：Repository派生查询（零SQL，适合简单固定场景） #

如果不想写自定义JPQL，可以直接按照Spring Data JPA的方法命名规则定义查询方法，框架会自动生成对应SQL：

1. 在Repository接口中添加派生查询方法：

public interface DeviceAccessCredentialsRepository extends PagingAndSortingRepository<DeviceAccessCreditals, DeviceAccessCredKey> {
    List<DeviceAccessCreditals> findByDeviceAccessCredKey_DeviceNameOrDeviceAccessCredKey_DeviceNameAndDeviceAccessCredKey_ServiceOrDeviceAccessCredKey_ServiceAndDeviceAccessCredKey_VendorOrDeviceAccessCredKey_VendorAndDeviceAccessCredKey_CredentialUseAndDeviceAccessCredKey_PasswordType(
        String deviceName, String deviceNameAllMatch,
        String service, String serviceAllMatch,
        String vendor, String vendorAllMatch,
        String credentialUse,
        String passwordType
    );
}

2. 调用时给OR匹配的固定值传字符串"ALL"即可：

List<DeviceAccessCreditals> result = deviceAccessCredRepository
    .findByDeviceAccessCredKey_DeviceNameOrDeviceAccessCredKey_DeviceNameAndDeviceAccessCredKey_ServiceOrDeviceAccessCredKey_ServiceAndDeviceAccessCredKey_VendorOrDeviceAccessCredKey_VendorAndDeviceAccessCredKey_CredentialUseAndDeviceAccessCredKey_PasswordType(
        deviceDetails.getName(), "ALL",
        deviceDetails.getService(), "ALL",
        deviceDetails.getVendor().toUpperCase(), "ALL",
        use,
        passwordType
    );

注意：因为查询字段都在嵌入式主键deviceAccessCredKey下，方法名中必须加deviceAccessCredKey_前缀做字段路径关联，避免和实体类其他同名字段冲突。

• 不要用findById实现带OR、动态条件的查询：主键查询只支持所有主键字段的精确等值匹配，只适合按主键拿单条记录的场景，灵活条件查询请用自定义查询、派生查询或Specification
• 你当前的复合主键包含password这类业务属性，长期维护很容易出问题，建议后续调整表结构，给表加独立的无业务含义主键，把原来的复合字段改成普通业务字段，降低查询、更新逻辑的耦合度
• 如果后续需要加更多动态条件（比如入参为空时跳过对应字段的过滤），可以直接用JPA Criteria API或Specification做动态条件拼接，不用硬写固定SQL

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