Refactorizando código espagueti

Clase 9 de 24 • Curso de PHP: Integración con HTML

Cristobal Nyram

student•

**trucaso de la clase **

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

¡Magia tenemos un arreglo muy chulo!

Edgar Castro

student•

Muy buen truco!

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

Luis Miguel Falcon Montaño

student•

Esta es la forma correcta:

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

Gabriela Ugarte Maco

student•

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

student•

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

student•

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

Juan David Rueda Quiroga

student•

en lugar de

array_push();

es más optimo usar

$tabla_del_9[] = $resultado

Luis Alberto Aguilar Muñoz

student•

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

Joel Dominguez Merino

student•

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! 😁

César Luis Amundaray Rodríguez

student•

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

Jossie Steeven Quintero Giron

student•

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

student•

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

Juan Omar Palma álvarez

student•

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

Cristobal Nyram

student•

¡RECUERDA!

¡NUNCA! NUNCA IMPRIMIR ETIQUETAS HTML EN PHP

Sergio Andrés Barragan

student•

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

José Alberto Ramírez Quiroz

student•

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

César Luis Amundaray Rodríguez

student•

Supongo que es porque puedes colocar el HTML directamente.

Kenneth Angulo L

student•

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. 🤡

Jhon Abril

student•

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

student•

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

student•

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>

Joserra Parada Fontaiña

student•

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>

Jorge Eduardo Hernández Oropeza

student•

Ok. Acabo de entender que para imagenes vendria siendo igual, algo como:

<img src="<?= $path_body_images ?>Search.svg" alt="Search Icon">

Jorge Eduardo Hernández Oropeza

student•

Cómo sería esto para el caso de las imagenes? Realmente no me gusta porque debo declarar el echo en esos momentos de buscar el path de la imagen.

Juan Sebastian Guarin

student•

Alguien podría indicarme si esta manera de usar echo es buena practica? o que me recomendarían para poder usar funciones y se puedan imprimir?

<?php

function tablas($numero){
    for ($i=1; $i <=10 ; $i++) { 
         $resultado = $numero * $i; 
        echo "<li>$numero X $i = $resultado</li>";
    }
}

?>
<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Buenas Practicas</title>
</head>
<body>
    <h2>Tabla del 9</h2>
    <ul>
      <?= tablas(9);?>
    </ul>

</body>
</html>