Problema con num_rows

[Fumaka0o]

Estoy teniendo problemas a la hora de crear un sistema de seguimiento de visitas.

el caso es que en el If que depende del resultado de $comprueba, siempre mete resultado en la base de datos pero como registro nuevo, no actualiza si ya existe. porfavor, kien tenga un momento y pueda echarle un ojo se lo agradeceria. estoy un poco perdido ya que no se como ponerlo, he intentado varios metodos pero ninguno me funciona como quisiera.

<?php

include("includes/conexion/conecta.php");
$resultados = mysql_query("SELECT ip FROM `visitas` WHERE ip='$ip'");
$comprueba = mysql_num_rows($resultados,$conecta);
if ($comprueba != 0){
$actualiza_visitas = "UPDATE `visitas` SET fecha='$fecha' contador='$contador' ultima_url='$ultima_url' WHERE idsession='$idsession'";
mysql_query($actualiza_visitas,$conecta);
}else{
$crea_visita = "INSERT INTO `visitas` (id, idsession, ip, fecha, contador, primera_url, ultima_url, proviene) VALUES ('', '$idsession', '$ip', '$fecha', '$contador', '$primera_url', '$ultima_url', '$proviene')";
mysql_query($crea_visita,$conecta);
}
include("includes/conexion/cierra_conecta.php");
?>


muchas gracias por vuestro tiempo y apoyo

[masterojitos]

haber tus error es el siguiente:

esta asi:
$actualiza_visitas = "UPDATE `visitas` SET fecha='$fecha' contador='$contador' ultima_url='$ultima_url' WHERE idsession='$idsession'";

le faltan comas, dejalo asi:
$actualiza_visitas = "UPDATE `visitas` SET fecha='$fecha', contador='$contador', ultima_url='$ultima_url' WHERE idsession='$idsession'";

Y NADA MAS, ya que no se que viene en las variables.....

a cierto, mejor tambien as asi:

