Mvc (Modelo Vista Controlador)

syntex · #1 (**permalink**) 13/07/2010, 14:06

hola gente,

Estoy desarrollando un Modelo Vista Controlador y bueno he comenzado ahora hacer este modelo y me gustaria que me pudiesen orientar con lo expuesto aqui.
GLOBALES:

Código PHP:

  <?php 
    define('RUTA_controllers', 'app/controllers/'); 
    define('RUTA_models', 'app/models/'); 
    define('RUTA_views', 'app/views/'); 
    define('RUTA_lib', 'app/lib/'); 
    define('RUTA_layouts', 'app/views/layouts/'); 
?>

CLASE PADRE PREPARADA

Código PHP:

  <?php 
class User_Base{ 
    function __construct(){ 
        global $Bd; 
        $this->Bd = $Bd; 
    } 
} 
?>

CONTROLLERS

Código PHP:

  <?php 
/** 
* Controller 
**/ 
class Usuarios_controller extends User_Base{ 
private $userid; 
private $user; 
public $texto_user_no; 
    function __construct(){ 
        parent::__construct();     
    } 
     public function login( $user ,$password) { 
        require_once(RUTA_models."portada/portada.php"); 
        $user = new usuarios($_POST[user],$_POST[password]); 
        $userid = $user->validateUser(); 
        if($userid == 0 ){ 
                  //****************************************** 
                 if ((empty($_POST[user]) or empty($_POST[password])) and $_POST[xs] !=""){ 
                    ob_start();         
                      require_once(RUTA_layouts."portada/portada.php"); 
                    $this->texto_user_no = ob_get_clean(); 
                } 
                //****************************************** 
               require_once(RUTA_views."portada/sin_loginear.php"); 
          }else{  
             require_once(RUTA_views."portada/logineado.php"); 
          } 
     } 
}     
?>

MODELS

Código PHP:

  <?php 
/** 
* Model 
**/ 
class usuarios extends User_Base{ 
    public $user; 
    private $password; 
    private $userid; 
    function __construct( $user, $password ) { 
        $this->user = $user; 
        $this->password = $password; 
        parent::__construct(); 
    } 
   public function validateUser( ) { 
        $R = $this->Bd->consultar("email,password","usuarios","WHERE (email = '".$_POST[user]."' and password = '".md5($_POST[password])."')",""); 
        $this->dato = $this->Bd->num_rows($R); 
        $this->Bd->limpiaconsulta($R); 
       if( $this->dato !=  0 )  { 
           return $this->dato; 
       } else { 
          return 0; 
       } 
   } 
}   
?>

VIEWS
SIN LOGINEARSE

Código HTML:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Documento sin título</title>
</head>

<body>
<?= $this->texto_user_no; ?>
<form action="index.php" method="post">
	<input type="text" name="user" value="<?= $_POST[user]; ?>" />
    <input type="text" name="password" value="<?= $_POST[password]; ?>" />
    <input type="submit" name="xs" value="enviar" />
</form>
</body>
</html>

LOGINEADO

Código HTML:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Documento sin título</title>
</head>

<body>
<p>El usuario <?= $user->user; ?> existe
</body>
</html>

layouts

Código HTML:

<h1>Usuario inválido</h1>
<span>Introduzca correctamente la cuenta de usuario.</span>

Espero que me puedan ayudar y mejorar todo lo reportado aqui

Gracias.

GatorV · #2 (**permalink**) 13/07/2010, 14:13

Pues dos cosas, no uses global, es mas dificil saber si cambias una variable y esta es global, y 2, no heredes todo de una sola clase, mejor diseña correctamente tu modelo UML y ve las relaciones, heredar todo de una clase es incorrecto ya que son objetos diferentes.

Saludos.

syntex · #3 (**permalink**) 13/07/2010, 14:17

Hola.

Y entonces si quisiera recuperar el valor de unla variable dentro de la clase como podria hacerlo que no fuera con global?

El constructor de la clase padre lo e definido para mantener las variables mas usadas dentro de las clases extendidas , no seria correcto???

Gracias

GatorV · #4 (**permalink**) 13/07/2010, 14:25

Puedes instanciar la clase dentro de las clases que sean necesarias.

No, por eso te recomiendo antes de programar, mejor diseña correctamente la estructura de tu sistema, y asigna responsabilidades a todo lo que necesites en base al alcance de cada clase.

syntex · #5 (**permalink**) 14/07/2010, 02:15

En la clase padre habia añadido la variable del objeto para la base de datos y así realizar la consulta en la clase extendida.

global $Bd; //[Variable objeto en la clase padre]

Pero entonces mejor instanciar la clase dentro de las clase padre así? Y destruir el global de la clase Padre?

MODELO

Código PHP:

  <?php
/**
* Model
**/
class usuarios extends User_Base{
    public $user;
    private $password;
    private $userid;
    function __construct( $user, $password ) {
        $this->user = $user;
        $this->password = $password;
        parent::__construct();
    }
   public function validateUser( ) {
        $R = $this->Bd->consultar("email,password","usuarios","WHERE (email = '".$_POST[user]."' and password = '".md5($_POST[password])."')","");
        $this->dato = $this->Bd->num_rows($R);
    $this->Bd->limpiaconsulta($R);
        $this->Bd->cerrarconexion();
       if( $this->dato !=  0 )  {
           return $this->dato;
       } else {
          return 0;
       }
   }
}  
?>

CLASE PADRE

Código PHP:

  <?php
class User_Base{
    public $Bd;
    private $host;
    private $dbuser;
    private $dbpass;
    private $select;
    function __construct(){
        include_once(RUTA_lib."class_bd.php");
        $host = "localhost";
        $dbuser = "xxxxxxxxx";
        $dbpass= "xxxxxxxxxxxxxxxxxxx";
        $select = "xxxxxxxxxxxxxxxxxxxx";
        $Bd = new conectarMySQL("mysql",$host,$dbuser,$dbpass,$select);
        $Bd->conectar();
        $this->Bd = $Bd;
    }
}
?>

Muchas gracias,

syntex · #6 (**permalink**) 14/07/2010, 02:50

En el controlador quiero mostrar un loyout de una vista html, no se si lo digo correcto por comenzar ahora a programar en MVC, y un poco pez.

La manera conveniente sería en el controlador.

CONTROLADOR

Código PHP:

  <?php
//******************************************
            if ((empty($_POST[user]) or empty($_POST[password])) and isset($_POST[xs])){
                ob_start();        
                  require_once(RUTA_layouts."portada/portada.php");
                $this->texto_user_no = ob_get_clean();
            }
            //******************************************
?>

loyout de una vista html

// require_once(RUTA_layouts."portada/portada.php"); //

Código PHP:

  <h1>Usuario inv&#225;lido</h1>
<span>Introduzca correctamente la cuenta de usuario.</span>

O añadir la vista template en una funcion en la clase para hacerla llamar en la vista???

Actualmente esta así

Código HTML:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Documento sin título</title>
</head>

<body>
<?= $this->texto_user_no; ?>
<form action="index.php" method="post">
	<input type="text" name="user" value="<?= $_POST[user]; ?>" />
    <input type="text" name="password" value="<?= $_POST[password]; ?>" />
    <input type="submit" name="xs" value="enviar" />
</form>
</body>
</html>