欢迎您访问程序员文章站本站旨在为大家提供分享程序员计算机编程知识!
您现在的位置是: 首页  >  后端开发

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

程序员文章站 2022-04-10 16:00:53
...
在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判断两个结果,我也认为没有意义,如果不去做判断的话,这样写有什么想法吗?
还请各位指教.

回复内容:

在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判断两个结果,我也认为没有意义,如果不去做判断的话,这样写有什么想法吗?
还请各位指教.

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、直观、可读性强

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