Is there a shortcut i'm missing?

[Lo'oris]

$name=$_GET['name'];
if (!$name)
$name="value";

i can't figure out how to shorten this thing. Is there some kind of
operator i don't know about?

[Jerry Stuckle]

Lo'oris wrote:

$name=$_GET['name'];
if (!$name)
$name="value";

i can't figure out how to shorten this thing. Is there some kind of
operator i don't know about?

$name = isset($_GET['name']) ? $_GET['name'] : null;

You should always test with isset() to see if a value passed to your
page is set or not. Otherwise you will get a notice if you have them
enabled.

--
==================
Remove the "x" from my email address
Jerry Stuckle
JDS Computer Training Corp.
js*******@attglobal.net
==================

[Christoph Burschka]

Jerry Stuckle wrote:

Lo'oris wrote:

>$name=$_GET['name'];
if (!$name)
$name="value";

i can't figure out how to shorten this thing. Is there some kind of
operator i don't know about?

$name = isset($_GET['name']) ? $_GET['name'] : null;

You should always test with isset() to see if a value passed to your
page is set or not. Otherwise you will get a notice if you have them
enabled.

If setting multiple variables from $_GET, you can also try this:

$parameters=array('name'=>"value",'example'=>"valu e1",'another'=>"value2");
foreach ($parameters as $parameter=>$value)
{
$$parameter=$_GET[$parameter]?$_GET['parameter:$value;
}

--
cb

[Jerry Stuckle]

Christoph Burschka wrote:

Jerry Stuckle wrote:

>Lo'oris wrote:

>>$name=$_GET['name'];
if (!$name)
$name="value";

i can't figure out how to shorten this thing. Is there some kind of
operator i don't know about?

$name = isset($_GET['name']) ? $_GET['name'] : null;

You should always test with isset() to see if a value passed to your
page is set or not. Otherwise you will get a notice if you have them
enabled.

If setting multiple variables from $_GET, you can also try this:

$parameters=array('name'=>"value",'example'=>"valu e1",'another'=>"value2");
foreach ($parameters as $parameter=>$value)
{
$$parameter=$_GET[$parameter]?$_GET['parameter:$value;
}

Which is only slightly less dangerous than running with register_globals
on. Someone can come in and set any variable in your script by setting
it in the query string. And if you miss initializing a variable you've
got a huge potential security breach.

One reason register_globals is no longer enabled by default.

--
==================
Remove the "x" from my email address
Jerry Stuckle
JDS Computer Training Corp.
js*******@attglobal.net
==================

[Hendri Kurniawan]

Jerry Stuckle wrote:

Christoph Burschka wrote:

>Jerry Stuckle wrote:

>>Lo'oris wrote:
$name=$_GET['name'];
if (!$name)
$name="value";

i can't figure out how to shorten this thing. Is there some kind of
operator i don't know about?

$name = isset($_GET['name']) ? $_GET['name'] : null;

You should always test with isset() to see if a value passed to your
page is set or not. Otherwise you will get a notice if you have them
enabled.

If setting multiple variables from $_GET, you can also try this:

$parameters=array('name'=>"value",'example'=>"val ue1",'another'=>"value2");

foreach ($parameters as $parameter=>$value)
{
$$parameter=$_GET[$parameter]?$_GET['parameter:$value;
}

Which is only slightly less dangerous than running with register_globals
on. Someone can come in and set any variable in your script by setting
it in the query string. And if you miss initializing a variable you've
got a huge potential security breach.

One reason register_globals is no longer enabled by default.

Not really. It think it's a clever way to do it. Save you some coding time.

If you see, he only allows the variable in the parameter to be changed.

But then again, it's only my oppinion.

Hendri

[Jerry Stuckle]

Hendri Kurniawan wrote:

Jerry Stuckle wrote:

>Christoph Burschka wrote:

>>Jerry Stuckle wrote:
Lo'oris wrote:
$name=$_GET['name'];
if (!$name)
$name="value";
>
i can't figure out how to shorten this thing. Is there some kind of
operator i don't know about?
>
$name = isset($_GET['name']) ? $_GET['name'] : null;

You should always test with isset() to see if a value passed to your
page is set or not. Otherwise you will get a notice if you have them
enabled.
If setting multiple variables from $_GET, you can also try this:

$parameters=array('name'=>"value",'example'=>"va lue1",'another'=>"value2");

foreach ($parameters as $parameter=>$value)
{
$$parameter=$_GET[$parameter]?$_GET['parameter:$value;
}

Which is only slightly less dangerous than running with
register_globals on. Someone can come in and set any variable in your
script by setting it in the query string. And if you miss
initializing a variable you've got a huge potential security breach.

One reason register_globals is no longer enabled by default.

Not really. It think it's a clever way to do it. Save you some coding time.

If you see, he only allows the variable in the parameter to be changed.

But then again, it's only my oppinion.

Hendri

Not at all. I key in

http://www.example.com?admin=1

And now in your program $admin=1. And what if $admin is the variable
which indicates I'm an admin?

This effectively does exactly what register_globals does - just limits
it to the $_GET variables.

--
==================
Remove the "x" from my email address
Jerry Stuckle
JDS Computer Training Corp.
js*******@attglobal.net
==================

[Hendri Kurniawan]

Jerry Stuckle wrote:

Hendri Kurniawan wrote:

>Jerry Stuckle wrote:

>>Christoph Burschka wrote:
Jerry Stuckle wrote:
Lo'oris wrote:
>$name=$_GET['name'];
>if (!$name)
> $name="value";
>>
>i can't figure out how to shorten this thing. Is there some kind of
>operator i don't know about?
>>
$name = isset($_GET['name']) ? $_GET['name'] : null;
>
You should always test with isset() to see if a value passed to your
page is set or not. Otherwise you will get a notice if you have them
enabled.
>

If setting multiple variables from $_GET, you can also try this:

$parameters=array('name'=>"value",'example'=>"v alue1",'another'=>"value2");

foreach ($parameters as $parameter=>$value)
{
$$parameter=$_GET[$parameter]?$_GET['parameter:$value;
}
Which is only slightly less dangerous than running with
register_globals on. Someone can come in and set any variable in
your script by setting it in the query string. And if you miss
initializing a variable you've got a huge potential security breach.

One reason register_globals is no longer enabled by default.

Not really. It think it's a clever way to do it. Save you some coding
time.

If you see, he only allows the variable in the parameter to be changed.

But then again, it's only my oppinion.

Hendri

Not at all. I key in

http://www.example.com?admin=1

And now in your program $admin=1. And what if $admin is the variable
which indicates I'm an admin?

This effectively does exactly what register_globals does - just limits
it to the $_GET variables.

Well in order to do that the $parameters variable have to include admin
is it.

Hendri

[Toby A Inkster]

Lo'oris wrote:

$name=$_GET['name'];
if (!$name)
$name="value";

i can't figure out how to shorten this thing. Is there some kind of
operator i don't know about?

The current PHP method is:

$name = $_GET['name'] ? $_GET['name'] : 'value';

It was proposed that PHP 6 should have a new function "ifsetor" which
would achieve the same thing using this syntax:

$name = ifsetor($_GET['name'], 'value');

However, this proposal was rejected in favour for making the middle part
of the current method optional, so you could write:

$name = $_GET['name'] ? : 'value';

However, if you want to use the above, you'll need to wait for PHP 6, and
your code will not be backwards-compatible with earlier versions of PHP.

Personally, I've written my own ifsetor() function, and just use that.

/**
* Implementation of proposed (but discarded) PHP6 built-in function.
*
* @param mixed $value Preferred value.
* @param mixed $fallback Fallback value.
* @return mixed isset($value) ? $value : $fallback
*/
function ifsetor ($value, $fallback)
{
if (isset($value))
return $value;
return $fallback;
}

I use this most often for loading settings. Assuming I have a bunch of
settings from a file stored in some array:

$mode = ifsetor($settings['mode'], 3); // default mode is 3

--
Toby A Inkster BSc (Hons) ARCS
Contact Me ~ http://tobyinkster.co.uk/contact
Geek of ~ HTML/SQL/Perl/PHP/Python*/Apache/Linux

* = I'm getting there!