Manejo de excepciones

[Salome]

Hola a tod@s....

Estoy apenas incursionando en el asunto de POO y estoy más enredada!

Les cuento, estoy validando en un método que la cuenta de correo no exista... y si existe debe hacer algo, tomé de ejemplo un método similar que tiene la aplicación, pero por algún motivo no pasa nada...

La idea es que si el email del contacto existe, no deje seguir ejecutando el ingreso del contacto.. al final no sé si deba hacerlo cuándo ya envío los datos para le ingreso o deba hacerlo en la vista, valido desde ahi si existe o no el email de ese contacto ¿que me recomiendan?

Así llamo al método...

Código PHP:

  
if ( false === $manage_contact->existsMail( $_POST['contact_email'] ) )
        
    /*si contact_email no existe, entonces continuo el ingreso...*/
    if ( false === $manage_contact->exists( $_POST['id_contact'] ) )
    {
        $manage_contact->insert( $new_contact );
    }
} 
   

Lo que si funciona es el método exists realiza el ingreso correctamente....

y el método para verificar que el mail del contacto no exista está asi, y no sé si esté bien... es más no tengo claro que intento hacer ahi

Código PHP:

  
 public function existsMail( $contactMail )
    {
        if ( false !== empty( $contactMail ) )
        {
            throw new Exception( "Company id can't be empty" );
        }

        try
        {
            $query = "SELECT
                mos_ap_contact.email
                FROM
                mos_ap_contact
                WHERE
                mos_ap_contact.email = '{$contactMail}'
            ";

            $result = $this->dataBase->query( $query );

            $data = $this->dataBase->fetchAssoc( $result );

            if ( false === empty( $data["email"] ) )
            {
                return true;
            }
        }
        catch ( Exception $exception )
        {
            throw new Exception( "Can't look if the contact exists ({$exception->getMessage()})" );
        }

        return false;
    } 
   

[emiliodeg]

bueno no esta del todo mal, esta casi bien

Código PHP:

public function existsMail( $contactMail )
    {
        if ( false !== empty( $contactMail ) )
        {
            throw new Exception( "Company id can't be empty" );
        }
/*esto lo sacamos del try pq nunca causara un excepcion ya q es una simple asignacion*/
$query = "SELECT
                mos_ap_contact.email
                FROM
                mos_ap_contact
                WHERE
                mos_ap_contact.email = '{$contactMail}'
            ";
 
        try
        {
            
 
            $result = $this->dataBase->query( $query );
 
            $data = $this->dataBase->fetchAssoc( $result );
 
            if ( false === empty( $data["email"] ) )
            {
                return true;
            }
        }
        catch ( Exception $exception )
        {
            throw new Exception( "Can't look if the contact exists ({$exception->getMessage()})" );
        }
 
        return false;
    }
 

siempre deberias usar un bloque try cathc cuando consutlas algo a otra funcion pero no es necesario q anides trys y catchs

este metodo si es parte del modelo del sistema no deberia tener try catch diretametne tirar las excepciones y el controlador manejarlas

espero t sirva la ayuda