Forum PHP.pl > licznik osob online

uczaj21

7.05.2009, 20:50:27

Prosze o pomoc mam skrypt php pokazujacy ilosc osob na stronie online oraz ilosc osob odwiedzonych strone i nr ip.Problem polega na liczniku osob online na www zawsze pokazuje 1 osobe chodz jest ich wieksza ilosc.Tylko ta opcja zle dziala inne dzialaja prawidlowo

[PHP] pobierz, plaintext 
<?php
$ip = $_SERVER['REMOTE_ADDR'];
 
function badIP($ip)
{
 $end_line = "r\n";
 if(file_exists("banned_ip.txt")){
   $fd = fopen("banned_ip.txt", "r+");
 }
 else{
   $fd = fopen("banned_ip.txt", "a+");
 }
 if(!$fd) return false;
 while (!feof ($fd)) {
   $line = trim(fgets($fd));
   if($line == $ip){
     fclose($fd);
     return true;
   }
 }
 fseek($fd, filesize("banned_ip.txt"));
 fputs($fd, $ip.$end_line);
 fclose($fd);
 return false;
}
 
function getHits()
{
 if(($fp = @fopen("counter.txt", "r+")) === false)
   return false;
 $count = fgets($fp);
 if(!badIP($_SERVER['REMOTE_ADDR'])){
   $count = $count + 1;
   fseek($fp, 0);
   fputs($fp, $count);
 }
 fclose($fp);
 return $count;
}
$minut = 1;
$teraz = time()+60*$minut;
$bazaip = file("ip.txt");
$ile = count($bazaip);
$ip = $_SERVER["REMOTE_ADDR"];
 
for ($i=0;$i<$ile;$i++) {
$pol = explode("",$bazaip[$i]);
if ($pol[2]<time()) {$bazaip[$i]="";}
if ($pol[1]==$ip) {
$bazaip[$i]="$ip$teraz <br>\n"; $jest=1;
}
}
 
if ($jest<>1) $bazaip[]="$ip $teraz\n";
$f = fopen("ip.txt","w");flock($f,2);
for ($i=0;$i<$ile+1;$i++) {
if ($bazaip[$i]<>"") { fwrite($f, $bazaip[$i]); $ileip++; }
}
flock($f,3); fclose($f);
 
echo '<font size="2"><u>Gosci na naszej stronie online : </u></font>'.'<b><font color="#099FF0">'.$ileip.'&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</b></font>';
echo('<font size="2"><u>Ta strona została odwiedzona : </u></font>'.'<font color="#099FF0"><b>'.getHits().'&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</b></font>');
echo '<font size="2"><u>&nbsp;Twoj numer ip : &nbsp;</u></font>'.'<b><font color="#099FF0">'.$ip.'&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</font></b>';
?>
[PHP] pobierz, plaintext

okitoki

7.05.2009, 21:46:04

Kod

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

function badIP($ip)
 {
 $end_line = "r\n";
 if(file_exists("banned_ip.txt"))
 $fd = fopen("banned_ip.txt", "r+");
 else
 $fd = fopen("banned_ip.txt", "a+");
 if(!$fd) return false;
 while (!feof ($fd))
 {
 $line = trim(fgets($fd));
 if($line == $ip)
 {
 fclose($fd);
 return true;
 }
 }

 fseek($fd, filesize("banned_ip.txt"));
 fputs($fd, $ip.$end_line);
 fclose($fd);
 return false;
 }

function getHits()
 {
 if(($fp = fopen("counter.txt", "r+")) === false)
 return false;
 $count = fgets($fp);
 if(!badIP($_SERVER['REMOTE_ADDR']))
 {
 $count = $count + 1;
 fseek($fp, 0);
 fputs($fp, $count);
 }
 fclose($fp);
 return $count;
 }

$minut = 1;
$teraz = time()+60*$minut;
$bazaip = file("ip.txt");
$jest=0;
$ileip=0;

