tags:

views:

88

answers:

4

I'm working on a login class in PHP. I have the following line inside a class definition:

private $salty = sha1('salty');

...which is giving me the following error:

"Parse error: parse error, expecting ','' or ';'' in C:\xampp\htdocs\test\includes\jaLogin.php on line 26"

I've tried using this line outside of the class definition and it works fine, it's only when I use it inside the class definition that I get a problem.

If I remove the word "private" I get a slightly different error:

*Parse error: parse error, expecting `T_FUNCTION' in C:\xampp\htdocs\test\includes\jaLogin.php on line 26*

I feel like I'm missing something obvious...

+8  A: 

You can't use function-calls to initialize class member variables.

class YourClass
{
    private $salty;

    public function __construct()
    {
        $this->salty = sha1('salty');
    }
}

is the one way to initialize your variable.

EDIT

Even, e.g., a simple concatenation of two constant strings is not allowed (protected $_string = 'Hello ' . 'World!';). The evaluation of class properties happens at compile time, so the usage of constructs that depend on run-time information is illegal.

[...] 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.

(Properties)

Stefan Gehrig
Bingo! Right on the mark! Thanks for that. Any idea why this is?
Mr_Chimp
It's because your class is just a data structure. You always need to call a function to set them to some values.In your case it's the constructor function.
toto
Added some explanation. Actually it's a problem of when in the execution process the member variables are evaluated.
Stefan Gehrig
That makes a lot of sense. Thanks.
Mr_Chimp
+1  A: 

Just declare the variable as null and then initialize it in your constructor.

private $salty = null

function __construct() {
    $this->salty = $sha1('salty');
}
pablasso
A: 

$sha1() won't function because you're trying to handle a function as a variable.

And when you're working on a better login mechanism why not start using:

$this->salty = hash("SHA512", "salty");

Additionally you can create a salt hash and add it with your password. This addition gives you a little overhead because you will have to save the salt in a db-table too and retreive and combine it when you are validation (user)credentials.

Hope it helps you!

Ben Fransen
It doesn't, but thanks for trying!
Mr_Chimp
A: 

In what way doesn't it help you?

Ben Fransen
I didn't try to "handle a function as a variable". You added the $ in "$sha1()" yourself. You do have a point when you say that I should be using hash() instead of sha1(). BTW, try to reply to the comment that you're replying to, otherwise we get a thread full of "answers" which aren't.
Mr_Chimp
Ah, I'm sorry for creating a new answer. I'm new here.. I was reffering to the answer of Pablasso, not your post.
Ben Fransen
Oh! Well, that makes sense...I hadn't even spotted that in Pablasso's answer! No worries, pull up a chair, make your self comfortable, etc!
Mr_Chimp