$comprueba = mysql_num_rows($resultados);
if ($comprueba >= 0){

es decir, ya no le mandes la variable de conexion... ya no necesario.

Suerte.

[Fumaka0o]

Muchas gracias por tu rapidez de respuesta, he probado con lo que dices pero sigo en las mismas. incluso ahora no arroja ningun error pero tampoco graba en la base de datos. me estoy volviendo loco con esto y seguro que es una tonteria.

$resultados = mysql_query("SELECT ip FROM `visitas` WHERE ip='$ip'");
$comprueba = mysql_num_rows($resultados);
if ($comprueba >= 0){
$actualiza_visitas = "UPDATE `visitas` SET fecha='$fecha', contador='$contador', ultima_url='$ultima_url' WHERE idsession='$idsession'";
mysql_query($actualiza_visitas);
}
else{
$crea_visita = "INSERT INTO `visitas` (id, idsession, ip, fecha, contador, primera_url, ultima_url, proviene) VALUES ('', '$idsession', '$ip', '$fecha', '$contador', '$primera_url', '$ultima_url', '$proviene')";
mysql_query($crea_visita);
}

voy a ducharme seguramente luego lo vea desde otro punto de vista.

[masterojitos]

ps la verdad, ayi las sentencias ahora si estan bien....

ya solo queda depender de las variables que usas, quizas un tipo de campo sea incorrecto o algo asi....

lo mas optimo es hacer:
mysql_query($actualiza_visitas) or die(mysql_error());

o has un hecho de las cadenas sql para que veas que trae.

Suerte.

[fodsite]

¿Cómo obtienes los valores de las variables?

Ten cuidado porque generalmente en un servidor Windows puedes recibir variables solo usando '$variable', independiente que sea POST o GET.

En un Servidor Web Linux, por defecto debes indicar la forma de recibir la variable:

'$_GET[variable]'
'$_POST[variable]'

Lo anterior depende del metodo con que envies tu form.

Saludos.

[Keysher]

Para matizar el último comentario, no depende del sistema operativo, sino de la directiva Register_Globals, configurable en el php.ini y que se aconseja que esté desactivada para no acceder directamente a variables de los arrays superglobales.

Con respecto al error, comprueba que la consulta SQL esté correcta, que IP tenga el valor correcto.

[Fumaka0o]

gracias a los dos pero... no funciona :-s

ya no se que hacer, aqui os dejo el codigo al completo y si podeis decirme algo os lo agradeceria, ahora mismo ni muestra error ni guarda en BD, solo muestra los datos que se supone que tendria que guardar

if(isset($_SESSION['contador'])) {
$contador = $_SESSION['contador']++;
$contador = $_SESSION['contador'];
} else {
$_SESSION['contador'] = 1;
$contador = $_SESSION['contador'];
}

if (getenv(HTTP_X_FORWARDED_FOR)) {
$ip = getenv(HTTP_X_FORWARDED_FOR);
} else {
$ip = getenv(REMOTE_ADDR);
}

$fecha=date("F j, Y, g:i a");

$primera_url = "$_SERVER[REQUEST_URI]";

$ultima_url = "$_SERVER[REQUEST_URI]";

$proviene = getenv("HTTP_REFERER");


////////estas lineas son de comprobacion, para ver los datos que voy a meter////////
echo "$ip<br>";
echo "$contador<br>";
$idsession = session_id();
echo "$idsession<br>";
echo "$primera_url<br>$ultima_url<br>$proviene";
////////////////////////////////////////////////////////////////////////////////////

include("includes/conexion/conecta.php");

$resultados = mysql_query("SELECT ip FROM visitas WHERE idsession='$idsession'") or die(mysql_eror());
$comprueba = mysql_num_rows($resultados) or die (mysql_error());

if ($comprueba >= 0){

$actualiza_visitas = "UPDATE visitas SET fecha='$fecha', contador='$contador', ultima_url='$ultima_url' WHERE idsession=".'$idsession';
mysql_query($actualiza_visitas) or die(mysql_eror());

}else{

$crea_visita = "INSERT INTO visitas (id, idsession, ip, fecha, contador, primera_url, ultima_url, proviene) VALUES ('', '$idsession', '$ip', '$fecha', '$contador', '$primera_url', '$ultima_url', '$proviene')";
mysql_query($crea_visita) or die(mysql_eror());

}

mysql_free_result($comprueba);

include("includes/conexion/cierra_conecta.php");

Si lo veis necesario os cuelgo tambien la estructura de la tabla.

saludos y muchas gracias a todos por vuestro tiempo

[masterojitos]

chamo pero tu codigo que enredado que es..... pero vale quer aqui te va lo que puedo apoyarte.....

Código PHP:

  if(isset($_SESSION['contador'])) { 
$_SESSION['contador']++; 
$contador = $_SESSION['contador']; 
} else { 
$contador = 1; 
$_SESSION['contador'] = $contador; 
} 
 
$ip = $_SERVER['REMOTE_ADDR']; 
 
$fecha=date("F j, Y, g:i a"); 
 
$primera_url = "$_SERVER[REQUEST_URI]";//???? 
 
$ultima_url = "$_SERVER[REQUEST_URI]";//???? 
 
$proviene = getenv("HTTP_REFERER");//???? 
 
 
////////estas lineas son de comprobacion, para ver los datos que voy a meter//////// 
echo "$ip<br>"; 
echo "$contador<br>"; 
$idsession = session_id(); 
echo "$idsession<br>"; 
echo "$primera_url<br>$ultima_url<br>$proviene"; 
//////////////////////////////////////////////////////////////////////////////////// 
 
include("includes/conexion/conecta.php"); 
 
$resultados = mysql_query("SELECT ip FROM visitas WHERE idsession='$idsession'") or die(mysql_error());//es error no eror 
$comprueba = mysql_num_rows($resultados);//aqui no va "or die" 
 
if ($comprueba >= 0){ 
 
$actualiza_visitas = "UPDATE visitas SET fecha='$fecha', contador='$contador', ultima_url='$ultima_url' WHERE idsession='$idsession'";//has bien la sentencia 
mysql_query($actualiza_visitas) or die(mysql_error());//es error no eror 
 
}else{ 
 
$crea_visita = "INSERT INTO visitas (id, idsession, ip, fecha, contador, primera_url, ultima_url, proviene) VALUES ('', '$idsession', '$ip', '$fecha', '$contador', '$primera_url', '$ultima_url', '$proviene')"; 
mysql_query($crea_visita) or die(mysql_error());//es error no eror 
 
} 
 
//mysql_free_result($comprueba);//esto es para un query no para un mysql_num_rows 
 
include("includes/conexion/cierra_conecta.php");  
   

Bueno, ese es mi aporte...... espero te sirva

SUERTE.

[Fumaka0o]

Despues de seguir urgando he conseguido hacer que funcione, aqui dejo las sentencias para quien lo quiera consultar.

include("includes/conexion/conecta.php");

$resultados = mysql_query("SELECT * FROM visitas WHERE idsession='$idsession'") or die(mysql_error());

if (mysql_num_rows($resultados) <= 0){

$crea_visita = "INSERT INTO visitas (id, idsession, ip, fecha, contador, primera_url, ultima_url, proviene) VALUES ('', '$idsession', '$ip', '$fecha', '$contador', '$primera_url', '$ultima_url', '$proviene')";
mysql_query($crea_visita) or die(mysql_error());

}else{

$actualiza_visitas = "UPDATE visitas SET fecha='$fecha', contador='$contador', ultima_url='$ultima_url' WHERE idsession='$idsession'";
mysql_query($actualiza_visitas) or die(mysql_error());

}

mysql_free_result($resultados);

include("includes/conexion/cierra_conecta.php");

este hilo puede cerrarse

[masterojitos]

ehhhh y que cambiaste ah????
que donde estaba tu error????

que no te sirvieron los otros post o no sabes agradecer????

en fin.........

[Fumaka0o]

perdona, con la emocion se me ha ido, no te me ralles chewaca ;)

gracias por vuestras ayudas

deberia haber mas gente como vosotros en el mundo