fix(hw19): Добавил проверку на валидные поля.

otusteamedu · Mar 17, 2024 · e60c8b6 · e60c8b6
1 parent 2697086
commit e60c8b6
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 10 deletions.
diff --git a/app/src/Domain/ValueObject/AbstractValueObject.php b/app/src/Domain/ValueObject/AbstractValueObject.php
@@ -25,7 +25,7 @@ public function getValue(): string
     /**
      * @param string $value
      * @throws Exception
-     * @return mixed Возвращаемое значение после валидации
+     * @return void Возвращаемое значение после валидации
      */
-    abstract protected function validation(string $value): mixed;
+    abstract protected function validation(string $value): void;
 }
diff --git a/app/src/Domain/ValueObject/Email.php b/app/src/Domain/ValueObject/Email.php
@@ -2,16 +2,19 @@
 
 namespace App\Domain\ValueObject;
 
-use Exception;
+use InvalidArgumentException;
 
 class Email extends AbstractValueObject
 {
     /**
      * @param string $value
-     * @return mixed
+     * @return void
+     * @throws InvalidArgumentException
      */
-    protected function validation(string $value): mixed
+    protected function validation(string $value): void
     {
-        return;
+        if (!filter_var($value, FILTER_VALIDATE_EMAIL)) {
+            throw new InvalidArgumentException('Неверный формат email адреса');
+        }
     }
 }
diff --git a/app/src/Domain/ValueObject/Message.php b/app/src/Domain/ValueObject/Message.php
@@ -2,16 +2,20 @@
 
 namespace App\Domain\ValueObject;
 
-use Exception;
+use InvalidArgumentException;
 
 class Message extends AbstractValueObject
 {
     /**
      * @param string $value
-     * @return mixed
+     * @return void
+     * @throws InvalidArgumentException
      */
-    protected function validation(string $value): mixed
+    protected function validation(string $value): void
     {
-        return;
+        // Проверяем, что сообщение не пустое
+        if (empty($value)) {
+            throw new InvalidArgumentException('Сообщение не может быть пустым');
+        }
     }
 }