for ($i=0;$i<count($bazaip);$i++)
{
 if(eregi("([0-9.]{1,})( )([0-9]{1,})",$bazaip[$i],$pol))
 {
 if ($pol[3]<time()) $bazaip[$i]="";
 if ($pol[1]==$ip)
 {
 $bazaip[$i]="$ip $teraz\n";
 $jest=1;
 }
 }
}

if ($jest<>1) $bazaip[]="$ip $teraz\n";
$f = fopen("ip.txt","w");flock($f,2);
for ($i=0;$i<count($bazaip);$i++)
 if ($bazaip[$i]<>"")
 {
 fwrite($f, $bazaip[$i]);
 $ileip++;
 }
flock($f,3); fclose($f);

echo 'Gosci na naszej stronie online : '.''.$ileip.'      ';
echo('Ta strona została odwiedzona : '.''.getHits().'      ');
echo ' Twoj numer ip :  '.''.$ip.'      ';
?>

uczaj21

8.05.2009, 20:01:12

Dzieki serdecznie działa a czy mógł byś wyjaśnić te zmiany w kodzie

okitoki

8.05.2009, 20:09:14

praktycznie nie wiele zmodyfikowałem twój kod, funkcja eregi powoduje szczegółowe rozbicie zmiennej $bazaip[$i] na IP i time. co powoduje że nie dostają się tam żadne inne znaki typu enter, ma to znaczenie potem w porównaniu

uczaj21

10.05.2009, 07:54:20

zauważyłem że po zmodifikowaniu kodu licznik odwiedzin po odswiezaniu strony nabija kolejne wartosci nie jest odporny na odswiezanie a byl odporny

okitoki

10.05.2009, 11:45:56

bo masz w kodzie dużo błędów, i to nie moich

Kod

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

function badIP($ip)
{
$end_line = "\r\n"; // tu miałes blad

!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 if(file_exists("banned_ip.txt"))
 $fd = fopen("banned_ip.txt", "r+");
 else
 $fd = fopen("banned_ip.txt", "a+");
 if(!$fd) return false;
 while (!feof ($fd))
 {
 $line = trim(fgets($fd));
 if($line == $ip)
 {
 fclose($fd);
 return true;
 }
 }

 fseek($fd, filesize("banned_ip.txt"));
 fputs($fd, $ip.$end_line);
 fclose($fd);
 return false;
 }

function getHits()
 {
 if(($fp = fopen("counter.txt", "r+")) === false)
 return false;
 $count = fgets($fp);
 if(!badIP($_SERVER['REMOTE_ADDR']))
 {
 $count = $count + 1;
 fseek($fp, 0);
 fputs($fp, $count);
 }
 fclose($fp);
 return $count;
 }

$minut = 1;
$teraz = time()+60*$minut;
$bazaip = file("ip.txt");
$jest=0;
$ileip=0;

for ($i=0;$i<count($bazaip);$i++)
{
 if(eregi("([0-9.]{1,})( )([0-9]{1,})",$bazaip[$i],$pol))
 {
 if ($pol[3]<time()) $bazaip[$i]="";
 if ($pol[1]==$ip)
 {
 $bazaip[$i]="$ip $teraz\n";
 $jest=1;
 }
 }
}

if ($jest<>1) $bazaip[]="$ip $teraz\n";
$f = fopen("ip.txt","w");flock($f,2);
for ($i=0;$i<count($bazaip);$i++)
 if ($bazaip[$i]<>"")
 {
 fwrite($f, $bazaip[$i]);
 $ileip++;
 }
flock($f,3); fclose($f);

echo 'Gosci na naszej stronie online : '.''.$ileip.'      ';
echo('Ta strona została odwiedzona : '.''.getHits().'      ');
echo ' Twoj numer ip :  '.''.$ip.'      ';
?>

uczaj21

10.05.2009, 14:58:06

