PHP OO Estructura MVC y POO

natalivg · #1 (**permalink**) 04/10/2013, 07:24

Hola, hace tiempo vengo leyendo este foro y me ha si de mucha ayuda pero ahora quería consultarles algo.

Pues he revisado el modelo MVC y la POO y tengo esto actualmente:

Estructura de Carpetas

Config
controller
css
img
js
model
view

En config tengo esta clase para manejar la coneccion a la base de datos, use PDO por que me parecio mas sencillo ademas de solo tener que cambiar unas cuantas lineas para usar otro motor de BD.

Código PHP:

Ver original<?php
class config
{
var $dsn = 'mysql:dbname=mitabla;host=localhost';
var $user = 'root';
var $password = '';
var $dbms='PDO';
    function connect()
    {
        switch($this->dbms)
        {
        case 'PDO':
        try 
            {
                    $connection= new PDO($this->dsn,$this->user,$this->password);
        
            }
              catch (PDOException $e) 
              {
               echo 'Connection Error: ' . $e->getMessage();
               }
        return $connection;
        break;
        }
    }
}
?>

En la carpeta model tengo las consultas a la base de datos.

Código PHP:

Ver original<?php
require_once '../config/config.php';
class modelo_cargos
{
   function modelo_modificar_cargos($idcargoc,$descripcionc)
    {
        $config= new config();
        $conn=$config->connect();
        $sql="UPDATE cargos SET descripcionc=? WHERE idcargoc=?";
        $descripcioncl=$conn->quote($descripcionc);
        $sth = $conn->prepare($sql);
        $sth->bindParam(1, $descripcioncl,PDO::PARAM_STR, 20);
        $sth->bindParam(2, $idcargoc,PDO::PARAM_INT);
        $sth->execute();
    }
 
}
?>

- Aquí mi primera duda, si yo quisiera hacer un paginador la lógica iría en el
modelo cierto? o me equivoco.
- Y segundo aqui uso quote para entrecomilalr cadenas desagradables y las
guarda asi en la bd y despues con una expresion regular quito las comillas algo
asi:

Ingreso Ca'de''na \"co'\"mpleja y lo guardo como 'Ca''de''''na "co''"mpleja' y aplicandole la expresion regular a la vista Cadena compleja, hacer esto esta bien o hay una forma mejor.

En controller tengo esto:

Código PHP:

Ver original<?php
require_once '../model/modelo_cargos.php';
class controlador_cargos
{
function controlador_modificar_cargos($idcargoc,$descripcionc)
  {
  $modelo_cargos= new modelo_cargos();
  $data=$modelo_cargos->modelo_modificar_cargos($idcargoc,$descripcionc);
  }
}
?>

Aqui solo recibo la consulta y tengo entendido que aqui no deberia haber nada de html correcto?.
Y la view lo tengo asi:

Código PHP:

Ver original<!DOCTYPE html>
<html lang="es">
<head>
<meta charset="ISO-8859-1">
<meta name="Administración Cargos" content="Cargos">
<meta name="keywords" content="Cargos Administarción">
<link rel="stylesheet" href="../css/estilos.css">
<link href="../css/bootstrap.css" rel="stylesheet">
<link href="../css/bootstrap-responsive.css" rel="stylesheet">
<link href="../css/docs.css" rel="stylesheet">
<link href="../js/google-code-prettify/prettify.css" rel="stylesheet">
<script src="../js/mensajes.js"></script>
<script src="../js/limites.js"></script>
<title>Administración Cargos</title>
</head>
<body id="body">
<div id="centrar">
    <header>
     <h1 id="header" >Administración de Cargos</h1>   
    </header>
    <div class="container" id="col">
