[SOLUCIONADO] PHP orientado a objetos

[user_fdw]

Hola a todos.

Si tengo un objeto, con algunos metodos para desplegar vistas, algo asi como un controlador, que forma me recomienda para desplegar las vistas, haciendo require o mejor con header('location:...')???

De antemano muchas gracias.

[MaNuX0218]

Antes yo te invito a saber que es cada cosa, creo que con eso responderá tu pregunta:

Require(): http://php.net/manual/es/function.require.php

Header(): http://php.net/manual/es/function.header.php

Saludos.

[user_fdw]

Hola, gracias por tu respuestas!

tengo nocion de lo q hace cada una, es solo q en el caso q planteo, no se cual seria mas recomendable usar, cual me sugeririas?

[chronos682]

Sería con requiere, no entiendo cómo podrías incluir un objeto con header location.

[user_fdw]

hola chronos682, gracias por tu respuesta!

pero si es una buena practica hacer un require de vista dentro de un objeto?

[chronos682]

Que es lo que quieres hacer exactamente con el objeto o cuál es la función del mismo? El require lo necesitas para instanciar el objeto? O específicamente para que?

[user_fdw]

el objeto es un controlador, por decirlo de alguna manera, en ese objeto, tengo varios metodos, entre ellos, hay algunas q son para desplegar vistas, sera bueno q las vistas las despliegue con require? lo digo por como el codigo de la vista se incluiria en el objeto al hacer el requiere. mi intencion es desplegar vistas de la mejor manera aplicable en ese caso, ya sea por require o por redireccionar a esa vista con header.

o si usted tiene alguna sugerencia se lo agradeceria mucho chronos682.

[chronos682]

Podrias poner el codigo del objeto para ver como funciona porque sigo aun sin entender bien.

[user_fdw]

este es el codigo chronos682

