PHP excluding specific variable values from foreach statement

Question

I am trying to exclude fields from a form from being required. Currently the entire array payment runs through this for each statement. I am trying to keep the fields extra, extra2 and extra3 from being required. The can be set to null or defined as whatever string if empty. This is what I have currently:

  foreach($p as $key => $value) {
    $value = trim($value);
    if($value == '') {
      $keyName = preg_replace('/([A-Z])/', " $1", $key);
      $this->_errors['Payment ' . $keyName] = __('Payment ','cart66') . $keyName . __(' required','cart66');
      $this->_jqErrors[] = "payment-$key";
    }

This is what I have tried, but to no avail:

 foreach($p as $key => $value) {
    $value = trim($value);
    if($value == '' && $p != 'extra' || 'extra2' || 'extra3') {
      $keyName = preg_replace('/([A-Z])/', " $1", $key);
      $this->_errors['Payment ' . $keyName] = __('Payment ','cart66') . $keyName . __(' required','cart66');
      $this->_jqErrors[] = "payment-$key";
    }
else if ($p['extra'] == '') {
 $_p['extra'] = NULL; 
}
else if ($p['extra2'] == '') {
 $_p['extra2'] = NULL; 
}
else if ($p['extra3'] == '') {
 $_p['extra3'] = NULL; 
}

}

It's my syntax isn't it? The database itself is set to accept null and is not primary or unique.

Okonomiyaki3000 · Accepted Answer · 2012-06-05 04:06:15Z

3

One good way would just be to check at the top of the loop and continue if you're in on a field that should be excluded.

$exclude = array('field1', 'field2', ...);
foreach ($p as $key => $value)
{
    if (in_array($key, $exclude)) { continue; }

    // your code... 
}

answered Jun 5, 2012 at 4:06

Okonomiyaki3000

3,70625 silver badges23 bronze badges

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

4 Comments

Rae Wuest Over a year ago

That went the opposite way, it excluded everything but the fields I want to exclude, but I can absolutely handle it from here. Thank you very much. Is this a different approach to the array_diff Benjamin was offering?

Okonomiyaki3000 Over a year ago

Yes, he is suggesting that you actually strip out the unneeded elements from your array before looping and then (if necessary) reinsert them afterward. Probably some performance benefit to doing it that way if you have quite large arrays but for typical input arrays the difference should be negligible so use whichever gives you the most readable, maintainable code.

Luís Almeida Over a year ago

should I put the "!" in my if condition to so what I really want right?

Okonomiyaki3000 Over a year ago

@LuísAlmeida I don't know. It depends on what you want.

Collectives™ on Stack Overflow

PHP excluding specific variable values from foreach statement

1 Answer 1

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related