<?php
require_once '../controller/controlador_cargos.php';
$controlador_cargos= new controlador_cargos();
$idcargoc = isset($_POST["idcargoc"]) ?$_POST["idcargoc"] : NULL;
$descripcionc = str_replace("'","",isset($_POST["descripcionc"]) ?$_POST["descripcionc"] : NULL);
$subm = isset($_POST["subm"]) ?$_POST["subm"] : NULL;
$buscar = isset($_POST["buscar"]) ?$_POST["buscar"] : NULL;
?>
<form action="vista_cargos.php" method="post" enctype="multipart/form-data" class="form-horizontal">
<div class="control-group">
    <div class="controls">
      <input type="hidden"  value="<?php echo $idcargoc; ?>" name="idcargoc">
    </div>
  </div>
  <div class="control-group">
    <label class="control-label" id="bold">Cargo</label>
    <div class="controls">
      <input type="text" placeholder="Ingrese el Cargo" class="span2" value="<?php echo $descripcionc; ?>" name="descripcionc"  onKeyDown="contador(this.form.descripcionc,this.form.remLen,20);" onKeyUp="contador(this.form.descripcionc,this.form.remLen,20);">
    </div>
  </div>
    <div class="control-group">
    <div class="controls">
       <input class="span2" placeholder="" name="buscar" type="text" value="<?php  echo $buscar ?>"/>
       <input type="submit" name="subm" value="Buscar" class="btn btn-btn"/>
       <input type="submit" name="subm" value="Ultimos Registros" class="btn btn-info" />
           <div class="btn-group">
              <a class="btn btn-primary" href="#"><i class="icon-user icon-white"></i></a>
              <a class="btn btn-primary dropdown-toggle" data-toggle="dropdown" href="#"><span class="caret"></span></a>
              <ul class="dropdown-menu">
                <li><input type="submit" name="subm" value="Guardar" class="btn-block btn-success" onClick="return ConfirmSave();"/></li>
                <li><input type="submit" name="subm" value="Modificar" class="btn-block btn-warning"  onClick="return ConfirmModify();"/></li>            
              </ul>
            </div>
    </div>
  </div>    
    </form>
<?php switch($subm):?>
<?php case 'Modificar':?>
<?php if(preg_match("/^[0-9a-zA-ZñÑáéíóú]+$/",$descripcionc)):?>
<?php $controlador_cargos->controlador_modificar_cargos($idcargoc,$descripcionc);?>
<script>
document.location = ('vista_cargos.php');
Modify();
</script>
<?php else:?>
<div class="alert alert-error" id="error">
<?php echo "Solo se permite texto,revise los datos ingresados.";?>
</div>
<?php endif;?>
<?php break;?>
<?php endswitch;?>
</div>
</div>
    <script type="text/javascript" src="../js/widgets.js"></script>
    <script src="../js/jquery.js"></script>
    <script src="../js/bootstrap-transition.js"></script>
    <script src="../js/bootstrap-alert.js"></script>
    <script src="../js/bootstrap-modal.js"></script>
    <script src="../js/bootstrap-dropdown.js"></script>
    <script src="../js/bootstrap-scrollspy.js"></script>
    <script src="../js/bootstrap-tab.js"></script>
    <script src="../js/bootstrap-tooltip.js"></script>
    <script src="../js/bootstrap-popover.js"></script>
    <script src="../js/bootstrap-button.js"></script>
    <script src="../js/bootstrap-collapse.js"></script>
    <script src="../js/bootstrap-carousel.js"></script>
    <script src="../js/bootstrap-typeahead.js"></script>
    <script src="../js/bootstrap-affix.js"></script>
    <script src="../js/holder/holder.js"></script>
    <script src="../js/google-code-prettify/prettify.js"></script>
    <script src="../js/application.js"></script>
</body>
</html>

Aqui uso la sintaxis alternativa porque a mi parecer se separa mejor el codigo PHP del HTML.

Eso es lo que he hecho hasta ahora, quisiera saber si esta bien o si tengo que mejorar algo o de plano lo estoy haciendo mal

.

Gracias de antemano.

bathorz · #2 (**permalink**) 05/10/2013, 14:53

No te hagas mucho problema, para mí tu código va bien. Una sugerencia. Colocar en config.php solamente elementos de configuración del sitio y retirar la conexión a la BBDD. Ej:

Código PHP:

Ver original// Rutas
require_once('Clases/clases.php');
// BBDD
define("DSN",'mysql:dbname=mitabla;host=localhost');
define(USER, 'root');
define(PASS, '');
// Mensajes del sitio
define("SQL100", "....");

Yo no utilizo un mvc estricto, pero tiene algo:

Código txt:

Ver originalFront/   <-- sería la Vista
   index.php
   archivos.php
   ...
Back/    <-- sería el Controlador
   procIndex.php
   procArchivos.php
   ...
