yii2中yii\base\event中可改良之处 [ 技术分享 ]

yii2源码中，个人感觉有一处可以改良的地方，在yii\base\event中的事件定义中，解除事件绑定的函数off()定义源码如下：

public static function off($class, $name, $handler = null)
{
        $class = ltrim($class, '\\');
        if (empty(self::$_events[$name][$class])) {
            return false;
        }
        if ($handler === null) {
            unset(self::$_events[$name][$class]);
            return true;
        }

        $removed = false;
        foreach (self::$_events[$name][$class] as $i => $event) {
            if ($event[0] === $handler) {
                unset(self::$_events[$name][$class][$i]);
                $removed = true;
            }
        }
        if ($removed) {
            self::$_events[$name][$class] = array_values(self::$_events[$name][$class]);
        }
        return $removed;
    }

显然在$handler !== null的情况下，函数首先会删除对应的self::$_events[$name][$class]数组的变量，由于采用unset() 函数删除，不会改变数组索引值。作者又在最后通过 array_value()函数处理下。 然后这些直接可以使用array_splice()完成。个人觉得这样会更加的高效，大家有什么意见呢？ 改完的代码如下：

public static function off($class, $name, $handler = null)
{
        $class = ltrim($class, '\\');
        if(empty(self::$_events[$name][$class])){
            return false;
        }
        if ($handler === null){
            unset(self::$_events[$name][$class]);
            return true;
        }
        foreach (self::$_events[$name][$class] as $i => $event){
            if($event[0] === $handler){
                array_splice(self::$_events[$name][$class], $i, 1);
                return true;
            }
        }
        return false;
}