我想知道我对不同事件的Observer模式的实现,例如’userLoggedOut’,’userRegistered’,’userLoggedIn’是否正确,下面是我的简化代码:
的index.php
$login = new \Observer\Observable\Login;
$userData = $login->getData();
$login->attach( new \Observer\Observer\Email );
$login->notify( 'userLoggedIn' );
$login->notify( 'userLoggedOut', $userData );
$login->notify( 'userRegistered' );
的login.php
namespace Observer\Observable;
use Observer\Interfaces;
class Login implements Interfaces\Observable {
private $observers = array();
function attach( Interfaces\Observer $object ) {
$this->observers[] = $object;
}
function getObservers() {
return $this->observers;
}
function notify( $action, $data = null ) {
foreach( $this->observers as $observer ) :
if( method_exists( $observer, $action ) ) $observer->$action( $data );
endforeach;
}
}
Email.php
namespace Observer\Observer;
use Observer\Interfaces;
class Email implements Interfaces\Observer {
function userLoggedIn( $data = null ) {
echo "email user logged in";
}
function userLoggedOut( $data = null ) {
echo "email user logged out";
}
function userRegistered( $data = null ) {
echo "email user registered";
}
}
我的问题是在处理多个动作时是否有更好的方法来实现Observer模式.
最佳答案 我不认为你的实施是错误的,但它并不适合我.你的Observable必须知道事件处理函数并实现它们(没有某种接口以保证它们在那里.为了继续你的方式,我要确保任何附加到Login的东西应该实现一个接口,保证那些动作函数将在那里.从那里,Login不必学习任何关于观察者的东西,它只是调用函数.
我将采取的另一种方法是特定事件名称的处理程序,如下所示:
class Observable {
protected static $event_names = array();
protected $observers = array();
function __construct() {
foreach (static::$event_names as $event_name) {
$this->observers[$event_name] = array();
}
}
function register($event, $object, $handler) {
if (array_key_exists($event, $this->observers)) {
$this->observers[$event][] = array($object, $handler);
} else {
echo "Invalid event \"$event\"!";
}
}
function trigger($event, $data = null) {
foreach ($this->observers[$event] as $observer) {
$observer[0]->$observer[1]($data);
}
}
}
class Login extends Observable {
protected static $event_names = array("userLoggedIn", "userLoggedOut", "userRegistered");
}
然后你是观察员会注册这样的事件:
class SomeListener {
function __construct() {
$login_instance->register("userLoggedIn", $this, "myLoggedInHandler");
}
function myLoggedInHandler($data = null) {
echo "User Logged In.";
}
}
这只是一个意见,但它是一种不同的方法,但它只需要知道什么事件触发(你的方法可以被认为也可以做,但是Observable应该简单地只调用处理程序方法IMO).