Base/
   config.php
   plantilla.php
   Clases/  <-- podría ser el Modelo
      clBD.php
      clArchivos.php
      clPaginacion.php
      ...
   Css/
      estilos.css
      paginador.css
   Funciones/
      funArchivos.php
      funPaginacion.php
      ...
   Js/
      sitio.js
      jquery.js

En lo que se refiere al paginador parece que el mío anda por todos lados:

Código txt:

Ver originalarchivos.php
   <formulario>
procArchivos.php
   recibe los datos del formulario
   instancia la clase clArchivos
   $sql = "..."
   // $numPag:pag actual, $limit:cant result x pag
   clPaginacion::Paginar($sql, $numPag, $limit);
 
clPaginacion.php
   public static $result;
   public static function Paginar($sql,$numPag,$limit) {
      // nº de registro de inicio
      $inicio = ($numPag-1) * $limit;
      // 1ª consulta
      $oStm = clDB::accederBBDD($sql,'ROWS');
      // cuenta los registros
      self::$iCantReg = $iRst = $oStm;
      // carga una variable con la cant paginas
      self::$fCantPags = ceil($iRst/$limit);
      // sql 2ª consulta, agrega limites
      $sql.=" LIMIT ".$inicio.",".$limit."; ";
      // 2ª consulta y retorno de resultados
      return clPaginacion::$result = clDBpdo::accederMySQL($sSQL,'fetch_all_assoc');
 
procArchivos.php
   // los datos paginados para procesar
   $arrayDatos = clPaginacion::$rResult;
   // función que pinta el índice del paginador
   paginador(parametros);
 
archivos.php
   [resultados]
   <paginador>

carlos_belisario · #3 (**permalink**) 05/10/2013, 20:32

Buenas, bienvenido al foro, ahora, has entendido el concepto?? aunque no hay forma correcta o incorrecta porque es solo una forma de organizar código no esta mal darte un poco de concejos

1) Modelo = lógica de negocio (no es solo la conexión y consultas a la base de datos)
2) Vista = lo que ve el usuario, no debería de tener nada de lógica
3) controlador = controla el flujo, carga el modelo que realiza la lógica y renderiza la vista

ahora viendo tu código

Cita:

Iniciado por natalivg

Código PHP:

Ver original<?php
class config
{
var $dsn = 'mysql:dbname=mitabla;host=localhost';
var $user = 'root';
var $password = '';
var $dbms='PDO';
    function connect()
    {
        switch($this->dbms)
        {
        case 'PDO':
        try 
            {
                    $connection= new PDO($this->dsn,$this->user,$this->password);
        
            }
              catch (PDOException $e) 
              {
               echo 'Connection Error: ' . $e->getMessage();
               }
        return $connection;
        break;
        }
    }
}
?>

config debería de referirse a la configuración de la app, la conexión debería de ser otra historia, (interesante siempre el que conozcas un ORM). Otra cosa
var esta OBSOLETO es de php 4, en php5 se usa la visibilidad
otra cosa

Cita:

Iniciado por natalivg

