PHP foreach with conditional

Question

<? foreach ($cases as $case) :?>
<? if (isset($case['case_status']) && ($case['case_status'] == 'Incomplete')) :?>
<div>
    <div class="alert alert-error">    
        <?= $warning ?>
    </div>

I'm trying to have that foreach print out the div only once if the condition is met. The problem is, for every case that meets the condition the div is printed.

I have tried using in_array, but I don't think I understand the syntax. Is in_array correct? Is there a better way to do this?

Variables are free. Use one to remember if you have already printed the warning. — Jon
– Jon, Commented Aug 12, 2013 at 22:49
Add <?php break; ?> at the end of (but inside) the if to stop the foreach. — Tomas Creemers
– Tomas Creemers, Commented Aug 12, 2013 at 22:53

user4035 · Accepted Answer · 2013-08-12 22:54:43Z

1

You can use a flag to mark, that you have already printed the div and add an extra condition to if:

<? 
$flag = false;
foreach ($cases as $case) :?>
<? if (isset($case['case_status']) 
       && ($case['case_status'] == 'Incomplete')
       && ($flag == false)) :?>
<div>
    <div class="alert alert-error">    
        <?= $warning ?>
    </div>
    <?php $flag = true; ?>

Or if you don't need to do anything in that loop, you can just break it after the first encountered element.

<? 
foreach ($cases as $case) :?>
<? if (isset($case['case_status']) && ($case['case_status'] == 'Incomplete')) :?>
<div>
    <div class="alert alert-error">    
        <?= $warning ?>
    </div>
    <?php break; ?>

answered Aug 12, 2013 at 22:54

user4035

24k11 gold badges69 silver badges101 bronze badges

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

4 Comments

Maximus2012 Over a year ago

won't this prevent further legitimate errors from printing once the flag has been set to true?

user4035 Over a year ago

@Maximus2012 The PO said, that he doesn't want to print this block. The flag will do it. The other conditions will stay unmodified. If he doesn't use this foreach for anything else, except for finding, if the element is present in the array, break will the best solution.

Tomas Creemers Over a year ago

@Maximus2012: that is what is asked: "I'm trying to have that foreach print out the div only once if the condition is met."

Maximus2012 Over a year ago

oh...ok. Thanks for the clarification. I guess I missed that part of the original question.

Skaparate · Accepted Answer · 2013-08-12 23:02:11Z

1

<?php
  $printed = array();
  foreach ($cases as $case) :?>
<? if (!in_array( $case['case_status'] ) && isset($case['case_status']) && ($case['case_status'] == 'Incomplete')) : ?>
<div>
    <div class="alert alert-error">    
        <?= $warning ?>
    </div>
<?php
    $printed[] = $case['case_status'];
  endif;
?>

This way you are saving every printed case_status and comparing it with in_array.

edited Aug 12, 2013 at 23:02

answered Aug 12, 2013 at 22:56

Skaparate

4793 silver badges14 bronze badges

Collectives™ on Stack Overflow

PHP foreach with conditional

2 Answers 2

4 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

4 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related