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();
?>