乌龙！mybatis-plus的@TableId注解不生效，原来竟是因为它！

【先来个小测试】

大家觉得下面的sql返回什么？

select * from table1 where null=1

 

答案：无返回。因为null=1是个false的表达式。这就像我们写where 1=2一样。

 

【↓↓正文开始↓↓】

本次迭代的需求开发完成，将开发分支merge到test分支，部署测试环境提测后，QA提了一个bug，附下面log截图。

通过logtrace排查程序，定位到如下代码。代码很简单，调用mybatis-plus的getById函数按主键查询数据。PayMerchantBankCardFlow这个实体类里在主属性上有标记@TableId，况且这个实体类在本次开发迭代中并未改动。那么，今天，mybatis-plus底层拼接sql时，怎么没有把主键字段拼出来呢？

PayMerchantBankCardFlow bankCardFlow = payMerchantBankCardFlowManager.getById(cardBindDTO.getFlowNo());

 

@TableName(value = "pay_merchant_bankcard_flow",autoResultMap = true)
public class PayMerchantBankCardFlow implements Serializable {

    private static final long serialVersionUID = 5112092241305418545L;

    /**请求流水号*/
    @JsonSerialize(using= ToStringSerializer.class)
    @TableId(type = IdType.ID_WORKER)
    private Long flowNo;

 

这确实令人费解！当时我在参加一个代码评审会，本着fixbug优先的原则，临时在PayMerchantBankCardFlowManager里重写了getById。然后发布让QA继续后续的功能测试。

public class PayMerchantBankCardFlowManager{

    @Override
    public PayMerchantBankCardFlow getById(Serializable id) {
        return getOne(Wrappers.query(new PayMerchantBankCardFlow().setFlowNo((Long) id)));
    }

 

bugfix就算完事了吗？不，作为靠谱的程序员，我们有必要刨根寻底查明原因。

一个小伙在本地通过junit测试，发现getById是没问题的。当然，凭借我们历往对mybatis-plus的掌握程度，这个@TableId必然不会有问题的。

但是结论呢？为什么本地没bug而测试环境有bug呢？

这时就考验技术人员的综合能力了。

还是组内的jason同学协助给破解了。

原来，在test分支里，PayMerchantBankCardFlow#flowNo的@TableId注解被别的开发分支给merge没了。这下就真相大白了。

最终修正PayMerchantBankCardFlow实体类，revert临时改动的代码，这个乌龙事件得以消停。

 

一个技术点：在springboot容器启动时，mybatis-plus会检查未设置@TableId的实体类。发现后会有WARN日志。2023-11-28 15:11:51.284 [TID:N/A] [] [main] WARN  c.b.mybatisplus.core.metadata.TableInfoHelper:? - Warn: Could not find @TableId in Class: com.emax.channel.provider.modules.paymerchantbankcardflow.entity.PayMerchantBankCardFlow.