problemas con funcion php

[matt_1985]

Lectores del foro

Estoy intentando llamar una funcion a la hora de hacer un insert into , lo hago de la
siguiente forma y no me funciona:

Código PHP:

// saber ip real 
function getIP() {
    if (isset($_SERVER['HTTP_X_FORWARDED_FOR'])) {
       $ip = $_SERVER['HTTP_X_FORWARDED_FOR'];
    }
    elseif (isset($_SERVER['HTTP_VIA'])) {
       $ip = $_SERVER['HTTP_VIA'];
    }
    elseif (isset($_SERVER['REMOTE_ADDR'])) {
       $ip = $_SERVER['REMOTE_ADDR'];
    }
    else {
       $ip = "unknown";
    }
   
    //echo "Tu IP es: $ip";
}
 
//aca hago el insert 
$ip=getIP();
$sql="INSERT INTO tabla (id,dato1,ip) VALUES ('$id','$dato1','$ip')";
 

espero puedan ayudarme .

Saludos y espero hayan pasado unas excelentes fiestas de año nuevo.

[Patriarka]

y que error te da?
y hace un echo $ip

[matt_1985]

al hacer el echo me arroja bien la ip , pero al intentar insertar el valor me aparece vacio en la BD.

gracias

[Patriarka]

y estas ejecutando la consulta?
mysql_query($sql);

[matt_1985]

Si la consulta funciona pero el problema es la funcion que no me carga el dato , estoy pasando por un proxy y no lograba conseguir la ip real con la funcion $_SERVER['REMOTE_ADDR'] .

igual pongo el resto del codigo

Código PHP:

$ip=getIP();
$sql="INSERT INTO tabla (id,dato1,ip) VALUES ('$id','$dato1','$ip')";
$resultado=mysql_query($sql,$link);
 

saludos

[Patriarka]

PERO, anda o no anda????

[matt_1985]

El dato de la funcion no me carga . ahi esta mi problema . el campo de la ip aparece vacio .

[Patriarka]

BUENO hace esto:
echo $sql;

y pega el resultado en tu phpmyadmin y fijate que error te da?

[matt_1985]

Bueno resolvi mi problema de la sigueinte forma

Código PHP:

function getIP() {
    if (isset($_SERVER['HTTP_X_FORWARDED_FOR'])) {
       $ip = $_SERVER['HTTP_X_FORWARDED_FOR'];
    }
    elseif (isset($_SERVER['HTTP_VIA'])) {
       $ip = $_SERVER['HTTP_VIA'];
    }
    elseif (isset($_SERVER['REMOTE_ADDR'])) {
       $ip = $_SERVER['REMOTE_ADDR'];
    }
    else {
       $ip = "ip desconocida";
    }
   
    //echo "Tu IP es: $ip";
    return $ip; 
}
 
 
$ip_real=getIP($ip);
 
$sql="INSERT INTO tabla (id,dato1,ip) VALUES ('$id','$dato1','$ip_real')";
$resultado=mysql_query($sql,$link);
 

bueno espero a alguien le sirva .

saludos

[Martriay]

probá al final de tu función escribir esto:

return $ip;

y decime si funciona

Igual te recomiendo que no uses una función para eso, que corra el código solito, son pocas líneas y al ser elseif uno tras otro se saltea bastante del código, y no pareciera que lo reutilices, no hace falta una función.
Son mis consejos.

[Patriarka]

$ip_real=getIP($ip);

$ip no esta definida

[Martriay]

Repito, o borrás la función y dejás el contenido, es decir, esto:

Código PHP:

  * * if (isset($_SERVER['HTTP_X_FORWARDED_FOR'])) {
* * * *$ip = $_SERVER['HTTP_X_FORWARDED_FOR'];
* * }
* * elseif (isset($_SERVER['HTTP_VIA'])) {
* * * *$ip = $_SERVER['HTTP_VIA'];
* * }
* * elseif (isset($_SERVER['REMOTE_ADDR'])) {
* * * *$ip = $_SERVER['REMOTE_ADDR'];
* * }
* * else {
* * * *$ip = "unknown";
* * }
 
$sql="INSERT INTO tabla (id,dato1,ip) VALUES ('$id','$dato1','$ip')"; 
   

o dejas la funcion, pero con un return

Código PHP:

  function getIP() {
* * if (isset($_SERVER['HTTP_X_FORWARDED_FOR'])) {
* * * *$ip = $_SERVER['HTTP_X_FORWARDED_FOR'];
* * }
* * elseif (isset($_SERVER['HTTP_VIA'])) {
* * * *$ip = $_SERVER['HTTP_VIA'];
* * }
* * elseif (isset($_SERVER['REMOTE_ADDR'])) {
* * * *$ip = $_SERVER['REMOTE_ADDR'];
* * }
* * else {
* * * *$ip = "unknown";
* * }
* *
* *return $ip;
}
*
 
$ip=getIP();
$sql="INSERT INTO tabla (id,dato1,ip) VALUES ('$id','$dato1','$ip')"; 
   

[matt_1985]

Martriay,

deje la función tal como mostre arriba en la solución de mi problema.

saludos

[Martriay]

Ah, no lo había visto

igual, te sigo recomendando que elijas una de las opciones que te dije, o cambiar esto:

Código PHP:

  $ip_real=getIP($ip); 
   

por

Código PHP:

  getIP(); 
   

así no creás una variable de más (ocupás memoria al pedo)

y la sql debería quedarte aśi:

Código PHP:

  $sql="INSERT INTO tabla (id,dato1,ip) VALUES ('$id','$dato1','$ip')"; 
   

son consejos, si te funciona te funciona, pero yo lo haría así

saludos