Question

I have something simple I’m trying to accomplish with less repetition.

By default, I want a div to be shown, however if $x == 1, then check to see if $y != 1, and if $y doesn’t, then don’t show the block.

However the best I can come up with is the following:

if($x) {

    if($y != 1) {

        echo '<div>display block</div>';        
    }

} else {

    echo '<div>display block</div>';

}

This seems a bit repetitive.

I know I can tweak it a bit and do something like:

$displayBlock = '<div>display block</div>';

if($x) {

    if($y != 1) {

        echo $displayBlock;     
    }

} else {

    echo $displayBlock;

}

But even still, I have a feeling that there is a way to do this whole if if else thing which I can’t see right now.

How do you accomplish the above with less if statements? So: if $x != 1 (default), then show the displayBlock. if $x == 1, and $y != 1, then show the display block. If $x == 1 && $y == 1, then do not show the displayBlock.

Answer 1

if (!$x || $y != 1) echo $displayBlock;