[PHP]Blokowanie licznika Opcje

[Mateoo88]

Witam, mam skrypt licznika odwiedzin, bardzo prostego. Licznik działa dobrze a nawet można żec aż za dobrze Nalicza każde wejsce na strone nawet gdy się strone odswieży to też nalicza. Czy może ktoś mi pomóc lub nakierować co zrobić aby przyblokować ten licznik tak by naliczał wejscia z jednego adresu co 12h. Z góry dzięki

[PHP] pobierz, plaintext 
<?php
$count_my_page = ("hitcounter.txt");
$hits = file($count_my_page);
$hits[0] ++;
$fp = fopen($count_my_page , "w");
fputs($fp , "$hits[0]");
fclose($fp);
echo $hits[0];
?>
[PHP] pobierz, plaintext 

[Ilware]

sposobów zabezpieczeń jest kilka

1- zapisywanie cookie na czas 12h i sprawdzanie czy już takie cookie istnieje - jeżeli nie to inkrementujesz licznik.
2- zapisywanie ip odwiedzającego do bazy i sprawdzanie po ip

+ ew
3 - zabezpieczenie na sesji ale to tylko na okres jednego wejścia na stronę

[Mateoo88]

Zależy mi na prostocie więc wybieram ciasteczka.

do kodu dodałem:

[PHP] pobierz, plaintext 
setcookie("visited", "1", time()+3600*3);
[PHP] pobierz, plaintext 

wszystko wygląda tak:

[PHP] pobierz, plaintext 
<?php
setcookie("visited", "1", time()+3600*3);
$count_my_page = ("hitcounter.txt");
$hits = file($count_my_page);
$hits[0] ++;
$fp = fopen($count_my_page , "w");
fputs($fp , "$hits[0]");
fclose($fp);
echo $hits[0];
?>
[PHP] pobierz, plaintext 

Wyskakuje mi błąd : Warning: Cannot modify header information - headers already sent by (output started at /home/y7828sm/public_html/index.php:8) in /home/y7828sm/public_html/counter.php on line 2

Mam małą więdze n/t php i nie bardzo wiem co robie źle. Prosze o pomoc

Ten post edytował Mateoo88 12.01.2012, 14:11:17

[Ilware]

http://wortal.php.pl/Wortal/Artykuly/PHP/P...ch-programistow

temat wałkowany miliardy razy, nie możesz wysłać nagłówków jak wyślesz już jakieś znaki do przeglądarki.

oprócz tego dodanie tylko tej linii Ci nic nie da, po prostu teraz przy każdym odświeżeniu będzie się tworzyło ciasteczko.musisz sprawdzić czy ciasto istnieje jeżeli nie to tworzysz nowe i inkrementujesz licznik, jeżeli istnieje to nic nie robisz.

Ten post edytował Ilware 12.01.2012, 14:19:39

[Bateria]

[PHP] pobierz, plaintext 
<?php
	if(!isset($_COOKIE['licznik'])){
		$count_my_page = ("hitcounter.txt");
		$hits = file($count_my_page);
		$hits[0] ++;
		$fp = fopen($count_my_page , "w");
		fputs($fp , "$hits[0]");
		fclose($fp);
		echo $hits[0];
		setcookie('licznik', 'licznik', time()+43200);
	}
?>
[PHP] pobierz, plaintext 

[Mateoo88]

Wyskakuje błąd w 10 linijce kodu czyli:

[PHP] pobierz, plaintext 
setcookie('licznik', 'licznik', time()+43200);
[PHP] pobierz, plaintext 

Dalej nalicza za każdym odświeżeniem strony

[Bateria]

Zrób to tak jak napisałem i jak będzie dalej błąd to napisz:
1. Do jakiegoś pliku .php wklej kod:

[PHP] pobierz, plaintext 
<?php
	$count_my_page = ("hitcounter.txt");
	$hits = file($count_my_page);
	if(!isset($_COOKIE['licznik'])){
		$hits[0] ++;
		setcookie('licznik', 'licznik', time()+43200);
	}
	$fp = fopen($count_my_page , "w");
	fputs($fp , "$hits[0]");
	fclose($fp);
	echo $hits[0];
?>
[PHP] pobierz, plaintext 

2. Stwórz plik 'hitcounter.txt' i wpisz w nim wartość 0, nie zostawiaj go pustego.
3. Wyczyść ciacha przeglądarki i odpal skrypt.

ps. Testujesz to na localhost czy wgrałeś na jakiś serwer?

Ten post edytował Bateria 12.01.2012, 15:03:11

[Mateoo88]

Pliki wgrywam na serwer.
Niestety zrobiłem tak jak napisałeś wyżej i dalej wyskakuje błąd : Warning: Cannot modify header information - headers already sent by (output started at /home/y7828sm/public_html/index.php:7) in /home/y7828sm/public_html/counter.php on line 6

