[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

[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