getting values from complex array

tito

I have array(5 10 15, 10 25 35, 18 65 44)
each value holds multiple subvalues
i need to do a comparison for each subvalue within a value.
like
if 5 > 90
or if 10 > 90
or if 15 > 90

then move to the next value and do the same

is this possible?

LordShryku

[man]Explode[/man] your array values to get subvalues into a new array.

foreach($array as $value) {
   $sub = explode(' ', $value);
   foreach($sub as $number) {
      # check condition met here
   }
}

tito

That works perfect, thanks.

Now, I am trying to get the count of all the results, so I did the below, but that returns the wrong count.

foreach($array as $value) { 
   $sub = explode(' ', $value); 

$count=0;
$count=count($sub);
$count++;



   foreach($sub as $number) { 
      # check condition met here 
   } 
}

LordShryku

You want to initialize your count outside of your first loop. Otherwise you're resetting it with each iteration. Also, using $count++, you're adding one to it, and that isn't neccesary.

$count = 0;
foreach($array as $value) {
   $sub    = explode(' ', $value);
   $count += count($sub);
   foreach($sub as $number) {
      # check condition met here
   }
}

tito

The echo returns different values every time, in the form of 123 or 2345

$count1 = 0;
foreach($_POST['predminutes1'] as $value1) {
   $sub1 = explode(' ', $value1);
$count1 += count($sub1);

echo $count1;

   foreach($sub1 as $number1) {

if ($number1 >'90'){

eval(print_standard_error('error_invalidminutes'));
}
   }
}

tito

I think that I am getting messed up by the explode delimiter. Some values in the array do no have it as they are single.

I input them as 1 2 3
but if the value is only 1 there are no spaces.

tito

I finally got it by doing $count++;
But that leaves me another problem: the delimiter. If there are no spaces, then that value is appended with the next one in the count and both are counted as 1

