PHP foreach with more condition

Question

Is it possible to set foreach function more than 2 condition. Example like below:

$stat_cps = $_POST['stat_cp'];
$acc_idss = $_POST['acc_ids'];

$string = 'NG';
foreach ($stat_cps as $url)
{
     if(strpos($string, $url) !== FALSE)
     {
          echo "One of the field is NG";
          return true;
     }
}

echo "All field is OK";

What I want is:
if $stat_cps or $acc_idss contains NG then echo "One of the field is NG";

On the above code, it's only working for $stat_cps

*stat_cps and acc_idss is from radio button form.

Anyone can give the suggestion?

$newarray= array_merge($stat_cps, $acc_idss); now foreach on $newarray — Dimag Kharab
– Dimag Kharab, Commented Feb 26, 2016 at 6:59
what value come this varibales $stat_cps and $acc_idss ? — Vadivel S
– Vadivel S, Commented Feb 26, 2016 at 7:00

RomanPerekhrest · Accepted Answer · 2016-02-26 07:17:31Z

2

"One-line" solution with array_merge, implode and strpos functions:

...
$hasNg = strpos("NG", implode(",", array_merge($stat_cps,$acc_idss)));
...

// check for 'NG' occurance
echo ($hasNg !== false)? "One of the field is NG" : "string 'NG' doesn't exists within passed data";

answered Feb 26, 2016 at 7:17

RomanPerekhrest

93.1k4 gold badges75 silver badges112 bronze badges

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

1 Comment

TommyBs Over a year ago

nice concise solution, that's what I like about this site, you always see things you didn't think of it combining various features of a language

TommyBs · Accepted Answer · 2016-02-26 07:13:09Z

1

You can use array_merge to combine both your arrays:

http://php.net/manual/en/function.array-merge.php

$stat_cps = $_POST['stat_cp'];
$acc_idss = $_POST['acc_ids'];
$merged = array_merge($stat_cps,$acc_idss);
$string = 'NG';
foreach ($merged as $url)
{
     if(strpos($string, $url) !== FALSE)
     {
          echo "One of the field is NG";
          return true;
     }
}

This will remove duplicate string keys but it doesn't look like that should be an issue here if you just want one match

From you comment try doing

 $stat_cps = $_POST['stat_cp'];
$acc_idss = $_POST['acc_ids'];
$merged = array_merge($stat_cps,$acc_idss);
$match = false;
$string = 'NG';
foreach ($merged as $url)
{
     if(strpos($string, $url) !== FALSE)
     {
          $match =  true;
     }
}

if($match){
       echo "One of the field is NG";
}else{
      echo "Everything is OK";
}

edited Feb 26, 2016 at 7:13

answered Feb 26, 2016 at 6:58

TommyBs

9,7484 gold badges37 silver badges74 bronze badges

5 Comments

HiDayurie Dave Over a year ago

Hi... when I put echo "All field is OK"; outside foreach it always give result OK even though one of the field contains NG.

TommyBs Over a year ago

If it's outside the foreach then it will always display regardless as it's not applied to that logic

TommyBs Over a year ago

From your comment if you just want a match (your return true is doing nothing) try using the second code example that sets a flag based on the existence (called $match)

HiDayurie Dave Over a year ago

How about if I have database query there? Actually what I want is when the 2 string (with multiple values from radio button) have NG status, then it will update the database table NG, but if All OK then OK

TommyBs Over a year ago

Then add your mysql query to the if($match){} statement where it echoes "One of the field is NG" I'm sorry and I don't want to seem rude but if you're struggling to write a MySQL query that's a separate question. I'm not going to write everything for you as you should make an attempt yourself otherwise it just feels like you're getting someone else to do your work.

dhruv jadia · Accepted Answer · 2016-02-26 07:03:12Z

0

$stat_cps = $_POST['stat_cp'];
$acc_idss = $_POST['acc_ids'];

$process=0;
$string = 'NG';
foreach ($stat_cps as $url)
{
     if(strpos($string, $url) !== FALSE)
     {
          $process=1;
     }
}
foreach ($acc_idss as $url2)
{
     if(strpos($string, $url2) !== FALSE)
     {
          $process=1;
     }
}
if($process==1){
  echo "One of the field is NG";
  return true;
}

answered Feb 26, 2016 at 7:03

dhruv jadia

1,6802 gold badges15 silver badges28 bronze badges

6 Comments

dhruv jadia Over a year ago

@HiDayurie Dave : check this

HiDayurie Dave Over a year ago

Hi, actually in that condition I have database query to save the field. If using 2 foreach like your code I think I need to have 2 database query. Is it right?

dhruv jadia Over a year ago

no need to take 2 database query.U can use single query

HiDayurie Dave Over a year ago

Example, if NG then save the query to be NG, if OK then save the query to be OK.

dhruv jadia Over a year ago

u can also set condition in one query.No need to perform 2 queries.

|

Mohd Zubair Khan · Accepted Answer · 2016-02-26 07:04:33Z

0

#header  user in_array()
    $stat_cps = $_POST['stat_cp'];
    $acc_idss = $_POST['acc_ids'];

    $string = 'NG';
    foreach ($stat_cps as $url)
    {
         if(in_array($string, $url) !== FALSE)
         {
              echo "One of the field is NG";
              return true;
         }
    }

answered Feb 26, 2016 at 7:04

Mohd Zubair Khan

2823 silver badges17 bronze badges

1 Comment

RomanPerekhrest Over a year ago

this doesn't solve the initial problem "... more than 2 condition"

Collectives™ on Stack Overflow

PHP foreach with more condition

4 Answers 4

1 Comment

5 Comments

6 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

5 Comments

6 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related