Jony's Blog

最近几个单都是用 PHPCMS 做二次开发，包括现在正在进行的这个大单，有多大？给客户添数据后，光是栏目数就已经设置到3000+并且上升势头不减……

以前没遇到过一个网站有那么多栏目的情况，估计 PHPCMS 的开发人员也没遇到过，因为我发现栏目越多，添加和修改栏目的速度越慢，栏目数过1000的时候，基本修改一次栏目就得花上1分钟时间，现在 3000+ 栏目已经要4，5分钟了……查阅了下 PHPCMS 的代码发现了问题所在。

PHPCMS 的栏目相关处理方法都是在 include\admin\category.class.php 里，这个类文件中有一个被频繁使用的方法 repair，问题就出在这里。无论是添加、删除还是其他的一些栏目操作，大多都会执行一次这个方法（有个地方连续执行两次，不解，应该是开发人员的失误）。而这个方法的主要工作就是将整个站的所有栏目遍历一次遍，对每个栏目都更新一下 arrparentid, parentdir, arrchildid 三个字段的内容。妈妈呀，那3000多个栏目岂不是要执行3000多次 update？怪不得……

我想开发人员之所以这么设计，应该是考虑到一个栏目的路径以及父栏目的变更对其父栏目和子栏目的影响，就计划要整体更新一遍防止出错。不过这种全盘更新的方式有失斟酌呀！一般只需要对影响到到的某几个栏目分支的所有栏目更新一遍就可以了。于是我将这个方法修改如下：

function repair($id=0,$pid=-1,$opid=-1)
{
    @set_time_limit(600);
    if($id==0){
        if(is_array($this->category))
        {
            foreach($this->category as $catid => $cat)
            {
                if($catid == 0) continue;
                $arrparentid = $this->get_arrparentid($catid);
                $parentdir = $this->get_parentdir($catid);
                $arrchildid = $this->get_arrchildid($catid);
                $child = is_numeric($arrchildid) ? 0 : 1;
                $this->db->query(“UPDATE `$this->table` SET arrparentid=’$arrparentid’,parentdir=’$parentdir’,arrchildid=’$arrchildid’,child=’$child’ WHERE catid=$catid”);
                if($cat[‘module’]==’phpcms’) $this->url($catid);
            }
        }
        $this->cache();
        return TRUE;
    }

    if($pid<1) $pid=$id;     if($opid<1) $opid=$id;     $arrpid = explode(",",$this->get_arrparentid($pid).”,”.$pid);
    $arropid = explode(“,”,$this->get_arrparentid($opid).”,”.$opid);
    $arrchildid = explode(“,”,$this->get_arrchildid($id));
    $allid = array_merge($arrpid,$arropid);
    $allid = array_merge($allid,$arrchildid);
    $allid = array_flip($allid);

    foreach($allid as $catid=>$v)
    {
        if($catid == 0) continue;
        $arrparentid = $this->get_arrparentid($catid);
        $parentdir = $this->get_parentdir($catid);
        $arrchildid = $this->get_arrchildid($catid);
        $child = is_numeric($arrchildid) ? 0 : 1;
        $this->db->query(“UPDATE `$this->table` SET arrparentid=’$arrparentid’,parentdir=’$parentdir’,arrchildid=’$arrchildid’,child=’$child’ WHERE catid=$catid”);
        if($this->category[$catid][‘module’]==’phpcms’) $this->url($catid);
    }

    $this->cache();
    return TRUE;
}

这个方法修改完毕后，在去这个类文件的 add 和 edit 方法中修改 $this->repair(); 这一行，主要是有三个参数依次为栏目ID，最后的父级ID，之前的父级ID。三个参数都不设置则和原来的效果完全一致，只填栏目ID则只更新该栏目以及其子栏目的前面所说的那三个字段内容，填写最后的父级ID，则会将所指定栏目最后所在栏目分支里的所有栏目更新一遍，填写之前的父级ID，则会将所指定栏目更改父级栏目之前所在的栏目分支里所有栏目更新一遍。注意，如果要更新相关栏目分支里的栏目，栏目ID这个参数必须同时设置。针对我这边的情况，我将 add 方法里的 repair 方法调用修改成 $this->repair($catid,$parentid);，edit 方法里则改成 $this->repair($catid,$parentid,$oldparentid);。

另外 add 方法里执行 repair 方法之前还得插入如下代码，以保证新添加的栏目可以更新到缓存里去。

$this->category[$catid][‘parentid’] = $category[‘parentid’];
$this->category[$catid][‘arrparentid’] = $category[‘arrparentid’];
$this->category[$catid][‘parentdir’] = $category[‘parentdir’];
$this->category[$catid][‘arrchildid’] = $catid;