0

Estoy intentando llenar un Select con números entre 1 y 100, además de eso evitar que se agreguen números repetidos y números negativo.

No entiendo porque me está arrojando un error al momento de la verificación.

let num = document.getElementById('num').value;
let lista = document.getElementById('lista');
let res = document.querySelector('div#res');
let valores = [];

function isNumero(n) {
    if (Number(n) >= 1 && Number(n) <= 100) {
        return true;
    } else {
        return false;
    }
}

function inLista(n, l) {
    if (l.indexOf(Number(n)) != -1) {
        return true;
    } else {
        return false;
    }
}

function addNum() {
    if ((isNumero(num) && !inLista(num,valores)) {
        valores.push(Number(num));
        let item = document.createElement('option');
        item.text = `Valor ${num} agregado.`;
        lista.appendChild(item);
    } else {
        alert('Valor no agregado.');
    }
}

> Cita en bloque

3
  • function addNum() {..} no veo le envies ningun parametro?
    – nestdan
    Commented el 23 abr. 2022 a las 15:20
  • @nestdan num es una variable global que definió en la linea 1 Commented el 23 abr. 2022 a las 17:43
  • @ @Fedex7501 gracias
    – nestdan
    Commented el 23 abr. 2022 a las 18:03

1 respuesta 1

0

No se qué error estás obteniendo exactamente pero puedes optimizar un poco tu código

function isNumero(n) {
    return n >= 1 && n <= 100;
}

function addNum() {
    const parsedNum = Number.parseInt(num);
    if (
        isNumero(parsedNum) &&
        !valores.includes(parsedNum)
    ){
        valores.push(parsedNum);
        let item = document.createElement('option');
        item.text = `Valor ${num} agregado.`;
        lista.appendChild(item);
    } else {
        alert('Valor no agregado.');
    }
}

Edit: además creo que deberías poner let num = document.getElementById('num').value; dentro de la función addNum para que el valor se actualize

2
  • Muchas gracias por tu repuesta, ya puedo agregar el valor a la lista sin problema. En efecto, tenía que colocar ``` let num = document.getElementById('num').value; ``` dentro de la función addNum(). También tomé tu consejo de optimizar el código.
    – Devarctico
    Commented el 24 abr. 2022 a las 22:45
  • Óptimo. Te agracecería que aceptaras la respuesta como correcta entonces. Commented el 24 abr. 2022 a las 23:09

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.