Counter resetet sich

[qzeta]

Hallo,
hat jemand eine Ahnung warum sich der angehängte Counter immer wieder resetet...? Passiert bei ca. 1200 Klicks auf www.gemberlin.com

[qzeta]

ups...

[Nightflyer]

Auch wenn der Code sichtlich alt und unübersichtlich ist kann ich keinen Fehler entdecken der das verursachen könnte. Vielmehr denke ich dass es mit eine sog. "race-condition" zusammenhängt. User A liest Counter aus und schreibt neuen Wert in Datei. Gleichzeitig versucht User B den Wert auszulesen was jedoch nicht geht da User A grade am Schreiben ist. Zu lösen wäre dieses Problem mit einer Sperre ( siehe flock() ).

PS: Rechte an den Lieder hast du?

[qzeta]

ja stimmt, das script ist ziemlich alt. hatte ich vor einiger zeit irgendwo im web ausgegraben. bisher hatte ich keine probleme entdeckt mit gleichzeitigem auslesen und schreiben - sonst würde doch ein "undefined" als wert ausgegeben? das problem ist eher, dass die counter.txt bereits zum zweiten mal auf 0 gesetzt wurde.
Jetzt habe ich die counter.txt in ein geschüztes verzeichnis gesteckt... vieleicht hilft das ja...? oder habe ich da was falsch verstanden?

die musik ist bei der GEMA angemeldet...

danke für die hilfe

[qzeta]

ok... scheint so als hätte ich definitiv etwas falsh verstanden.
der counter resetet sich immernoch andauernd.
habe ein neues script gefunden. hier für alle die es interessiert...

PHP-Code:

<?
function schreiben($datei,$mode) { 
   global $REMOTE_ADDR; 
   $fp = @fopen($datei,$mode); 
   flock($fp,2); 
   fwrite($fp,$REMOTE_ADDR."|".time()."\n"); 
   flock($fp,3); 
   fclose($fp); 
} 

function ip_sperre() { 
   global $REMOTE_ADDR; 
   $reloadlogdatei = "counter/ip.txt"; 
   $anzahleintraege = 50; // nach sovielen ip´s werden die alten gelöscht
   $zeitlimit = 7200; // solange bleibt eine ip gesperrt (in sec.)
    
   // prüfen ob datei existiert wenn nicht erstellen
   if(!file_exists($reloadlogdatei)) { 
      $datei = fopen($reloadlogdatei,"w+"); 
      fclose($datei); 
   } 
   $fp = file($reloadlogdatei); 
   $size = sizeof($fp); 

   // beschreibt die Datei, wenn Liste voll 
   if($size >= $anzahleintraege){ 
      schreiben($reloadlogdatei,"w"); 
      // Prüft, ob eine IP in der Liste steht 
   }
   else { 
     $ausgabe=false;
for ($i=0;$i<$size;$i++) { 
   $zeile = explode("|",$fp[$i]); 
   if ($zeile[0] == $REMOTE_ADDR AND $zeile[1] > (time()-$zeitlimit)){ 
      // ip wird gefunden 
      $ausgabe = true; 
   }
}  
      // protokolliert die ip 
      schreiben($reloadlogdatei,"a+"); 
   } 
   return $ausgabe; 
}


$fn = "counter/counter.txt";
if (file_exists($fn))
{
// counterstand auslesen
$fp = fopen($fn,"r");
$zahl = fgets($fp,10);
fclose($fp);
}
else
// beim ersten aufruf..
$zahl = 0;

// counterstand erhöhen...
echo "wert=$zahl";

if (ip_sperre()==false)
{
// neu schreiben
$zahl = $zahl+1;
$fp = fopen($fn,"w");
flock($fp,2);
fputs($fp,$zahl);
flock($fp,3);
fclose($fp);
}
?>

[Nightflyer]

Ich kann die Verwendung einer IP-Sperre nicht nachvollziehen. Die IP der meisten User wechselt ja sowieso andauernd. Sonnvoller wäre IMHO einfach ein Cookie beim Nutzer absetzen und danach zu prüfen. Hast du PHP5? Wenn ja mach ich dir schnell ein kleines Script welches funktioniert

