Проверка нужна, так как если условия не выполнены, то кнопка не подсвечивает и не переходит на другой уровень. В данном случае есть просто кнопка и другие элементы, и после того, как условие выполняется, то переход совершается. А не так: если условие проходит, то запускается функция.
Почему не написать так?
onItemClick = (item) => {
const id = Number(item.target.id) // const {id, firstChild } = item.target
if (id === answer) {
item.target.firstChild.className = "li-btn right"
}
}
Если сделать так, то будет работать? Разве здесь не просто this.state.userAction меняется на const id?