Está bien mi código?

[mathiasmontiel]

Hola cómo están? Estoy aprendiendo PHP por cuenta propia desde hace ya un tiempo, pero solo cosas superficiales, y según estuve investigando creo que agarré unas malas costumbres al escribir PHP estructurado. Poco a poco estoy corrigiendo esas malas costumbres pero ahora me siento muy inseguro al escribir código y más todavía con POO y las clases, funciones, etc.

Tengo este pequeño trozo de PHP, es solo una práctica y quisiera saber si hay algo que se pueda corregir en cuanto a la manera de escribir el código. El código original tiene más funciones y validaciones pero quité varias partes porque todo es casi lo mismo y sería muy largo.
Mi intención es escribir código más limpio ya que por querer apresurarme aprendí mal y recién ahora me di cuenta. Estoy leyendo algunos artículos pero no detallan mucho sobre el código sucio y como soy novato a veces no me doy cuenta cuando algo está mal.

El código es muy sencillo creo que ni hace falta explicar:

Código PHP:

  <?php
require_once('modelo.php');
require_once('vista.php');

function controlador() {

    if (isset($_POST['enviar'])) {
        
        $datos = helper();
        $usuario = set_obj();
        foreach ($datos as $campo=>$valor) {
            $$campo = $valor;
        }
        if (empty($texto)){
            retornar_vista(VISTA_ERROR);
        } else {
            retornar_vista(VISTA_TEXTO);
        }
        
    } elseif (!empty($_GET['url'])) {

        $event=$_GET['url'];    
        $datos = helper();
        $usuario = set_obj();
        
        switch ($event) {
            case url:
                $usuario->get($url);
                $data = array(
                    'nombre'=>$usuario->url
                );
                retornar_vista(VISTA_URL, $data);
                break;
            case help:
                $usuario->set($user_data);
                $data = array(
                    'nombre'=>$usuario->help
                );
                retornar_vista(VISTA_HELP, $data);
                break;
        }        
    } else {
        retornar_vista(VISTA_HOME);
    }
}

function set_obj() {
    $obj = new Usuario();
    return $obj;
}

function limpiar($tags){
    $tags = strip_tags($tags);
    $tags = stripslashes($tags);
    $tags = htmlentities($tags);
    return $tags;
} 

function helper() {
    $datos = array();
    if ($_POST) {
        if (array_key_exists('titulo', $_POST)) {
            $titulo = limpiar($_POST['titulo']);
            $datos['titulo'] = $titulo;
        }
        if (array_key_exists('descripcion', $_POST)) {
            $descripcion = limpiar($_POST['descripcion']);
            $datos['descripcion'] = $descripcion;
        }
        $datos['timestamp'] = time();
    } elseif ($_GET) {
        if (array_key_exists('url', $_GET)) {
            $datos = $_GET['url'];
        }
    }
    return $datos;
}

controlador();
?>   

[abimaelrc]

Si hace lo que quieres, entonces no hay que preocuparse por que está bien. En todo caso ¿te está dando algún error? Si no es así, entonces solo preocupate por aprender a trabajar con la seguridad de los datos que cuando se inserte donde quieras guardar la información, es decir, que no haya un ataque XSS o una inyección sql. Te recomiendo que búsques bajo estos términos en internet. Si quieres aprender sobre algún estandar en codificar puedes usar http://www.phptherightway.com/

[mathiasmontiel]

Si, funciona perfecto. Por ahora solo estoy tratando de sacarme las malas costumbres ya que antes hacía todo a mi manera sin conocer muchas cosas básicas del PHP. Lo de la seguridad también, se lo importante que es aunque no profundicé mucho todavía.
Voy a ver el link que me pasaste, parece interesante, seguro me servirá. Gracias.