[qzeta]

ja habe php 5.2.2 zur verfügung. wäre echt fein die sache mit einem cookie zu regeln. danke für die angebotene hilfe

[Nightflyer]

So, noch nicht geprüft ob es überhaupt funktioniert aber so in etwa sollte es in PHP5 schon aussehen

PHP-Code:

<?php

class counter{
    
    var $counter;
    var $cookiename = $_SERVER['HTTP_HOST'];
    var $counterfilename = 'counter.txt';
    
    function checkCookie(){
        
        return isset( $_COOKIE[ $this->cookiename ] );
        
    }
    
    function set_Cookie(){
        
        setcookie( $this->cookiename );
        
    }
    
    function getCounter(){
        
        return file_get_contents( $this->counterfilename );
        
    }
    
    function counterCountAndSave(){
        
        file_put_contents( $this->counterfilename, $this->counter + 1 );
        
    }
    
    function returnCounterValue(){
        
        return "&wert=".file_get_contents( $this->counterfilename );
        
    }
    
    function __construct(){
        
        if( $this->checkCookie() === false ){
            
            $this->set_Cookie();
            
        }else{
            
            return false;
            
        }
        
        $this->counter = $this->getCounter();
        $this->counterCountAndSave();
        
    }
    
}

$counter = new counter;
echo $counter->returnCounterValue();

[qzeta]

ich bin ja nicht gerade eine php-leuchte...
verstehe ich es richtig, dass dieses script ein kompletter counter ist...?
oder muß ich in meinem script die ip-sperre dadurch ersetzen.
bin ein bißchen verwirrt

[Nightflyer]

So, habe das Script noch ein bischen verfeinert und getestet. Müsste funktionieren. Ja das Script ist ein kompletter Counter - eine IP-Sperre gibt es nicht mehr stattdessen basiert es wie vorgeschlagen auf einer Cookie-Sperre.

PHP-Code:

<?php

error_reporting(E_ALL);

class counter{
    
    var $counter;
    var $cookiename;
    var $counterfilename;
    
    function checkCookie(){ //Prüfen ob Cookie vorhanden
        
        return (bool) isset( $_COOKIE[ str_replace( ".", "_", $this->cookiename ) ] );
        
    }
    
    function set_Cookie(){ //Neues Cookie setzen
        
        return (bool) setcookie( str_replace( ".", "_", $this->cookiename ), 0, time()+3600*24*7 );
        
    }
    
    function getCounter(){ //Counter auslesen
        
        $counter = file_get_contents( $this->counterfilename );
        
        if( $counter === false ){
            
            touch( $this->counterfilename );
            return 0;
            
        }
        
        return (int) $counter;
        
    }
    
    function counterCountAndSave(){ //Counter hochzählen und speichern
        
        file_put_contents( $this->counterfilename, $this->counter + 1 );
        
    }
    
    function returnCounterValue(){ //Counter zurückgeben
        
        return "&wert=".( $this->counter + 1 );
        
    }
    
    function __construct(){ 
        
        $this->cookiename = $_SERVER['HTTP_HOST'];
        $this->counterfilename = 'counter.txt';
        
        if( $this->checkCookie() === false ){
            
            if( $this->set_Cookie() === false ){
                
                die("Cookie konnte nicht gesetzt werden");
                
            }
            
        }else{
            
            return false;
            
        }
        
        $this->counter = $this->getCounter();
        $this->counterCountAndSave();
        
    }
    
}

$counter = new counter;
echo $counter->returnCounterValue();

[qzeta]

danke für das script echt wahnsinnig, wie du sowas in kürzester zeit erstellen kannst...
beim ersten aufruf der seite funktionierte es auch wunderbar...
bei jedem weiteren aufruf wird nur noch die "1" angezeigt...?
lösch ich die cookies, stimmt der wert wieder...usw...

www.gemberlin.com

[Nightflyer]

fein, dann ist ja alles in Butter

[Nightflyer]

echt wahnsinnig, wie du sowas in kürzester zeit erstellen kannst...

Nur Übung...