Script funciona cuando quiere

[lvfp]

Hola:

Tengo un contador de visitas en un página que en local funciona perfectamente, pero en el servidor funciona cuando quiere. Este es el código, supongo que tendrá muchas cosas mejorables (no soy ni mucho menos experto en php)


<?php
session_start();
$con = mysql_connect("localhost","root","");
if (!$con)
{
die('Could not connect: ' . mysql_error());
}
?>

<?php
mysql_select_db("MiDB", $con);
$result = mysql_query("SELECT numero FROM visitas WHERE contador ='contador'");
while($row = mysql_fetch_array($result))
{
$conta = $row['visitas'];
}
if(isset($_SESSION['contador'])){

}else{
$conta = $conta + 1;
mysql_query("UPDATE visitas SET visitas =" . $conta ."
WHERE contador = 'contador'");}
echo $conta ;
$_SESSION['contador'] = $conta;
mysql_close($con);
?>

Tengo una base de datos con una tabla llamada visitas con dos campos, numero y contador, numero es donde almaceno la cantidad de visitas.

Este script está incluido en el pié de la página con un include, el pie de página a su vez esta con otro include en cada página de la web.

La web es www.franbalseirofotografia.com


Gracias.

[cuasatar]

¿Estas verificando en remoto que la sesión si se esta iniciando ? ¿Que condición usas para cerrar la sesión?

if(isset($_SESSION['contador'])){
}

No entiendo para que haces esto. No me parece lógico verificar si existe una sesión para no hacer ningún tipo de acción.

$result = mysql_query("SELECT numero FROM visitas WHERE contador ='contador'");
while($row = mysql_fetch_array($result))
{
$conta = $row['visitas'];
}

Esto esta mal. Nunca se debe llamar en un fetch array a la tabla sino al campo de la tabla que se desea ver. Supongo que en número estara el número de visitas de la pagina, esto deberia ir así.

Código PHP:

  $conta = $row['numero'];  
   

Espero que por este lado puedas revisar tu script para poder mejorarlo.

[webankenovi]

<?php
session_start();
$con = mysql_connect("localhost","root","");
if (!$con)
{
die('Could not connect: ' . mysql_error());
}

mysql_select_db("MiDB", $con);

$sql= "SELECT numero FROM visitas WHERE contador= 'contador' ";
$mysql_query = mysql_query($sql) or die ();
$row = mysql_fetch_array($mysql_query);

$conta = $row['visitas'];
if($conta)
{
die('error: ' . mysql_error());
}else{

$conta = $conta + 1;
mysql_query("UPDATE visitas SET visitas = $conta WHERE contador = 'contador'");

$_SESSION['contador'] = $conta;
}

if(isset($_SESSION['contador'])){

AQUI SUPUESTAMENTE SI NO HAY SESION NO HACES NADA CREO ENTENDER

}else{

aqui se muestra lo que kieras si hay sesion

}


LA SESSION DEBES HACERLA ANTES DE EL ISSET DE SEESION te lo he puesto el el if del $conta
mysql_close($con);

?>


prueba haber y lo cuentas ademas deberias usar mysql_real_escape_strings y htmlentities

[lvfp]

Gracias por vuestras respuestas (ya os decía que no soy un experto en php) :)

Mi idea es comprobar si es la primera visita a la web en esta sesión,

si lo es, al valor de visitas acumulado hasta entonces en la DB le suma 1, lo muestra en el contador y actualiza la DB

si no lo es, no hace nada y muestra el valor de la base de datos

Hay un error en e código que puse:

mysql_query("UPDATE visitas SET visitas =" . $conta ."

lo correcto, como yo lo tengo es:

mysql_query("UPDATE numero SET visitas =" . $conta ."


Gracias y un saludo

[lvfp]

Bien, recomponiendo un poco el script me queda así:

Código PHP:

  <?php 
 
    session_start(); 
 
$con = mysql_connect("localhost","root",""); 
if (!$con) 
  { 
  die('Could not connect: ' . mysql_error()); 
  } 
?> 
 
<?php 
mysql_select_db("miDB", $con); 
$result = mysql_query("SELECT numero FROM visitas"); 
$row = mysql_fetch_array($result); 
  
  $conta = $row['visitas']; //aquí saco las visitas hasta este momento 
 
if(isset($_SESSION['contador'])){ //si esta variable estaba inicializada no hace nada 
         
}else{ //si no suma una visita y lo graba en la DB 
        $conta = $conta + 1; 
        mysql_query("UPDATE visitas SET numero =" . $conta); 
} 
 
$_SESSION['contador'] = 1;  
echo $conta; // 
mysql_close($con); 
 
?>   

Ahora funciona en todos los navegadores menos en chrome que falla. La cuestión es que he comprobado que a veces pasa por las dos ramas del IF , con lo que alguna vez actualiza sumando una nueva visita en la misma sesión.

Empiezo a ponerme de los nervios

[waldragon]

Yo la estoy viendo en IE y suma una visita por cada pagina que veo, eso es lo que quieres?

[Uncontroled_Duck]

Código PHP:

<?php
session_start();
 
//conect db
$con = mysql_connect("localhost", "root", "root");
if( !$con )
    die('Could not connect: ' . mysql_error());
?> 
 
<?php
//select db
$db = mysql_select_db("forosdelweb", $con); 
if( !$db )
    die('Not select: ' . mysql_error()); // <-- tambien puedes comprobar si la DB está seleccionada correctamente.
 
$result = mysql_query("SELECT numero FROM visitas");
$row = mysql_fetch_array($result);
 
//$conta = $row['visitas']; <-- si le pones el nombre del campo en vez del de la tabla igual te trae datos...
$conta = $row['numero']; //aquí saco las visitas hasta este momento | Ahora si!
 
if( !isset($_SESSION['contador']) ) // con añadir ! antes de isset quitamos la primera parte del if
//{
    //si esta variable estaba inicializada no hace nada 
//}
//else
{
    ////si no está definda suma una visita y lo graba en la DB 
    $conta = $conta + 1;
    mysql_query("UPDATE visitas SET numero =" . $conta);
}
 
$_SESSION['contador'] = 1;
echo $conta; // 
mysql_close($con);
?>
 

Saludos,

[lvfp]

Muchas gracias, parece que ahora funciona.

Tengo que aclarar que aunque tu comentario sarcástico de la línea 19 tiene razón el código lo tenía bien, lo que pasa es que cambié ciertos nombres para el código de ejemplo y puse mal el nombre del campo.

Y ahora que funciona el cliente me ha dicho que mejor lo sacamos :)


Saludos y gracias a todos.