Contador de visitas en php no funciona como deberia

Mazterrr · #1 (**permalink**) 02/11/2015, 18:52

Saludos amigos tengo un contador de visitas que consegui en internet pero no esta trabajando como deberia pues sigue contando las visitas al refrescar la pagina.
Aqui les dejo tal cual como lo tengo funcionando:

TENGO TRES ARCHIVOS:

pagina1.html
pagina2.php
pagina3.php

pagina1.html

Código:

<html> 
<head></head> 
<body> 
<a href="pagina2.php">1 - Creación de la tabla.</a><br> 
<a href="pagina3.php">2 - Página que cuenta la visita</a> 
</body> 
</html>

pagina 2.php

Código:

<?php 
$conexion=mysql_connect("localhost","MI_USUARIO","MI_CLAVE") or 
  die("Problemas en la conexion"); 

mysql_select_db("MI_BASE_DE_DATOS",$conexion) or 
  die("Problemas en la selección de la base de datos"); 

$registros=mysql_query("drop table if exists visitas",$conexion) or 
  die("Problemas en el select:".mysql_error()); 

$registros=mysql_query("create table visitas ( 
                            ip varchar(12), 
                            fecha date, 
                            hora time 
                         )",$conexion) or 
  die("Problemas en el select:".mysql_error()); 

header("Location: pagina1.html"); 
?>

pagina3.php

Código:

<html> 
<head></head> 
<body> 

<?php 
$conexion=mysql_connect("localhost","MI_USUARIO","MI_CLAVE") or 
  die("Problemas en la conexion"); 

mysql_select_db("MI_BASE_DE_DATOS",$conexion) or 
  die("Problemas en la selección de la base de datos"); 

$ip=$_SERVER['REMOTE_ADDR']; 
$fecha=date("Y-m-d"); 
$hora=date("H:i:s"); 
$registro=mysql_query("select * from visitas where ip='$ip' and fecha='$fecha'",$conexion) or  
  die("Problemas en el select:".mysql_error()); 
if ($reg=mysql_fetch_array($registro)) 
{ 
  echo "Usted ya visitó hoy día el sitio.<br>"; 
} 
else 
{ 
  $registros=mysql_query("insert into visitas(ip,fecha,hora) values    ('$ip','$fecha','$hora')",$conexion) 
     or die("Problemas en el select:".mysql_error()); 
} 
$registro=mysql_query("select count(*) as cantidad from visitas",$conexion) or 
  die("Problemas en el select:".mysql_error()); 
$reg=mysql_fetch_array($registro); 
echo "Cantidad de visitas:".$reg['cantidad']; 
?> 
</body> 
</html>

TENGO UNA TABLA LLAMADA "visitas" que esta almacenando lo siguiente:

IP fecha hora
186.89.185.2 2015-11-03 00:34:40
186.89.185.2 2015-11-03 00:34:43
186.89.185.2 2015-11-03 00:34:45
186.89.185.2 2015-11-03 00:34:49

Como les dije cada entrada a pagina3.php esta sumando una visita sin ninguna restriccion.

xfxstudios · #2 (**permalink**) 02/11/2015, 19:03

y si lo cambias asi:

Código PHP:

Ver original<html> 
<head></head> 
<body> 
 
<?php 
$conexion=mysql_connect("localhost","MI_USUARIO","MI_CLAVE") or 
  die("Problemas en la conexion"); 
 
mysql_select_db("MI_BASE_DE_DATOS",$conexion) or 
  die("Problemas en la selección de la base de datos"); 
 
$ip=$_SERVER['REMOTE_ADDR']; 
$fecha=date("Y-m-d"); 
$hora=date("H:i:s"); 
$registro=mysql_query("select * from visitas where ip='$ip' and fecha='$fecha'",$conexion) or  
  die("Problemas en el select:".mysql_error()); 
//contamos los registros encontrados
$total = mysql_num_rows($registro);
//si existe mostramos el error
if ($total >= 1) 
{ 
  echo "Usted ya visitó hoy día el sitio.<br>"; 
}
//si no continuamos 
else 
{ 
  $registros=mysql_query("insert into visitas(ip,fecha,hora) values    ('$ip','$fecha','$hora')",$conexion) 
     or die("Problemas en el select:".mysql_error()); 
} 
$registro=mysql_query("select count(*) as cantidad from visitas",$conexion) or 
  die("Problemas en el select:".mysql_error()); 
$reg=mysql_fetch_array($registro); 
echo "Cantidad de visitas:".$reg['cantidad']; 
?> 
</body> 
</html>

otra cosa debes adaptar tu codigo a mysqli porque mysql esta sin soporte ya y puedes tener ciertos problemillas a futuro. Saludos

AlejandraLara · #3 (**permalink**) 02/11/2015, 19:13

Ignorando el hecho de que usas mysql y no alguna otra api:

Lo que pasa es que en ningún momento estás limitando que la ip solo la cuenta una vez cada cierto tiempo, antes de insertar los valores verifica que no exista la ip dentro de cierto rango de fechas. Algo así:

La query: SELECT ip FROM tabla WHERE fecha = '$hoy' AND ip = '$ip'

Código PHP:

Ver original$num = mysql_num_rows($ip);
if( $num < 1 ){
 //insertar
}else{
 //no se inserta porque ya hay un registro con la ip
 echo 'Visitado hoy';
}
 
mysql_free_result($query);

Y shalala.

Mazterrr · #4 (**permalink**) 02/11/2015, 19:24

Cita:

Iniciado por AlejandraLara

Ignorando el hecho de que usas mysql y no alguna otra api:

Lo que pasa es que en ningún momento estás limitando que la ip solo la cuenta una vez cada cierto tiempo, antes de insertar los valores verifica que no exista la ip dentro de cierto rango de fechas. Algo así:

La query: SELECT ip FROM tabla WHERE fecha = '$hoy' AND ip = '$ip'

Código PHP:

Ver original$num = mysql_num_rows($ip);
if( $num < 1 ){
 //insertar
}else{
 //no se inserta porque ya hay un registro con la ip
 echo 'Visitado hoy';
}
 
mysql_free_result($query);

Y shalala.

Que otras api se usan para este tipo de cosas? la verdad desconozco bastante del tema pero puedo buscar información al respecto.

AlejandraLara · #5 (**permalink**) 02/11/2015, 19:34

Usa mysqli, mysql está obsoleta ya desde hace tiempo :c

mortiprogramador · #6 (**permalink**) 02/11/2015, 19:51

Saludo

Con las sugerencias que te dieron, ya debe solventarse el tema.

En cuanto a mysql-mysqli.
Sí bien mysql está obsoleta, por ahora sigue funcionando,
y dudo mucho que la desactiven por ahora,
pues mysqli en temas de puertos,
aún sigue dependiendo de mysql.