Besucherzähler - irgendwo werden Umbrüche eingefügt

The User

Guten Mittag.

Ich habe in PHP einen Besucherzähler geschreiben, und wenn eine IP in den letzten 30 Minuten schon da war, wird eine Zahl angehoben, das ganze jedoch nicht als neuer Besuch gewertet.
Nun gibt es aber ein Problem: vor dem neuen Eintrag wird immer ein Zeilenumbruch eingefügt. Ich habe stundenlang über dem Code gebrütet und die Stelle nicht gefunden.
Könnt ihr mir helfen?

Code

<?
// datum, zeit und ip herausfinden
$IP = $_SERVER['REMOTE_ADDR'];
$datei = file("pfad");
$timestamp = time();
$d2 = $datei[2]; // hier steht, wie viele Einträge da sind
$min = (date("H",$timestamp) * 60 + (date("i",$timestamp)));
$day = date("Y",$timestamp);
$datum = date("Y.m.d",$timestamp);
$i = $d2 + 3; // hinterste Zeile
$b = 0; //speichert, ob die IP gefunden wurde
$zwischen = array();
do
{
	$i--;
	$zwischen = explode("|",$datei[$i]);
	if(!($i > 0 && $zwischen[0] == $datum && $min - $zwischen[1] <= 30 )) {break;} //Abbruch wenn Eintrag von Gestern oder mehr als 30minuten her
	if($zwischen[2] == $IP)
	{
		$b = 1;
	}
}
while($b == 0); //wird versucht ip übereinstimmung zu finden
if($b == 1) //bei IP-Übereinstimmung
{
	$zeile = $zwischen[0] . "|" . $min . "|" . $zwischen[2] . "|" . ($zwischen[3] + 1) . "|" . $zwischen[4];
//neue Uhrzeit wird eingefügt sowie der Zähler um eins erhöht
	$datei[$i] = $zeile;
	$datei[$d2 + 3] = "*/";
	$datei[$d2 + 4] = "?>";
	$datei[2] = $d2;
	$datei[0] = "<?";
	$datei[1] = "/*";
}
else //ohne Übereinstimmung
{
// neuer Eintrag wird generiert
	$zwischen = array($datum,$min,$IP,1);
	$zeile = $datum . "|" . $min . "|" . $IP . "|" . 1 . "|" . $min;
	$datei[$d2 + 3] = $zeile;
	$datei[$d2 + 4] = "*/";
	$datei[$d2 + 5] = "?>";
	$datei[2] = $d2 + 1;
	$datei[0] = "<?";
	$datei[1] = "/*";
}
$datei3 = "";
//Array wird übernommen
for($i = 0; $i < $d2+6; $i++)
{
	$datei3 .= $datei[$i]."\n";
}
// Eintragung in die Datei
$datei2 = fopen("pfad","w");
fwrite($datei2,$datei3);
fclose($datei2);
?>

Alles anzeigen

Ich finde die Stelle einfach nicht, wo der zusätzliche Umbruch eingeschleust wird.

Findet jemand den Fehler?

Ich würde mich sehr über Hilfe freuen, da blickt man ja nicht sofort durch.

Danke im Voraus
The User

K.Y.L.T.

Kleiner Rat : Kommentiere das nächste mal das Script durch, durch diese kurzen Variabeln kann sich ja keiner mehr durchfinden.

Mein Tipp liegt in disem Codebereich:
for($i = 0; $i < $d2+6; $i++)
{
$datei3 .= $datei[$i]."\n";
}
versuch doch mal das umzubauen:
for($i = 0; $i < $d2+6; $i++)
{
if($datei[$i] != "")
$datei3 .= $datei[$i]."\n";
}
oder so und dann nochmal auszuprobieren

phpfan

Was mich interessieren würde, wäre der genaue Aufbau der Datei. Im Moment würde ich mal behaupten, dass das Script viel zu kompliziert aufgebaut ist und man das locker vereinfachen könnte. Da man aber weder die Datei kennt, noch bei den Variablen wirklich auf die Schnelle durch den Quellcode steigt, habe ich keine Lust, mir da mehr Gedanken darüber zu machen.

The User