$count1 = 0;
foreach($_POST['predminutes1'] as $key=>$value1) {

   $sub1 = explode(' ', $value1);
	$count1 = count($sub1);
	$count1++;
	echo $count1;

LordShryku

Maybe I'm confused as to what you're looking for. If you want a count of all the results, then you don't want to echo until you are done with your loop. By echoing inside of the loop, you're just echoing the count of each subarray.

Also, with explode, if it does not encounter it's delimiter in the string, it creates the array with 1 element. Which is all that you would want to evaluate, and should keep the count correct.

What am I missing?

Here's the test I just tried. Counted fine for me...

<?php
$array = array('5 10 15', '10 25 35', '18 65 44', '5', '12 14', '18');
# counts          3           3           3        1      2      1  = 13 total

$count = 0;
foreach($array as $value) {
   $sub    = explode(' ', $value);
   print_r($sub); # debugging: show the array
   $count += count($sub);
   foreach($sub as $number) {
      # check condition met here
   }
}

echo $count." total values\n";
?>

Results:

aaron@junkiexl <(hacks)> # php phpb.php
Array
(
    [0] => 5
    [1] => 10
    [2] => 15
)
Array
(
    [0] => 10
    [1] => 25
    [2] => 35
)
Array
(
    [0] => 18
    [1] => 65
    [2] => 44
)
Array
(
    [0] => 5
)
Array
(
    [0] => 12
    [1] => 14
)
Array
(
    [0] => 18
)
13 total values
aaron@junkiexl <(hacks)> #

tito

ok. What's happening it is counting the 0 values. I understand about the echo and the loop, but if I don't do that, the error message takes over before it displays, so I only look for the final value of the loop.

In the below example, the count should have been 3

$count1 = 0;
foreach($_POST['predminutes1'] as $value1) {
   $sub1 = explode(' ', $value1);
print_r($sub1); # debugging: show the array
$count1 += count($sub1);
echo $count1." total values\n";

   foreach($sub1 as $number1) {

if ($number1 >'90'){

eval(print_standard_error('error_invalidminutes'));
}
   }
}

I get this

Array (
[0] => 45
[1] => 90 ) 2 total values
Array (
[0] => ) 3 total values
Array (
[0] => ) 4 total values
Array (
[0] => 200 ) 5 total values

tito

yep, it is counting the zero values. I just removed from the loop

Array ( [0] => 45 [1] => 90 ) Array ( [0] => 15 ) Array ( [0] => 10 ) Array ( [0] => 200 ) Array ( [0] => 16 ) Array ( [0] => ) 7 total values

LordShryku

Ah, ok, so you're error printout seems to be a cold stop. You should echo in that condition then. Also, considering you're working with null values, if you don't want them counted, you'll have to check them.

$count = 0;
foreach($_POST['predminutes1'] as $value) {
   $sub = explode(' ', $value);
   foreach($sub as $number) {
      # check value to add to count
      if(!empty($number))
         $count++;

  if ($number > 90) {
     echo $count." total values\n";
     eval(print_standard_error('error_invalidminutes'));
  }
   }
}
# This will show if the condition is never met
echo $count." total values\n";

That'll echo the count up until the point that the condition is met. In the event that the condition is never met, you would echo after the loop

tito

Hey we are getting somewhere. It echo the right value when outside the loop (if condition not met). I see if before the redirect, but first echo if condition is met is still no good. I guess I will make two arrays.

I am doing this for a soccer prediction where people have to put the score, then the minutes scored. And I am checking first that xtimes minutes input is not greater than score input and am also checking for people not put more than 90 as value for minutes. (if overtime they have to tick another box)

Thanks for help.

tito

I need some more help please. I've got all of the above under control. I just have a last validation to do.

I got two arrays that are using the same value for $key which is the $playerid

I would like to know the proper syntax to use for the second array
it does not work when I do $k1 ! =$key1 even when I keep them in the same loop.

if ($k1=='$key from first array'){

#First array
if (!empty($_POST['predminutes1'])){
$count1 = 0;
foreach($_POST['predminutes1'] as $key1=>$gvalue1) {

   $gsub1 = explode(' ', $gvalue1);
   foreach($gsub1 as $gnumber1) {
      if(!empty($gnumber1))
        $count1++; 
if ($count1 > $predscore1) {

     eval(print_standard_error('error_invalidscorers1'));
}

  }

   }

}

#second array

if (!empty($_POST['predplayer1'])){
foreach ($_POST['predplayer1'] as $k1=>$v1) {
if ($k1 != '$key from first array'){
eval(print_standard_error('error_invalidpenalty1'));
}
}
}

Array one is a textbox where I put minutes scored and, with your help, validated it against total goals.

Array two is the penalties that I want to make sure that there is a minutes value
already in checkbox. Otherwise, someone will put correct amount of minutes for correct amount of goals and break loose with the penalties ending up with team 1 having a score of 2 with 2 values of minutes in checkboxes and 12 values of penalties goals. So I must stop that.

LordShryku

There's a few ways you can go about this. Now, if I understand right, any key present in predminutes1 should also be present in predplayer1 (Or is that the other way around)? You could do the check within the first loop (suggested), or you could do it by stripping array keys later.

Within the first loop, using [man]array_key_exists[/man]:

# change first conditional to make sure both arrays are present
if(!empty($_POST['predminutes1']) && !empty($_POST['predplayer1'])) {
   $count1 = 0;
   foreach($_POST['predminutes1'] as $key1 => $gvalue1) {
      $gsub1 = explode(' ', $gvalue1);
      foreach($gsub1 as $gnumber1) {
         if(!empty($gnumber1))
            $count1++;

     if($count1 > $predscore1)
        eval(print_standard_error('error_invalidscorers1'));

     # check here for key presence and error if it's not there
     if(!array_key_exists($key1, $_POST['predplayer1']))
        eval(print_standard_error('error_invalidpenalty1'));
  }
   }
}

Or, outside the loop using [man]array_keys[/man] and [man]array_diff[/man]:

if(!empty($_POST['predminutes1'])) {
   $count1 = 0;
   foreach($_POST['predminutes1'] as $key1 => $gvalue1) {
      $gsub1 = explode(' ', $gvalue1);
      foreach($gsub1 as $gnumber1) {
         if(!empty($gnumber1))
            $count1++;

     if($count1 > $predscore1)
        eval(print_standard_error('error_invalidscorers1'));
  }
   }
}

$keys1 = array_keys($_POST['predminutes1']);
$keys2 = array_keys($_POST['predplayer1']); # assuming you've checked to make sure this is present
$diff  = array_diff($keys2, $keys1);
if(count($diff) > 0)
   eval(print_standard_error('error_invalidpenalty1'));

The first way would probably be faster since you'd be doing the check within the original loop and saving work later on, but it checks that keys in predminutes1 are also present in predplayer1. The second way checks that keys in predplayer1 are in predminutes1.

(Sorry if this is confusing. It's early and my lack of coffee might make me hard to understand for a few hours)

tito

That's the other way around.

While the values in textbox (predminutes) need not to have checkbox checked (predplayer), if checkbox predplayer is checked, then there must be a value in checkbox predminutes.

LordShryku

The second way should work for you then...

tito

No it does not

if (!empty($_POST['predminutes1'])){
$count1 = 0;
foreach($_POST['predminutes1'] as $key1=>$gvalue1) {

   $gsub1 = explode(' ', $gvalue1);
   foreach($gsub1 as $gnumber1) {
      # check value to add to count
      if(!empty($gnumber1))
        $count1++; 
if ($count1 > $predscore1) {

     eval(print_standard_error('error_invalidscorers'));


}

  }

   }

}

if (!empty($_POST['predplayer1'])){
$keys1 = array_keys($_POST['predminutes1']);
$keys2 = array_keys($_POST['predplayer1']); # assuming you've checked to make sure this is present
$diff  = array_diff($keys2, $keys1);

if(count($diff) > 0)
{
            eval(print_standard_error('error_invalidpenalty1'));
}

}

tito

if all checkboxes are checked, regardless minutes is empty it goes thru

when no checkbox is checked I get this error

Warning: array_keys(): The first argument should be an array in /interschedule.php on line 550

Warning: array_diff(): Argument #1 is not an array in /interschedule.php on line 551

tito

Interesting. Here is what I got. First line is $key1, second line is $key2 and third line is $diff

Array ( [0] => 0000000031 [1] => 0000000032 [2] => 0000000033 [3] => 0000000034 [4] => 0000000035 [5] => 0000000036 ) 
Array ( [0] => 0000000036 ) 
Array ( [0] => 0000000031 [1] => 0000000032 [2] => 0000000033 [3] => 0000000034 [4] => 0000000035 )

code I used:

$keys1 = array_keys($_POST['predminutes1']); 
$keys2 = array_keys($_POST['predplayer1']); # assuming you've checked to make sure this is present 
$diff  = array_diff($keys1, $keys2); 
print_r($keys1);
echo "<br>";
print_r($keys2);
echo "<br>";
print_r($diff);
if(count($diff) > 0) {
   eval(print_standard_error('error_invalidpenalty1'));
}

Problem I've got $keys1 for the minutes is posting all 5 textboxes, even if some of them has no values, whereas $key2 posts only the values that are checked in checkbox. Therefore, count will always be greater than 0

tito

And I think this is the reason. I was getting an error when sending in empty values in for predminutes, so I did:

$preddata1 = serialize((is_array($_POST['predminutes1']) ? $_POST['predminutes1']: array()));

and looks like that is inserting also the empty values in database

a:6:{s:10:"0000000031";s:0:"";s:10:"0000000032";s:2:"40";s:10:"0000000033";s:2:"12";s:10:"0000000034";s:0:"";s:10:"0000000035";s:0:"";s:10:"0000000036";s:2:"28";}