Wip

src/EventInterface.php

@@ -7,10 +7,6 @@
 
 namespace Behavioral\Observer;
 
-/**
- * Interface EventInterface
- * @package Behavioral\Observer
- */
 interface EventInterface
 {
     /**

src/FootballEvent.php

@@ -9,10 +9,6 @@
 
 namespace Behavioral\Observer;
 
-/**
- * Class FootballEvent
- * @package Behavioral\Observer
- */
 class FootballEvent implements EventInterface
 {
     const GOAL = 'Goal!!!';

src/FootballObserver.php

@@ -9,10 +9,6 @@
 
 namespace Behavioral\Observer;
 
-/**
- * Class FootballObserver
- * @package Behavioral\Observer
- */
 class FootballObserver implements ObserverInterface
 {
     /**

src/FootballSubject.php

@@ -9,10 +9,6 @@
 
 namespace Behavioral\Observer;
 
-/**
- * Class FootballSubject
- * @package Behavioral\Observer
- */
 class FootballSubject implements SubjectInterface
 {
     /**
@@ -35,9 +31,14 @@ public function __construct(string $name)
 
     /**
      * @param ObserverInterface $observer
+     * @throws \Exception
      */
     public function attachObserver(ObserverInterface $observer): void
     {
+        if (array_key_exists($observer->getName(), $this->observers)) {
+            throw new \InvalidArgumentException('Observer is already exist');
+        }
+
         $this->observers[$observer->getName()] = $observer;
     }
 
@@ -47,7 +48,7 @@ public function attachObserver(ObserverInterface $observer): void
     public function detachObserver(string $name): void
     {
         if (!array_key_exists($name, $this->observers)) {
-            throw new \InvalidArgumentException();
+            throw new \InvalidArgumentException('Observer is not exist');
         }
 
         unset($this->observers[$name]);

src/ObserverInterface.php

@@ -7,10 +7,6 @@
 
 namespace Behavioral\Observer;
 
-/**
- * Interface ObserverInterface
- * @package Behavioral\Observer
- */
 interface ObserverInterface
 {
     /**

src/SubjectInterface.php

@@ -7,10 +7,6 @@
 
 namespace Behavioral\Observer;
 
-/**
- * Interface SubjectInterface
- * @package Behavioral\Observer
- */
 interface SubjectInterface
 {
     /**

tests/ObserverTest.php

@@ -15,10 +15,6 @@
 use Behavioral\Observer\SubjectInterface;
 use PHPUnit\Framework\TestCase as PHPUnit_Framework_TestCase;
 
-/**
- * Class ObserverTest
- * @package Behavioral\Observer\Tests
- */
 class ObserverTest extends PHPUnit_Framework_TestCase
 {
     /**