Ternary Operator Problem

[Phil Powell]

[PHP]
$sortBy = ($_POST['sortBy']) ? $_POST['sortBy'] : ($_GET['sortBy']) ?
$_GET['sortBy'] : 1;
[/PHP]

This script should set $sortBy to the $_POST or the $_GET or the
default of 1 in one line.

I do not want to have to do this:

[PHP]
if ($_POST['sortBy']) {
$sortBy = $_POST['sortBy'];
} elseif ($_GET['sortBy']) {
$sortBy = $_GET['sortBy'];
} else {
$sortBy = 1;
}
[/PHP]

This is too bloated for the 2000-line script I already have, and I
want to master ternary operators as I have never done so in 7 years.

I simply want the ternary operator equivalent of:

if - elseif - else

NOT

If - else

as the default..

Thanx
Phil

[Pedro Graca]

["Followup-To:" header set to comp.lang.php.]
Phil Powell wrote:

[PHP]
$sortBy = ($_POST['sortBy']) ? $_POST['sortBy'] : ($_GET['sortBy']) ?
$_GET['sortBy'] : 1;
[/PHP]

Try more parenthesis:

$c1 = isset($_POST['sortBy']);
$c2 = isset($_GET['sortBy']);

$sortBy = ($c1) ? ($_POST['sortBy']) : ( ($c2) ? ($_GET['sortBy']) : (1) );

--
USENET would be a better place if everybody read: : mail address :
http://www.catb.org/~esr/faqs/smart-questions.html : is valid for :
http://www.netmeister.org/news/learn2quote2.html : "text/plain" :
http://www.expita.com/nomime.html : to 10K bytes :

[Tim Van Wassenhove]

In article <1c**************************@posting.google.com >, Phil Powell wrote:

[PHP]
$sortBy = ($_POST['sortBy']) ? $_POST['sortBy'] : ($_GET['sortBy']) ?
$_GET['sortBy'] : 1;
[/PHP]
$val = isset($_POST['val']) ? $_POST['val'] : (isset($_GET['val']) ? $_GET['val'] : 1);
I do not want to have to do this:

[PHP]
if ($_POST['sortBy']) {
$sortBy = $_POST['sortBy'];
} elseif ($_GET['sortBy']) {
$sortBy = $_GET['sortBy'];
} else {
$sortBy = 1;
}
[/PHP]

Who cares about lines anyway? Imho the if - elseif - else structure is
more readable, because more people will understand what is going on...
(As you said yourself that you hadn't heard about ternary operator..)

--
http://home.mysth.be/~timvw

[Five Cats]

In message <1c**************************@posting.google.com >, Phil
Powell <so*****@erols.com> writes

[PHP]
$sortBy = ($_POST['sortBy']) ? $_POST['sortBy'] : ($_GET['sortBy']) ?
$_GET['sortBy'] : 1;
[/PHP]

This script should set $sortBy to the $_POST or the $_GET or the
default of 1 in one line.

I do not want to have to do this:

[PHP]
if ($_POST['sortBy']) {
$sortBy = $_POST['sortBy'];
} elseif ($_GET['sortBy']) {
$sortBy = $_GET['sortBy'];
} else {
$sortBy = 1;
}
[/PHP]
This might be useful for the _POST & _GET parts of your query::
http://uk.php.net/manual/en/function...-variables.php


This is too bloated for the 2000-line script I already have, and I want
to master ternary operators as I have never done so in 7 years.
Maybe it's time to split that script into some component parts?

I simply want the ternary operator equivalent of:

if - elseif - else

NOT

If - else

as the default..

--
Five Cats
Email to: cats_spam at uk2 dot net

[Chung Leong]

"Phil Powell" <so*****@erols.com> wrote in message
news:1c**************************@posting.google.c om...

[PHP]
$sortBy = ($_POST['sortBy']) ? $_POST['sortBy'] : ($_GET['sortBy']) ?
$_GET['sortBy'] : 1;
[/PHP]

This script should set $sortBy to the $_POST or the $_GET or the
default of 1 in one line.

Use $_REQUEST--it holds the content of $_POST, $_GET, and $_COOKIE.

[Phil Powell]

Tim Van Wassenhove <eu**@pi.be> wrote in message news:<2g************@uni-berlin.de>...

In article <1c**************************@posting.google.com >, Phil Powell wrote:

[PHP]
$sortBy = ($_POST['sortBy']) ? $_POST['sortBy'] : ($_GET['sortBy']) ?
$_GET['sortBy'] : 1;
[/PHP]

$val = isset($_POST['val']) ? $_POST['val'] : (isset($_GET['val']) ? $_GET['val'] : 1);

I do not want to have to do this:

[PHP]
if ($_POST['sortBy']) {
$sortBy = $_POST['sortBy'];
} elseif ($_GET['sortBy']) {

$sortBy = $_GET['sortBy'];

} else {

$sortBy = 1;

}
[/PHP]

Who cares about lines anyway?

[snip]

The person who inherits your 50 PHP scripts each averaging 1700 lines
each, might care.

Phil

[Tim Van Wassenhove]

In article <1c**************************@posting.google.com >, Phil Powell wrote:

Tim Van Wassenhove <eu**@pi.be> wrote in message news:<2g************@uni-berlin.de>...

In article <1c**************************@posting.google.com >, Phil Powell wrote:

> [PHP]
> $sortBy = ($_POST['sortBy']) ? $_POST['sortBy'] : ($_GET['sortBy']) ?
> $_GET['sortBy'] : 1;
> [/PHP]

$val = isset($_POST['val']) ? $_POST['val'] : (isset($_GET['val']) ? $_GET['val'] : 1);

> I do not want to have to do this:
>
> [PHP]
> if ($_POST['sortBy']) {
> $sortBy = $_POST['sortBy'];
> } elseif ($_GET['sortBy']) {

$sortBy = $_GET['sortBy'];

> } else {

$sortBy = 1;

> }
> [/PHP]

Who cares about lines anyway?

[snip]

The person who inherits your 50 PHP scripts each averaging 1700 lines
each, might care.

Then that person should get himself a decent editor, so he can collapse
code blocks etc...

All that matters is the readability, and i think the if - else structure
is more readable.

--
http://home.mysth.be/~timvw

[Phil Powell]

It's one code block, 1700 lines each, 20 files.

They're files consisting of 20 - 30 classes each. All in one code
block. Total lines: avg 1700.

Phil

Tim Van Wassenhove <eu**@pi.be> wrote in message news:<2g***********@uni-berlin.de>...

In article <1c**************************@posting.google.com >, Phil Powell wrote:

Tim Van Wassenhove <eu**@pi.be> wrote in message news:<2g************@uni-berlin.de>...

In article <1c**************************@posting.google.com >, Phil Powell wrote:
> [PHP]
> $sortBy = ($_POST['sortBy']) ? $_POST['sortBy'] : ($_GET['sortBy']) ?
> $_GET['sortBy'] : 1;
> [/PHP]

$val = isset($_POST['val']) ? $_POST['val'] : (isset($_GET['val']) ? $_GET['val'] : 1);

> I do not want to have to do this:
>
> [PHP]
> if ($_POST['sortBy']) {
> $sortBy = $_POST['sortBy'];
> } elseif ($_GET['sortBy']) { $sortBy = $_GET['sortBy']; > } else { $sortBy = 1; > }
> [/PHP]

Who cares about lines anyway?

[snip]

The person who inherits your 50 PHP scripts each averaging 1700 lines
each, might care.

Then that person should get himself a decent editor, so he can collapse
code blocks etc...

All that matters is the readability, and i think the if - else structure
is more readable.