Question

I need to access $layers from inside the function isOk($layer), but everything i tried the outside function variable $layers is ok, but inside the function, even with global is return null.

Here is the code:

$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');

I always use Object orientation, but this was something so simple that i made with a simple function… Why $layers is not being ‘received’ correcly inside the function?

Answer 1

Watch this…

HalfAssedFramework.php

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

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

example.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');

Run example.php directly, and it should work. Run HalfAssedFramework.php, and it won’t.

The problem is scope. When example.php is included inside the run function, all the code inside it inherits the function’s scope. In that scope, $layers isn’t global by default.

To fix this, you have a couple of options:

• If you know example.php will never run directly, you can say global $layers; at the beginning of the file. I’m not sure whether this will work if the script is run directly, though.
• Replace $layers with $GLOBALS['layers'] everywhere.
• Add $layers as an argument to isOk.
• Put this code within a class, as suggested by Geoffrey.