PHP function with variable as default value for a parameter

Question

By default a PHP function uses $_GET variables. Sometimes this function should be called in an situation where $_GET is not set. In this case I will define the needed variables as parameter like: actionOne(234)

To get an abstract code I tried something like this:

function actionOne($id=$_GET["ID"])

which results in an error:

Parse error: syntax error, unexpected T_VARIABLE

Is it impossible to define an default parameter by using an variable?

Edit

The actionOne is called "directly" from an URL using the framework Yii. By handling the $_GET variables outside this function, I had to do this on an central component (even it is a simple, insignificant function) or I have to change the framework, what I don't like to do.

An other way to do this could be an dummy function (something like an pre-function), which is called by the URL. This "dummy" function handles the variable-issue and calls the actionOne($id).

John Parker · Accepted Answer · 2011-05-02 16:13:35Z

56

No, this isn't possible, as stated on the Function arguments manual page:

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

Instead you could either simply pass in null as the default and update this within your function...

function actionOne($id=null) {
    $id = isset($id) ? $id : $_GET['ID'];
    ....
}

...or (better still), simply provide $_GET['ID'] as the argument value when you don't have a specific ID to pass in. (i.e.: Handle this outside the function.)

edited May 2, 2011 at 16:13

answered May 2, 2011 at 16:05

John Parker

54.5k11 gold badges133 silver badges131 bronze badges

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

6 Comments

markus Over a year ago

Same here, the solution breaks encapsulation, the check should happen outside the function.

The Bndr Over a year ago

I updated my post. In my specific case, your solution looks like the best way... (I will check this out soon) In general I probably would use the solution form @Galen. What do you mean with "simply provide $_GET['ID'] as the argument value"? Is this according to your example?

The Bndr Over a year ago

isn't it better to use "$id===null?$id=$_GET["ID"]:null;" instead of $id = isset($id) ? $id : $_GET['ID'];? Because you avoid one assignment of $id, if $id is still set ($id=$id)!?! ..or is there no different?

John Parker Over a year ago

@The Bndr I meant that you'd assign $id before you call the function, rather than in the function itself. In terms of your second point - if you're worried about the overhead of a variable assignment, I suspect you shouldn't be using PHP. :-)

The Bndr Over a year ago

@middaparka maybe! ;-) but my version of assigning $id is also usable?! Or is your version the "better practice"?

|

markus · Accepted Answer · 2011-05-02 16:08:52Z

4

You should get that id before you call the function. Checking for the existence of the parameter breaks encapsulation. You should do something like that:

if (isset($_GET["ID"])
{
    $id = $_GET["ID"];
}
else
{
    //$id = something else
}

function doSomethingWithID($id)
{
    //do something
}

answered May 2, 2011 at 16:08

markus

40.8k23 gold badges100 silver badges143 bronze badges

Comments

Galen · Accepted Answer · 2011-05-02 16:30:52Z

4

function actionOne( $id=null ) {
    if ($id === null) $id = $_GET['ID'];
}

But, i would probably do this outside of the function:

// This line would change, its just a for instance
$id = $id ? $id : $_GET['id'];
actionOne( $id );

edited May 2, 2011 at 16:30

answered May 2, 2011 at 16:04

Galen

30.2k9 gold badges75 silver badges90 bronze badges

2 Comments

The Bndr Over a year ago

hi thank you.... there is no other way? ...without handling this problem inside or outside this function?

markus Over a year ago

I still think you should provide the good practice as the main example and the worse practice as the alternative!

pablo henrique · Accepted Answer · 2019-05-23 15:09:12Z

2

You could use constant variable

define('ID',$_GET["ID"]);
function($id = _ID_){
    //code
}

answered May 23, 2019 at 15:09

pablo henrique

3423 silver badges10 bronze badges

Comments

Naftali · Accepted Answer · 2011-05-02 16:06:28Z

1

Yes it is impossible.

The default has to be a static variable:

function actionOne( $id='something') {
    //code
}

answered May 2, 2011 at 16:06

Naftali

147k41 gold badges247 silver badges304 bronze badges

3 Comments

markus Over a year ago

a static variable? and you're passing a string? how does that solve the problem?

markus Over a year ago

The only thing which is called a dynamic variable in PHP afaik is a variable-variable and that wouldn't solve the problem either.

markus Over a year ago

Your solution is wrong, you're passing a string as default value, how does that address the original question?

Mihail Gershkovich · Accepted Answer · 2016-08-25 22:29:59Z

1

Easy peanuts!
(Might contain minor mistakes, errors or typos!)

You need a helper function, which will call you main function recursively, but having NULL as default:

Wrong: function actionOne($id=$_GET["ID"])

Right:

function actionOne($id) {...}
function actionOnewithID($id=NULL) {
  if (NULL==$id){actionOne($_GET["ID"]);}
  else {actionOne($id);
}

And if you need to return a value:

function actionOne($id) {...}
function actionOnewithID($id=NULL) {
  if (NULL==$id){return(actionOne($_GET["ID"]));}
  else {return(actionOne($id));
}

I hope this helps!

edited Aug 25, 2016 at 22:29

answered Aug 25, 2016 at 22:15

Mihail Gershkovich

5264 silver badges9 bronze badges

Comments

Hosserin Ibrahim · Accepted Answer · 2022-08-23 13:56:54Z

1

shortest way is:

function actionOne($id = null)
{
     $id = $id ?? $_GET["ID"];
     ...
}

answered Aug 23, 2022 at 13:56

Hosserin Ibrahim

2541 silver badge11 bronze badges

Collectives™ on Stack Overflow

PHP function with variable as default value for a parameter

7 Answers 7

6 Comments

Comments

2 Comments

Comments

3 Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

7 Answers 7

6 Comments

Comments

2 Comments

Comments

3 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related