1

les paso a comentar, estoy queriendo validar unos campos y para esto decidi usar el patron strategy, asi que me dispuse a crear 2 interfaces una para checkear los valores del login

<?php
interface IvalidatorLogin {
    public function validate_username($username);
    public function validate_password($password);
    public function response($username, $password,$email,$is_admin,$status);
}

como veran el metodo response lleva mas argumentos que los que necesita, ya que lo que busco es reutilizar username y passsword para validar tambien el registro de una cuenta nueva, la cual comparte estos parametros.

<?php
interface IvalidatorRegister {
    public function validate_email($email);
    public function validate_admin($is_admin);
    public function validate_status($status);
    public function response($username, $password,$email,$is_admin,$status);
}

En las implementaciones tengo lo siguiente para el ValidatorLogin

<?php
    class ValidatorLogin implements IvalidatorLogin{
        protected $username;
        protected $password;

        public function response($username, $password, $email, $is_admin, $status){
            $username = $this->validate_username($username);
            $password = $this->validate_password($password);

            return $username && $password ? true : false;
        }

        public function validate_username($username){
            if (isset($username) && !empty($username) && $username !== '') {
                if (preg_match('/^[a-zA-Z0-9ñÑ ]+$/', $username)){
                    return true;
                }
            }
            return false;
        }

        public function validate_password($password){
            if (isset($password) && !empty($password) && $password !== '') {
                if (preg_match('/^[a-zA-Z0-9ñÑ ]+$/', $password)){
                    return true;
                }
            }
            return false;
        }

    }

El validatorRegister queda de esta manera

<?php
class ValidatorRegister implements IvalidatorRegister,IvalidatorLogin
{
    protected $username;
    protected $password;
    protected $email;
    protected $is_admin;
    protected $status;

    public function response($username, $password,$email,$is_admin,$status)
    {
        $username = $this->validate_username($username);
        $password = $this->validate_password($password);
        $email = $this->validate_email($email);
        $is_admin = $this->validate_admin($is_admin);
        $status = $this->validate_status($status);
        return $username && $password && $email && $is_admin && $status ? true : false;
    }

    public function validate_username($username)
    {
        if (isset($username) && !empty($username) && $username !== '') {
            if (preg_match('/^[a-zA-Z0-9ñÑ ]+$/', $username)) {
                return true;
            }
        }
        return false;
    }

    public function validate_password($password)
    {
        if (isset($password) && !empty($password) && $password !== '') {
            if (preg_match('/^[a-zA-Z0-9ñÑ ]+$/', $password)) {
                return true;
            }
        }
        return false;
    }

    public function validate_email($email)
    {
        if (isset($email) && !empty($email) && $email !== '') {
            return true;
        }
        return false;
    }
    public function validate_admin($is_admin)
    {
        if($is_admin == 1 || $is_admin == 0){
            return true;
        }
        return false;
    }

    public function validate_status($status){
        if($status == 1 || $status == 0){
            return true;
        }
        return false;
    }
}

El contexto quedo asi:

<?php
    class Validator {
        public $estrategia;

        public function __construct($estrategia)
        {
           $this->estrategia = $estrategia;
            //$this->password = $password;
        }
        public function setEstrategia($estrategia){

            $this->estrategia = $estrategia;


        }

        public function ejecutar ($username,$password,$email = null,$is_admin = null, $status = null){
            return $this->estrategia->response($username,$password,$email,$is_admin,$status);
        }
    }

Lo que me hace ruido es que en el contexto validator tenga que dejar por default los argumentos que en el login no uso. y que en el metodo de la interfaz IvalidatoLogin tenga que declarar argumentos que no voy a usar.Estoy incursionando en estas practicas de los patrones de diseño y cualquier critica constructiva que pueda mejorar el codigo me serviria muchisimo.Saludos!

0

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.