Konstruktor w klasie Opcje

[Szymad]

Witam, mam kod. Uwaga, nazwy zmiennych kopiowałem z jednej funkcji tworząc kolejne, dlatego nazwy są takie same, a odnoszą się do czego innego.

[PHP] pobierz, plaintext 
class Equal{
 
        public $CONFIRM_ID = 1;
        public $ACCOMPANYING_ID = 0;
 
 
 
        public $oldCountCONFIRM = 0;
        public $oldCountACCOMPANYING = 0;
        public $newCountCONFIRM = 0;
        public $newCountACCOMPANYING = 0;
 
        public $oldMagazineCONFIRM = 0;
        public $oldMagazineACCOMPANYING = 0;
        public $newMagazineCONFIRM = 0;
        public $newMagazineACCOMPANYING = 0;
 
        public $differenceCONRIRM = 0;
        public $differenceACCOMPANYING = 0;
 
        public function __constructor(){
 
 
                $this->getProductsConfirm();
                $this->getProductsAccompanying();
                $this->getMagazineConfirm();
                $this->getMagazineAccompanying();
 
        }
 
        protected function getProductsConfirm(){
 
            /**
            * 
            * POBIERA INFORMACJE, KTÓRE PRODUKTY SŁUŻĄ 
            * ZA UPOMINKI DLA POTWIERDZONYCH
            **/
            $link = getConnect();
            $query2 = "SELECT * FROM `ad_config`";
			$ac = mysqli_query($link,$query2);
			$temp = mysqli_fetch_row($ac);
			$for_confirm = $temp[2];
 
 
 
 
            return $for_confirm;
 
        }
 
        protected function getProductsAccompanying(){
 
            /**
            * 
            * POBIERA INFORMACJE, KTÓRE PRODUKTY SŁUŻĄ 
            * ZA UPOMINKI DLA TOWARZYSZĄCYCH
            **/
            $link = getConnect();
            $query2 = "SELECT * FROM `ad_config` WHERE `ID`=2";
			$ac = mysqli_query($link,$query2);
			$temp = mysqli_fetch_row($ac);
			$for_confirm = $temp[2];
 
            return $for_confirm;
 
        }
 
        protected function getMagazineConfirm(){
 
            /**
            * 
            * POBIERA INFORMACJE, JAKI JEST STAN MAGAZYNOWY PRODUKTÓW,
            * KTÓRE PRODUKTY SŁUŻĄ 
            * ZA UPOMINKI DLA POTWIERDZONYCH
            *
            **/
            $CONFIRM_ID = $this->getProductsConfirm();
 
            if(($CONFIRM_ID)>0){
            $link = getConnect();
            $query2 = "SELECT * FROM `products` WHERE `ID`=".$CONFIRM_ID;
			$ac = mysqli_query($link,$query2);
			$temp = mysqli_fetch_row($ac);
			$for_confirm = $temp[2];
 
            $this->$oldCountCONFIRM = $for_confirm;
 
            return $for_confirm;
 
            }else{
 
                throw new Exception("[INNER_OBJ] Unknown ID of for Confirm Products"); 
            }
 
 
 
        }
 
        protected function getMagazineAccompanying(){
 
            /**
            * 
            * POBIERA INFORMACJE, JAKI JEST STAN MAGAZYNOWY PRODUKTÓW,
            * KTÓRE PRODUKTY SŁUŻĄ 
            * ZA UPOMINKI DLA TOWARZYSZĄCYCH
            *
            **/
            $CONFIRM_ID = $this->getProductsAccompanying();
 
            if(($CONFIRM_ID)>0){
            $link = getConnect();
            $query2 = "SELECT * FROM `products` WHERE `ID`=".$CONFIRM_ID;
			$ac = mysqli_query($link,$query2);
			$temp = mysqli_fetch_row($ac);
			$for_confirm = $temp[2];
            $this->$oldCountACCOMPANYING = $for_confirm;
            return $for_confirm;
 
            }else{
 
                throw new Exception("[INNER_OBJ] Unknown ID of for Confirm Products"); 
            }
 
 
 
        }
 
 
 
 
 
 
    public function equal(){
 
 
        echo "\n\tStan w magazynie dla potwierdzonych ".$this->$oldCountCONFIRM;
        echo "\n\tStan w magazynie dla potwierdzonych ".$this->$oldCountACCOMPANTING;
    }
    }
[PHP] pobierz, plaintext 

Otrzymuję: Fatal error: Cannot access empty property in /home/szymadjg/public_html/Lukasz/attendance/edit.php on line 146

Co robię źle?

Ten post edytował Szymad 17.11.2012, 20:59:16

[Crozin]

Ale skąd mamy wiedzieć, skoro nawet nie widzimy/nie wiemy gdzie jest magiczna linia numer 146? Poza tym wrzucaj SSCCE, a nie 150 linii kodu.

[Szymad]

W pliku to ta linia

[PHP] pobierz, plaintext 
  echo "\n\tStan w magazynie dla potwierdzonych ".$this->$oldCountCONFIRM;
[PHP] pobierz, plaintext 

[cudny]

$oldCountCONFIRM pewnie jest null lub 0

[CuteOne]

$this->$oldCountCONFIRM
powinno być
$this->oldCountCONFIRM

[Szymad]

Bardzo dziękuję.

[thek]

Tak tylko jeszcze w kwestii estetyczno-logicznej... W definicji klasy i jej atrybutów nie powinno się im nadawać wartości "na sztywno", czyli: public $CONFIRM_ID = 1; bo to nie miejsce na to. Od tego służy konstruktor. Inna sprawa to sensowność wykonywania niektórych operacji. Zauważ, że getProductsAccompanying() oraz getProductsConfirm() w konstruktorze są kompletnie zbędne. Nic nie robią, nic nie zwracają do użycia, nic nie inicjalizują. Są jednak wykorzystywane wewnątrz pozostałych 2 metod i tam faktycznie to co zwracają jest używane. To co dodatkowo lekko razi także to rzucanie wyjątku w else.