Forum PHP.pl > [php]Licznik odwiedzin

Pomoc - Szukaj - Użytkownicy - Kalendarz

redelek

21.02.2008, 09:43:19

Witam,

Zrobiłem sobie taki licznik odwiedzin,

[PHP] pobierz, plaintext 
<?php
require_once('browser.php');
 
$db = mysql_connect("localhost", "user","pass");
mysql_select_db("licznik",$db);
$wynik = mysql_query ("SELECT * FROM licznik_odw");
 
 
for ($count = 0; $row = mysql_fetch_row ($wynik); ++$count)
{}
 
$now = getdate();
$month = $now[mon];
$mday = $now[mday];
$year = $now[year];
$hours = $now[hours];
$min = $now[minutes];
$sec = $now[seconds];
$id = $count++;
$ip = "$REMOTE_ADDR";
$br = new Browser;
$sys=$br->Platform;
$przeg=$br->Name.' '.$br->Version;
$time = "$year-$month-$mday $hours:$min:$sec";
 
for ($i=0; $i < strlen($id); $i++)
		{
				$sign = substr($id, $i, 1);
				print("<img src=\"image/$sign.gif\" border=\"0\" alt=\"$sign\">");
		}
//print "$id";
$wynik = mysql_query ("SELECT ip FROM licznik_odw ");
list($ip_db) = mysql_fetch_array($wynik);
//echo "<br>".$ip_db;
//echo "<br>".$ip;
If ($ip == $ip_db ){
	  echo "<br><b>IP już odnotowany :</b> ".$ip_db."";
	  }else {
$counter = "INSERT INTO licznik_odw (ID, przegladarka, IP, system, time) VALUES ('$id','$przeg','$ip','$sys','$time')";
$wynik = mysql_query($counter);
mysql_close($db);
}
?>
[PHP] pobierz, plaintext

Problem jest w tym , że jak ktoś się łączy to ma nadal wyświetlaną starą cyferkę dopiero jak odświeży stronę cyfra się zmienia i pojawia się komunikat że jego IP jest już w bazie. Macie może pomysł gdzie zrobiłem błąd że tak się dzieje ?

Pozdrawiam
Redelek

nospor

21.02.2008, 09:45:18

No ale zastanow sie przez chwile...
najpierw wyswietlasz stan licznikow a dopiero potem wkladasz. No to sie nie dziw ze stan licznikow przed wlozeniem jest inny.

ps:

[PHP] pobierz, plaintext 
<?php
for ($count = 0; $row = mysql_fetch_row ($wynik); ++$count)
{}
?>
[PHP] pobierz, plaintext

a czemu to ma niby sluzyc?

redelek

21.02.2008, 10:12:16

Cytat(nospor @ 21.02.2008, 09:45:18 )

No ale zastanow sie przez chwile...
najpierw wyswietlasz stan licznikow a dopiero potem wkladasz. No to sie nie dziw ze stan licznikow przed wlozeniem jest inny.

ps:

[PHP] pobierz, plaintext 
<?php
for ($count = 0; $row = mysql_fetch_row ($wynik); ++$count)
{}
?>
[PHP] pobierz, plaintext

a czemu to ma niby sluzyc?

A bo mi nie wyświetlał licznika i kolega mi podpowiedział że to powinno pomóc

Czyli mówisz że najpierw wyświetlić dane a potem dodać czy źle Cię zrozumiałem?

nospor

21.02.2008, 10:15:32

Cytat

Czyli mówisz że najpierw wyświetlić dane a potem dodać

No to przeciez na odwrot! Najpierw dodac, potem wyswietlic. Mowilem przeciez: "zastanow sie przez chwile"

Jak moze wyswietlic sie cos, skoro to dodajesz dopiero po wyswietleniu?

redelek

21.02.2008, 11:29:37

Cytat(nospor @ 21.02.2008, 10:15:32 )

No to przeciez na odwrot! Najpierw dodac, potem wyswietlic. Mowilem przeciez: "zastanow sie przez chwile"

Jak moze wyswietlic sie cos, skoro to dodajesz dopiero po wyswietleniu?

