[JavaScript]Funkcja nie jest funkcją. Opcje

[MeGusta]

Witam,
Czemu pluje mi, że funkcja nie jest funkcją? Dodam, że jeśli umieszczę jej wywołanie poza forEach normalnie działa.

[HTML] pobierz, plaintext 
 
    this.drawInventory = function(item)
    {
    	document.getElementById('player_inv').innerHTML += item;
    	return false;
    };
 
   this.inventory.forEach(function(item) {
     	this.drawInventory(item);
    });
[HTML] pobierz, plaintext 

Błąd:
Uncaught TypeError: this.drawInventory is not a function

[markuz]

Przed tym forEach zrób var self = this; i w funkcji używaj self zamaist this. Bo this w tej funkcji to coś innego niż poza nia.

[MeGusta]

Żeby nie zakładać nowego tematu, w obiekcie mam zdefiniowane zmienne, this.name ... itd, w funkcji chce nadpisać te zmienne lecz coś nie działa.

[HTML] pobierz, plaintext 
this.playerData = function()
	{
		$.ajax({ type: 'POST', url: 'engine.php', data: { task: 1 }}).done(function(data)
		{
            d = data.split('|');
            this.name = d[1]; // nie nadpisuje, d[1] pełne
            this.level = d[2]; // nie nadpisuje
        });
 
	};
 
	this.playerData();
[HTML] pobierz, plaintext 

[markuz]

To samo co wyżej, this w funkcji którą podajesz w done nie dot. tego obiektu tylko ajaxa. Musisz zrobić np. var self = this; przed $.ajax a w funkcji używać self zamiast this.

[MeGusta]

Niestety nadal nie nadpisuje, mój cały kod:

[HTML] pobierz, plaintext 
var player = function(){
 
	this.name = 'xyz';
        this.level = 1;
 
        this.playerData = function()
	{
		var self = this;
		$.ajax({ type: 'POST', url: 'engine.php', data: { task: 1 }}).done(function(data){
 
            d = data.split('|');
 
            self.name = d[1];
            self.level = d[2];
 
        });
 
	};
 
this.playerData();
}
 
[HTML] pobierz, plaintext 

[markuz]

Ja bym to napisał w ten sposób:

[JAVASCRIPT] pobierz, plaintext 
var Player = function(options) {
    this.name = options.name || 0;
    this.level = options.level || 0;
    this.init();
};
 
Player.prototype = {
    constructor: Player,
    init: function() {
        var self = this;
        this.getData({ task: 1 }, function(d) {
            var data = d.split('|');
            self.name = data[1];
            self.level = data[2];
        });
    },
    getData: function(data, onDone) {
        $.ajax({ type: 'POST', url: 'engine.php', data: data}).done(onDone);
    }
};
 
var player = new Player();
[JAVASCRIPT] pobierz, plaintext 

Pisałem z palca, mogą być literówki. Jak nie działa, to sprawdź czy na pewno otrzymujesz dane, jaki mają format itp. Zrób console.log(data) i pokaż co tam trzymasz.

[MeGusta]

Teraz wypluwa:

[HTML] pobierz, plaintext 
player.js:3 Uncaught TypeError: Cannot read property 'name' of undefined
[HTML] pobierz, plaintext 

3 linijka:

[HTML] pobierz, plaintext 
this.name = options.name || null;
[HTML] pobierz, plaintext 

[markuz]

No. I co teraz zrobisz? Jakieś pomysły?

[MeGusta]

Szczerze to średnio. Nie mam pojęcia.

[markuz]

Ok. To wywał options z argumentu i przypisz domyślne wartości dla name i level tj. this.name = null; this.level = 0;

[MeGusta]

Kurcze, no juz w morde nie wiem, jak nie nadpisywało tak nie nadpisuje.

[HTML] pobierz, plaintext 
var Player = function(options) {
 
    this.name = null;
    this.level = 0;
 
    this.init();
 
};
 
 
Player.prototype = {
    constructor: Player,
    init: function() {
        var self = this;
        this.getData({ task: 1 }, function(d) {
 
            var data = d.split('|');
            self.name = data[0];
            self.level = data[1];
            console.log(self.name) // zwraca prawidłowo
        });
    },
    getData: function(data, onDone) {
        $.ajax({ type: 'POST', url: 'engine.php', data: data}).done(onDone);
    },
    getName: function()
    {
        return this.name;
    }
};
 
var player = new Player();
 
console.log(player.getName()); // zwraca null
[HTML] pobierz, plaintext 

Jakieś pomysły?

Ten post edytował MeGusta 20.08.2016, 13:22:16

[markuz]

Linijka 34 wykona się wcześniej niż 17,18, 19, itd. Zapytanie AJAX trochę trwa, a skrypt wykona się bardzo szybko.

[MeGusta]

Jak mógłbym rozwiązać ten problem aby działało to w ten sposób w jaki chce.

[markuz]

Z konstruktora usuń this.init();
Do init: function() dodaj parametr (callback) np. onDone i wywołaj go po wyczytaniu danych.
Wywołaj init po utworzeniu obiektu Player i w parametrze umieść funkcje w której sobie pobierasz name tj.
player.init(function() {
console.log(player.getName());
});

Ten post edytował markuz 20.08.2016, 14:23:46

[viking]

Całe jquery 3 bardzo mocno opiera się na Promise. Jeżeli zaimplementujesz u siebie wtedy pierwszym krokiem będzie jquerowy ajax albo natywne fetch do pobrania danych i dopiero wyciąganie danych o użytkowniku.

[MeGusta]

Panowie, nie wiem czy dobrze zrozumiałem, ale zrobiłem coś takiego, działać działa, ale pytanie czy to jest poprawnie?
W funkcji getData w callbacku wywołuje funkcje która będzie (jest) odpowiedzialna za wyświetlanie już pobranych zmiennych, tak myślę i na dłuższą drogę to chyba nie ma sensu, to funkcji korzystających z tych zmiennych będzie dużo więcej, a callbacka do każdej nie bd dodawał. Ktoś poratuje?

[HTML] pobierz, plaintext 
var Player = function() {
 
    this.name = null;
    this.level = 0;
 
    this.init();
 
};
 
 
Player.prototype = {
    constructor: Player,
    init: function() {
        var self = this;
        this.getData({ task: 1 }, function(d) {
            var data = d.split('|');
            self.name = data[0];
            self.level = data[1];
        });
    },
    getData: function(data, onDone) {
        var self = this;
        $.ajax({ type: 'POST', url: 'engine.php', data: data}).done(onDone, function(){ self.loadGame(); });
    },
    loadGame: function()
    {
        $('#player_name').text('Nazwa: ' + this.name);
        $('#player_lvl').text('Level: ' + this.level);
    },
};
[HTML] pobierz, plaintext 

[markuz]

Czy chcesz aby obiekt Player sterował też zachowaniem gry - tak jak to napisałeś (metoda loadGame w Player). Jeżeli tak, to może warto w ogóle porzucić obiekty albo zrobić 1 o nazwie Game ew. wywalić logikę Game do nowego obiektu Game a Player zostawić jak jest.

Co do pytania to przenieś wywołanie metody self.loadGame(); do init() po przypisaniu name i level, getData ma służyć tylko do komunikacji z engine.php.