Skip to main content
Corrección de código
Origen Enlace
HeytalePazguato
  • 11k
  • 3
  • 13
  • 34
let opcion

// Función catalogo que se ejecutará únicamente al ser llamada
function catalogo() {
  // Como tenías muchas variables similares es mejor utilizar un arreglo
    productos = [];
    productos[0] = parseInt(prompt("Ingrese cuantos juegos de COD quiere llevarse")); || 0;
  productos[1] = parseInt(prompt("Ingrese cuantos juegos de Fifa 23 quiere llevarse")); || 0;
  productos[2] = parseInt(prompt("Ingrese cuantos juegos de Lego Marvel quiere llevarse")); || 0;
  productos[3] = parseInt(prompt("Ingrese cuantos juegos de UFC 4 quiere levarse")); || 0;
  productos[4] = parseInt(prompt("Ingrese cuantos juegos de The Last of Us quiere llevarse")); || 0;
  productos[5] = parseInt(prompt("Ingrese cuantos juegos de R6 quiere llevarse")); || 0;
  
  // resultado contará los elementos del arreglo
  let resultado = productos.reduce((a, b) => {
    return a + b;
    });
  
  // Si el resultado de la suma de todos los elementos del arreglo productos es 0
  if (resultado == 0) {
    alert("No seleccionó ningun articulo ")
    return 0;
  }
  // En caso contrario, si la suma es cualquier valor diferente de 0
  else {
    alert(`Se seleccionaron ${resultado} artículos`)
    return productos;
  }
}

// La primera parte de tu código
do{
    // Obtenemos el valor de entrada del usuario, eliminamos diacríticos y convertimos a minúsculas
    opcion = prompt(" ¿Desea comprar algun producto? , si es asi escriba (si) sino escriba (no) " ).normalize("NFD").replace(/\p{Diacritic}/gu, "").toLowerCase();
    
    if (!['si', 'no'].includes(opcion)) {
        alert(" No pusiste lo especificado ")
    }else{
        if (opcion == "si") {
            catalogo()
            break;
        }if (opcion == "no") 
            alert(" Usted decidio no comprar ")
            break;
    }
    // Como ya revisas que opción sea "si" o "no" adentro del while
    // No hace falta revisarlo aquí, con hacer un do -> while infinito es suficiente
}while (true)

Edición

Gracias al comentario de @MarcePuente es importante mencionar que /\p{Diacritic}/gu está disponible a partir del 2021, si se utilizan versiones anteriores puedes utilizar /[\u0300-\u036f]/g.

También agregué || 0 al final de cada línea donde se obtiene el número de artículos, en caso de que el usuario oprima Cancel se generaba un NaN (Del inglés Not a Number) y el resultado del alert era NaN, ahora si hay algún NaN se cambiará por 0 gracias a la modificación.

let opcion

// Función catalogo que se ejecutará únicamente al ser llamada
function catalogo() {
  // Como tenías muchas variables similares es mejor utilizar un arreglo
    productos = [];
    productos[0] = parseInt(prompt("Ingrese cuantos juegos de COD quiere llevarse"));
  productos[1] = parseInt(prompt("Ingrese cuantos juegos de Fifa 23 quiere llevarse"));
  productos[2] = parseInt(prompt("Ingrese cuantos juegos de Lego Marvel quiere llevarse"));
  productos[3] = parseInt(prompt("Ingrese cuantos juegos de UFC 4 quiere levarse"));
  productos[4] = parseInt(prompt("Ingrese cuantos juegos de The Last of Us quiere llevarse"));
  productos[5] = parseInt(prompt("Ingrese cuantos juegos de R6 quiere llevarse"));
  
  // resultado contará los elementos del arreglo
  let resultado = productos.reduce((a, b) => {
    return a + b;
    });
  
  // Si el resultado de la suma de todos los elementos del arreglo productos es 0
  if (resultado == 0) {
    alert("No seleccionó ningun articulo ")
    return 0;
  }
  // En caso contrario, si la suma es cualquier valor diferente de 0
  else {
    alert(`Se seleccionaron ${resultado} artículos`)
    return productos;
  }
}

// La primera parte de tu código
do{
    // Obtenemos el valor de entrada del usuario, eliminamos diacríticos y convertimos a minúsculas
    opcion = prompt(" ¿Desea comprar algun producto? , si es asi escriba (si) sino escriba (no) " ).normalize("NFD").replace(/\p{Diacritic}/gu, "").toLowerCase();
    
    if (!['si', 'no'].includes(opcion)) {
        alert(" No pusiste lo especificado ")
    }else{
        if (opcion == "si") {
            catalogo()
            break;
        }if (opcion == "no") 
            alert(" Usted decidio no comprar ")
            break;
    }
    // Como ya revisas que opción sea "si" o "no" adentro del while
    // No hace falta revisarlo aquí, con hacer un do -> while infinito es suficiente
}while (true)
let opcion

// Función catalogo que se ejecutará únicamente al ser llamada
function catalogo() {
  // Como tenías muchas variables similares es mejor utilizar un arreglo
    productos = [];
    productos[0] = parseInt(prompt("Ingrese cuantos juegos de COD quiere llevarse")) || 0;
  productos[1] = parseInt(prompt("Ingrese cuantos juegos de Fifa 23 quiere llevarse")) || 0;
  productos[2] = parseInt(prompt("Ingrese cuantos juegos de Lego Marvel quiere llevarse")) || 0;
  productos[3] = parseInt(prompt("Ingrese cuantos juegos de UFC 4 quiere levarse")) || 0;
  productos[4] = parseInt(prompt("Ingrese cuantos juegos de The Last of Us quiere llevarse")) || 0;
  productos[5] = parseInt(prompt("Ingrese cuantos juegos de R6 quiere llevarse")) || 0;
  
  // resultado contará los elementos del arreglo
  let resultado = productos.reduce((a, b) => {
    return a + b;
    });
  
  // Si el resultado de la suma de todos los elementos del arreglo productos es 0
  if (resultado == 0) {
    alert("No seleccionó ningun articulo ")
    return 0;
  }
  // En caso contrario, si la suma es cualquier valor diferente de 0
  else {
    alert(`Se seleccionaron ${resultado} artículos`)
    return productos;
  }
}

