[PHP]licznik błąd w fclose Opcje

[Alessio]

eh zaraz mnie cos trafi,

[PHP] pobierz, plaintext 
<?php
class Licznik
{
private $licz;
private $plik;
 
public function __contruct()
  {
    if(!(file_exists("licznik.txt") || !isset($_COOKIE['zliczony'])))
    {
      $this->plik = fopen("licznik.txt", "w+");
      fputs($this->plik = "-");
      $this->licz = 1;
 
    }
    else 
    {
      $this->licz = filesize("licznik.txt");
      $this->plik = fopen("licznik.txt");
 
    }
    if (!$_COOKIE['zliczony'])
    {
      flock($this->plik,2);
      fputs($this->plik,"-");
      flock($this->plik,3);
      setcookie('zliczony',1);
    }
  }
 
public function __destruct()
  {
  fclose($this->plik);
  }
public function pokazLicznik()
  {
  return $this->licz;
  }
}
$Licznik = new Licznik();
echo "Stronę tą odwiedzono ".$Licznik->pokazLicznik()." razy";
 
?>
 
[PHP] pobierz, plaintext 

co tu jest źle
Warning: fclose(): supplied argument is not a valid stream resource in C:\wamp\www\obiektowy\test.php on line 33
czemu? może mi ktoś poprawić ten kod

Ten post edytował Alessio 16.10.2009, 19:46:14

[Fifi209]

Bo jeżeli wykonuje się destruktor to niszczy obiekt ($this) z tego co wiem.

[Zyx]

Panowie, z całym szacunkiem, ale powypisywaliście straszliwe głupoty. Obiekt niszczony jest po zakończeniu działania przez destruktor, to po pierwsze, a po drugie on tu akurat nie ma nic do rzeczy. I co ma niby kolejność jego napisania/wystąpienia do błędów?

Alessio -> Czy nie zastanowiłeś się, dlaczego błąd pojawia się dopiero w destruktorze, skoro teoretycznie powinien wywalić się wcześniej na fputs() i flock(), jeśli plik nie został otwarty? Przyjrzyj się nazwie konstruktora, bo na mój gust zjadłeś tam jedną literkę i on się po prostu nie wykonuje...

Ten post edytował Zyx 16.10.2009, 20:41:38

[Fifi209]

Alessio -> Czy nie zastanowiłeś się, dlaczego błąd pojawia się dopiero w destruktorze, skoro teoretycznie powinien wywalić się wcześniej na fputs() i flock(), jeśli plik nie został otwarty? Przyjrzyj się nazwie konstruktora, bo na mój gust zjadłeś tam jedną literkę i on się po prostu nie wykonuje...

Jestem ślepy, ale moim zdaniem ma dobrze jeżeli chodzi o konstruktor, w szczególności że się wykonuje kolego.

[Zyx]

Ech...

__contruct <---- tak ty napisałeś
__construct <---- tak wymaga PHP

Sprawdź dokładnie, bo Twoja metodologia testowania wydaje mi się mocno naciągana, albo masz jakieś dziwne PHP .

PS. Uwaga ortograficzna: "TĘ stronę", a nie "TĄ stronę"

Ten post edytował Zyx 16.10.2009, 20:48:33