Forum PHP.pl

Mam pewien problem. Napisałem licznik oparty na pliku tekstowym. Na początku działał jak należy, a po dodaniu komentarzy wartość licznika nie wzrasta. O co chodzi?

[PHP] pobierz, plaintext 
<?php
$plik= fopen("licznik.txt", "r+"); //otwiera plik licznik.txt
$wartosc= file_get_contents("licznik.txt"); // odczytuje wartość z pliku licznik.txt
$wartosc++; // zwieksza wartość odczytanego pliku o 1
echo "Oglądano nas: $wartosc razy."; // wyświetla wartość odczytanego i zwiększonego wcześniej pliku licznik.txt
fwrite($plik, $wartosc); // zapisuje zwiększoną wartość do pliku licznik.txt
fclose($plik); // zamyka otowrzony plik licznik.txt
?>
[PHP] pobierz, plaintext 

Proszę o pomoc.

No to mozna by sie domyslic, ze chodzi ocos z tymi komentarzami... niestety nie byles na tyle domyslny by wkleic kod odpowiedzialny za komentarze i wogole wyjasnic co mają komentarze do licznika wiec.... domysl sie sam co wiec...

napisałem sobie komentarze, aby wiedzieć krok po kroku co robi skrypt.. po usunięciu tych komentarzy nadal nie działa :/ .. tylko czemu? mógłbyś mnie naprowadzić co jest źle?

aaaaa, mowisz o komentarzach php myslalem ze napisales dodatkowy moduł komentarzy do swego serwisu....
Wyrażaj się jaśniej na przyszlosc

A jak usuniesz teraz komentarze to zacznie dzialac? Niemozliwe. KOmentarze nie mają wplywu na kod. Moze dodając komentarze dodales cos innego przypadkiem?

Kod co tu podales jest ok. Przy zalozeniu ze istnieje plik licznik.txt

edit: wlacz sobie wyswietlanie wszystkich bledow. moze masz jakies:
http://nospor.pl/php-faq-n29.html#faq-2

plik licznik.txt istnieje, dodałem skrypt od błędów, ale żadnego mi nie wywaliło. a może to wina krasnala?

[PHP] pobierz, plaintext 
<?php
$plik= fopen("licznik.txt", "r+"); //otwiera plik licznik.txt
var_dump($plik);
$wartosc= file_get_contents("licznik.txt"); // odczytuje wartość z pliku licznik.txt
echo "Przed: $wartosc ";
$wartosc++; // zwieksza wartość odczytanego pliku o 1
echo "Po: $wartosc ";
echo "Oglądano nas: $wartosc razy."; // wyświetla wartość odczytanego i zwiększonego wcześniej pliku licznik.txt
fwrite($plik, $wartosc); // zapisuje zwiększoną wartość do pliku licznik.txt
fclose($plik); // zamyka otowrzony plik licznik.txt
?>
[PHP] pobierz, plaintext 

Pokaz co wyswietlilo

resource(2) of type (stream) Przed: 1 Po: 1 Oglądano nas: 1 razy.

po odświezaniu nie wzrasta wartość

dobre, dobre....
a tak:

[PHP] pobierz, plaintext 
<?php
$plik= fopen("licznik.txt", "r+"); //otwiera plik licznik.txt
var_dump($plik);
$wartosc= file_get_contents("licznik.txt"); // odczytuje wartość z pliku licznik.txt
$wartosc = (int)$wartosc;
echo "Przed: $wartosc ";
$wartosc++; // zwieksza wartość odczytanego pliku o 1
echo "Po: $wartosc ";
$wartosc = $wartosc + 1; // zwieksza wartość odczytanego pliku o 1
echo "Po2: $wartosc ";
 
echo "Oglądano nas: $wartosc razy."; // wyświetla wartość odczytanego i zwiększonego wcześniej pliku licznik.txt
fwrite($plik, $wartosc); // zapisuje zwiększoną wartość do pliku licznik.txt
fclose($plik); // zamyka otowrzony plik licznik.txt
?>
[PHP] pobierz, plaintext 

resource(2) of type (stream) Przed: 2 Po: 3 Po2: 4 Oglądano nas: 4 razy.

po jednym odświeżeniu:
resource(2) of type (stream) Przed: 4 Po: 5 Po2: 6 Oglądano nas: 6 razy.

