Refactorizando código espagueti

Contenido del curso

Introducción

¿Cómo combinar HTML con PHP?

Manejo de formularios

Proyecto: formulario de contacto

Conclusión

24
Es tu turno de poner todo en práctica
01:50 min

Tomar examen

Cursos Empresas Blog Live Conf Eventos Precios

0:0018:19

Refactorizando código espagueti

Curso de PHP: Integración con HTML

Clase 9 de 24

Clase 9 / 24

José Alberto Ramírez Quiroz

Hace 4 años

¿Por que no usar echo para imprimir HTML, afecta en el tema de seguridad?

César Luis Amundaray Rodríguez

Hace 4 años

Supongo que es porque puedes colocar el HTML directamente.

Kenneth Angulo L

Hace 4 años

Tecnicamente se puede, y lo hemos hecho en los ejercicios de las clases, de manera didactica, para el aprendizaje. . Una profesora dijo una vez, solo porque algo es posible, no es que debamos hacerlo !! . 📜 Pero cuando hablamos de buenas practicas, la regla, la orden, el mandato, la obligacion que tenemos al desarrollar es NO HACERLO. . Por que ? buena pregunta ! . Y aqui es donde cada persona experimentada, avanzada en la practica, podra tener sus propias opiniones. 🏆 . Se me ocurren algunas:

Orden,
Legibilidad del codigo.
Facilidad de mantenimiento.
Una importantisima para hacer codigo modular es “separation of concerns”, tengo archivos php que se encargen de la logica, y lo que tenga que ver con mostrar en pantalla se lo dejo a las vistas o plantillas (tambien archivos php pero solo ellos tienen pemritido generar html)
.

Ya que tocas el tema seguridad, eso es otro tema que da para otro curso completo del gran Retax Master !!. . Sobre la seguridad puedo decir hace mas danio lo que viene hacia el servidor que lo que sale del server. a no ser que alguien para pruebas haga un echo del array que contiene la configuracion de la base de datos y lo deje expuesto en produccion. 🤡

Comentarios (42)42 comentarios

Cristobal Nyram

**trucaso de la clase **

echo "<pre>"
echo  vardump($arreglo);
echo "</pre>"

¡Magia tenemos un arreglo muy chulo!

Edgar Castro

Muy buen truco!

Solamente que el vardump() no requiere de "echo".

Luis Miguel Falcon Montaño

Esta es la forma correcta:

echo "<pre>";
var_dump($tabla_del_9);
echo "</pre>";

Gabriela Ugarte Maco

Excelente! me encantó la refactorización. Realmente hay muchas maneras de hacerlo, pero siempre usaremos la más legible . gracias Profe!!

Yerson Felipe Ramirez Garnica

Yo lo hice de la siguiente manera :D

<?php 
$multiplicacion = readline("Que numero para multiplicar");
$array = array(1,2,3,4,5,6,7,8,9,10);
foreach ($array as $valor) {
   $resultado = $multiplicacion * $valor . "\n";
   echo "$multiplicacion x $valor = $resultado";
}
?>

david bryan alcocer cañete

Muy buen ejemplo, resaltando la importancia de hacer código limpio.

Juan David Rueda Quiroga

en lugar de

array_push();

es más optimo usar

$tabla_del_9[] = $resultado

Luis Alberto Aguilar Muñoz

Pero neesitarias otro ciclo para recorrer la posicion, en cambio array_push guarda el valor en la ultima posicion.

Joel Dominguez Merino

Muy buen ejemplo de como refactorizar un código lo más legible posible. La idea de crear un arreglo de puro string donde esta tanto el texto y la operación no me agrada del todo pero entiendo que es mejor eso a tener un poquito de lógica en el HTML. Gracias Carlos! 😁

Una clase retadora. Hay que procurar escribir siempre tomando en cuenta estas recomendaciones.

Jossie Steeven Quintero Giron

