[JS] Dodawanie obiektów do tablicy

Question

Cześć, robie prostą gre w js i napotkałem trudności podczas tworzenia obiektów, będe wdzięczny jak ktoś rzuci okiem :) https://jsfiddle.net/p4zkuvdc/

(index):89 Uncaught TypeError: Cannot read property 'strzal' of undefined
at o.<anonymous> ((index):89)
at o.emit (index.js:150)
at t.update (Ticker.js:319)
at _tick (Ticker.js:125)

Moge też wysłać cały projekt w archiwum chcącej pomóc osobie :)

http://chirurgiaplastycznaodsrodka.pl/sample1.rar

Comments

Tutaj można pobrać :)

http://chirurgiaplastycznaodsrodka.pl/sample1.rar

---

W kod nie patrzę bo na mobilku niewygodnie jest z tych codepen i jsfiddle korzystac, ale bląd jednoznacznie wskazuje, że próbujesz odnieść się do właściwości strzal na nieisniejącym obiekcie (undefined). Prześledź dokładnie kod i upewnij się, że masz w tym miejscu utworzony obiekt z propertką strzal albo zobacz czy nie masz może jakieś literówki?

---

Problem w tym, że najpierw napisałem kod strukturalnie (działa), a teraz chce dodać więcej przeciwników więc przerabiam go na obiektowy. Prześledziłem już cały kod kilka razy i pomimo, że rozumiem błąd, nie wiem co poprawić, żeby zadziałało :/
Jak będziesz mieć później czas to prosze pamiętaj odwiedzić ten wątek, żeby zobaczyć czy wiesz co zmienić  

Myśle, że błąd jest trywialny tylko robie to pierwszy raz i rzuciłem się troche na za głęboką wode (projekt do szkoły). Korzystam z PixiJS

Pozdrawiam :)

---

Błąd pewnie polega na tym, że próbuje wywołać funkcje strzal() zanim zostaje stworzony obiekt (?)

Answer 1

Odnośnie samego kodu nie będę nic pisał, bo jest masa rzeczy do poprawy, wiec od razu przejdę do błędu.

 for (i = 0; i <= przeciwnicy.length; i++) {
                    przeciwnik = przeciwnicy[i];
                    //console.log(przeciwnik);
                    przeciwnik.strzal();
                    przeciwnik.move();
                }

powyższa pętla jest błędna, dlatego wyskakuje ci błąd. Zamiast mniejsze równe powinno być samo mniejsze,  o tak.

 for (i = 0; i < przeciwnicy.length; i++) {
                    przeciwnik = przeciwnicy[i];
                    //console.log(przeciwnik);
                    przeciwnik.strzal();
                    przeciwnik.move();
                }

 

Comments

Dzięki, jak bede przy kompie to zobacze czy dziala. Będe też wdzięczny za pokazanie tych błędów i zaproponowania lepszego rozwiązania - oczywiście jeżeli będziesz miał czas i ochote :)

Dopiero zaczynam nauke js i zależy mi by już na początku nie wpajać złych praktyk

Pozdrawiam :)

Answer 2

Odświeżam