[RESOLVED] Need help with detecting value

[zzz]

I have a form that draws a list of check boxes based on an array:

foreach ($myArray as $key => $value) {
echo  '<input type=checkbox name="'.$key.'" value="'.$key.'" ';
    	if (in_array($key, $unionMem)) {
				echo ' checked';
		}
echo '>'.$value;
}

After I post a form I need to detect whether any of the checkboxes were selected or not an set a variable based on that. I've tried like this but it does not seem to work properly.

foreach ($myArray as $key => $value) {

if ($_POST[$key] != '') {
	$um = 'Empty';
} else {
	$union_array[$i++] = $_POST[$key];
   	$dbValue  = implode(", ", $union_array);
	$um = 'NotEmpty';
}
}

I'm stuck...🆒

[leatherback]

It helps if you explain WHAT is not working.

in any case.. Doing an implode in the middle of the loop doesn't make sense.

[firemankurt]

Try ...

foreach ($myArray as $key => $value) {

if ($_POST[$key] ) {
    $um = 'Empty';
} else {
    $union_array[$i++] = $_POST[$key];
       $dbValue  = implode(", ", $union_array);
    $um = 'NotEmpty';
}
} 

or ...

foreach ($myArray as $key => $value) {

if (isset($_POST[$key])) {
    $um = 'Empty';
} else {
    $union_array[$i++] = $_POST[$key];
       $dbValue  = implode(", ", $union_array);
    $um = 'NotEmpty';
}
} 

[zzz]

Combining the two suggestions I came up with the following:

i = 0;
$union_array = array();

foreach ($myArray as $key => $value) { 
   $union_array[$i++] = $_POST[$key]; 
}

if (???????) { // need to know if there are any values in $union_array
   $dbValue  = implode(", ", $union_array); 
   $um = 'NotEmpty'; 
} else {        

   $um = 'Empty';
} 

So, if I could detect if there are any values in my $union_array then I can implode them for db storage and set $um value to show that there were results and if not -- show it's empty.

I think I'm on the right track but am not sure how to check array...

[leatherback]

/* please be aware that including the values of posted data straight into your queries results in serious security risks. You should NEVER trust external data, unless it is for yourself (And even then, you run the risk of messing your own data up).*/
$union_array = array();

foreach ($myArray as $key => $value) 
  {
  $union_array[] = $_POST[$key];
  }

if (count($union_array) > 0)
  {
   $dbValue  = implode(", ", $union_array);
   $um = 'NotEmpty';
  }
else 
  {        

  $dbValue = "";
   $um = 'Empty';
  }

[zzz]

I parse all my input value through a class that strips malicious code...

Unfortunatelly the suggestion did not work. My guess, it is because I run $union_array[] = $_POST[$key]; from within a nother array's loop. Whether there's a value posted or not, I get a count == 5 which is the number of the the values in $myArray.

I guess, what I need to do is detect the output of the $dbValue. Currently, whether it has values or not, it returns five commas -- the implode connectors.

So, when I select just one value, I can get ,Value2,,, which is probably not correct.

Arrays is my weakest link...

:glare:

[leatherback]

Hi,

So my suggestion DID work. But didn't do what you were hoping for, as you have a flaw in your logic 🙂

I do not think your problem is with arrays, as they are nothing but containers of variables. Your problem is with the logic.

I guess you want something like this:

$union_array = array();

foreach ($myArray as $key => $value)
  {
  if((array_key_exists($key, $_POST)) && ($_POST[$key] <> ""))
    {
    $union_array[] = $_POST[$key];
    }
  }

if (count($union_array) > 0)
  {
   $dbValue  = implode(", ", $union_array);
   $um = 'NotEmpty';
  }
else
  {        

  $dbValue = "";
   $um = 'Empty';
  } 

[zzz]

You're right! Logic was to blame.

Works perfect now!