mybatis框架中的queryWrapper的or查询，联想到MySQL中and 和or的关系

统计的原生SQL应该是这样的：

SELECT
	COUNT( 1 ) 
FROM
	table_name 
WHERE
	( 
		id = '679135XXXXXXX1212' 
		AND ( STATUS = 1 OR STATUS = 2 ) 
		AND isRemoved = 0
	)

服务端使用mybatis框架的QueryWrapper【query包装器】来统计子设备，之前的查询语法是这样的：

public int method(String id, int isRemoved) {
    
    
        QueryWrapper queryWrapper = new QueryWrapper<>();
        queryWrapper.eq("id", id);
        queryWrapper.eq("status", DeviceConstant.DeviceStatus.WORK)
                .or().eq("status", DeviceConstant.DeviceStatus.FAULT);
        return mapper.selectCount(queryWrapper);
 }

但是生成出来的SQL语句是

SELECT
	count(1) 
FROM
	table_name 
WHERE
	( 
		id = '679135XXXXXXX1212' AND STATUS = 1  
		OR STATUS = 2 AND isRemoved = 0
	);

正如我们所知道的那样，MySQL中的 and的优先级高于or，因而，上面的SQL语句可以转化为

SELECT
	count(1) 
FROM
	table_name 
WHERE
	( 
		(id = 679135XXXXXXX1212 AND STATUS = 1 )  
		OR (STATUS = 1 AND isRemoved = 0) 
	);

于是，就违背了我们最初的SQL语句，自然查询出来的有可能不对，导致统计的结果不对，因为这样的SQL语句表明：查询出设备Id等于‘679135XXXXXXX1212’且状态等于1，或者所有未移除的设备的状态等于2。

因此，mybatis的queryMapper的查询语法需要修改，如下：

public int method(String id, int isRemoved) {
    
    
        QueryWrapper queryWrapper = new QueryWrapper<>();
        queryWrapper.eq("id", id);
       queryWrapper.and(wrapper -> wrapper.eq("status", DeviceConstant.DeviceStatus.WORK)
                .or().eq("status", DeviceConstant.DeviceStatus.FAULT)
        );
        return mapper.selectCount(queryWrapper);
 }

这种方式生成的SQL语句如下

SELECT
	COUNT( 1 ) 
FROM
	table_name 
WHERE
	( 
		id = ? 
	 	AND ( ( STATUS = ? OR STATUS = ? ) ) 
		AND isRemoved = ? 
	)

这样的SQL才是我们想要的查询语句。

这是一个很细节的语法，一旦不注意，就陷入了mybatis的陷阱中。mybatis本身没有问题，本身是个优秀的框架。