Почему значение undefined свойства?

Question

[lotrop]

Есть такой код:

Player = {
    height : 96,
    width : 60,
    x : 0,
    y : 0,
    sprite : new Sprite('player', this.x, this.y, 60, 96, 4, 16),
    draw : function(){
        this.sprite.draw(this.x, this.y);
    },
    update : function(){
        this.sprite.update(this.x, this.y);
    }
};

В функции Sprite печатаю список полученных аргументов и на месте this.x, this.y получаю undefined. Остальные параметры передаются хорошо. Это значит я не могу при первом объявлении пользоваться внутренними свойствами объекта?

Answer 1

[Дмитрий Кравченко]

у вас в месте объявления контекст window (или другой функции, в которой вызван этот код), но не объекта Player

Player = {
    height : 96,
    width : 60,
    x : 0,
    y : 0,
    draw : function(){
        this.sprite.draw(this.x, this.y);
    },
    update : function(){
        this.sprite.update(this.x, this.y);
    }
};
Player.sprite  = new Sprite('player', Player.x, Player.y, Player.width, Player.height, 4, 16),

Comments

[lotrop]

А почему когда я использую Player.update(), то все у меня отлично работает? Там же тоже используется this.x и this.y?

[lotrop]

Player, конечно объявляется в контексте window? но ведь функции Player.update(), и Player.draw() работают.

[Дмитрий Кравченко]

lotrop: когда вызываете функцию именно в таком виде (Player.update()), она вызывается в контексте объекта Player и this указывает на Player.

var update = Player.update;
update(); // будет ошибка, что-то типа Cannot get property update of undefined
//функцию отвязали от объекта Player и вызвали в контексте window, 
//window.sprite не определено, undefined. Так же, как и this.x, this.y в вашем коде

[Дмитрий Кравченко]

lotrop: запись update() в последнем моем примере равносильна window.update(), this указывает на window

[lotrop]

А нельзя именно внутренним способом, примерно, как я пытался сделать все это?

[Дмитрий Кравченко]

lotrop: вам подойдет использование функции конструктора

function Player(width, height, x, y){
this.width = width;
this.height = height;
this.x = x || 0;
this.y = y || 0;
this.sprite = new Sprite('player', this.x, this.y,  this.width, this.height, 4, 16);

return this;
}
Player.prototype.draw = function(){
this.sprite.draw(this.x, this.y);
}
Player.prototype.update = function(){
this.sprite.update(this.x, this.y);
}


var player = new Player(60,96,0,0);

[lotrop]

Спасибо, конструктором я делал) А вот мне было интересно в литеральном стиле объявить)

[Дмитрий Кравченко]

lotrop: тогда только через функцию-инициализатор, что в итоге даст тот же результат, что и код выше

Answer 2

[Назар Мокринский]

this.x вы вызываете в контексте вышестоящего кода.
Это классическая ошибка. Вопрос не в том, когда, в в том, где.
Поскольку начальные значения у вас одинаковые можете просто 0, 0 написать, как вы сделали с остальными значениями.

Answer 3

[Mikhail Osher]

Вы издеваетесь? Я вижу подобные вопросы чуть ли не каждую неделю. Пользуйтесь поиском.

Comments

[lotrop]

Молодец!