in_array

seby

i have this code inside a while loop mysql query:

if(in_array($game['gid'], array($server['games'])))
{
	$games .= '<input type="checkbox" name="games[]" value="'.$game['gid'].'" checked /> ' .$game['game'];
}
else
{
	$games .= '<input type="checkbox" name="games[]" value="'.$game['gid'].'" /> ' .$game['game'];
}

i want to make the checkbox checked if its in the array of numbers.
$server['games'] contains numbers, ie: 1,2,3 etc, it only works if i manually put numbers in the array.. why is this? perhaps i am overseeing something?

Weedpacket

Is $server['games'] already an array? If that's so, then you don't need the extra array(). With that there what you're asking amounts to if($game['gid']==$server['games']), because there's only one element in the array you're looking in for $game['gid'], and that one element is the array $server['games'].

seby

$server['games'] contains a string of number seperated with commas, ie: $string = "1,2,3,4";

i haven't read too much on arrays, but i'm wondering why can't array(); take values from a string of numbers.

e.g.

array(1,2,3); // returns: Array ( [0] => 1 [1] => 2 [2] => 3 )

shouldn't it be the same as:

$string = "1,2,3";
array($string); // returns: Array ( [0] => 1,2,3 )

how come? like i said, perhaps i dont understand arrays well enough to understand.

Weedpacket

No; $string is a string, it's not an array - and who's to say where the elements begin and end - it could be on a comma, it could be on a tab, it could be on a space, it could be on a newline, it could be on anything. For that matter, you might not want the string to be broken at all. For that reason, PHP can't automatically turn a string into an array if you don't tell it where to break the string.

And you do that by using the [man]explode[/man] function.

What
$string="1,2,3";
$a = array($string);
does is (and it's the Right Thing) is set $a to be an array containing a single element - the string "1,2,3".

seby

thanx.

if (in_array($game['gid'], explode(",", $server['games'])))

that did it.