yii CDbCriteria类的compare方法的使用

喜欢ヅ旅行 2022-06-08 11:18 211阅读 0赞

一.注意慎用yii CDbCriteria类的compare方法，这里面可能会有坑。

二.场景：线上的一个报表查询，发现当选择其中的一个筛选时，查询出来的范围不对，后跟代码后发现这里有个坑。

三.由来：一般看compare方法的解释是: “这个方法比较特殊，他会根据你的参数自动处理成addCondition或者addInCondition”, 所以想着使用 $criteria->compare(‘id’,$id);的这种写法是比较好的，即能匹配整型或字符串类型的参数，也能匹配数组类型的参数，可谓一举两得，但是当$id为空数组时，该查询条件直接返回，不会作为连接的查询。

四.举例如下:

$criteria =newCDbCriteria;

$criteria->compare(‘name’,$name);

$criteria->compare(‘id’,$id);

$criteria->compare(‘date’,$date);

正常情况下的，转化为对应的sql会如下: select … from … where name = ‘$name’ and id = ‘$id’ and date = ‘$date’…— 或 select … from … where name = ‘$name’ and id in (id1,id2…) and date = ‘$date’…

-- 但是当$id = array()时，打印query_log时，却会发现输出的sql是select … from … where name = ‘$name’ and date = ‘$date’… 少了一个id的筛选查询，所以数据范围就不对了

那么，why?

五.看源码: 我们进入到CDbCriteria类，找到compare()方法

public function compare($column, $value, $partialMatch=false, $operator=’AND’, $escape=true){

  1.   if(is\_array($value))\{
  3.         if($value===array())return $this;

return $this->addInCondition($column,$value,$operator);

}else{

$value=”$value”;

if(preg_match(‘/^(?:/s*(<>|<=|>=|<|>|=))?(.*)$/‘,$value,$matches)){

$value=$matches[2];

$op=$matches[1];

}else{

$op=’’;

if($value===’’)return $this;

if($partialMatch){

if($op===’’)return $this->addSearchCondition($column,$value,$escape,$operator);

if($op===’<>’)return $this->addSearchCondition($column,$value,$escape,$operator,’NOT LIKE’);

}elseif($op===’’){

$op=’=’;

$this->addCondition($column.$op.self::PARAM_PREFIX.self::$paramCount,$operator);

$this->params[self::PARAM_PREFIX.self::$paramCount++]=$value;

return $this;

}

看compare方法的3,4,5行代码，当是数组，且是空数组时，直接return了。。。真相大白..

六.总结：

知道原因后，以后在使用compare()方法作为筛选条件时，一定要注意参数的类型，如果是空数组的时候，就要加上另外的逻辑判断；

或者当参数为空数组时直接用addInCondition()来替代.

发表评论

表情：
评论列表 （有 0 条评论，211人围观）

还没有评论，来说两句吧...

相关阅读