Das klappt leider nicht.
Selbst wenn ich das Extremmittel ereg_replace("\n","",$datei[$i]); in die for-Schleife einfüge.

Ich werde verückt!

The User

phpfan

Jau, mit Kanonen auf Spatzen schießen. Wozu ereg_replace und nicht trim? Und bitte mein Posting von13:21 Uhr beachten.

The User

Die Datei?
<?
/*
zahl der einträge
datum|minute|ip|startminute
...
*/
?>

Viele liebe Grüße
The User

phpfan

Frage: Was sollen die<?, /* */ und ?> in der Datei? Warum arbeitest du nicht einfach mit Timestamps?

The User

Die sind nur, damit mans nicht einfach so auslesen kann.
Mit trim werden immer noch die Umbrüche eingefügt.

Aber der Fehler muss frühestens bei der for-Schleife liegen.
Ich hab mir mal die Arrays ausgeben lassen.

Es klappt wirklich alles, es kommen nur immer diese Umbrüche.

Viele liebe Grüße
The User

phpfan

Zitat von The User

Die sind nur, damit mans nicht einfach so auslesen kann.

Was wäre daran so dramatisch, wenn man so eine Datei auslesen könnte? Man kann diese Datei auch einfach in einen geschützen Bereich ablegen und gut ist.
Auf die Schnelle zusammengebastelt und ungetestet:

PHP

<?php
   $ip = $_SERVER['REMOTE_ADDR']; 

   $filename = "counter.txt";

   $new_data = array();

   if (file_exists($filename))
   {
      $data = file($filename);

      $count = trim(array_shift($data));


      $new_data = array();

      $ip_found = false;


      foreach ($data as $line)
      {
         $line = trim($line);

         $values = explode("|", $line);

         if ($values[0] == $ip)
         {
            $ip_found = true;

            if ($values[1] < time() - 1800)
               $count++;


            $values[1]= time();

            $line = implode("|", $values);
         }

         $new_data[] = $line;
      }

      if (! $ip_found)
      {
         $new_data[] = $ip . "|" . time();
         $count++;
      }

      array_unshift($new_data, $count);
    }
    else
    {
       $new_data = array(1, $ip . "|" . time());
    }    

    $fp = @fopen($filename, "w");

    if ($fp)
    {
       fputs ($fp, implode("\n", $new_data));

       fclose($fp);
    }
    else
      echo "Datei konnte nicht zum Schreiben geöffnet werden";
?>

Alles anzeigen

The User

Joah, das hat was!

Klappt.
Wie schütze ich eigentlich ein Verzeichnis ohne htaccess.
Wenn ich googel find ich nur mit.

Danke
The User

phpfan

Zitat von The User

Wie schütze ich eigentlich ein Verzeichnis ohne htaccess

Man könnte da was mit Session basteln, das funktioniert aber dann nicht mit Textdateien. Die einfachste Lösung ist ein Verzeichnis außerhalb des Dokument-Root-Verzeichnisses. Ob das bei dir geht, kannst du daran feststellen, dass du nicht sofort im richtigen Verzeichnis bist, wenn du dich mit deinem FTP-Client einloggst.

The User

phpfan, du bist der Größte!
Hab noch nen paar Kleinigkeiten am Zähler verändert und es klappt!

Aber wie ist das mit Verzeichnisschutz?
Also bei normalen Verzeichnissen?
Hast du vielleicht nen Link?

Danke, danke, danke
The User

EDIT: Ah, ich lande in nem anderen Verzeichnis, interessiert mich aber dennoch.

phpfan

Ohne htaccess funktioniert das mit Text-Dateien nicht. Aber warum willst du diese Lösung nicht?

The User

Ne, ich frage nur aus Interesse, hab ja nichts gegen diese Lösung.
Denn Zugriffe auf Seiten, Skripts etc. zu sperren, vermag ja selbst ich.
Aber ganze Verzeichnisse haben auch etwas...
Naja, so ein externes Verzeichnis ist ja wohl doch so ziemlich die einfachste Lösung.
Aber nochmal für die Hilfe, sowohl beim Zähler als auch mit dem Verzichnisschutz.

Viele liebe Grüße
The User

Tags