in_array not working explode

Question

enter image description here

I am not able to figure out why my condition is not working while the ip address is in the array. Why condition is failing as shown in image

<?php $valid_ip_list = explode(',',$this->valid_ips);
           echo $client_ip = $_SERVER['REMOTE_ADDR'];
           print('<pre>');
           print_r($valid_ip_list);
            if(in_array($client_ip ,$valid_ip_list))
                {
                    echo 'I am here';
                }
           else
                {
                    echo 'Condition fail';
                }
?>

Problem solved with the help of array_map('trim', explode(',', $valid_ips))

Check for white space characters in the ip list values... use var_dump() rather than echo and print_r to get more information — Mark Baker
– Mark Baker, Commented Dec 6, 2012 at 10:35
what does strlen($this->valid_ips[2]) give you? Or better, change print_r to var_dump and post the results. — Salman Arshad
– Salman Arshad, Commented Dec 6, 2012 at 10:48

Salman Arshad · Accepted Answer · 2012-12-06 11:14:41Z

3

This should help

$valid_ips = '192.100.100.61,192.100.100.2,127.0.0.1';

// authorized
if (in_array($_SERVER['REMOTE_ADDR'], array_map("trim", explode(',', $valid_ips)))) {
    //... 
}

// unauthorized
else {
    //...
}

edited Dec 6, 2012 at 11:14

Salman Arshad

274k85 gold badges450 silver badges540 bronze badges

answered Dec 6, 2012 at 10:39

maček

78k37 gold badges172 silver badges200 bronze badges

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

3 Comments

Pramod Kumar Sharma Over a year ago

ips will be dynamic every time so i can not give static

Pramod Kumar Sharma Over a year ago

is there any way to trim every element of array in explode

Salman Arshad Over a year ago

How does re-arranging the code help? It is still essentially the same.

Collectives™ on Stack Overflow

in_array not working explode

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related