require/include: constant vs. variable

pimmy

Hi,

I am stuck with a silly problem which I cant resolve. Please look at the example:

settings.php

Expand|Select|Wrap|Line Numbers

 
<?php

define('VALUE_STRING', 'This is my string');

$VALUE_ARRAY = array('This', 'is', 'my', 'array');

?>

values.class.php

Expand|Select|Wrap|Line Numbers

 
<?php

Class Values

{

/* Here I have some constructor */
 
public function getValues() {
 
        error_log("First:  " . VALUE_STRING);

        error_log("Second: " . implode(', ', $VALUE_ARRAY));

}

?>

index.php

Expand|Select|Wrap|Line Numbers

 
<?php

$values = new Values();

$values->getValues();

?>

When I execute index.php I get the following in the error log:

Expand|Select|Wrap|Line Numbers

 
[Fri Nov 02 11:57:59 2007] [error] [client 9.196.182.69] First:  This is my string

[Fri Nov 02 11:57:59 2007] [error] [client 9.196.182.69] PHP Notice:  Undefined variable: VALUE_ARRAY in /usr/local/apache/htdocs/values.class.php on line 9

[Fri Nov 02 11:57:59 2007] [error] [client 9.196.182.69] PHP Warning:  implode() [<a href='function.implode'>function.implode</a>]: Bad arguments. in /usr/local/apache/htdocs/values.class.php on line 9

[Fri Nov 02 11:57:59 2007] [error] [client 9.196.182.69] Second:

Does anyone know why I cant get the array values, but I can get the constant?

Cheers,

Pimmy

code green

A constant has global scope whereas variables and arrays do not.

pimmy

Quote:

Originally Posted by code green

A constant has global scope whereas variables and arrays do not.

What about having inlude/require line at the beginning?

values.class.php

Expand|Select|Wrap|Line Numbers

 
<?php

require_once(settings.php);
 
Class Values

{

/* Here I have some constructor */
 
public function getValues() {
 
error_log("First: " . VALUE_STRING);

error_log("Second: " . implode(', ', $VALUE_ARRAY));

}

?>

I thought that this would be enough to make the array visible to the function, but aparently it makes no difference. Still get the same error.

Cheers,

Pimmy

code green

Quote:

What about having inlude/require line at the beginning?

Any file that includes settings.php can see the array, but a function or class is a seperate entity again.
If you want a function or class method to see the array it has to be included inside the function.
But normally you would simply pass the variable name in to the function as a parameter

pimmy

It is quite interesting, but this doesnt work either:

Expand|Select|Wrap|Line Numbers

 
<?php

Class Values

{

/* Here I have some constructor */
 
public function getValues() {
 
require_once(settings.php);
 
error_log("First: " . VALUE_STRING);

error_log("Second: " . implode(', ', $VALUE_ARRAY));

}

?>

pimmy

Quite interesting, but this doesnt work either:

Expand|Select|Wrap|Line Numbers

 
<?php

Class Values

{

/* Here I have some constructor */
 
public function getValues() {
 
require_once(settings.php);
 
error_log("First: " . VALUE_STRING);

error_log("Second: " . implode(', ', $VALUE_ARRAY));

}

?>

Atli

Hi pimmy.

Please use [code] tags when posting code. It makes it much easier to understand your code.

As to your problem.
You could use the global keyword.

pimmy

Quote:

Originally Posted by Atli

Hi pimmy.

Please use [code] tags when posting code. It makes it much easier to understand your code.

As to your problem.
You could use the global keyword.

Ooops, sorry. Rushed into the forum without checking the features.
Thanks for the tip, I will give the GLOBAL flag a try.

cheers,
Pimmy

code green

[PHP]<?php
Class Values
{
/* Here I have some constructor */

public function getValues() {

require_once(settings.php);

error_log("First: " . VALUE_STRING);
error_log("Second: " . implode(', ', $VALUE_ARRAY));
}
?>[/PHP]I can't see why this won't work, unless settings.php is included eleswhere in the script.
I think 'once' has global scope throughout the script.

pimmy

For some reason it behaves like that. I've tried only require as well, and got the error for already defined constants.

I will try GLOBAL, looks promissing.

Have a great weekend guys.