strano errore su contatore visite

[sevenjeak]

Salve utenti di AV,

Ho appena creato un contatore visite che memorizza le visite su un foglio di testo e una sessione che impedisce allo stesso visitatore di effettuare la stessa visita piu di una volta, il codice è questo:

      <?php
         $visit = "Counter.txt";
         if (!isset($_SESSION["visit"])) {
            $_SESSION["visit"] = 1;
            $visit_open = fopen($visit, "r+");
            $visit_read = fread($visit_open, filesize("Counter.txt"));
            fseek($visit_open, 0);
            $visit_write = fwrite($visit_open, $visit_read + 1);
            fclose($visit_open);
         }
            $visit1_open = fopen($visit, "r");
            $visit1_read = fread($visit1_open, filesize("Counter.txt"));
            echo $visit1_read;
            fclose($visit1_open);
      ?>

Lo script funziona fino a quando conta 9 visite, se le conta 10 lo script mi restituisce 1, per poi rifunzionare quando le visite sono 11.

di cosa si tratta?

Sinceramente non ho provato lo script, ma non mi piace come struttura.
Fai in modo che intanto apra il file interessato con una WR (non r+), poi fai l'if e scegli se fare un fread o un fwrite.

Ciao

[dreadnaut]

Il codice sembra corretto, benche' contorto, quindi e' probabile che ci siamo dimenticati di una qualche conversione automatica del php, che fa sparire lo 0. Che accade quando arrivi a 19-20-21?

In compenso, potresti semplificare assai il codice usando file_get_contents() e se usi php5 anche file_put_contents(). E.g.

$visit = "Counter.txt";

$numero_visite = file_get_contents($visit);

if (!isset($_SESSION["visit"])) {
  ++$numero_visite;
  $_SESSION["visit"] = true;

  // In php5 il blocco seguente potrebbe essere sostituito da un semplice
  // file_put_contents($visit, $numero_visite);

  if ($visit_open = fopen($visit, "w")) {
    fwrite($visit_open, $numero_visite);
    fclose($visit_open);
  }
} 

[ageofempiresgif]

Apparte che non si capisce perchè crei una variabile con dentro il nome del file eppoi riscrivi il nome del file testualmente :P ...
Comunque ho trovato l'errore!

Provate ad aggiungere un bell'

echo "Filesize: ".filesize("Counter.txt")."<br>"; 

dopo la chiusura del file aperto con "r+" e vedete la sorpresina... continua a leggere 1 carattere anche se ne contiene 2! ...fino alla successiva riesecuzione della pagina. Quindi ciò che dice svacant è giusto, però bisogna capire il perchè! Good Game ^^

Usa lo standard come fanno tutti i cristiani:

            $visit_open = fopen($visit, "r");
            $visit_read = fread($visit_open, filesize("Counter.txt"));
            fclose($visit_open);
            $visit_open = fopen($visit, "w");
            $visit_write = fwrite($visit_open, $visit_read + 1);
            fclose($visit_open); 

[sevenjeak]

Ok, grazie, ho modificato lo script cosi:

      <?php
         $visit = "Counter.txt";
         $numero_visite = file_get_contents($visit);
         if (!isset($_SESSION["visit"])) {
            $_SESSION["visit"] = 1;
            if ($visit_open = fopen($visit, "w")) {
               fwrite($visit_open, ++$numero_visite);
            }
         }
         echo $numero_visite;
      ?>

[Xsescott]

se usi le sessioni ricordati di avviarle con session_start();

[sevenjeak]

Grazie, cmq avevo inserito sulla mia pagina gia la sessio_start() con la session_register() visto che inizzializzo $_SESSION["visit"] a 1, ma ora mi hai fatto venire un dubbio, la session_register() serve o ne posso anche fare a meno?

[mycarlo]

Ma session_register() è deprecata ormai, non trovi sia più semplice utilizzare l'array $_SESSION?

[sevenjeak]

quello infatti lo usato, volevo solamente sapere, io ho scritto questo codice:

session_start();
session_regiser();
...
$_SESSION["visit"] = 1;
... 

Funziona se non metto session_register()?

[andreafallico]

Si, anche se non metti session_register funziona.