[php + mysql] licznik.... czemu nie działa ? [BŁĘDY POPRAWIONE - DZIAŁA] Opcje

[Ermes]

JUŻ NIE AKTUALNE POPRAWNY KOD NA DOLE STRONY

Chciałbym żeby ktoś to obejrzał i spróbował mi wyjaśnić czemu to nie chce działac :/
Ten skrypt miał zliczać odwiedziny ludzi, ale dodatkowo miał zapamiętywać po ip kto odwiedził stronę i nie zliczac jego dalszych poczynań (np. odświerzanie strony co minuta) przez następne 24h (żeby nie wychodziły kosmiczne liczby odwiedzin) no i jak wsadziłem to na serwer okazło się że pokazuje tylko 1 i nic wiecej... a kilku lidzi odwiedziło stronę
powiem jeszcze, żeby nie było niejasności, że skrypt poprawnie stworzył bazę i tabele, ale już nie chciał odawać niczego tak jak by mu się coś urwało przy insertownaiu... HELP!

[PHP] pobierz, plaintext 
<?
//****************** połączenie z bazą
$sql_connect = mysql_connect(localhost, 'admin', 'x*x*x*x*x');
	if ($sql_connect) {
		$sql_select = mysql_query('USE odwiedziny');
		if (!$sql_select) {
//***************** jeśli nie ma takiej bazy to niech ją utworzy						 
			$sql_create = mysql_query('CREATE DATABASE odwiedziny');
			$sql_select = mysql_query('USE odwiedziny');
//***************** zaznaczenie bazy, stworzenie tabel i wpisanie rekordu z 0 w li
czniki	
		$sql_create_table = mysql_query('CREATE TABLE liczniki (id char(1) NOT NULL, ilosc char(6), PRIMARY KEY(id))');
			$sql_create_table = mysql_query('CREATE TABLE ip_time (id char(1) NOT NULL, ip char(15), time char(10), PRIMARY KEY(id))');
			$sql_query = mysql_query('INSERT INTO liczniki VALUES (1, 0)');
			$sql_query = mysql_query('INSERT INTO liczniki VALUES (2, 0)');
		}
	}
//***************** odczytanie wszystkich ip jakie otwierały stronę i zliczenie il
e ich było
	$sql_query = mysql_query('SELECT ip FROM ip_time');
	$num_rows = mysql_num_rows($sql_query);
//****************** 1 sytuacja
//***************** jeśli nie było jeszcze żadnego to ustawnienie licznika na 1 i 
zapisanie w bazie 
//***************** dodatkowo zapisanie numeru id, godziny i ip który otworzył str
onę
	if ($num_rows == 0) {
		$num_rows++;
		$licznik = 1;
		$sql_query = mysql_query('UPDATE liczniki SET ilosc=1 WHERE id=1');
		$ip = $_SERVER['REMOTE_ADDR'];
		$time = time();
		$sql_query = mysql_query('INSERT INTO ip_time VALUES ('.$num_rows.', '.$ip.', '.$time.'');
//***************** wyświetlenie wartosci licznika
		echo $licznik;
	}
//***************** 2 sytuacja
//***************** jeśli było już kilka ip które otworzyły stronę to sprawdza czy
 takie ip juz nie wystapiło w bazie
	else {
		$ip = $_SERVER['REMOTE_ADDR'];
		$time = time();
		$sql_query = mysql_query('SELECT time FROM ip_time WHERE ip='.$ip.'');
		if ($sql_query == '') {
//***************** jesli nie wystapiło za zapisuje ip godzine i numer id i zwieks
za licznik
			$num_rows++;
			$sql_query = mysql_query('INSERT INTO ip_time VALUES ('.$num_rows.', '.$ip.', '.$time.')');
			$sql_query = mysql_query('SELECT ilosc FROM liczniki WHERE id=1');
			$licznik = mysql_fetch_assoc($sql_query);
			$licznik++;
			$sql_query = mysql_query('UPDATE liczniki SET ilosc='.$licznik.' WHERE id=1');
			echo $licznik;
		}
//************** 3 sytuacja
//************** jeśli jednak wystąpiło to sprawdza godzinę przi danym ip i porówn
uje czy minęło już 24h od otwarcia strony
		else {
			$ip_time = mysql_fetch_assoc($sql_query);
			$ip_time['time'] = $ip_time['time']+86400;
//*************** jak minęło 24h to odnawia godzinę przy ip i zwiększa licznik
			if ($ip_time['time'] < $time) {
				$num_rows++;
				$sql_query = mysql_query('UPDATE ip_time SET time='.$time.' WHERE ip='.$ip.'');
				$sql_query = mysql_query('SELECT ilosc FROM liczniki WHERE id=1');
				$licznik = $sql_query;
				$licznik++;
				$sql_query = mysql_query('UPDATE liczniki SET ilosc='.$licznik.' WHERE id=1');
				echo $licznik;
			}
//***************** jeśli nie minęło 24h to poprostu odczytuje licznik
			else {
				$sql_query = mysql_query('SELECT ilosc FROM liczniki WHERE id=1');
				$licznik = mysql_fetch_assoc($sql_query);
				echo $licznik;
			}
		}
	}
//****************** zamyka połączenie
 mysql_close($sql_connect);
?>
[PHP] pobierz, plaintext 

z tymi komentarzami chyba da się nieco zrozumiec ?

Ten post edytował Ermes 9.02.2007, 23:51:00

[wipo]

