Ya estoy inscrito ¿Todavía no tienes acceso? Nuestros Planes
Ya estoy inscrito ¿Todavía no tienes acceso? Nuestros Planes
Solucionado (ver solución)
Solucionado
(ver solución)
2
respuestas

[Duda] No se porque esta fallando mi codigo pero ya no genera el numero secreto

let numeroSecreto = 0;
let intentos = 0;



function asignarTextoElemento(elemento, texto){
    let elementoHTML = document.querySelector(elemento);
    elementoHTML.innerHTML = texto;
    return;
}
function verificarIntento(){
    let numeroUsuario = parseInt(document.getElementById(`valorUsuario`).value);
    
    
    if(numeroSecreto === numeroUsuario){
        asignarTextoElemento(`p`,`Acertaste el numero en ${intentos} ${(intentos === 1) ? `intento` : `intentos`}`);
        document.getElementById(`reiniciar`).removeAttribute(`disabled`);
       
    } else {
    //El usuario no acerto 
        if(numeroUsuario > numeroSecreto){
            asignarTextoElemento(`p`,`El numero secreto es menor`);
        } else{
            asignarTextoElemento(`p`,`El numero secreto es mayor`)
        }
        intentos++;
        limpiarCaja();
    }
    return;
}

function limpiarCaja() {
    let valorCaja = document.querySelector(`#valorUsuario`).value = ``;
}
function generarNumeroSecreto() {
    return Math.floor(Math.random()*10)+1;    
}

function condicionesIniciales() {
    asignarTextoElemento(`h1`,`Juego del numero secreto`)
    asignarTextoElemento(`p`,`Indica un numero del 1 al 10`)
    
    numeroSecreto = generarNumeroSecreto;
    intentos = 1;
    
}

function reiniciarCodigo() {
    //limpiar input
    limpiarCaja();
    //indicar mensaje de intervalo de numeros
    //generar el numero aleatorio
    condicionesIniciales();
    //deshabilitar el botón de nuevo juego 
    document.querySelector('#reiniciar').setAttribute('disable','true')
    //inicializar el numero de intentos
    
}

condicionesIniciales()
2 respuestas
solución!

El problema principal está dentro de la función condicionesIniciales():

numeroSecreto = generarNumeroSecreto;

Lo que pasa aquí es que generarNumeroSecreto es una función, para corregirlo, simplemente haría falta los paréntesis () , siendo así:

numeroSecreto = generarNumeroSecreto();

Otro de los errores menores también se encuentra dentro de la función reiniciarCodigo(), específicamente en esta línea:

document.querySelector('#reiniciar').setAttribute('disable','true')

El problema es que el atributo correcto es "disabled", no "disable". Así que debes corregirlo a:

document.querySelector('#reiniciar').setAttribute('disabled', 'true');

Tienes razón muchas gracias ayer dandole una revisión de nuevo note los parentesis pero me faltaba lo de disabled,