Php check if 5 variables are the same value

Question

In php I need to check if 5 variables are the same value, and then proceed.

Instead of doing

if($a == 'logged' && $b == 'logged' && $c == 'logged' && $d == 'logged' && $e == 'logged') 
{ 
   // Code to run here
}

Is there a different way to do it? Like:

if($a,$b,$c,$d == 'logged')

Something like that possible..?

not sure but could $a == $b == $c == $d == $e == 'logged' this work — kpp
– kpp, Commented Apr 30, 2014 at 12:00
@kpp: Parse error: syntax error, unexpected '==' (T_IS_EQUAL) — Andresch Serj
– Andresch Serj, Commented Apr 30, 2014 at 12:02
@kpp before writing it as a comment, you could test it yourself and see... — Digital Chris
– Digital Chris, Commented Apr 30, 2014 at 12:06
if (($a == $b) == ($c == "logged")) { echo "yup"; } this worked for me — Ivan Yonkov
– Ivan Yonkov, Commented Apr 30, 2014 at 12:07

Jérôme · Accepted Answer · 2014-04-30 12:03:13Z

9

You can use something like this

$values = array($a, $b, $c, $d, 'logged');

if(count(array_unique($values)) === 1) {
    //All elements are the same
}

answered Apr 30, 2014 at 12:03

Jérôme

2921 silver badge6 bronze badges

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

12 Comments

arkascha Over a year ago

This is elegant, but much less efficient and hard to read / understand / debug. But a really interesting idea!

Ivan Yonkov Over a year ago

@arkascha this could be wrapped into helper function with proper name, so the coder will know what it's doing :)

arkascha Over a year ago

@RoyalBg Yep, that is true indeed. But: that makes this even less efficient, besides having to create an array on-the-fly then also a function call has to be done.

Ivan Yonkov Over a year ago

@arkascha not true at all: function compareValues($value) { return count(array_unique(func_get_args()))==1; } so you can pass $a = 'asd'; $b = 'asd'; $c = 'asd'; var_dump(compareValues('asd', $a, $b, $c));

arkascha Over a year ago

Sorry, I don't get your point here. That version is different from your answer above, but is has similar problems. Just that now you don't create an array and call a function, now you call four functions, which is even less efficient! Function calls are really expensive.

|

arkascha · Accepted Answer · 2014-04-30 12:04:12Z

2

I recommend against trying to shorten that. There is no more compact notation available to my knowledge apart from optical effects, none that the php parser accepts. The efficiency would not be better, since internally the same comparisons have to be done anyway.

Instead I usually try to enhance the readability of the code instead and use reversed notation, since human time is much more expensive than computer power these days :-)

if (   'logged' == $a 
    && 'logged' == $b
    && 'logged' == $c 
    && 'logged' == $d 
    && 'logged' == $e )  { 
  // Code
}

answered Apr 30, 2014 at 12:04

arkascha

43.2k8 gold badges61 silver badges97 bronze badges

1 Comment

Ivan Yonkov Over a year ago

Although this might be good advice, it's really not an answer of the question, so it should not be posted as an answer

Collectives™ on Stack Overflow

Php check if 5 variables are the same value

2 Answers 2

12 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

12 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related