// La primera parte de tu código
do{
    // Obtenemos el valor de entrada del usuario, eliminamos diacríticos y convertimos a minúsculas
    opcion = prompt(" ¿Desea comprar algun producto? , si es asi escriba (si) sino escriba (no) " ).normalize("NFD").replace(/\p{Diacritic}/gu, "").toLowerCase();
    
    if (!['si', 'no'].includes(opcion)) {
        alert(" No pusiste lo especificado ")
    }else{
        if (opcion == "si") {
            catalogo()
            break;
        }if (opcion == "no") 
            alert(" Usted decidio no comprar ")
            break;
    }
    // Como ya revisas que opción sea "si" o "no" adentro del while
    // No hace falta revisarlo aquí, con hacer un do -> while infinito es suficiente
}while (true)

Edición

Gracias al comentario de @MarcePuente es importante mencionar que /\p{Diacritic}/gu está disponible a partir del 2021, si se utilizan versiones anteriores puedes utilizar /[\u0300-\u036f]/g.

También agregué || 0 al final de cada línea donde se obtiene el número de artículos, en caso de que el usuario oprima Cancel se generaba un NaN (Del inglés Not a Number) y el resultado del alert era NaN, ahora si hay algún NaN se cambiará por 0 gracias a la modificación.

Origen Enlace
HeytalePazguato
  • 11k
  • 3
  • 13
  • 34

Buen día,

Cuando deseas que una sección de código se ejecute únicamente en determinados momentos es mejor crear una función y llamar a la función cuando se necesite ejecutar dicha sección de código.

Por ejemplo, podrías crear una función catalogo para mostrar todos los alert únicamente cuando el usuario escriba "si".

Por cierto, como pudiste darte cuenta, el usuario puede ingresar múltiples combinaciónes de "si" y "no", para no tener que escribir todas las combinaciones (Que por cierto te hace falta considerar si "sí" lleva tilde o si escribe "sI") puedes procesar la entrada del usuario y convertirla siempre a minúsculas y remover las tildes.

Para remover las tildes puedes utilizar normalize:

opcion.normalize("NFD").replace(/\p{Diacritic}/gu, "")

Esta línea elimina los diacríticos y convierte las letras a su valor normalizado, por ejemplo íóü se convertiría en iou.

Luego puedes convertir la opción a minúsculas para no trabajar con tantas combinaciones:

opcion.normalize("NFD").replace(/\p{Diacritic}/gu, "").toLowerCase()

Y para evaluar únicamente "si" y "no" puedes utilizar includes:

['si', 'no'].includes(opcion.normalize("NFD").replace(/\p{Diacritic}/gu, "").toLowerCase())

Ahora, como necesitas que se repita cuando el usuario escriba algo diferente a "si" o "no" entonces negamos la sentencia enterior con !:

!['si', 'no'].includes(opcion.normalize("NFD").replace(/\p{Diacritic}/gu, "").toLowerCase())

Ejemplo completo:

let opcion

// Función catalogo que se ejecutará únicamente al ser llamada
function catalogo() {
  // Como tenías muchas variables similares es mejor utilizar un arreglo
    productos = [];
    productos[0] = parseInt(prompt("Ingrese cuantos juegos de COD quiere llevarse"));
  productos[1] = parseInt(prompt("Ingrese cuantos juegos de Fifa 23 quiere llevarse"));
  productos[2] = parseInt(prompt("Ingrese cuantos juegos de Lego Marvel quiere llevarse"));
  productos[3] = parseInt(prompt("Ingrese cuantos juegos de UFC 4 quiere levarse"));
  productos[4] = parseInt(prompt("Ingrese cuantos juegos de The Last of Us quiere llevarse"));
  productos[5] = parseInt(prompt("Ingrese cuantos juegos de R6 quiere llevarse"));
  
  // resultado contará los elementos del arreglo
  let resultado = productos.reduce((a, b) => {
    return a + b;
    });
  
  // Si el resultado de la suma de todos los elementos del arreglo productos es 0
  if (resultado == 0) {
    alert("No seleccionó ningun articulo ")
    return 0;
  }
  // En caso contrario, si la suma es cualquier valor diferente de 0
  else {
    alert(`Se seleccionaron ${resultado} artículos`)
    return productos;
  }
}

// La primera parte de tu código
do{
    // Obtenemos el valor de entrada del usuario, eliminamos diacríticos y convertimos a minúsculas
    opcion = prompt(" ¿Desea comprar algun producto? , si es asi escriba (si) sino escriba (no) " ).normalize("NFD").replace(/\p{Diacritic}/gu, "").toLowerCase();
    
    if (!['si', 'no'].includes(opcion)) {
        alert(" No pusiste lo especificado ")
    }else{
        if (opcion == "si") {
            catalogo()
            break;
        }if (opcion == "no") 
            alert(" Usted decidio no comprar ")
            break;
    }
    // Como ya revisas que opción sea "si" o "no" adentro del while
    // No hace falta revisarlo aquí, con hacer un do -> while infinito es suficiente
}while (true)