[PHP] pobierz, plaintext 
<?php
$plik= fopen(&#092;"licznik.txt\", \"r+\"); //otwiera plik licznik.txt
 
$wartosc= file_get_contents(&#092;"licznik.txt\"); // odczytuje wartość z pliku licznik.txt
$wartosc = (int)$wartosc;
$wartosc = $wartosc + 1; // zwieksza wartość odczytanego pliku o 1
echo &#092;"Oglądano nas: $wartosc razy.\"; // wyświetla wartość odczytanego i zwiększonego wcześniej pliku licznik.txt
fwrite($plik, $wartosc); // zapisuje zwiększoną wartość do pliku licznik.txt
fclose($plik); // zamyka otowrzony plik licznik.txt
?>
[PHP] pobierz, plaintext 

działa ) dzięki

bez liniki:
$wartosc = (int)$wartosc;

oraz ze zmienioną linijką:
$wartosc++ na $wartosc= $wartosc+1
też działa

Ten post edytował kamil_lk 19.03.2009, 11:43:43

Ktoś lub coś musiał dopisać jakieś spacje i tabulacje na koniec pliku licznik.txt i z tego powodu PHP nie potrafił tego zinkrementować po wczytaniu - przetestowałem Twój skrypt i gdy zawartość pliku była poprawna, wszystko działało. Dopisałem na końcu dwa zejścia do nowej linii wraz z tabulacją i przestało. Wczytując dane z plików, musisz pamiętać o sprawdzeniu ich poprawności i odfiltrowaniu syfu, o który naprawdę nietrudno. Inaczej jesteś narażony na takie właśnie kwiatki.

Przy okazji odkryłem ciekawą własność operatora ++. Otóż okazuje się, że potrafi on poprawnie inkrementować również alfabet .

http://pl.php.net/manual/pl/language.operators.increment.php


Zachodzi wowczas domyslna konwersja na inta dlatego dziala

Jako ciekawostkę powiem, że lepiej jest zrobić ++$zmienna (szybsze), no i zamiast fopen, fwrite i fclose używać file_put_contents .

[PHP] pobierz, plaintext 
<?php
$ciastko= setcookie("count","1");
if ($ciastko !="1") {
$plik= fopen("licznik.txt", "r+"); //otwiera plik licznik.txt
$wartosc= file_get_contents("licznik.txt"); // odczytuje wartość z pliku licznik.txt
$wartosc= $wartosc+1; // zwieksza wartość odczytanego pliku o 1
echo "Oglądano nas: $wartosc razy."; // wyświetla wartość odczytanego i zwiększonego wcześniej pliku licznik.txt
fwrite($plik, $wartosc); // zapisuje zwiększoną wartość do pliku licznik.txt
fclose($plik); // zamyka otowrzony plik licznik.txt
}
else {
$plik= fopen("licznik.txt", "r+"); //otwiera plik licznik.txt
$wartosc= file_get_contents("licznik.txt"); // odczytuje wartość z pliku licznik.txt
echo "Oglądano nas: $wartosc razy."; // wyświetla wartość odczytanego i zwiększonego wcześniej pliku licznik.txt
fclose($plik);
}
?>
[PHP] pobierz, plaintext 

moglibyście powiedzieć czy jest to dobrze napisane? chodzi mi o to żeby licznik był odporny na odświeżanie

Zrób może tak (wydaje mi się, że lepiej będzie na sesjach.):

[PHP] pobierz, plaintext 
<?php
ob_start();
session_start();
 
    if (!isset($_SESSION['zaliczono'])) {
 
        $plik= fopen("licznik.txt", "r+"); //otwiera plik licznik.txt
        $wartosc= file_get_contents("licznik.txt"); // odczytuje wartość z pliku licznik.txt
        $wartosc= $wartosc+1; // zwieksza wartość odczytanego pliku o 1
        echo "Oglądano nas: $wartosc razy."; // wyświetla wartość odczytanego i zwiększonego wcześniej pliku licznik.txt
        fwrite($plik, $wartosc); // zapisuje zwiększoną wartość do pliku licznik.txt
        fclose($plik); // zamyka otowrzony plik licznik.txt
        $_SESSION['zaliczono'] = true;
 
    } else {
 
        $plik= fopen("licznik.txt", "r+"); //otwiera plik licznik.txt
        $wartosc= file_get_contents("licznik.txt"); // odczytuje wartość z pliku licznik.txt
        echo "Oglądano nas: $wartosc razy."; // wyświetla wartość odczytanego i zwiększonego wcześniej pliku licznik.txt
        fclose($plik);
    }
?>
[PHP] pobierz, plaintext 

Ten post edytował b4x 19.03.2009, 21:53:19

a czy tamten kod jest poprawny?

Co do tamtego kodu:

1. Ustawiasz ciastko przed zapisaniem wartości - czyli nawet nie dodałeś wartości tylko "na start" dałeś że odwiedził stronę (pierw sprawdź, i jeśli nie odwiedził - ustaw cookie).
2. Sprawdzasz ciasteczko czy jego wartość != 1 funkcja setcookie" title="Zobacz w manualu PHP" target="_manual, odwołaj się do $_COOKIE['count'].

Czyli poprawnie powinno wyglądać jakoś tak:

[PHP] pobierz, plaintext 
<?php
   if (!isset($_COOKIE['count'])) {
 
       $plik= fopen("licznik.txt", "r+"); //otwiera plik licznik.txt
       $wartosc= file_get_contents("licznik.txt"); // odczytuje wartość z pliku licznik.txt
       $wartosc= $wartosc+1; // zwieksza wartość odczytanego pliku o 1
       echo "Oglądano nas: $wartosc razy."; // wyświetla wartość odczytanego i zwiększonego wcześniej pliku licznik.txt
       fwrite($plik, $wartosc); // zapisuje zwiększoną wartość do pliku licznik.txt
       fclose($plik); // zamyka otowrzony plik licznik.txt
       setcookie("count","1"); // ustawiasz ciasteczko jak już zwiększysz ilość ;)
 
   } else {
 
       $plik= fopen("licznik.txt", "r+"); //otwiera plik licznik.txt
       $wartosc= file_get_contents("licznik.txt"); // odczytuje wartość z pliku licznik.txt
       echo "Oglądano nas: $wartosc razy."; // wyświetla wartość odczytanego i zwiększonego wcześniej pliku licznik.txt
       fclose($plik);
   }
?>
[PHP] pobierz, plaintext 

A tak poza tym wygląda ok.

Ten post edytował b4x 19.03.2009, 22:03:41