小心使用ThinkPHP中的addAll方法

作者在项目里写一个简单的创建问题接口时，一不小心就踩到了ThinkPHP的两个坑。将它称之为坑，实则是对文档的不熟悉。接口代码如下：

public function createProblems(){    $problems = I('json.');    if (empty($problems)) {        $this->error($problems, 'json格式不符合规范');    }    foreach ($problems as $problem) {        $data = D('Problem')->create($problem, self::OP_INSERT);        if (!$data) {            $this->error($problem, D('Problem')->getError());        }        $temp[] = $data;    }    $ret = D('Problem', 'Service')->addProblems($temp);    if ($ret === false) {        $this->error(null, '导入失败');    }    $this->success(null);}

接口完成的功能是批量创建问题，参数为json数组，addProblem()方法中就是一个addAll操作。

第一个坑-自动填充

使用自动填充可能会覆盖表单提交项目。其目的是为了防止表单非法提交字段。使用Model类的create方法创建数据对象的时候会自动进行表单数据处理。

官方文档明确说了自动填充会覆盖表单，所以即使你post过来的参数中给出了具体数值，使用create方法之后也可能会被覆盖。千万注意！！！

第二个坑-addAll方法

addAll方法中不能出现null值，否则其他数据会自动向前移动，导致添加失败。举例：

[    {        "appId": 1,        "serviceId": 2,        "createUser":null,        "status": 1,        "priority": 2    }]

其中，createUser字段为null，插入时的sql语句会变成insert into va_problem （appId, serviceId, createUser, status, priority）values (1, 2, 1, 2)。null值没了，导致插入失败，这应该是ThinkPHP3.2.3的一个bug。这篇博客有相关的讨论。

引入坑的过程

problemModel里面有对createUser做自动填充。

通过api创建问题时，首先自动填充会覆盖，所以表单中的createUser值失效，这是第一个坑。然后，由于自动填充createUser调用的是get_username()函数，通过api调用时，session(username)取不到值，所以create之后字段变成"createUser":null，引入了第二个坑。