Klasa zwraca samą siebie Opcje

[tadeurz]

Mam problem, nie wiem czy jest to wogóle możliwe.Mam 2 klasy Element i Box, chce żeby wywołanie metody Element.save() zwróciło samą siebie do metody Box.storeElements()

[PHP] pobierz, plaintext 
class Box{
 public static function storeElement($element){
  if( $element->type == 'red' ) self::goToRedElements();
   else self::goToDefaultElements();
 }
}
class Element{
 private type = 'default';
 public function save(){
  $this->type = 'red';
  box::storeElement(this); //tutaj jak zwrócić instancje obiektu Element ?
 }
}
/**************************/
 $element = new Element();
 $element->save();
 
//nie chce tego robić tak:
 $element = new Element();
 $element->save();
 //z metody save() wylatuje wywołanie box::storeElement();
 box::storeElement($element);
[PHP] pobierz, plaintext 

Ten post edytował tadeurz 4.03.2013, 02:00:53

[nospor]

nie: this
a: $this
...

Jak zwykle w manualu takie rzeczy są bardzo dobrze wyjaśnione.

[tadeurz]

Oczywiście masz racje.
Co ciekawsze teraz tak na świeżo jak przeglądniemy się zwróconej notce i dokładnie przeczytamy:

[PHP] pobierz, plaintext 
Notice: Use of undefined constant this - assumed 'this' in /xxx/xxxxx.php on line xx
[PHP] pobierz, plaintext 

I zastanowimy dlaczego pisze coś o stałej, cały problem rozwiązany.

[domo]

po pierwsze źle definiujesz składową 'type' klasy Element
zamiast

[PHP] pobierz, plaintext 
private type = 'default';
[PHP] pobierz, plaintext 

powinno być

[PHP] pobierz, plaintext 
private $type = 'default';
[PHP] pobierz, plaintext 

w metodzie save() ponownie źle przekazujesz parametr.
zamiast:

[PHP] pobierz, plaintext 
 public function save(){
  $this->type = 'red';
  box::storeElement(this); //tutaj jak zwrócić instancje obiektu Element ?
 } 
[PHP] pobierz, plaintext 

powinno być

[PHP] pobierz, plaintext 
 public function save(){
  $this->type = 'red';
  box::storeElement($this); //tutaj jak zwrócić instancje obiektu Element ?
 }
[PHP] pobierz, plaintext 

Kolejnym błędem jest odwoływanie się do prywatnej składowej 'type' klasy Element w sposób jaki to robimy dla publicznych składowych.

[PHP] pobierz, plaintext 
 public static function storeElement($element){
  if ($element->type == 'red' ) self::goToRedElements();   // type jest prywatna!
   else self::goToDefaultElements();
 }
[PHP] pobierz, plaintext 

Jeśli chcesz mieć dostęp tylko do odczytu wartości 'type' musisz dodać metodę do klasy Element zwracającą tę wartość.

np:

[PHP] pobierz, plaintext 
public function getType()
{
return $this->type;
}
[PHP] pobierz, plaintext 

Poniżej wklejam poprawnie napisany kod:

[PHP] pobierz, plaintext 
<?php
 
class Box
{	
	public static function storeElement($element){
		if ($element->getType() == 'red') {
			self::goToRedElements();
		} else {
			self::goToDefaultElements();
		}		
	}
}
class Element
{
	private $type = 'default';
 
	public function getType()
	{
		return $this->type;
	}
 
	public function save(){
		$this->type = 'red';
		box::storeElement($this); //tutaj jak zwrócić instancje obiektu Element ?
	}
}
/**************************/
$element = new Element();
$element->save();
 
//nie chce tego robić tak:
$element = new Element();
$element->save();
//z metody save() wylatuje wywołanie box::storeElement();
box::storeElement($element);
 
?>
[PHP] pobierz, plaintext 

Ten post edytował domo 4.03.2013, 11:00:04

[tadeurz]

Wszytkie Twoje uwagi są słuszne.