search

Home  >  Q&A  >  body text

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

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

1

2

3

4

5

6

7

8

9

<code>    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']);

    }</code>

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

PHPzPHPz2893 days ago303

reply all(2)I'll reply

  • PHP中文网

    PHP中文网2017-04-10 17:52:34

    1

    2

    3

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

     

    </code>

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

    1

    2

    3

    4

    <code>if (is_array($routhing) && isset($routing['directory'])) {

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

    }

    </code>

    思路很清晰了把?

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

    1

    2

    3

    <code>1、因为不涉及复杂的语句

    2、简洁

    3、直观、可读性强</code>

    reply
    0
  • PHPz

    PHPz2017-04-10 17:52:34

    1

    2

    3

    4

    5

    <code class="php">// && vs ||

    !isset($a) && $a=1;

    echo $a;//1

    isset($a) || $a =2;

    echo $a;//1</code>

    reply
    0
  • Cancelreply