PHP OO Colocar excepciones TRY-CATCH

[plitre_sys19]

Buenas noches! Estoy tratando de colocarle excepciones a un código que estoy utilizando:

Código PHP:

  spl_autoload_register(function($clase){
        include strtolower($clase).'.php';
    });
    Class Categoria{

        private $idCategoriaArea;
        private $nombreCategoria;

        public function __construct($idCategoriaArea=null,$nombreCategoria=null)
        {
            $this->idCategoriaArea=$idCategoriaArea;
            $this->nombreCategoria=$nombreCategoria;

        }

        public function getIdCategoria(){
            return $this->idCategoriaArea;
        }
        public function getNombCategoria(){
            return $this->nombreCategoria;
        }
        public function setIdCategoria($idCategoriaArea){
            $this->idCategoriaArea=$idCategoriaArea;
        }
        public function setNombCategoria($nombreCategoria){
            $this->nombreCategoria=$nombreCategoria;
        }
public function ObtenerTodasCategoria(){
            try {
                    $categoria=array();
                    $db=new Conexion();
                    $conexion=$db->conectar();
                    $consulta=$conexion->prepare('SELECT idCategoriaArea, nombreCategoria FROM categoriaarea ORDER BY idCategoriaArea desc');
                    $consulta->execute();
                    $resultado = $consulta->get_result();
                    while($fila=$resultado->fetch_assoc())
                    {
                        $cat=new Categoria($fila['idCategoriaArea'],$fila['nombreCategoria']);
                        array_push($categoria, $cat);
                    }
                    return $categoria;
            } catch (Exception $e) {
                echo $e->getMessage();
            }}  
   

este es el html:

Código HTML:

<option value="" selected>Seleccione</option>
				<?php
					require_once('categoria.class.php');
						$obj=new Categoria();
						$categoria=$obj->ObtenerTodasCategoria();
						foreach ($categoria as $cat){

							echo '<option value="'.$cat->getIdCategoria().'">'.utf8_decode($cat->getNombCategoria()).'</option>';
						}
				?>

			</select> 

Pero mi duda es recomendable colocarlo en la función de la clase? o en el foreach del formulario?
Me gustaría si por casualidad ven algo en el código en especifico me comentarán

[hhs]

Es sencillo tus clases deben de lanzar excepciones, no manejarlas. Así que mueve el try/catch fuera de tu clase

[plitre_sys19]

Hola hhs, movi el try catch fuera de la clase, y leyendo el manual de php, coloque el throw new Exception cuando se ejecuta la consulta asi:

Código PHP:

  $consulta->execute() or throw new Exception('Error en la consulta.');  
   

y en el html coloque el try catch:

Código HTML:

<select id='combo'>
<option value="" selected>Seleccione</option>
<?php
require_once('categoria.class.php');
 try {
$obj=new Categoria();
$categoria=$obj->ObtenerTodasCategoria();
foreach ($categoria as $cat){
echo '<option value="'.$cat->getIdCategoria().'">'.utf8_decode($cat->getNombCategoria()).'</option>';
}
} catch (Exception $e) {
                echo $e->getMessage();
            }
?>
</select> 

Asi debería es que se debe de trabajar con el try catch?

[hhs]

Es correcto.
Solo un detalle en esta linea de código:

Código PHP:

$consulta->execute() or throw new Exception('Error en la consulta.');
 

Lo adecuado es que la clase Conexion dispare la excepción si sucede algo inesperado en el método execute, pero si es de terceros el código y no quieres meterte en líos déjalo así.

[plitre_sys19]

Iniciado por hhs

Lo adecuado es que la clase Conexion dispare la excepción si sucede algo inesperado en el método execute, pero si es de terceros el código y no quieres meterte en líos déjalo así.

El código lo programe yo, en ese caso como podría ser para que la clase dispare la excepción?

[hhs]

Publica el código de tu clase, pero es igual que en tu primer caso, no importa cual de las dos clases lance una excepción el try/catch la manejara.

[plitre_sys19]

Iniciado por hhs

Publica el código de tu clase, pero es igual que en tu primer caso, no importa cual de las dos clases lance una excepción el try/catch la manejara.

este es el código de la clase categoria.php

Código PHP:

  <?php 
 
spl_autoload_register(function($clase){ 
        include strtolower($clase).'.php'; 
    }); 
    Class Categoria{ 
 
        private $idCategoriaArea; 
        private $nombreCategoria; 
 
        public function __construct($idCategoriaArea=null,$nombreCategoria=null) 
        { 
            $this->idCategoriaArea=$idCategoriaArea; 
            $this->nombreCategoria=$nombreCategoria; 
 
        } 
 
        public function getIdCategoria(){ 
            return $this->idCategoriaArea; 
        } 
        public function getNombCategoria(){ 
            return $this->nombreCategoria; 
        } 
        public function setIdCategoria($idCategoriaArea){ 
            $this->idCategoriaArea=$idCategoriaArea; 
        } 
        public function setNombCategoria($nombreCategoria){ 
            $this->nombreCategoria=$nombreCategoria; 
        } 
 
        public function insertar(){ 
                $link=new Conexion(); 
                $conexion=$link->conectar(); 
                $consulta=$conexion->prepare('INSERT INTO categoriaarea (nombreCategoria) VALUES (?)'); 
                $consulta->bind_param('s',$this->nombreCategoria); 
                if($consulta->execute()) 
                { 
                    $conexion->close(); 
                    $consulta->close(); 
                    return true; 
                } 
                else{ 
                    $conexion->close(); 
                    $consulta->close(); 
                    return false; 
                } 
 
        } 
 
        public function Buscar($value){ 
                $link=new Conexion(); 
                $conexion=$link->conectar(); 
                $consulta=$conexion->prepare('SELECT * FROM categoriaarea WHERE nombreCategoria=?'); 
                $consulta->bind_param('s',$value); 
                $consulta->execute(); 
                $consulta->store_result(); 
                $num=$consulta->num_rows; 
                if($num>=1){ 
                    $conexion->close(); 
                    $consulta->close(); 
                    return true; 
                } 
                else{ 
                    $conexion->close(); 
                    $consulta->close(); 
                    return false; 
                } 
        } 
 
        public function ObtenerTodasCategoria(){ 
     
                    $categoria=array(); 
                    $db=new Conexion(); 
                    $conexion=$db->conectar(); 
                    $consulta=$conexion->prepare('SELECT idCategoriaArea, nombreCategoria FROM categoriaarea ORDER BY idCategoriaArea desc'); 
                    $consulta->execute(); 
                    $resultado = $consulta->get_result(); 
                    while($fila=$resultado->fetch_assoc()) 
                    { 
                        $cat=new Categoria($fila['idCategoriaArea'],$fila['nombreCategoria']); 
                        array_push($categoria, $cat); 
                    } 
                    return $categoria; 
        } 
    } 
?>   

y este es el código de mi clase conexión.php

Código PHP:

  <?php 
     
    Class Conexion{ 
 
        private $server='localhost'; 
        private $usuario='root'; 
        private $pass=''; 
        private $mainDataBase='sistema'; 
         
             /*Realiza la conexión a la base de datos.*/ 
       public function conectar(){ 
          $link=new mysqli($this->server, $this->usuario, $this->pass,$this->mainDataBase); 
 
         //  //Si sucede algún error la función muere e imprimir el error 
        if($link->connect_error){ 
            die("Error en la conexion : ".$link->connect_errno."-".$link->connect_error); 
        } 
 
        //evitando problemas con acentos 
        $link->query("SET NAMES 'utf8'"); 
         
        return $link; 
        } 
        
    } 
 
?>