Código PHP:

  <?php 
    require '../modelo/sesion.php'; 
    new sesion("../index.php");    

    require '../modelo/usuario.php';
    $usuario=new cls_usuario();

    class controlador_admin{
        private $accion;
        private $contenido;
        private $msj;

        function __construct($usuario) {

            $this->accion=isset($_REQUEST['accion'])?$_REQUEST['accion']:null;
            $this->contenido=array();

            $row=$usuario->ver_mi_info();

            if ($this->accion=="vista_gestion_usuarios" || $this->accion==null) {
            
                $this->contenido[0]="gestion_usuarios.php";
                $this->contenido[1]="?accion=vista_gestion_usuarios";
            
            } else if ($this->accion=="vista_transferencias") {
            
                $this->contenido[0]="transferencia_proyecto.php";
                $this->contenido[1]="?accion=vista_transferencias";
            
            } else if ($this->accion=="vista_trayectos") {
            
                $this->contenido[0]="trayectos.php";
                $this->contenido[1]="?accion=vista_trayectos";
            
            } else if ($this->accion=="vista_bitacora") {
            
                $this->contenido[0]="bitacora.php";
                $this->contenido[1]="?accion=vista_bitacora";

            } else if ($this->accion=="vista_mi_perfil") {
            
                $this->contenido[0]="mi_perfil.php";
                $this->contenido[1]="?accion=vista_mi_perfil";

                $row=$usuario->ver_mi_info();
            
            } else if ($this->accion=="vista_cambiar_clave") {
            
                $this->contenido[0]="cambiar_clave.php";
                $this->contenido[1]="?accion=vista_cambiar_clave";

            } else if ($this->accion=="agregar_usuario") {
            
                $this->msj=$usuario->agregar_usuario($_POST['nombre'], $_POST['apellido'], $_POST['tipo_usu'], 
                    $_POST['correo'], $_POST['usuario'], $_POST['clave']);

                $this->contenido[0]="gestion_usuarios.php";
                $this->contenido[1]="?accion=vista_gestion_usuarios";
            
            } else if ($this->accion=="eliminar_usuario") {
            
                $this->msj=$usuario->eliminar_usuario($_POST['id']);

                $this->contenido[0]="gestion_usuarios.php";
                $this->contenido[1]="?accion=vista_gestion_usuarios";
            
            } else if ($this->accion=="modificar_usuario") {

                $_POST['tipo_usu']=isset($_POST['tipo_usu']) ? $_POST['tipo_usu'] : null ;

                if ($_POST['tipo_usu']) {
                    $this->msj=$usuario->modificar_usuario($_POST['id'], $_POST['nombre'], $_POST['apellido'], 
                    $_POST['tipo_usu'], $_POST['correo'], $_POST['usuario']);
                } else {
                    $this->msj=$usuario->modificar_usuario($_POST['id'], $_POST['nombre'], $_POST['apellido'], 
                        null, $_POST['correo'], $_POST['usuario']);                    
                }

                $this->contenido[0]="gestion_usuarios.php";
                $this->contenido[1]="?accion=vista_gestion_usuarios";
            
            } else if ($this->accion=="modificar_mi_info") {

                $this->msj=$usuario->modificar_usuario($_POST['id'], $_POST['nombre'], $_POST['apellido'], null,
                $_POST['correo'], $_POST['usuario']);                    

                $this->contenido[0]="mi_perfil.php";
                $this->contenido[1]="?accion=vista_mi_perfil";
            
            } else if ($this->accion=="cambiar_clave") {
            
                $this->msj=$usuario->modificar_mi_clave($_POST['id_usuario'], $_POST['clave_actual'],
                    $_POST['clave_nueva']);

                $this->contenido[0]="cambiar_clave.php";
                $this->contenido[1]="?accion=vista_cambiar_clave";
            
            } else if ($this->accion=="filtrar") {

                $this->contenido[0]="gestion_usuarios.php";
                $this->contenido[1]="?accion=vista_gestion_usuarios";
                
                isset($_REQUEST['parametro'])?$_REQUEST['parametro']:null;

                if (isset($_REQUEST['parametro']) && $_REQUEST['parametro_txt'] != "") {

                    if ($_REQUEST['parametro']=="nombre_perfil") {
                        $usuario->listar_usuarios("SELECT * FROM usuario INNER JOIN perfil 
                            WHERE usuario.id_perfil=perfil.id_perfil AND perfil.".$_REQUEST['parametro']." 
                            LIKE '$_REQUEST[parametro_txt]%' ORDER BY perfil.".$_REQUEST['parametro']." ASC");
                    }else{
                        $usuario->listar_usuarios("SELECT * FROM usuario INNER JOIN perfil 
                            WHERE usuario.id_perfil=perfil.id_perfil AND usuario.".$_REQUEST['parametro']." 
                            LIKE '$_REQUEST[parametro_txt]%' ORDER BY usuario.".$_REQUEST['parametro']." ASC");
                    }

                } else {
                    $usuario->listar_usuarios("SELECT * FROM usuario INNER JOIN perfil 
                        WHERE usuario.id_perfil=perfil.id_perfil ORDER BY ".$_REQUEST['parametro']." ASC");
                }

                return false;
            } else if ($this->accion=="cerrar_sesion") {
            
                session_start();
                session_destroy();
                
                header("location:../index.php");
                return false;
            } 

            require '../vista/vista_admin/vista_admin.php';
        }
    }

    $controlador_admin=new controlador_admin($usuario);
 ?>   

aclaro que en gran medida el codigo ha sido condicionado por otras personas para q sea asi, yo se q hay muchas fallas, yo no lo hubiese programado asi, pero me adapto.

mi curiosidad surge por saber buenas practicas.

[chronos682]

Ya creo mas o menos entender que es lo que hace. Se supone que al llamar la clase desde tu código esta determina que acción es la que se quiere hacer y de acuerdo a eso genera un array con dos elementos el primero en el que está el nombre del archivo al que apuntará y el segundo las variables que se pasarán por get a través de la url, al final del código (supongo) lo que quieres hacer es que junte los dos elementos p ej: trayectos.php?accion=vista_trayectos y se dirija a esa página. Si es así ahí si tienes que usar el header location pero recuerda que tienes que invocarlo antes de escribir cualquier código html en pantalla para que funcione o sino te va a dar error.

[user_fdw]

fino chronos682, aclaraste mi duda, muchas gracias por tu ayuda y tu tiempo!

saludos.

[hhs]

mi curiosidad surge por saber buenas practicas.

Espero que tengas tiempo por que tienes mucho trabajo por delante, comenzado por hacer una buena refactorización del código que presentas.