Счетчик посещений cookies

[Kisa1993]

Помогите, попробовала сама написать счетчик, вот что вышло:

<?php
$all;
if (isset($_COOKIE['Schet'])) $cook = 0;
else $all++;
setcookie("Schet",$cook,0x6FFFFFFF);
echo "<p>Всего пользователей посетило страницу: <strong>$all</strong></p>"
?>

Желаемый принцип работы: Создаем пустую переменную "all", проверяем создана ли кука "Schet" если да то значение куки = 0, если нет прибавляем к переменной $all единицу, (по идеи в переменной all уже лежит 1) затем создаем куку "Schet", и затем в результате выводим $all. При втором посещении т.к. у нас уже есть кука, 1 прибавляться не будет.

В моем случае вывелось: Всего пользователей посетило страницу: 1.

Если другой человек зайдет без куки счетчик увеличится?

Как можно оптимизировать мой код?

[LunatiK]

Если другой человек зайдет без куки счетчик увеличится?

А данные посещений где-нибудь сохраняются?

[Kisa1993]

Ну по идеи в $all

[LunatiK]

оно уничтожается после выполнения скрипта

[Kisa1993]

а как сделать чтобы не уничтожалось?

[crautcher]

в базу сохраните или в файл и будет вам счетчик

[Kisa1993]

<?php
$db = mysql_connect ("localhost","test","test");
mysql_select_db ("schetchik",$db);
?>
<?php
$all;
if (isset($_COOKIE['Schet'])) $cook = 0;
else $all++;
setcooki("Schet",$cook,0x6FFFFFFF);
echo "<p>Всего пользователей посетило страницу: $all</p>";

$result = mysql_query ("UPDATE schet SET schet='$all'");
?>

Создала базу schetchik, в ней таблицу schet.

$result должна обновлять данные в таблице schet из переменной all.

Выводит след. ошибки:

Notice: Undefined variable: all in z:\home\test1.ru\www\dark\schet.php on line 6

[Денвер: показать возможную причину ошибки]

Notice: Undefined variable: all in z:\home\test1.ru\www\dark\schet.php on line 8

Fatal error: Call to undefined function: setcooki() in z:\home\test1.ru\www\dark\schet.php on line 9

[LunatiK]

Правильно выводит. Переменная $all не определена. и Функции setcooki() не существует. Внимательней пишите и проверяйте написанное, ну и не пугайтесь этих надписей они ваши помощники.

[NeoXidizer]

<?php $all; ?>

Желаемый принцип работы: Создаем пустую переменную "all", проверяем создана ли кука "Schet" если да то значение куки = 0, если нет прибавляем к переменной $all единицу, (по идеи в переменной all уже лежит 1) затем создаем куку "Schet", и затем в результате выводим $all. При втором посещении т.к. у нас уже есть кука, 1 прибавляться не будет.

В моем случае вывелось: Всего пользователей посетило страницу: 1.

Если другой человек зайдет без куки счетчик увеличится?

Как можно оптимизировать мой код?

нельзя так объявлять переменную в процедурном коде. Выведет Undefined variable: all. Нужно всегда указывать начальное значение, например null, 0, и т.д.

Я бы сделал алгоритм следующим:

Проверяем, есть ли у пользователя кука

Если нет, то создаем куку, которая хранится до закрытия браузера

Увеличиваем в базе данных текущее значение счетчика на 1

базой данных может быть как и база в СУБД, так и простым текстовым файлом, на ваш выбор

тобишь как-то так:

if (!isset($_COOKIE['visit'])) {
открываем файл
получаем из него текущее значение счетчика
увеличиваем значение счетчика на 1
записываем результат
закрываем файл
} else {
устанавливаем куку, которая живет лишь текущую сессию.
}

Edited June 21, 2012 by NeoXidizer

[Kisa1993]

А как открыть файл, и получить значение?

[NeoXidizer]

http://ru2.php.net/manual/ru/book.filesystem.php

вам нужны fopen, fgets, fputs, fclose

Находите функцию, читаете описание и смотрите примеры

если владеете английским, то читаете комментарии

либо

http://ru2.php.net/manual/ru/function.file-get-contents.php

http://ru2.php.net/manual/ru/function.file-put-contents.php

Edited June 21, 2012 by NeoXidizer

[Kisa1993]

<?php
if (!isset($_COOKIE['visit']))
{
$handle = fopen("text.txt", "r");
$buffer = fgets($handle, 4096);

Как увеличить счетчик на 1 и сохранить?

fclose($handle);
}
else
{
setcookie("visit","0");
}
?>

[NeoXidizer]

Как увеличить счетчик на 1 и сохранить?

вот конечный код

if (!isset($_COOKIE['visit'])) {
	$filename = 'text.txt';
	if (file_exists($filename)) {
		$handle = fopen($filename, "r");
		$counter = (int)fgets($handle);
		fclose($handle);
		$handle = fopen($filename, "w");
		fputs($handle, ++$counter); // записываем в файл текущее значение счетчика +1
		fclose($handle);
	} else {
		$handle = fopen($filename, "w+");
		if (!$handle)
			die('Файл невозможно создать');
		fputs($handle, "1"); // Если файла изначально не существует, создаем его с содержимым "1"
		fclose($handle);
	}
	setcookie("visit");
}

а еще можно так

if (!isset($_COOKIE['visit'])) {
	$filename = 'texts.txt';
	if (file_exists($filename)) {
		$counter = (int)file_get_contents($filename);
		file_put_contents($filename, ++$counter);
	} else
		file_put_contents($filename, "1");
	setcookie("visit");
}

или даже так

if (!isset($_COOKIE['visit'])) {
	$filename = 'texts.txt';
	if (file_exists($filename))
		file_put_contents($filename, (int)file_get_contents($filename)+1);
	else
		file_put_contents($filename, "1");
	setcookie("visit");
}

Edited June 21, 2012 by NeoXidizer

[crautcher]

$count = intval(file_get_contents('counter.dat'));
file_put_contents('counter.dat' , ++$count);

[NeoXidizer]

[добавьте кнопку удалить сообщение]

Edited June 21, 2012 by NeoXidizer

[Kisa1993]

Огромное всем спасибо, сама бы до конца жизни разбиралась.