Człowieku kto to pisał?
Chińczyk z czarnego lądu?

[Ermes]

jeśli ci się nie podoba to popraw, bo głównie oto mi chodzi

[Babcia@Stefa]

Człowieku kto to pisał?
Chińczyk z czarnego lądu?

Zgadam się, troche dużo tego kodu ^^,
Jak na licznik powinno tego być mało (ja mam licznik w php, mysql w paru linijkach).

Dziękuję, Babcia@Stefa

[Ermes]

hm... może dodam trochę komentarza to się wam trochę rozjaśni o co w nim biega ;]

OK! już sam sobie poradziłem z paroma błedami w kodzie

dam tutaj teraz poprawnie działający kod jeśli ktoś bedzie chciał go użyć
Przypomnę skrypt zlicza odwiedziny na stronie zapamiętując kto ją odwiedził i nie zlicza ponownego wejścia na stronę z danego ip przed upływem 24 godzin.

[PHP] pobierz, plaintext 
<?
/* Połaczenie z bazą */
$sql_connect = mysql_connect(localhost, 'admin', 'xxxxx');
	if ($sql_connect) {
	/* Sprawdza czy baza odwiedziny istnieje */
		$sql_select = mysql_query('USE odwiedziny');
		if (!$sql_select) {
		/* jeśli takabaza nie istnienie to ją tworzy */
			$sql_create = mysql_query('CREATE DATABASE odwiedziny');
			$sql_select = mysql_query('USE odwiedziny');
			/* tworzy tabele w bazie */
			$sql_create_table = mysql_query('CREATE TABLE liczniki (id char(1) NOT NULL, ilosc char(6), PRIMARY KEY(id))');
			$sql_create_table = mysql_query('CREATE TABLE ip_time (id char(1) NOT NULL, ip char(15), time char(10), PRIMARY KEY(id))');
			/* ustawia wartość poczatkową dla licznika = 0*/
			$sql_query = mysql_query('INSERT INTO liczniki VALUES (1, 0)');
		}
	}
	/* pobiera wszystkie ip które odwiedziły stronę i ilczy ile ich jest */
	$sql_query = mysql_query('SELECT ip FROM ip_time');
	$num_rows = mysql_num_rows($sql_query);
	/* jeśli jeszcze wcześnie żaden komputer nie odwiedził strony to ustawia licznik na 1 i zapisuje aktualny IP
	godzinę i numer id w bazie */
	if ($num_rows == 0) {
		$num_rows++;
		$licznik = 1;
		$sql_query = mysql_query('UPDATE liczniki SET ilosc="1" WHERE id="1"');
		$ip = $_SERVER['REMOTE_ADDR'];
		$time = time();
		/*zapisuje dane o odwiedzajacym */
		$sql_query = mysql_query('INSERT INTO ip_time VALUES ('.$num_rows.', "'.$ip.'", '.$time.')');
		/* wyświetla licznik */
		echo $licznik;
	}
	else {
	/* jesli już ktoś odwiedził stronę wcześniej to pobiera aktualne IP i czas */
		$ip = $_SERVER['REMOTE_ADDR'];
		$time = time();
		/* sprawdza czy to ip już wystapiło w bazie czy nie */
		$sql_query = mysql_query('SELECT time FROM ip_time WHERE ip="'.$ip.'"');
		/* jeśli nie to zapisuje informacje o nowym ip w tabeli (czas i ip)
		oraz zwiększa licznik o 1*/
		$ip_time = mysql_fetch_assoc($sql_query);
		if ($ip_time['time'] == '') {
			$num_rows++;
			$sql_query = mysql_query('INSERT INTO ip_time VALUES ('.$num_rows.', "'.$ip.'", '.$time.')');
			$sql_query = mysql_query('SELECT ilosc FROM liczniki WHERE id=1');
			$licznik = mysql_fetch_assoc($sql_query);
			$licznik['ilosc']++;
			$sql_query = mysql_query('UPDATE liczniki SET ilosc='.$licznik['ilosc'].' WHERE id=1');
			/* wypisuje stan licznika */
			echo $licznik['ilosc'];
		}
		else {
		/* jeśli jednak IP wystąpiło już w bazie to sprawdza czy od czasu odwiedziny upłyneło juz 24h czy nie */
			$ip_time['time'] = $ip_time['time']+86400;
			if ($ip_time['time'] < $time) {
			/* jeśli upłunęło to updatuje informacje o czasie odwiedzin w bazie i zwiększa licznik o 1*/
				$num_rows++;
				$sql_query = mysql_query('UPDATE ip_time SET time='.$time.' WHERE ip="'.$ip.'"');
				$sql_query = mysql_query('SELECT ilosc FROM liczniki WHERE id=1');
				$licznik = mysql_fetch_assoc($sql_query);
				$licznik['ilosc']++;
				$sql_query = mysql_query('UPDATE liczniki SET ilosc='.$licznik['ilosc'].' WHERE id=1');
				echo $licznik['ilosc'];
			}
			else {
			/* jeśli nie upłyneło 24h od odwiedzin poprostu odczytuje stary stan licznika */
				$sql_query = mysql_query('SELECT ilosc FROM liczniki WHERE id=1');
				$licznik = mysql_fetch_assoc($sql_query);
				echo $licznik['ilosc'];
			}
		}
	}
 mysql_close($sql_connect);
?>
[PHP] pobierz, plaintext 

Ten post edytował Ermes 10.02.2007, 00:07:01