Почему не срабатывает шаблон Observer php?

Question

[dllweb]

Привожу код

<?php

interface Observer {
	public function putText($text);
}

interface Observable {

	public function addObserver(Observer $o);
	public function notifyObservers();


}



class PutTextOB implements Observable {

	private $_text;
	private $_observers = array();

	public function __construct($t){

		$this->_text = $t;

	}


	public function addObserver(Observer $o){
		$this->_observers[] = $o;
 	}

 	public function notifyObservers(){
 		foreach ($this->_observers as $instance) {
 			$instance->putText($this->_text);
 		}
 	}



	public function setText(){

		$this->notifyObservers();
		file_put_contents("one.txt", $this->_text, FILE_APPEND);


	}

}



// observables objects

class onePutText implements Observer {

	public function __construct(Observable $o){
		$o->addObserver($this);
	}

	public function putText($text){
		file_put_contents("changedOne.txt", $text, FILE_APPEND);
	}

}

class twoPutText implements Observer {


	public function __construct(Observable $o){
		$o->addObserver($this);
	}


	public function putText($text){
		file_put_contents("changedTwo.txt", $text, FILE_APPEND);
	}

}








$set = new PutTextOB("lol");
$set->setText();
$set->notifyObservers();

?>

Всё это описано в одном файле, так же файлы *.txt лежат в том же каталоге, если по отдельности вызывать эти объекты и вызывать метод putText то всё пишется, но в случае паттерна, должна добавиться запись, которая увы, никак не добавляется во остальные два текстовых файла.

Answer 1

[KorsaR-ZN]

А где вы прикрепляете слушателей?
Как он будет работать, если его события никто не слушает!

Comments

[dllweb]

эт не оно ?

class twoPutText implements Observer {

public function __construct(Observable $o){
$o->addObserver($this);
}

[dllweb]

KorsaR-ZN, Давай сэкономим время, поясни как нужно

[dllweb]

KorsaR-ZN:

<?php

interface Observer {
	public function putText($text);
}

interface Observable {

	public function addObserver(Observer $o);
	public function notifyObservers();


}



class PutTextOB implements Observable {

	private $_text;
	private $_observers = array();

	public function __construct($t){

		$this->_text = $t;

	}


	public function addObserver(Observer $o){
		$this->_observers[] = $o;
 	}

 	public function notifyObservers(){
 		foreach ($this->_observers as $instance) {
 			$instance->putText($this->_text);
 		}
 	}



	public function setText(){

		file_put_contents("one.txt", $this->_text."\n", FILE_APPEND);


	}

}



// observeables objects

class onePutText implements Observer {

	public function putText($text){
		file_put_contents("changedOne.txt", $text."\n", FILE_APPEND);
	}

}

class twoPutText implements Observer {

	public function putText($text){
		file_put_contents("changedTwo.txt", $text."\n", FILE_APPEND);
	}

}








$set = new PutTextOB("lol");
$set->setText();
$set->addObserver(new onePutText());
$set->addObserver(new twoPutText());
$set->notifyObservers();

[dllweb]

KorsaR-ZN: Разобрался в общем, спасибо за внимание, ставлю тебе как решение

[KorsaR-ZN]

dllweb: То, но это немного не корректная реализация, лучше сделать наоборот.
Если вы хотите использовать Вашу реализацию, то Вам нужно так.

Пример

$set = new PutTextOB("lol");

$ob1= new onePutText($set);
$ob2 = new twoPutText ($set)

$set->setText();

Более правильная реализация, но для этого вам из onePutText и twoPutText, нужно удалить конструктор.

Пример

$set = new PutTextOB("lol");
$set->addObserver(new onePutText());
$set->addObserver(new twoPutText());

$set->setText();

После вызова setText, вам не нужно Вызывать notifyObservers, т.к он вызывается уже внутри метода.

[dllweb]

KorsaR-ZN: Я взял за пример второй вариант в комментарии выше, хоть можно и в конструктор onePutText("послать что нибудь ещё") и уже модифицировать сообщение

[KorsaR-ZN]

dllweb: Ну второй вариант лучше, т.к Вы с класса снимаете зависимость от Observable, и можете использовать его самостоятельно, а потом при необходимости зарегистрировать его в Observable.