CodeIgniter框架Router.php源码有问题吗?

Question

在ci框架的systemcoreRouter.php文件中,line132行开始代码如下:

    is_array($routing) && isset($routing['directory']) && $this->set_directory($routing['directory']);
    $this->_set_routing();

    // Set any routing overrides that may exist in the main index file
    if (is_array($routing))
    {
        empty($routing['controller']) OR $this->set_class($routing['controller']);
        empty($routing['function'])   OR $this->set_method($routing['function']);
    }

首先,第一行的:
is_array($routing) && isset($routing['directory']) && $this->set_directory($routing['directory']);
我不确定这样写是否有意义,这样一行表达式的结果(true or false)不添加判断的话,得到的结果(true or false)有什么用意呢?
另外在if判断语句里,使用empty判断两个结果,我也认为没有意义,如果不去做判断的话,这样写有什么想法吗?
还请各位指教.

Answer 1

is_array($routing) && isset($routing['directory']) && $this->set_directory($routing['directory']);

&&这个符号知道是 true + true 对把？
也就是说 第一个 和 第二个 一定都要执行成功才会执行第三个，这三个都要执行成功，否这就不执行。我们可以换一个写法：

if (is_array($routhing) && isset($routing['directory'])) {
    $this->set_directory($routing['directory']);
}

思路很清晰了把？

为什么要这样做 && 这样做？

1、因为不涉及复杂的语句
2、简洁
3、直观、可读性强

Answer 2

// && vs ||
!isset($a) && $a=1;
echo $a;//1
isset($a) || $a =2;
echo $a;//1