Cuando estudien algún framework como Laravel, aplicaremos este tipo de separación. En los controladores se escribe la lógica y el procesamiento de datos y el la vistas reciben esos datos y los muestran al cliente

Antonio Cortes Avila

Hace 2 años

Impresionante se me vino a la mente hasta la segunda refactorizacion. Lo ultimo no se me hubiera ocurrido nunca

Juan Omar Palma álvarez

con algunas incerciones m{as logr{e que al cambiar el valor $numero se muestre la tabla de multiplicar de la variable $numero ![](

¡RECUERDA!

¡NUNCA! NUNCA IMPRIMIR ETIQUETAS HTML EN PHP

Sergio Andrés Barragan

Un tema básico, con una explicación de muy alto nivel. Felicitaciones profesor!!

Damian Yoel Szalai

Hace 15 días

Muy bueno y util! extraños los jugos en el medio del video! :)

Cursos Cliniweb

Hace 9 meses

-____- -____-

Jhon Abril

Tengo una duda. ¿Hay alguna buena práctica para que una función retorne HTML o simplemente ponerlo todo dentro de las comillas dobles? Ejemplo:

function makeContent($message) { return "Tu mensaje: $message"; }

Tonathiuh Díaz Miranda

Hola! implementaste dos ciclos en lugar de uno, duplicaste la complejidad del programa, cuál es el balance de performance que podríamos determinar en este ejemplo para satisfacer una correcta lectura de código y el mínimo de ciclos necesarios, es decir, 1?

Joserra Parada Fontaiña

Esta es mi propuesta :

<?php

function multiplicar($j) {
$Multiplicar=[];
for ($i=1; $i <= 10; $i++) {
    $resultado = $j*$i;
    $texto = "$j x $i = $resultado";
    array_push($Multiplicar, $texto);
};
return($Multiplicar);
};
?>

<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Asi se hace</title>
</head>
<body>
    
    <?php $multiplicando = 4 ?>
    <h2>Tabla del <?= $multiplicando ?></h2>
    <ul>
        <?php foreach(multiplicar($multiplicando) as $valor ): ?>
            <li> <?= $valor ?> 
        <?php endforeach; ?>   
    </ul>

</body>
</html>

Mi propuesta es :

?php

function multiplicar($j) {
$Multiplicar=[];
for ($i=1; $i <= 10; $i++) {
    $resultado = $j*$i;
    $texto = "$j x $i = $resultado";
    array_push($Multiplicar, $texto);
};
return($Multiplicar);
};
?>

<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Asi se hace</title>
</head>
<body>
    
    <?php $multiplicando = 4 ?>
    <h2>Tabla del <?= $multiplicando ?></h2>
    <ul>
        <?php foreach(multiplicar($multiplicando) as $valor ): ?>
            <li> <?= $valor ?> 
        <?php endforeach; ?>   
    </ul>

</body>
</html>

Introducción

¿Cómo se renderiza una página web en mi computadora?

Levantando un servidor local

¿Cómo combinar HTML con PHP?

PHP como preprocesador de HTML

Imprime texto y etiquetas HTML

Condicionales

Ciclos

¿Cómo pasar variables de PHP a JavaScript?

Evita el código espagueti

Refactorizando código espagueti

Manejo de formularios

Cómo obtener una solicitud al servidor con PHP

Envío de un formulario a través de GET

Envío de un formulario a través de POST

Envío de imágenes

Guardando imágenes en el servidor

Tipos de inputs

Otros tipos de inputs

Valida si un formulario ha sido enviado

Sanitizando datos de mi formulario

Validando datos

Proyecto: formulario de contacto

Creando el maquetado

Haciendo la lógica de recepción del formulario

Integrando PHPMailer

Implementando el servidor de correos de Gmail

Conclusión

Es tu turno de poner todo en práctica

Refactorizando código espagueti

Progreso del curso

Introducción

¿Cómo combinar HTML con PHP?

Manejo de formularios

Proyecto: formulario de contacto

Conclusión