Syntax error while defining an array as a property of a class

Question

...

public $aSettings = array(
  'BindHost' => "127.0.0.1",
  'Port' => 9123,
  'MaxFileSize' => (5 * (1024 * 1024)), // unexpected "(" here
  'UploadedURL' => "http://localhost",
  'UploadPath' => dirname(__FILE__) . "/upload",
  'UploadMap' => dirname(__FILE__) . "/uploads.object",
  'RegisterMode' => false
);

...

This is my code, straight from a class. The problem I have is the "unexpected ( on line 22", line 22 being MaxFileSize.

I can't see a problem with it, is this a Zend Engine limitation? Or am I blind.

Works fine for me -> codepad.org/EIorteTQ .... im guessing its a problem defining non-constant variable in a class — Manse
– Manse, Commented Feb 10, 2012 at 9:40
Have you tried to just remove the outer ( )? Or just all ( ) on that line, as there is no need for them when doing multiplication... — Svish
– Svish, Commented Feb 10, 2012 at 9:42
silly's answer to a similar question is useful in this regard. While it uses the same approach as Michael Krelin's, it shows a way to include the initialization logic within the class definition. — Alex Coventry
– Alex Coventry, Commented Feb 22, 2013 at 23:48

Michael Berkowski · Accepted Answer · 2014-09-03 10:47:29Z

8

You cannot use non-constant values while initializing class properties in PHP versions earlier than 5.6.
These are initialized at compile time, at which PHP will do no calculations or execute any code. (5 * (1024 * 1024)) is an expression that requires evaluation, which you cannot do there. Either replace that with the constant value 5242880 or do the calculation in __construct.

PHP 5.6, introduced in 2014, allows "constant scalar expressions" wherein a scalar constant or class property can be initialized by an evaluated expression in the class definition rather than the constructor.

edited Sep 3, 2014 at 10:47

Michael Berkowski

271k47 gold badges450 silver badges395 bronze badges

answered Feb 10, 2012 at 9:42

deceze♦

525k89 gold badges807 silver badges954 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Blake Over a year ago

Thanks, I wasn't aware of that. I'll move the operations to my construct.

Michael Krelin - hacker · Accepted Answer · 2012-02-10 09:42:07Z

3

I suspect this is not the whole code and this is a definition of a static variable inside a class, where you're quite limited in expressions and can't calculate a lot.

If I'm right, you may want to do something like that instead:

class thingamajig {
    public static $aSettings;
};
thingamajig::$aSettings = array ( ... );

P.S. Sorry, I've just read your prose where you confirm it's a part of a class static variable. So you can't just ignore out-of-place keyword.

answered Feb 10, 2012 at 9:42

Michael Krelin - hacker

144k25 gold badges200 silver badges177 bronze badges

2 Comments

Blake Over a year ago

Indeed it is not all of the code, it was taken from a class file, I removed the surrounding code.

Michael Krelin - hacker Over a year ago

Yeah, read that now. At first it was too obvious to finish reading :)

pjumble · Accepted Answer · 2012-02-10 09:42:52Z

3

I assume what you're showing is actually a class property (because of the public keyword). Initialization of class properties in PHP must be constant.

This declaration may include an initialization, but this initialization must be a constant value--that is, it must be able to be evaluated at compile time and must not depend on run-time information in order to be evaluated.

http://www.php.net/manual/en/language.oop5.properties.php

answered Feb 10, 2012 at 9:42

pjumble

17k6 gold badges45 silver badges52 bronze badges

Comments

Timur · Accepted Answer · 2012-02-10 09:46:33Z

1

When you define variable in class, you cannot assign expression to it. (5 * (1024 * 1024)) is an expression. 6164480 is not.

answered Feb 10, 2012 at 9:46

Timur

6,7381 gold badge30 silver badges37 bronze badges

2 Comments

deceze Over a year ago

Technically 6164480 is an expression as well, just a very simple one. :o)

Timur Over a year ago

Hmm... Yep, @deceze described it a lot more accurate :)

Jon · Accepted Answer · 2014-03-13 11:48:23Z

0

This limitation no longer exists as of PHP 5.6

The new feature that enables the previously-disallowed syntax is called constant scalar expressions:

It is now possible to provide a scalar expression involving numeric and string literals and/or constants in contexts where PHP previously expected a static value, such as constant and property declarations and default function arguments.
class C {
    const THREE = TWO + 1;
    const ONE_THIRD = ONE / self::THREE;
    const SENTENCE = 'The value of THREE is '.self::THREE;

    public function f($a = ONE + self::THREE) {
        return $a;
    }
}

echo (new C)->f()."\n"; echo C::SENTENCE; ?>
The above example will output:
4 The value of THREE is 3

answered Mar 13, 2014 at 11:48

Jon

439k85 gold badges756 silver badges820 bronze badges

Comments

thenetimp · Accepted Answer · 2012-02-10 09:43:43Z

-3

Public is a declaration only used in objects. This is not an object, remove public and it's fine.

answered Feb 10, 2012 at 9:43

thenetimp

9,9205 gold badges31 silver badges42 bronze badges

2 Comments

Blake Over a year ago

The code was taken FROM an object, hence ignore the public keyword.

thenetimp Over a year ago

OK, well I did take it out and php -l came back with no errors, so the error is elsewhere even though it's reporting there.

Collectives™ on Stack Overflow

Syntax error while defining an array as a property of a class

6 Answers 6

1 Comment

2 Comments

Comments

2 Comments

This limitation no longer exists as of PHP 5.6

Comments

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

1 Comment

2 Comments

Comments

2 Comments

This limitation no longer exists as of PHP 5.6

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related