1

我需要$layers从函数内部访问isOk($layer)，但是我尝试外部函数变量$layers的一切都可以，但是在函数内部，即使使用全局也是返回 null。

这是代码：

$layers = array();
foreach($pcb['PcbFile'] as $file){
    if(!empty($file['layer'])){
        $layers[$file['layer']] = 'ok';
    }
}
// Prints OK!
var_dump($layers);

function isOk($layer){
    global $layers;
// Not OK! prints NULL
    var_dump($layers);
    if(array_key_exists($layer, $layers))
        return ' ok';
    return '';
}
// NOT OK
echo isOk('MD');

我总是使用面向对象，但这太简单了，我用一个简单的函数做了……为什么$layers在函数内部没有被正确地“接收”？

4

2 回答 2

6

看这个...

HalfAssedFramework.php

<?php
class HalfAssedFramework {
    public static function run($path) {
        include $path;
    }
}

HalfAssedFramework::run('example.php');

例子.php

<?php

$layers = array();
foreach($pcb['PcbFile'] as $file){
    if(!empty($file['layer'])){
        $layers[$file['layer']] = 'ok';
    }
}
// Prints OK!
var_dump($layers);

function isOk($layer){
    global $layers;
// Not OK! prints NULL
    var_dump($layers);
    if(array_key_exists($layer, $layers))
        return ' ok';
    return '';
}
// NOT OK
echo isOk('MD');

直接运行example.php，它应该可以工作。运行HalfAssedFramework.php，它不会。

问题是范围。当example.php被包含在run函数中时，它里面的所有代码都继承了函数的作用域。在该范围内，$layers默认情况下不是全局的。

要解决此问题，您有几个选择：

  • 如果你知道example.php永远不会直接运行，你可以global $layers;在文件的开头说。不过，如果脚本直接运行，我不确定这是否会起作用。
  • 随处替换$layers。$GLOBALS['layers']
  • $layers作为参数添加到isOk.
  • 按照Geoffrey的建议，将此代码放在一个类中。
于 2012-08-10T03:48:12.370 回答
2

没有完全回答这个问题，但是您是否考虑过不使用全局变量？全局变量确实不是那么酷：它们使您的代码更难阅读、更难理解，因此也更难维护。

考虑这样的事情：

<?php

class LayerCollection
{
    private $layers;

    public function __construct($layers)
    {
        $this->layers = $layers;
    }

    public static function fromPcbFile($data)
    {
        $layers = array();

        foreach ($data['PcbFile'] as $layer) {
            if (!empty($layer)) {
                $layers[$layer] = true;
            }
        }

        return new self($layers);
    }

    public function hasLayer($layer)
    {
        return array_key_exists($layer, $this->layers);
    }
}

$layers = LayerCollection::fromPcbFile($pcb);
var_dump($layers->hasLayer('MD'));

不是更好看吗？然后，您可以继续丰富LayerCollection，因为您需要更多与图层交互的方式。这并不完美，因为有still一个静态方法存在（使测试更难，工厂更适合该工作），但这是重构的良好开端。

更多关于为什么全局变量是邪恶的：https ://softwareengineering.stackexchange.com/questions/148108/why-is-global-state-so-evil

于 2012-08-10T03:38:56.190 回答