jeśli są jakieś błedy to prosze o zniwelowanie ich ogolnie skrypt to ściągłem z internetu a nie jestem az na takim poziomie aby dojrzec i usunac bledy zalezy mi na tym aby skrypcik dzialal prawidlowo.Dzisiaj zauwazylem jak bylem na uczelni i wszedłem na stronke gdzie zainstalowalem skrypt komp postawiony byl na linuksie i co zauwazylem ze licznik online wcale nie dzialal a licznik odwiedzin odswiezal sie

przyszedłem do domu i z ciekawosci sprawdzilem u siebie na win xp i co dziwne licznik online dziala i licznik odwiedzin rowniez

okitoki

10.05.2009, 19:17:11

no cóż, tych błędów ja nie zrobiłem, one już są w skrypcie który tu wstawiłeś, a problem był na linuxie, bo jest lepiej zabespieczony niż windows,

uczaj21

10.05.2009, 19:39:38

rozumiem ze to nie jest pana wina z tymi bledami lecz prosze o dokonanie zmian aby usunac te bledy Z gory dziekuje

okitoki

10.05.2009, 19:46:53

ostatni skrypt który ci wysłałem ma już poprawione te błędy

uczaj21

10.05.2009, 21:30:21

ok dzieki przetestuje skrypt i sie odezwie z gory dziekuje

phpion

11.05.2009, 08:35:38

Cytat(uczaj21 @ 10.05.2009, 20:39:38 )

rozumiem ze to nie jest pana wina z tymi bledami lecz prosze o dokonanie zmian aby usunac te bledy Z gory dziekuje

<ot>
Zabrzmiało to jak jakiś zleceniodawca, który namierzył błędy po poprzednim programiście i prosi aktualnego o ich poprawienie...
</ot>

okitoki

11.05.2009, 12:55:23

wiem że mój czas i wiedza jest cenna

. no cóż czasami trzeba też innym pomóc

uczaj21

19.05.2009, 20:27:33

Witam ponownie po testowaniu skryptu zauwazylem kolejny mankament tyczacy sie licznika odwiedzin.Zauwazylem iz kiedy wejde na stronke to licznik doda kolejna wartosc lecz tylko raz jak raz zapisze sobie nr ip w pliku banned_ip.txt to juz na zawsze a powinno z tego pliku kasowac sie nr ip po jakims czasie a tu sie nie kasuje i jak juz wejde np na drugi dzien to i tak licznik nie naliczy tego?

phpion

19.05.2009, 20:31:26

Cytat(uczaj21 @ 19.05.2009, 21:27:33 )

kiedy wejde na stronke to licznik doda kolejna wartosc lecz tylko raz jak raz zapisze sobie nr ip w pliku banned_ip.txt to juz na zawsze a powinno z tego pliku kasowac sie nr ip po jakims czasie a tu sie nie kasuje i jak juz wejde np na drugi dzien to i tak licznik nie naliczy tego?

Masz ewidentne problemy z przekazaniem własnych myśli.

Jeżeli już znalazłeś błąd to czy próbowałeś go wyeliminować? Wydaje mi się, że czekasz aż ktoś zrobi to za ciebie... takie coś nazywa się zleceniem prac.

nieraczek

19.05.2009, 20:34:55

O nie - znowu uczaj21 słynacy z tego, że czeka aż ktoś mu coś zrobi po czym nic już więcej w danym temacie nie napisze - tak jak zawsze.......

okitoki

19.05.2009, 21:17:17

Cytat(uczaj21 @ 19.05.2009, 21:27:33 )

Witam ponownie po testowaniu skryptu zauwazylem kolejny mankament tyczacy sie licznika odwiedzin.Zauwazylem iz kiedy wejde na stronke to licznik doda kolejna wartosc lecz tylko raz jak raz zapisze sobie nr ip w pliku banned_ip.txt to juz na zawsze a powinno z tego pliku kasowac sie nr ip po jakims czasie a tu sie nie kasuje i jak juz wejde np na drugi dzien to i tak licznik nie naliczy tego?

tak jest skonstruowana część tego skryptu zapisująca do pliku ip.txt, zerknij na to może się uda przerobić powiększając czas z minyty na dobe