Ejercicio 2 - Githubissues

Porfa no te olvides de poner los enunciados de los ejercicios en cada uno 🙏🏼

Revisar las redacciones:

 alert("A continuacion se le pedira una lista de paises y lugares suele ser muy pesado por eso cree un ejemplo");

Esta comprobación es innecesaria. Podes hacerla directamente en el if anterior:

if (paises.length === 0) {
      do {
        let paisesNew = prompt("ingrese un pais");
        if (paisesNew === null || paisesNew==="") {
          break
        } else {
            paises.push(paisesNew);
        }
      } while (true);
    }

Si podés usar acentos en los nombres de variables de js:

let tamano = paises.length; // -> let tamaño

Este HTML está mal formateado:

 div2.innerHTML = ` <h4>La lista tiene: ${tamano} elementos, que son los siguiente : <h4>`;

Acá tenes lógica muy confusa:

div2.innerHTML += `<li>Elemtento ultimo : ${paises[tamano]}</li>`;

Se entiende que tamano quedó con el length original (y no luego de agregar paris), entonces es (en realidad) el tamaño - 1 (que si representa la última posicion). Funciona, pero no es lo ideal. No te recomendaría crear esa variable tamano, deberías usar directamente paises.length.

Acá nuevamente el html está mal formateado:

div2.innerHTML += "<h3> La lista de elementos fue : </3> <ul> ";

No es lo ideal usar innerHTML precisamente por los problemas que te está causando. Por ahora está bien, pero en clase más adelante vamos a ver la manera correcta (document.createElement).

Acá el ejercicio no pide sumar, pide sustituir:

let agre = "Barcelona por que se pide en el ejercicio";
paises.splice(i, 0, agre);
div2.innerHTML += `<li> ${paises[i]} (se suma por el ejercicio)</li>`;

Acá te dejo la version "amable y constructiva" de chat gpt si te ayuda jajjaja, pero entendé que no lo digo de mala manera, solo que voy al punto:

Mejora del Issue de GitHub

¡Hola! 😊

He revisado tu código y he encontrado algunas áreas donde podríamos hacer algunas mejoras para que sea más legible y efectivo. ¡Vamos a hacerlo más amigable y fácil de entender!

Eliminación de Comprobación Innecesaria: La comprobación que realizas al principio parece ser innecesaria. Podemos integrarla directamente en la condición anterior. Por lo tanto, podríamos simplificar el código eliminando esta comprobación adicional.
Uso de Acentos en Nombres de Variables: Es cierto que puedes utilizar acentos en los nombres de variables de JavaScript. Vamos a cambiar tamano a tamaño para mejorar la legibilidad del código.
Corrección de Formato en HTML: Hay algunos problemas de formato en el HTML generado. Vamos a ajustarlos para que el HTML esté bien formateado y sea más claro.
Lógica Confusa: Encontré un fragmento de código con una lógica un poco confusa. Vamos a simplificarlo para que sea más fácil de entender y de mantener.
Evitar el Uso de innerHTML: Aunque por ahora el uso de innerHTML está bien, a largo plazo es mejor evitarlo debido a posibles problemas de seguridad y dificultades para el mantenimiento del código. En lugar de eso, te sugeriría que uses document.createElement.
Corrección de la Lógica de Sustitución: En una parte del código, estás realizando una adición en lugar de una sustitución, como se solicita en el ejercicio. Vamos a corregir eso para que cumpla con las instrucciones.

¡Gracias por tu trabajo en este proyecto! Estoy seguro de que estas mejoras harán que el código sea aún mejor. Si tienes alguna pregunta o necesitas más ayuda, ¡no dudes en preguntar! Estoy aquí para ayudar. 🚀

¡Gracias! 🙏

Joselobo753 / js-tp2-2-Rolingcode

Ejercicio 2 #1

Mejora del Issue de GitHub