How would I add isset() and keep the empty() on my code below?
$pagesize = (!empty($_GET['pagesize'])) ? $_GET['pagesize'] : 20;
UPDATE:
I am just wanting to make sure php doesn't produce any notices or warnings
Is this what you mean?
$pagesize = (isset($_GET['pagesize']) && !empty($_GET['pagesize'])) ?
$_GET['pagesize'] :
20;
http://us.php.net/manual/en/language.operators.logical.php
EDIT:
To be complete, empty already checks if something is set, so you don't need to use isset() as well.
I would also caution against using this code if it is going directly into a query or something similar. Consider using intval, is_numeric and similar functions.
! empty($var) and isset($var)
is redundant because there is no possible value (or lack thereof) for $var
that would cause ! empty($var)
to return true
while causing isset($var)
to return false
(and if ! empty($var)
is false
, then the conditional short-circuits, and isset($var)
is not evaluated). The only way the two would not be redundant is if you wanted to check if empty($var) and isset($var)
, which is the same as (but IMnsHO more confusing than) empty($var) and ! is_null($var)
.
I'm not sure exactly what you're after here. isset
will check if a value has been set and return true if it has. empty
will check if a value hasn't been set OR if it equates to false (eg: 0, "", null
) and return true if it does.
I can't see why you'd need to combine the two. To rewrite your example without empty
, you'd do this:
$pagesize = isset($_GET['pagesize']) && $_GET['pagesize']
? $_GET['pagesize']
: 20;
isset
.empty
will evaluate to true if the variable has not been set.