If statement in foreach loop not working correctly

Question

I have a foreach loop with an if statement inside it so that if $value equals 'Messages' it should increment my variable num1 by 1, but the if statement doesn't appear to be working correctly. Any help would be greatly appreciated.

EDIT: If I get rid of the if statement and just echo $value, it will echo out the value of every section in the array.

Code:

 <?php
     $num1 = 0;
     foreach($inventory['Category'] as $key => $value) {
         if ($value == 'Messages') {  
             $num1++;  
         }
     }
 ?>

Try just a var_dump($value == 'Messages') in the loop instead of the if - if it returns false when it should return true there is something wrong in the comparison. You could then try var_dump($value) to narrow down what that variable is returning. — alexteg
– alexteg, Commented Nov 8, 2011 at 18:40
Are you sure that $value will contains exactly 'Messages'? I mean, without space or anything else? Do a var_dump to double check. — Aurelio De Rosa
– Aurelio De Rosa, Commented Nov 8, 2011 at 18:40
Can you do a var_dump() on `$inventory['Category'] to verify that any values are Messages first? Otherwise the code looks fine. — Whetstone
– Whetstone, Commented Nov 8, 2011 at 18:41
I'm not sure what you're trying to do, but array_count_values might be of use to you. — lonesomeday
– lonesomeday, Commented Nov 8, 2011 at 18:44

CuriousMind · Accepted Answer · 2011-11-08 18:43:03Z

2

The only reason $num1 is not increasing could be because $value is not 'Messages'. Try echoing $value variable & see.

answered Nov 8, 2011 at 18:43

CuriousMind

34.3k29 gold badges102 silver badges142 bronze badges

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

Comments

Tim · Accepted Answer · 2011-11-08 18:49:44Z

0

Are you sure you don't want to do this:

<?php
 $num1 = 0;
 foreach($inventory as $key => $value) {
         if ($value == 'Messages') {  $num1++;  }
 }
 ?>

If you array has a key called $inventory['Messages'] this would find it. Your original sample looks for $inventory['Category']['Messages'].

answered Nov 8, 2011 at 18:49

Tim

14.4k6 gold badges42 silver badges63 bronze badges

1 Comment

nickb Over a year ago

This will find the value Messages in the array $inventory. This would not find $inventory['Messages'] - In order to do that, it would need to compare $key to 'Messages', not $value. For that reason, the OP's original sample does not look for $inventory['Category']['Messages'].

Collectives™ on Stack Overflow

If statement in foreach loop not working correctly

2 Answers 2

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related