[php]Licznik Opcje

[Glarden]

Mam licznik osób, które były na stronie, ale nie chcę, żeby dodawał do licznika osobę, która już na stronie była wcześniej. Rozpoznaje po IP. Skrypt wygląda tak:

[PHP] pobierz, plaintext 
<?php
function dodaj () {
	$file = file("licznik.txt");
	$ip = $_SERVER['REMOTE_ADDR'];
	if (!in_array($ip, $file)) {
		$plik = fopen("licznik.txt", "w");
		fputs($plik, $ip."rn");
		for ($i=0;$i<count($file);$i++) {
			fputs ($plik, $file[$i]);
		}
		fclose($plik);
	}
}
function odczytaj () {
	$file = file("licznik.txt");
	$ile = count($file);
	return $ile;
}
?>
[PHP] pobierz, plaintext 

I co nie zrobię, to jednak dodaje do tego pliku i przy każdym wejściu inkrementuje licznik. Co jest źle?

[Hacker]

polecam manual php

Elementy tablicy zawierają znak nowej linii.

[Glarden]

Dzięki, teraz już działa.
Ale mam jeszcze jedną prośbę. Zrobiłem licznik osób online, ale na razie mogę go wypróbować tylko na localhoscie, więc moglibyście sprawdzić, czy nie ma błędów? Oto skrypt:

[PHP] pobierz, plaintext 
<?php
function online_dodaj () {
	include ("baza.php");
	$c = time();
	$ses = session_id();
	$query = "select * from online where session_id='$ses'";
	$result = mysql_query ($query);
	$ile = mysql_num_rows ($result);
	if ($ile>0) {
		$query = "update online set time='$c' where session_id='$ses'";
		$result = mysql_query ($query);
	}
	else {
		$query = "insert into online values ('$ses', '$c')";
		$result = mysql_query ($query);
	}
	$czas = (time()-300);
	$query = "delete from online where time<$czas";
	$result = mysql_query ($query);
}
 
function online_odczytaj () {
	include ("baza.php");
	$query = "select * from online";
	$result = mysql_query ($query);
	$online = mysql_num_rows ($result);
	return $online;
}
?>
[PHP] pobierz, plaintext 

Kiedy ja wchodzę, to pokazuje, że jest jedna osoba (czyli poprawnie).

[Kicok]

Błędów w działaniu być nie powinno, tyle że kod jest nieco nieoptymalnie napisany. Jeśli chce ci się z tym jeszcze bawić, to zainteresuj się TYM przy odczytywaniu ilości osób online
oraz TYM przy aktualizowaniu wpisów w bazie danych (dzięki REPLACE liczbę zapytań z 4 możesz zmniejszyć do dwóch)

Ten post edytował Kicok 23.07.2007, 19:05:32

[Glarden]

Poczytam. Wielkie dzięki.