2010-06-07 56 views
0

我正在尝试将一些第三方跟踪代码集成到我的网站之一,但它是抛出一些错误,并且他们的支持没有多大用处,所以我想要尝试自己修复他们的代码。最让我有固定的,但是这个功能是给我的问题:X转发 - 为PHP中导致未定义的索引

private function getXForwardedFor() 
{ 
$s =& $this; 

$xff_ips = array(); 

$headers = $s->getHTTPHeaders(); 

if ($headers['X-Forwarded-For']) { 
$xff_ips[] = $headers['X-Forwarded-For']; 
} 

if ($_SERVER['REMOTE_ADDR']) { 
$xff_ips[] = $_SERVER['REMOTE_ADDR']; 
} 

return implode(', ', $xff_ips); // will return blank if not on a web server 
} 

在我的开发环境在那里我显示所有的错误我得到:1129

Notice: Undefined index: X-Forwarded-For in /sites/webs/includes/OmnitureMeasurement.class.php on line 1129 

线路是:

if ($headers['X-Forwarded-For']) { 

如果我打印出$标题,我会得到:

Array 
(
[Host] => www.domain.com 
[User-Agent] => Mozilla/5.0 (Windows; U; Windows NT 6.1; en-GB; rv:1.9.2.3) Gecko/20100401 Firefox/3.6.3 
[Accept] => text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8 
[Accept-Language] => en-gb,en;q=0.5 
[Accept-Encoding] => gzip,deflate 
[Accept-Charset] => ISO-8859-1,utf-8;q=0.7,*;q=0.7 
[Keep-Alive] => 115 
[Connection] => keep-alive 
[Referer] => http://www10.toptable.com/ 
[Cookie] => PHPSESSID=nh9jd1ianmr4jon2rr7lo0g553; __utmb=134653559.30.10.1275901644; __utmc=134653559 
[Cache-Control] => max-age=0 
) 

我看不到X-Forwarded-For在那里,我认为是造成问题。有什么我应该添加到功能考虑到这一点?

我在Fedora

回答

6

使用PHP 5.3和Apache 2这是不是一个真正的大问题,但好还是固定。它抱怨你试图访问不存在的数组键。 (即使查询使用if该键的阵列被认为是访问。)

变化

if ($headers['X-Forwarded-For']) 
    { $xff_ips[] = $headers['X-Forwarded-For']; } 

if (array_key_exists('X-Forwarded-For', $headers)) 
    { $xff_ips[] = $headers['X-Forwarded-For']; } 
+0

和再见错误消息。非常感谢。 (只要它让我接受!) – 2010-06-07 13:22:51

0

甚至比array_key_exists更好是isset因为后者是一个语言结构(执行更快),并可用于各种变量。将它作为一个例程来检查你不确定的变量是否在尝试从它们读取之前设置。

+0

哇,为什么反对票? :o – 2010-06-07 14:11:21

+0

在测试数组时,有*是* array_key_exists对于isset的说话,但是我不能记住我现在的生活。尽管如此,这是正确的,好的一点,绝对不值得赞赏。 +1 – 2010-06-07 14:38:49

+1

@pekka:isset()返回false是键存在,但有一个空值。 array_key_exists不。但是,从5.3开始,array_key_exists不再适用于对象 – 2010-06-07 22:20:37