[neshken]

Hola a todos,

Me encuentro un poco atareado en el siguiente predicamento: quiero hacer una clase que me permita en una tabla consultar, insertar, actualizar y eliminar. Pero que tome mi clase que he creado para conectar a mi base de datos; la cual es la siguiente:

Clase php dbCon.php

Código:

<?php
class DataBase
{
	// Atributos
	private $dbHost = "localhost";
	private  $dbName = "new_sivca";
	private  $dbUser = "root";
	private $dbPass = "root";

	// Metodos
	public function dbConectar()
	{
		try {
			$dbCon = new PDO("mysql:host={$this->dbHost};dbname={$this->dbName}",$this->dbUser,$this->dbPass);
			$dbCon->exec("SET CHARACTER SET utf8");
			$dbCon->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
			$dbCon->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);
			$dbCon->setAttribute(PDO::MYSQL_ATTR_USE_BUFFERED_QUERY, true);
			$dbCon->setAttribute(PDO::ATTR_PERSISTENT, true);
			return $dbCon;
		} catch (Exception $e) {
			die("<p>— Se ha presentado un error inexperado en la conexión a la base de datos. —</p>
			<b>Error</b>: ".$e->getMessage()."."
			."<br><b>Línea en la que ocurrió el error</b>: ".$e->getCode()
			." presente en el <b>Archivo</b>: <i>".$e->getFile()."</i>"
			."<br><b>Código de error</b>: ".$e->getCode()."."
			."<p>".$e->getTraceAsString()."</p>"
		);
		}finally{
			$dbCon = null;
		}
	}
}
?>

La tabla a al cual deseo hacer esto se llama "ramo"; consta de los siguientes campos: id, ramo, estado. Para ello he creado la clase Ramo. La cual la he definido de la siguiente manera:

Clase php class.Ramo.php

Código:

<?php
/**
 * Clase Ramos
 */

class Ramos
{

	public function getRamosLista()
	{
		try 
		{
			$resultado = array();
			$stm = $this->dbCon->prepare("CALL spRamos_lst()");
		  $stm->execute();
			$row = $stm->fetchAll(PDO::FETCH_OBJ);
		} catch (Exception $e) {
			die("Error: ".$e->getMessage());
		}
		
		#si hay registros
		if ($stm->rowCount()>0)
		{
			while($row){
				echo 
				"<tbody>
	<tr>
		<td>".$row->id."</td>
		<td>".$row->ramo."</td>
		<td>".$row->estado."</td>
	</tr>
</tbody>";
			}
		}
	}
}
?>

De ante mano agradezco cualquier sugerencia al respecto.

Extienda y listo...

class Ramos extends DataBase

[Triby]

La clase DataBase es solo para establecer la conexión?

Si es el caso, podrías crear la propiedad $dbCon en tu controlador principal y pasarla por parámetro a las otras clases. Además, dentro de la clase DataBase simplemente creas una variable y no una propiedad, porque le falta el prefijo $this->

Código PHP:

Ver original
class Ramos {
    private $dbCon;
    // Aquí las demás propiedades de la clase
 
    public function __construct(DataBase $dbCon) {
        $this->dbCon = $dbCon;
    }
    // Aquí el resto de métodos
}
 
// Primero conectas a base de datos
$dbCon = new DataBase();
$dbCon->dbConectar();
 
// Luego creas la instancia de Ramos
$ramos = new Ramos($dbCon);
 

[xerifandtomas]

Aparte de lo comentado por los compañeros para utilizar la conexión.

Decir que es una mala idea pintar las tablas o el html mejor dicho, directamente desde las clases, no por que no sea posible, pero esos métodos serán poco reusables a parte de costoso de mantener a la larga, es mejor retornar los datos y que en la vista se pinten como se quiera. Es decir la clase se debe encargar de retornar solo los datos y nada más. Así lo muestres donde y como lo muestres la clase no se tendrá que tocar.

Ya que mencionas indirectamente que la clase es para un CRUD yo crearía los siguientes métodos base:

Código PHP:

Ver original
class Ramos extends DataBase
{
    public function index($datos_paginacion)
    { 
        // Retorna todos los datos, con o sin paginación, según necesidades
    }
 
    public function show($idRegistro)
    { 
        // Retorna los datos de un registro por su id 
    }
 
    public function update($idRegistro)
    { 
        // Actualiza un registro por su id y retorna index() o show()
    }
 
    public function destroy($idRegistro)
    { 
        // Elimina un registro por su id y retorna index()
    }
 
    public function create($datos)
    { 
        // Añade un nuevo registro y retorna index() o show()
    }
}
 

Creo que poco más te puede hacer falta.