Licznik, mały a cieszy! Opcje

[Lisek54]

Tworzymy plik instaluj.php a tam kod:

[PHP] pobierz, plaintext 
<? $co = fopen('licznik.txt', 'w');
fputs($co, '1');
fclose($co); ?>
[PHP] pobierz, plaintext 

Później plik index.php:

[PHP] pobierz, plaintext 
<?
if($cook!='1'){
setcookie("cook","1");
$pliczek=fopen("licznik.txt", 'r');
$zmienna=fgets($pliczek);
fclose($pliczek);
$zmienna++;
$plik = fopen('licznik.txt', 'w');
fwrite($plik, $zmienna);
fclose($plik);
include("licznik.txt");
}
else
{
include("licznik.txt");
}
?>
[PHP] pobierz, plaintext 

Najpierw odpalamy instaluj.php gdy strona się załaduje usuwamy go (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Na plik licznik.txt nadaj chmod 777.

P.S jeden z moich pierwszych liczników :-)

Ten post edytował Lisek54 2.10.2005, 20:43:32

[lukir]

1)Podstawowy błąd jaki zrobiłeś to fragment:

[PHP] pobierz, plaintext 
<?php
if($cook!='1')
  setcookie("cook","1");
?>
[PHP] pobierz, plaintext 

Chodzi o zmienną $cook.
Mianowicie powinieneś odwołac się do wartości tej zmiennej w ten sposób:

[PHP] pobierz, plaintext 
<?php
if($_COOKIE['cook']!='1')
  setcookie("cook","1");
?>
[PHP] pobierz, plaintext 

Wtedy kod jest bezpieczniejszy.

2)I jeszcze setcookie():

[PHP] pobierz, plaintext 
<?php
if($_COOKIE['cook']!='1')
  setcookie("cook","1", time()+43200);
/*żeby ciasteczko zniknęło po jakimś czasie...
Dodatkowo funkcję setcookie() umieściłbym na końcu instrukcji if(), ponieważ
dopiero wtedy wartoś zostanie zwiększona.*/
?>
[PHP] pobierz, plaintext 

3)Powinieneś kontrolowac typ zmiennej $zmienna:

[PHP] pobierz, plaintext 
<?php
$pliczek=fopen("licznik.txt", 'r');
$zmienna=fgets($pliczek);
fclose($pliczek);
if(ereg("[0-9]", $zmienna)) {
  $zmienna++; //ma to sens tylko dla liczb
} else {
  echo("Błąd danych");
}
?>
[PHP] pobierz, plaintext 

4)Myślę, że nazewnictwo zmiennych nie jest satysfakcjonujące.
Może to i prosty skrypt, ale dobrym nawykiem jest logiczne i czytelne nadawanie nazw zmienym.

5)No i plik instaluj.php. Powinien wyglądac tak (żeby nie fałszowac wyniku (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) ):

[PHP] pobierz, plaintext 
<?php
$co = fopen('licznik.txt', 'w');
fputs($co, '0'); //ustawiamy wartośc licznika na 0 ;]
fclose($co);
?>
[PHP] pobierz, plaintext