<!DOCTYPE html>
<html lang="es">
<head>
<meta charset="ISO-8859-1">
<meta name="Administración Cargos" content="Cargos">
<meta name="keywords" content="Cargos Administarción">
<link rel="stylesheet" href="../css/estilos.css">
<link href="../css/bootstrap.css" rel="stylesheet">
<link href="../css/bootstrap-responsive.css" rel="stylesheet">
<link href="../css/docs.css" rel="stylesheet">
<link href="../js/google-code-prettify/prettify.css" rel="stylesheet">
<script src="../js/mensajes.js"></script>
<script src="../js/limites.js"></script>
<title>Administración Cargos</title>
</head>
<body id="body">
<div id="centrar">
<header>
<h1 id="header" >Administración de Cargos</h1>
</header>
<div class="container" id="col">
<?php
require_once '../controller/controlador_cargos.php';
$controlador_cargos= new controlador_cargos();
$idcargoc = isset($_POST["idcargoc"]) ?$_POST["idcargoc"] : NULL;
$descripcionc = str_replace("'","",isset($_POST["descripcionc"]) ?$_POST["descripcionc"] : NULL);
$subm = isset($_POST["subm"]) ?$_POST["subm"] : NULL;
$buscar = isset($_POST["buscar"]) ?$_POST["buscar"] : NULL;
?>
<form action="vista_cargos.php" method="post" enctype="multipart/form-data" class="form-horizontal">
<div class="control-group">
<div class="controls">
<input type="hidden" value="<?php echo $idcargoc; ?>" name="idcargoc">
</div>
</div>
<div class="control-group">
<label class="control-label" id="bold">Cargo</label>
<div class="controls">
<input type="text" placeholder="Ingrese el Cargo" class="span2" value="<?php echo $descripcionc; ?>" name="descripcionc" onKeyDown="contador(this.form.descripcionc,this.fo rm.remLen,20);" onKeyUp="contador(this.form.descripcionc,this.form .remLen,20);">
</div>
</div>
<div class="control-group">
<div class="controls">
<input class="span2" placeholder="" name="buscar" type="text" value="<?php echo $buscar ?>"/>
<input type="submit" name="subm" value="Buscar" class="btn btn-btn"/>
<input type="submit" name="subm" value="Ultimos Registros" class="btn btn-info" />
<div class="btn-group">
<a class="btn btn-primary" href="#"><i class="icon-user icon-white"></i></a>
<a class="btn btn-primary dropdown-toggle" data-toggle="dropdown" href="#"><span class="caret"></span></a>
<ul class="dropdown-menu">
<li><input type="submit" name="subm" value="Guardar" class="btn-block btn-success" onClick="return ConfirmSave();"/></li>
<li><input type="submit" name="subm" value="Modificar" class="btn-block btn-warning" onClick="return ConfirmModify();"/></li>
</ul>
</div>
</div>
</div>
</form>
<?php switch($subm):?>
<?php case 'Modificar':?>
<?php if(preg_match("/^[0-9a-zA-ZñÑáéíóú]+$/",$descripcionc)):?>
<?php $controlador_cargos->controlador_modificar_cargos($idcargoc,$descripci onc);?>
<script>
document.location = ('vista_cargos.php');
Modify();
</script>
<?php else:?>
<div class="alert alert-error" id="error">
<?php echo "Solo se permite texto,revise los datos ingresados.";?>
</div>
<?php endif;?>
<?php break;?>
<?php endswitch;?>
</div>
</div>
<script type="text/javascript" src="../js/widgets.js"></script>
<script src="../js/jquery.js"></script>
<script src="../js/bootstrap-transition.js"></script>
<script src="../js/bootstrap-alert.js"></script>
<script src="../js/bootstrap-modal.js"></script>
<script src="../js/bootstrap-dropdown.js"></script>
<script src="../js/bootstrap-scrollspy.js"></script>
<script src="../js/bootstrap-tab.js"></script>
<script src="../js/bootstrap-tooltip.js"></script>
<script src="../js/bootstrap-popover.js"></script>
<script src="../js/bootstrap-button.js"></script>
<script src="../js/bootstrap-collapse.js"></script>
<script src="../js/bootstrap-carousel.js"></script>
<script src="../js/bootstrap-typeahead.js"></script>
<script src="../js/bootstrap-affix.js"></script>
<script src="../js/holder/holder.js"></script>
<script src="../js/google-code-prettify/prettify.js"></script>
<script src="../js/application.js"></script>
</body>
</html>

el controlador carga la vista no la vista al controlador, por lo tanto hacer un require en una vista a un controlador es incorrecto

Cita:

Iniciado por natalivg

- Aquí mi primera duda, si yo quisiera hacer un paginador la lógica iría en el
modelo cierto? o me equivoco.

normalmente uso librerías de terceros de paginación, te doy un ejemplo en Yii lo que se hace es solicitar el provider de datos al modelo y pasarlo a la vista que llama al widget bien sea un grid o un list

Cita:

Iniciado por natalivg

- Y segundo aqui uso quote para entrecomilalr cadenas desagradables y las
guarda asi en la bd y despues con una expresion regular quito las comillas algo

no es necesario andar haciendo cosas con regex ni nada por el estilo, hay metodos que te brinda PDO, como consultas preparadas que te realizan el escape necesario a los caracteres especiales y te evitan SQLInjection.

ya creo que es toda mi observación, Saludos