That's kind of eww-y code as it is. Here's a cleaned up version:
<?php
/**
* @return string
* @desc Searches for the enduser's IP-Address
*/
function get_ipaddr() {
if (getenv("HTTP_CLIENT_IP") && strcasecmp(getenv("HTTP_CLIENT_IP"), "unknown"))
$ip = getenv("HTTP_CLIENT_IP");
else if (getenv("HTTP_X_FORWARDED_FOR") && strcasecmp(getenv("HTTP_X_FORWARDED_FOR"), "unknown"))
$ip = getenv("HTTP_X_FORWARDED_FOR");
else if (getenv("REMOTE_ADDR") && strcasecmp(getenv("REMOTE_ADDR"), "unknown"))
$ip = getenv("REMOTE_ADDR");
else if (isset($_SERVER['REMOTE_ADDR']) && $_SERVER['REMOTE_ADDR'] && strcasecmp($_SERVER['REMOTE_ADDR'], "unknown"))
$ip = $_SERVER['REMOTE_ADDR'];
else
$ip = "unknown";
return $ip;
}
$ip = get_ipaddr();
$b_ip = array(
'127.0.0.1',
'64.233.167.99'
);
foreach($b_ip as $v) {
if($v == $ip)
header('location: www.google.com');
}
?>By the way, the problem with your code is that
127.0.0.1 is a string. You didn't enclose it in quotes. Also -- PHP is pretty leniant on it's syntaxing, but I'm pretty sure it requires you to enclose
if statements in parenthensies. Plus, you used "Then" and "{" looks like you were trying to merge VB6 and PHP.