[PHP] licznik odwiedzin - dobrze napisany? Opcje

[dwwa]

napisałem prosty licznik odwiedzin oparty o bazę danych mysql, ale nie wiem czy to optymalne rozwiązanie i dobrze napisane... chodzi mi o licznik unikalnych odwiedzin. Na razie sprawdzam tylko ip, ale zaraz dopiszę jeszcze sprawdzanie ciastka.

[PHP] pobierz, plaintext 
// counter
	function counter() {
		$ip = $_SERVER['REMOTE_ADDR'];
		$check_ip_query = mysql_query("SELECT `ip` FROM `counter` WHERE `ip` = '$ip'");
		if (mysql_num_rows($check_ip_query) == 0) {
			mysql_query("INSERT INTO `counter` VALUES (null, '$ip')");
		}
		$counter = mysql_num_rows(mysql_query("SELECT `id` FROM `counter`"));
		return $counter;
	}
[PHP] pobierz, plaintext 

dobrze to jest napisane? działać działa

[nospor]

Do sprawdzania liczby rekordów służy COUNT w mysql a nie pobieranie wszystkiego do php i dopiero tam liczenie ile się pobrało - strasznie nieoptymalne.

[dwwa]

ale chodzi mi o samo rozwiązanie tego licznika, że oparty on jest o bazę

robię statystyki dla strony, dopisałem ilość unikalnych wizyt dziennie, tygodniowo i miesięcznie, co o tym sądzicie?

[PHP] pobierz, plaintext 
function counter($value) {
 
		switch($value) {
			case 'all':
				$counter = mysql_num_rows(mysql_query("SELECT `id` FROM `counter`"));
			break;
			case 'day':
				$day = date('Y-m-d H:i:s', time()-1440*60); // 24 hours
				$counter = mysql_num_rows(mysql_query("SELECT * FROM `counter` WHERE `date` > '$day'"));
			break;
			case 'week':
				$week = date('Y-m-d H:i:s', time()-10080*60); // 7 days
				$counter = mysql_num_rows(mysql_query("SELECT * FROM `counter` WHERE `date` > '$week'"));
			break;
			case 'mounth':
				$mounth = date('Y-m-d H:i:s', time()-43200*60); // 1 mounth
				$counter = mysql_num_rows(mysql_query("SELECT * FROM `counter` WHERE `date` > '$mounth'"));
			break;
		}
 
		$ip = $_SERVER['REMOTE_ADDR'];
		$date = date('Y-m-d H:i:s', time());
		$check_ip_query = mysql_query("SELECT `ip` FROM `counter` WHERE `ip` = '$ip'");
		if (mysql_num_rows($check_ip_query) == 0) {
			mysql_query("INSERT INTO `counter` VALUES (null, '$ip', '$date')");
		}
		return $counter;
	}
[PHP] pobierz, plaintext 

wyświetlam tak:

[PHP] pobierz, plaintext 
echo counter('value');
[PHP] pobierz, plaintext 

Ten post edytował dwwa 11.10.2011, 12:19:47

[nospor]

Kurcze, pytasz się o ty czy jest to optymalne rozwiązanie i czy jest dobrze napisane, to ci odpowiadam że nie jest. Na co ty, że nie takiej odpowiedzi oczekujesz :/

No to dobrze: jest to super napisane, w życiu nie widziałem nic lepszego, chętnie odkupię od ciebie ten skrypt za 2 tys euro.

[dwwa]

ok, poprawiłem na COUNT. Teraz będzie dobrze?

[PHP] pobierz, plaintext 
function counter($value) {
 
		switch($value) {
			case 'all':
				$counter = mysql_query("SELECT COUNT(`id`) FROM `counter`");
			break;
			case 'day':
				$day = date('Y-m-d H:i:s', time()-1440*60); // 24 hours
				$counter = mysql_query("SELECT COUNT(`id`) FROM `counter` WHERE `date` > '$day'");
			break;
			case 'week':
				$week = date('Y-m-d H:i:s', time()-10080*60); // 7 days
				$counter = mysql_query("SELECT COUNT(`id`) FROM `counter` WHERE `date` > '$week'");
			break;
			case 'mounth':
				$mounth = date('Y-m-d H:i:s', time()-43200*60); // 1 mounth
				$counter = mysql_query("SELECT COUNT(`id`) FROM `counter` WHERE `date` > '$mounth'");
			break;
		}
 
		$ip = $_SERVER['REMOTE_ADDR'];
		$date = date('Y-m-d H:i:s', time());
		$check_ip_query = mysql_query("SELECT `ip` FROM `counter` WHERE `ip` = '$ip'");
 
		if (mysql_num_rows($check_ip_query) == 0) {
			mysql_query("INSERT INTO `counter` VALUES (null, '$ip', '$date')");
		}
 
		$counter = mysql_fetch_row($counter);
		$counter = $counter[0];
		return $counter;
 
	}
[PHP] pobierz, plaintext 

Ten post edytował dwwa 11.10.2011, 12:30:54

[nospor]

Najpierw zliczasz, potem dodajesz. Wynik będziesz miał o jeden nieaktualny.

Jak ktoś już się pojawił miesiąc temu, to nawet jeśli dzisiaj wszedł, to już jako dzisiaj nie zostanie zliczony - no chyba, że jest to celowe.A, to jest celowe, bo to nie jest licznik online, a licznik odwiedzin (IMG:style_emoticons/default/smile.gif)

Powód edycji: [nospor]:

[dwwa]

czyli mam wszystko w miarę dobrze teraz?