Возможно стоит использовать объект для букв? С ним вообще не понадобится цикл, проверка будет за одно нажатие, мне кажется код проще получается
const randomLetters = {
q: true,
w: true,
e: true,
r: true,
t: true,
y: true,
};
const successAnswers = [];
document.onkeydown = function (e) {
const key = e.key.toLowerCase();
const latter = randomLetters[key];
if (latter === true) {
successAnswers.push(key);
delete randomLetters[key];
console.log(`randomLetters :>>`, randomLetters);
console.log(`successAnswers :>>`, successAnswers);
}
};
Ну а если всё таки нужен массив, то зачем вам пробегаться по массиву "самостоятельно", используйте indexOf, он вернёт нужный index элемента, если найдёт совпадение, если не найдёт вернёт -1
https://developer.mozilla.org/en-US/docs/Web/JavaS...