Kurcze nie mogę sobie poradzić, problem chyba jest w tym moim zabezpieczeniu które ma uniemożliwić zwiększanie licznika jeśli jest ten sam IP.
Jak to wyłączę to licznik pięknie śmiga , ale za to liczny odświeżanie strony:(.

Patrzę jak sroka w gnat i nie widzę tego

rozjaśnijcie proszę

Redelek

nospor

21.02.2008, 11:35:06

bo źle podchodzisz do sprawdzania czy jest.

Sprawdzac masz tak:

[SQL] pobierz, plaintext 
SELECT ip FROM tabela WHERE ip='twojeszukaneip'
[SQL] pobierz, plaintext

Jesli to zapytanie zwroci ci rekord, znaczy ze ip jest juz w bazie i masz wyswietlic komunikat: JESTES
Jesli nie zwroci ci rekordu, masz wowczas dodac i wyswietlic: DODALEM CIE DO BAZY. JAK WEJDZIESZ NASTEPNYM RAZEM TO ZOBACZYSZ "JESTES"

redelek

21.02.2008, 11:48:24

Cytat(nospor @ 21.02.2008, 11:35:06 )

bo źle podchodzisz do sprawdzania czy jest.

Sprawdzac masz tak:

[SQL] pobierz, plaintext 
SELECT ip FROM tabela WHERE ip='twojeszukaneip'
[SQL] pobierz, plaintext

No to ja mam to tu

[PHP] pobierz, plaintext 
<?php $wynik = mysql_query ("SELECT ip FROM licznik_odw");
list($ip_db) = mysql_fetch_array($wynik);
//echo "<br>".$ip_db;
If ($ip == $ip_db ){
	  echo "<br><b>IP już odnotowany :</b> ".$ip_db."";
	  }else {
$counter = "INSERT INTO licznik_odw (ID, przegladarka, IP, system, time) VALUES ('$id','$przeg','$ip','$sys','$time')";
$wynik = mysql_query($counter);?>
[PHP] pobierz, plaintext

Jeśli jest to nie zmienia , ale jeśli nie ma to też nie zwiększa cyferki dopiero za drugim razem ? źle myślę ?

nospor

21.02.2008, 11:59:22

no nie. ty masz inaczej. naprawde nie widzisz roznicy miedzy moim zapytaniem a twoim? Moje pobiera maksymalnie jeden rekord i to ten rekord ktorego szukamy. Twoje leci po wszystkich rekordach i dopiero w petli sprawdza czy jest czy nie ma. Ale zle to zrobiles to masz kwiatki. A ze mi sie nie chce ci tlumaczyc na podstawie twojego zlego kodu, to podalem ci inne (lepszze) rozwiązanie

redelek

21.02.2008, 12:14:30

Cytat(nospor @ 21.02.2008, 11:59:22 )

Wiem , że kombinuję, ale inaczej się nie nauczę. Nie chcę gotowca , nie liczę na to. Chcę skromnej pomocy która mnie naprowadzi w moim projekcie i zakręceniu

Poprawiłem na Twoje zapytanie hmmm ale jest to samo z czym walczę

Redelek

nospor

21.02.2008, 12:22:02

Cytat

Poprawiłem na Twoje zapytanie hmmm

Miej litosc... pokaz nowy kod ktory napisales...

redelek

21.02.2008, 12:29:47

Cytat(nospor @ 21.02.2008, 12:22:02 )

Miej litosc... pokaz nowy kod ktory napisales...

Wstydzę się bo będziesz krzyczała zaraz

[PHP] pobierz, plaintext 
<?php
$wynik = mysql_query ("select ip from licznik_odw where ip='$ip'");
list($ip_db) = mysql_fetch_array($wynik);
echo $ip_db;
If ($ip_db == $ip ){
	  echo "<br><b>IP już odnotowany :</b> ".$ip_db."";
	  }else {
$counter = "INSERT INTO licznik_odw (ID, przegladarka, IP, system, time) VALUES ('$id','$przeg','$ip','$sys','$time')";
$wynik = mysql_query($counter);
?>
[PHP] pobierz, plaintext

nospor

21.02.2008, 12:34:32

Cytat

Wstydzę się bo będziesz krzyczała zaraz

Nie taki nospor straszny jak go maluja

No i git. Zakladam ze zmienna $ip wypelniasz wczesniej? Jesli tak to co ci teraz nie dziala?

Wyswietlaj tez ewewntualne bledy:

[PHP] pobierz, plaintext 
<?php
$wynik = mysql_query ("select ip from licznik_odw where ip='$ip'") or die(mysql_error());
?>
[PHP] pobierz, plaintext

I tak z kazdym mysql_query()

edit: zmien ten kod na:

[PHP] pobierz, plaintext 
<?php
$wynik = mysql_query ("select ip from licznik_odw where ip='$ip'") or die(mysql_error());
 
 
If (mysql_num_rows($wynik) > 0 ){
	  echo "<br><b>IP już odnotowany :</b> ".$ip_db."";
	  }else {
$counter = "INSERT INTO licznik_odw (ID, przegladarka, IP, system, time) VALUES ('$id','$przeg','$ip','$sys','$time')";
$wynik = mysql_query($counter) or die(mysql_error());
}
?>
[PHP] pobierz, plaintext

redelek

21.02.2008, 12:42:55

Całość wygląda tak i jest nadal to samo:(

[PHP] pobierz, plaintext 
<?php 
 
require_once('browser.php');
 
$db = mysql_connect("localhost", "predel","niuniek2");
mysql_select_db("licznik",$db);
$wynik = mysql_query ("SELECT * FROM licznik_odw");
 
 
for ($count = 1; $row = mysql_fetch_row ($wynik); ++$count)
{}
 
$now = getdate();
$month = $now[mon];
$mday = $now[mday];
$year = $now[year];
$hours = $now[hours];
$min = $now[minutes];
$sec = $now[seconds];
$id = $count++;
$ip = "$REMOTE_ADDR";
$br = new Browser;
$sys=$br->Platform;
$przeg=$br->Name.' '.$br->Version;
$time = "$year-$month-$mday $hours:$min:$sec";
 
for ($i=0; $i < strlen($id); $i++)
		{
				$sign = substr($id, $i, 1);
				print("<img src=\"image/$sign.gif\" border=\"0\" alt=\"$sign\">");
		}
//print "$id";
 
$wynik = mysql_query ("select ip from licznik_odw where ip='$ip'") or die(mysql_error());
 
 
If (mysql_num_rows($wynik) > 0 ){
	  echo "<br><b>IP już odnotowany :</b> ".$ip_db."";
	  }else {
$counter = "INSERT INTO licznik_odw (ID, przegladarka, IP, system, time) VALUES ('$id','$przeg','$ip','$sys','$time')";
$wynik = mysql_query($counter) or die(mysql_error());
}
?>
[PHP] pobierz, plaintext

nospor

21.02.2008, 12:49:24

No ale te wyswietlanie tych cyferek zrob po dodaniu usera. Czyli po tym kodzie co zmieniles.

redelek

21.02.2008, 12:56:37

Cytat(nospor @ 21.02.2008, 12:49:24 )

No ale te wyswietlanie tych cyferek zrob po dodaniu usera. Czyli po tym kodzie co zmieniles.

To jest to samo

[PHP] pobierz, plaintext 
<?php
$wynik = mysql_query ("select ip from licznik_odw where ip='$ip'") or die(mysql_error());
If (mysql_num_rows($wynik) > 0 ){
	  echo "<br><b>IP już odnotowany :</b> ".$ip_db."";
	  }else {
$counter = "INSERT INTO licznik_odw (ID, przegladarka, IP, system, time) VALUES ('$id','$przeg','$ip','$sys','$time')";
$wynik = mysql_query($counter) or die(mysql_error());
 
 
}
	  for ($i=0; $i < strlen($id); $i++)
		{
				$sign = substr($id, $i, 1);
				print("<img src=\"image/$sign.gif\" border=\"0\" alt=\"$sign\">");
		}
?>
[PHP] pobierz, plaintext

nospor

21.02.2008, 13:01:15

ale musisz caly kod przeniesc na dół, włącznie z zapytaniem, włącznie z pętla co sie pytalem o nia, itd.

a tak na marginesie to liczbe rekordow tez sie liczy inaczej. Ty teraz przelatujesz po wszystkich rekordach i zwiekszasz $count o 1. tak sie nie robi. nalezy wykonac zapytanie:

[SQL] pobierz, plaintext 
SELECT count(*) FROM tabela
[SQL] pobierz, plaintext

Ono zwraca liczbe rekordow. I juz. zadnych petli ani innych cudakow co u ciebie sa

redelek

21.02.2008, 14:28:21

To zrobiłem tak i chyba działa, ale jeśli IP jest w bazie to wyświetla tylko napis a jak wyświetlić poprzedni stan licznika ?

[PHP] pobierz, plaintext 
<?php
require_once('browser.php');
 
$db = mysql_connect("localhost", "predel","niuniek2");
mysql_select_db("licznik",$db);
$wynik = mysql_query ("SELECT * FROM licznik_odw");
 
 
for ($count = 1; $row = mysql_fetch_row ($wynik); ++$count)
{}
 
$now = getdate();
$month = $now[mon];
$mday = $now[mday];
$year = $now[year];
$hours = $now[hours];
$min = $now[minutes];
$sec = $now[seconds];
$id = $count++;
$ip = "$REMOTE_ADDR";
$br = new Browser;
$sys=$br->Platform;
$przeg=$br->Name.' '.$br->Version;
$time = "$year-$month-$mday $hours:$min:$sec";
 
If (mysql_num_rows($wynik) > 0 ){
	  echo "<br><b>IP już odnotowany :</b> ".$ip."";
 
	  }else {
$counter = "INSERT INTO licznik_odw (ID, przegladarka, IP, system, time) VALUES ('$id','$przeg','$ip','$sys','$time')";
$wynik = mysql_query($counter) or die(mysql_error());
 
for ($i=0; $i < strlen($id); $i++)
		{
				$sign = substr($id, $i, 1);
				print("<img src=\"image/$sign.gif\" border=\"0\" alt=\"$sign\">");
		}
//print "$id";
 
$wynik = mysql_query ("select ip from licznik_odw where ip='$ip'") or die(mysql_error());
 
 
 
}
?>
[PHP] pobierz, plaintext

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.