0

Есть кусок кода

var someWords = ["clear", "business", "entertainment"];
function pickWord(someWords) {
    return someWords[Math.floor(Math.random() * someWords.length)];
}

Интерпретатор выдает

cannot read property length of undefined

Но как же ундефайнд если я даже имя аргумента переименовал по уже объявленной переменной??

4
  • 4
    могу поспорить что вызываешь функцию так: pickWord()
    – Grundy
    8 янв 2017 в 15:05
  • Так и было, но почему тогда интерпрератор выдает ошибку не на той строке где есть проблема?
    – Alex
    8 янв 2017 в 15:12
  • Это работает, потому что в любую функцию, можно передать любое количество аргументов
    – Grundy
    8 янв 2017 в 15:18
  • Все работает, спасибо за разъяснения.
    – Alex
    8 янв 2017 в 15:26

2 ответа 2

3

в данном случае имя параметра совпадает с именем глобальной переменной и перекрывает его внутри функции.

Чтобы не путаться с этим, можно переименовать параметр и получить следующее

var someWords = ["clear", "business", "entertainment"];
function pickWord(words) {
    return words[Math.floor(Math.random() * words.length)];
}

Таким образом, значение параметра someWords определяется тем, с каким параметром была вызвана функция pickWord.


Если функция была вызвана без параметров: pickWord() то значение всех объявленных параметров функции будет undefined.


Для решения необходимо либо передавать глобальную переменную в функцию

var someWords = ["clear", "business", "entertainment"];
function pickWord(words) {
    return words[Math.floor(Math.random() * words.length)];
}

pickWord(someWords);

Либо убрать параметр

var someWords = ["clear", "business", "entertainment"];
function pickWord() {
    return someWords[Math.floor(Math.random() * someWords.length)];
}

pickWord();

В данном случае будет использована глобальная переменная.

1

Не договариваете Вы чего-то. Все работает

var someWords = ["clear", "business", "entertainment"];

function pickWord(someWords) {
  return someWords[Math.floor(Math.random() * someWords.length)];
}

var word = pickWord(someWords);
console.log(word);

3
  • а вот так pickWord() не будет работать
    – Grundy
    8 янв 2017 в 15:10
  • Работает, да. Я вызывал функцию без агрумента. Но не ясно почему интерпретатор выдает ошибку не там где есть проблема, а на другой строке?
    – Alex
    8 янв 2017 в 15:15
  • Черт, я просто ослеп. Все правильно он выдает.
    – Alex
    8 янв 2017 в 15:17

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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