note 32498 modified in function.getenv by bjori

From: Date: Sat, 25 Mar 2006 10:58:19 +0000
Subject: note 32498 modified in function.getenv by bjori
References: 1  Groups: php.notes 
Request: Send a blank email to php-notes+get-106576@lists.php.net to get a copy of this message
This is just a reply to Anders Winther's fool-proof IP address fetcher. What he has is good, but uses a lot of superfluous lines. Basically, he could have done this: function getIP() { $ip; if (getenv("HTTP_CLIENT_IP")) $ip = getenv("HTTP_CLIENT_IP"); else if(getenv("HTTP_X_FORWARDED_FOR")) $ip = getenv("HTTP_X_FORWARDED_FOR"); else if(getenv("REMOTE_ADDR")) $ip = getenv("REMOTE_ADDR"); else $ip = "UNKNOWN"; return $ip; } Do you see how those numerous lines of code compressed into a few lines? --was-- This is just a reply to Anders Winther's fool-proof IP address fetcher. What he has is good, but uses a lot of superfluous lines. Basically, he could have done this: function getIP() { $ip; if (getenv("HTTP_CLIENT_IP")) $ip = getenv("HTTP_CLIENT_IP"); else if(getenv("HTTP_X_FORWARDED_FOR)) $ip = getenv("HTTP_X_FORWARDED_FOR"); else if(getenv("REMOTE_ADDR")) $ip = getenv("REMOTE_ADDR"); else $ip = "UNKNOWN"; return $ip; } Do you see how those numerous lines of code compressed into a few lines? http://php.net/manual/en/function.getenv.php

« previous php.notes (#106576) next »