A licznik nalicza kazde odświeżenie strony. Moja skromna wiedza o php dawno się skonczyła wiec prosze o kolejne wskazówki, z góry dziękuje

[Bateria]

Jak znów nie zadziała, pisz.

[PHP] pobierz, plaintext 
<?php
ob_start();
	$count_my_page = ("hitcounter.txt");
	$hits = file($count_my_page);
	if(!isset($_COOKIE['licznik'])){
		$hits[0] ++;
		setcookie('licznik', 'licznik', time()+43200);
	}
	$fp = fopen($count_my_page , "w");
	fputs($fp , "$hits[0]");
	fclose($fp);
	echo $hits[0];
ob_end_flush();
?>
[PHP] pobierz, plaintext 

Ten post edytował Bateria 12.01.2012, 15:18:52

[Mateoo88]

Niestety, to samo tyle, że błąd jest w 7 lini

[Bateria]

Czy aby Twoja przeglądarka ma na pewno włączoną obsługę ciasteczek?
Wejdź na adres:
http://bateria.ugu.pl/test.php
U mnie to w pełni działa, jeżeli u Ciebie nie będzie to działać wina leży po stronie przeglądarki.

Jak to zrobisz daj znać.

Ten post edytował Bateria 12.01.2012, 15:24:01

[Mateoo88]

Hmmm no widzisz u mnie działa Twoj link. a moj nie dziala. WWW zobacz sobie skrypt zainstalowalem pod dolna pomarańczową kreską ...

[Bateria]

Na PW napisałem ci moje gg. Napisz to coś pokombinujemy

Ten post edytował Bateria 12.01.2012, 15:35:19

[Ilware]

ob_start umieść na samym początku pliku przed wszystkim, uważaj żeby nie było nawet białego znaku.

[PHP] pobierz, plaintext 
<?php
ob_start();
?>
[PHP] pobierz, plaintext 

i powinno być okej

[Bateria]

No i powinno się na samym końcu

[PHP] pobierz, plaintext 
ob_end_flush();
[PHP] pobierz, plaintext 

Ja nie zadziała to napisz

[Mateoo88]

w pliku index.php

[PHP] pobierz, plaintext 
<?php
include ("counter.php");
?>
[PHP] pobierz, plaintext 

a w pliku counter.php:

[PHP] pobierz, plaintext 
    <?php
    ob_start();
    $count_my_page = ("hitcounter.txt");
    $hits = file($count_my_page);
    if(!isset($_COOKIE['licznik'])){
    $hits[0] ++;
    setcookie('licznik', 'licznik', time()+43200);
    }
    $fp = fopen($count_my_page , "w");
    fputs($fp , "$hits[0]");
    fclose($fp);
    echo $hits[0];
    ob_end_flush();
    ?>
[PHP] pobierz, plaintext 

mam jeszcze ten plik hitcounter.txt ale tam jest tylko:

[PHP] pobierz, plaintext 
1
[PHP] pobierz, plaintext 

________EDIT_______
Dodałem do index.php :

[PHP] pobierz, plaintext 
<?php ob_start(); ?>
[PHP] pobierz, plaintext 

i Działa !!
Wielkie dzieki Panowie : Bateria i Ilware
Dziekuje za poświęcony czas a żeby nie był on zmarnowany to prosze napiszcie mi co robi ten kod:

[PHP] pobierz, plaintext 
<?php ob_start(); ?>
[PHP] pobierz, plaintext 

Ten post edytował Mateoo88 12.01.2012, 15:43:44

[Micchaleq]

ob_start

Ten post edytował Micchaleq 12.01.2012, 15:49:38

[Mateoo88]

Jeszcze jedno pytanie co do kodu:
Jeżeli chcę by cookies odblokowywały się szybciej to fragment kodu:

[PHP] pobierz, plaintext 
setcookie('licznik', 'licznik', time()+43200);
[PHP] pobierz, plaintext 

zmienić na np

[PHP] pobierz, plaintext 
setcookie('licznik', 'licznik', time()+120);
[PHP] pobierz, plaintext 

Czy to tylko pomoże? Czy coś jeszcze trzeba zrobić?

[Bateria]

Już tłumaczę:

[PHP] pobierz, plaintext 
setcookie('nazwa_ciasteczka', 'wartosc_ciasteczka', time()+99);
[PHP] pobierz, plaintext 

Po trzecim przecinku ustalamy po jakim czasie ciastko ma być przestać aktwnę:
time() - wyświetla czas unixowy czyli ile sekund mineło od 1 stycznia 1970r.
time()+100 - ciasteczka dezaktywuje się po 100 sekundach od momentu jego dodania

[Mateoo88]

Dzięki za pomoc i odpowiedź!