No puedo

[juanma62]

Hola a todos:

Aquí os pongo un código que no me funciona y no se porqué:

<script type="text/javascript">
var acierto = [("a") || ("b") || ("A") || ("B")];
var letraA = document.getElementById("letraA");
var letraB = document.getElementById("letraB");
var letra = (letraA + letraB);

function comprobar() {
if (letra == acierto) {
alert("Has acertado");
} else {
alert("Estas equivocado");
}
}
</script>

<div id="contenedor">Desayuno<br />
<div id="datos">
<input type='text' id="letraA" name="letraA" />
<input type='text' id="letraB" name="letraB" />
<input type="button" value="Comprobar" id="comprobar" onclick="comprobar()" />
</div>
</div>

La idea es que el usuario meta una letra los input y si es a, A, b o B devuelva que es correcto y si no incorrecto.
Si alguien me puede ayudar os lo agradezco.

[marcelocuiza_de]

a simple vista hay varios errores y falencias..

Código Javascript:

function comprobar() {
var letraA = document.getElementById("letraA").value; //obtinenes el valor
var letraB = document.getElementById("letraB").value;//obtinenes el valor
var acierto=new Array("a","b","A","B");
var letra = (letraA + letraB);// concatenas
 
 for(var i = 0; i < acierto.length; i++) {
        if(acierto[i] == letraA || acierto[i] == letraB ) {   alert("Has acertado");  }
         else  { alert("Estas equivocado");}
    }
 
}
 

[juanma62]

Muchas gracias:

Yo lo había logrado con algo mucho más complejo. El que me has mandado funciona, pero me sale la ventana del acierto o del error 4 veces. Voy a ver si puedo arreglarlo.

Muchas gracias