Is there a better way?

[cooldude832]

I'm trying to come up with a list of banned numbers so to say that looks like this below

<?php
$badnums = array(1,2,4,5,7,8,23);
$num = $_SESSION['num'];
if($num != $badnums){
//Good
}
else{
//bad
}
?>

I know you can't do that so what I thought of was something else like

<?php
$badnums = array(1,2,4,5,7,8,23);
$num = $_SESSION['num'];
$badnums = implode("&&".$num."!= ".,$badnums);
if($num != $badnums){
//Good
}
else{
//bad
}
?>

Does that make sense? I think it would work but is there an easier way?

[teng84]

not clear? can you explain a bit and base on that code using in_array i guess is better

[cooldude832]

that would about do it

 

basically what i'm doing except i'm manually creating the additional comparison strings

 

[cooldude832]

it be

<?php
$badnums = array(1,2,4,5,7,8,23);
$num = $_SESSION['num'];
$badnums = implode("&&".$num."!= ".,$badnums);
if(!in_array($num,$badnums)){
//Good
}
else{
//bad
}
?>

[Aureole]

<?php
$banned= array(1, 3, 5, 7);
$num = $_SESSION['num'];

if (in_array($num, $banned))
{
echo "You are Bant.";
} else {
?>

Your HTML here.

<?php
}
?>

 

??

[teng84]

ya thats better that using explode or looping to reach the condition you need