2

В наличии три ссылки:

    <div class="menu">
    <a href="#" id="one" onclick="engine()">Rock</a>
    <a href="#" id="two" onclick="engine()">Paper</a>
    <a href="#" id="three" onclick="engine()">Scissors</a>

Задача, при клике на одну из ссылок менять(устанавливать) значение в глобальную переменную userChoice.

var userChoice; // глобальная переменная
function engineGame() { // ее вызываем при клике на ссылку
    document.getElementById('one').onclick = function() {
        userChoice = "rock";
        alert("до вызова функции userChoice = " + userChoice); // rock
    }
    document.getElementById('two').onclick = function() {
        userChoice = "paper";
        alert("до вызова функции userChoice = " + userChoice); // paper
    }
    document.getElementById('three').onclick = function() {
        userChoice = "scissors";
        alert("до вызова функции userChoice = " + userChoice); //scissors
    }
};
engineGame();
alert("после вызова функции userChoice = " + userChoice); // сразу выскакивает undefined :(

вопрос: 1. Добился ли я поставленной задачи? (установить значение в глобальную переменную) или это делается как-то иначе?

5
  • Все верно, понятно что undefined , событие то еще не выполнилось что бы переменная была переназначена .
    – Redr01d
    21 июн 2016 в 7:45
  • alert внутрь onclick перенесите 21 июн 2016 в 7:50
  • т.е. после того как я кликнул на ссылку и до момента перезагрузки станицы, переменной присвоено значение в соответствии с программой? Или спрошу по другому, я правильно установил значение в глобальную переменную по клику?
    – Evgeny
    21 июн 2016 в 7:51
  • Сделайте ещё одну ссылку <a href="#" id="three" onclick="alert(userChoice)">Test</a> и проверьте. Но вроде правильно. 21 июн 2016 в 8:38
  • 1
    Возможный дубликат вопроса: Как вернуть значение из события или из функции обратного вызова?
    – Grundy
    15 авг 2016 в 13:26

1 ответ 1

2

Код корректен только если события назначаются после загрузки кода ссылок. Назначения событий лучше делать в теле обратчика события onload объекта window. Так-же лучше использовать функцию addEventListener на случай если это не единственное событие по ссылке:


var userChoice = null;
window.addEventListener("load", function() 
{
    document.getElementById("one").addEventListener("click", function() 
    {
        userChoice = "rock";
        alert(userChoice); 
    });
    document.getElementById("two").addEventListener("click", function() 
    {
        userChoice = "paper";
        alert(userChoice); 
    });
    document.getElementById("tree").addEventListener("click", function() 
    {
        userChoice = "scissors";
        alert(userChoice); 
    });
};
Еще лучше сделать объект GameEngine и хранить выбор пользователя в его поле не плодя глобальных переменных, тем более с таким простым именем.

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.