php unexpected ')' parse error [duplicate]

Question

Possible Duplicate:
Can I use a function to return a default param in php?
Using function result as a default argument in PHP function

I am trying to set default value for a function. I want the function $expires default value to be time() + 604800.

I am trying

public function set($name,$value,$expires = time()+604800) {
    echo $expires;
    return setcookie($name, $value, $expires);
    }

But I get an error.

Parse error: syntax error, unexpected '(', expecting ')' in /var/www/running/8ima/lib/cookies.lib.php on line 38

How should I write it?

Wouldn't if (!$expires) { $expires = time() + 604800; } do it ? — user625860
– user625860, Commented Mar 1, 2012 at 9:37

Mark Baker · Accepted Answer · 2012-03-01 09:36:37Z

4

$expires = time()+604800

in the function definition.

Default value can't be the result of a function, only a simple value QUoting from the manual:

The default value must be a constant expression, not (for example) a variable, a class member or a function call.

Use:

public function set($name,$value,$expires = NULL) { 
    if (is_null($expires))
        $expires = time()+604800;
    echo $expires; 
    return setcookie($name, $value, $expires); 
}

answered Mar 1, 2012 at 9:36

Mark Baker

213k34 gold badges354 silver badges390 bronze badges

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

1 Comment

Luchian Grigore Over a year ago

+1 first answer that actually explains what's wrong.

Marc · Accepted Answer · 2012-03-01 09:37:01Z

1

You cannot use a function call in your params declaration.

Do it this way:

public function set($name,$value,$expires = null) {
    if(is_null($expires)) $expires = time()+604800;
    echo $expires;
    return setcookie($name, $value, $expires);
}

answered Mar 1, 2012 at 9:37

Marc

6,7919 gold badges51 silver badges79 bronze badges

Comments

knkill_prfl · Accepted Answer · 2012-03-01 09:36:28Z

0

try this:

public function set($name,$value,$expires = NULL) {
    if(is_null($expires)) {
        $expires = time() + 604800;
    }
    echo $expires;
    return setcookie($name, $value, $expires);
}

answered Mar 1, 2012 at 9:36

knkill_prfl

2171 gold badge4 silver badges9 bronze badges

1 Comment

Luchian Grigore Over a year ago

+1 if you can also explain why.

Nicola Peluchetti · Accepted Answer · 2012-03-01 09:36:46Z

0

You could do

public function set($name,$value,$expires = null) {
 if($expires === null){$expires = time()+604800);
 echo $expires;
 return setcookie($name, $value, $expires);
}

answered Mar 1, 2012 at 9:36

Nicola Peluchetti

76.9k32 gold badges150 silver badges197 bronze badges

Collectives™ on Stack Overflow

php unexpected ')' parse error [duplicate]

4 Answers 4

1 Comment

Comments

1 Comment

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

Comments

1 Comment

Comments

Linked

Related