[php]ip checken via function werkt niet?

[Roa]

Ey,

Ik check redelijk vaak het ip van mijn bezoekers/users, daarom leek het mij handig om een functie te maken die dat doet, dus ipv in ieder bestand dat ik het nodig heb dit:

PHP:

    //haal ip op
    if($HTTP_X_FORWARDED_FOR) 
      { 
       $ip = $HTTP_X_FORWARDED_FOR; 
      } 
      elseif($HTTP_VIA) 
      {     
       $ip = $HTTP_VIA; 
      } 
      elseif($REMOTE_ADDR) 
      { 
       $ip = $REMOTE_ADDR; 
      } 
      else 
      { 
       $ip = "unknown"; 
      }

gewoon dit:

PHP:

$session_ip = ip();

met een include waarin deze functie staat:

PHP:

funtion ip(){
    //haal ip op
    if($HTTP_X_FORWARDED_FOR) 
      { 
       $ip = $HTTP_X_FORWARDED_FOR; 
      } 
      elseif($HTTP_VIA) 
      {     
       $ip = $HTTP_VIA; 
      } 
      elseif($REMOTE_ADDR) 
      { 
       $ip = $REMOTE_ADDR; 
      } 
      else 
      { 
       $ip = "unknown"; 
      } 
return $ip;
}

Echter, als ik het dmv van een functie doe (of die nu in een aparte file staat en geinclude word of in dezelfde file, maakt niet uit) dan geeft hij ALTIJD unknown... Kan je in functions soms niet bij die informatie?? Ik wil namelijk niet die 3 dingen gaan meegeven als het niet hoeft!

dit werkt dus wel:

PHP:

function ip($ip1, $ip2, $ip3){
    //haal ip op
    if($ip1) 
      { 
       $ip = $ip1; 
      } 
      elseif($ip2) 
      {     
       $ip = $ip2; 
      } 
      elseif($ip3) 
      { 
       $ip = $ip3; 
      } 
      else 
      { 
       $ip = "unknown"; 
      } 
      return $ip;
}

echo ip($HTTP_X_FORWARDED_FOR, $HTTP_VIA, $REMOTE_ADDR);

[crisp]

1 (zoek)woord: scope

variabelen zoals $HTTP_X_FORWARDED_FOR moet je binnen een functie expliciet als global aanmerken of (zoals je zelf al ontdekt had) doorgeven als parameter.
Voor superglobals geldt dit trouwens niet (en ik zou je willen aanraden die te gaan gebruiken tenzij je host nog een echt antieke PHP versie draait)
Hier is wel genoeg over te vinden zowel hier in de FAQ als op php.net lijkt me...

[Roa]

Merci, that did the job (pffff, denk je dat je toch wel et één en ander weet van php )

[drm]

tjsa.