Question

我想知道我是否针对不同的事件实施了观察者模式，例如＆＃39; userLoggedOut＆＃39;＆＃39; userRegistered＆＃39;，＆＃39; userLoggedIn＆＃39;是正确的，下面是我的简化代码：

的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模式。

Answer 1

我不认为你的实施是错误的，但它并不适合我。你的Observable必须知道事件处理程序的功能并实现它们（没有某种界面来保证它们在那里。为了继续你的方式，我确保任何附加到{{{{ 1}}应该实现一个接口，保证那些动作函数将在那里。从那里，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");
}

这只是一个意见，但它是一种不同的方法，但它只需要知道什么事件发生火灾（你的方法可以被认为也可以做，但是Observable只需要调用处理程序方法IMO）。

PHP中的观察者模式 - 不同事件的正确实践

1 个答案: