0

Почему я получаю ошибку:

Uncaught TypeError: Cannot read property 'indexOf' of undefined

По идеи код должен вернуть оружие или написать что его нет, если такого нет в массиве swords

var armory = {};
var swords = ["broadsword", "Katana", "Claymore", "Scimitar"];

armory.retrieveSword = function (request) {
  return (this.swords.indexOf(request) >= 0) ?
          this.swords.splice(this.swords.indexOf(request), 1)[0] :
          console.log("No " + request + ", baby");
};

var isKnight = true;
var weapon = isKnight && armory.retrieveSword("Katana");
console.log(weapon);
2
  • var swords = ... -> armory.swords = ...
    – user176262
    15 авг 2016 в 5:09
  • Очевидно, потому что this.swords == undefined. А вот почему оно так - из вашего кода не ясно
    – Kromster
    15 авг 2016 в 5:13

3 ответа 3

3

Потому что внутри функции retrieveSword, this ссылается на объект armory, у которого нет свойства this.swords и по этому у вас undefined.

Вместо

var swords = ["broadsword", "Katana", "Claymore", "Scimitar"];

Вам нужно написать

armory.swords = ["broadsword", "Katana", "Claymore", "Scimitar"];
0

1) swords находится в глобальном объекте
2) this указывает на armory.retrieveSword

Чесно говоря мне сложно судить о том, каким код был задуман, но кажется можно поступить так:

var armory = function() {
    this.swords = ["broadsword", "Katana", "Claymore", "Scimitar"];
    var self = this;
    this.retrieveSword = function (request) {
        return (self.swords.indexOf(request) >= 0) ?
            self.swords.splice(self.swords.indexOf(request), 1)[0] :
                console.log("No " + request + ", baby");
    };
};

10
  • ой вэй, раз уж поменял способ создания объекта, надо было добавлять и измененный пример использования, не говоря уже о том, что в данном случае не обязательно сохранять this в конструкторе. В данном случае проблема не в потере контекста
    – Grundy
    15 авг 2016 в 7:06
  • Он не изменился...
    – cipher_web
    15 авг 2016 в 7:09
  • Ну ладно Вам, контекст this в функции retrieveSword изменился!
    – cipher_web
    15 авг 2016 в 7:11
  • очевидно изменился: теперь armory - это функция, которая может быть конструктором, в коде в вопросе armory - это сам объект, поэтому если использовать функцию как объект - код не будет работать как ожидается.
    – Grundy
    15 авг 2016 в 7:11
  • очевидно, что не изменился - он как был ссылкой на armory, так ей и остался
    – Grundy
    15 авг 2016 в 7:12
0

Спасибо разобрался. Теперь код работает как и было задумано, а именно:

код возвращает оружие если оно есть в массиве swords, и сообщение об ошибке если указанного оружия нет - метод armory.retrieveSword().

var armory = {
  swords: ["Broadsword", "Katana", "Claymore", "Scimitar"],
  addSword: function (swords) {
    this.swords = this.swords || [];
    this.swords.push(swords);
  }
};

armory.retrieveSword = function (request) {
  return (this.swords.indexOf(request) >= 0) ?
          this.swords.splice(this.swords.indexOf(request), 1)[0] :
          alert("No " + request + ", baby!");
};

var isKnight = true;
var weapon = isKnight && armory.retrieveSword("123");
console.log(weapon);

Ваш ответ

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

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