nested shorthand if

Question

I have a little trouble with a shorthanded if statement I can't figure out

($product == "vindo") ? $this->getNextVindoInList($id) : $this->getNextGandrupInList($id),

This works fine but I want to have another check in that statement. Like this:

if($product == "vindo") {
  if($number != 14) {
    $this->getNextVindoInList($id)
  }
} else {
 if($number != 22) {
    $this->getNextGandrupInList($id)
 }
}

Be careful, too much use will result in harder to read code!!! There is also minimal performance gain from it, it's neither faster or slower. — Brian
– Brian, Commented Sep 20, 2012 at 12:42

Travesty3 · Accepted Answer · 2018-10-16 19:40:40Z

14

For educational purposes, I will leave this answer intact. But it should be known that this is NOT RECOMMENDED. Nesting ternaries is a bad idea. It provides no performance benefit over explicit if-else statements and makes the code much more difficult to read.

That said, see below for how it can, but should not be done.

Two ways:

($product == "vindo" && $number != 14 ? $this->getNextVindoInList($id) : ($number != 22 ? $this->getNextGandrupInList($id) : '')

// Equivalent of:
if ($product == "vindo" && $number != 14)
    $this->getNextVindoInList($id);
else if ($number != 22)
    $this->getNextGandrupInList($id);

// OR

// Equivalent of your example:
($product == "vindo" ? ($number != 14 ? $this->getNextVindoInList($id) : '') : ($number != 22 ? $this->getNextGandrupInList($id) : ''))

edited Oct 16, 2018 at 19:40

answered Sep 20, 2012 at 12:40

Travesty3

14.5k7 gold badges64 silver badges103 bronze badges

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

1 Comment

Travesty3 Over a year ago

This is old. But someone just upvoted it, which made me come look at it, and want to slap my former self for even presenting this as a good idea. Please avoid nested ternaries. They are unnecessarily difficult to read.

Zevi Sternlicht · Accepted Answer · 2012-09-20 12:38:17Z

3

Try this!

($product == "vindo") ? ($number != 14 ? $this->getNextVindoInList($id) : null ) : (($number != 22) ? $this->getNextGandrupInList($id) : null)

answered Sep 20, 2012 at 12:38

Zevi Sternlicht

5,39921 silver badges31 bronze badges

Comments

phant0m · Accepted Answer · 2012-09-20 12:51:59Z

2

I will not present a solution with a nested ternary operator. Why? The code with the explicit if/else structure communicates intent. It shows what exactly is happening.

Why sacrifice readability for a few lines? That's quite a bad trade.

answered Sep 20, 2012 at 12:51

phant0m

17k6 gold badges52 silver badges84 bronze badges

Comments

PcPulsar · Accepted Answer · 2012-09-20 12:47:00Z

1

Your if statements can be simplefied by using this code:

if($product == "vindo" && $number != 14) {
  $this->getNextVindoInList($id)
} else if($number != 22) {
  $this->getNextGandrupInList($id)
}

The sorthand if isnt handy now because the else has an if statement too.

answered Sep 20, 2012 at 12:47

PcPulsar

4446 silver badges18 bronze badges

Collectives™ on Stack Overflow

nested shorthand if

4 Answers 4

1 Comment

Comments

Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

Comments

Comments

Comments

Related