Działa na tylko pierwszy element tablicy

Question

Witam, mam problem z tablicą. Obiekt rainpower to element tablicy rain. Znajduje się w nim 30 takich elementów. Gdy go użyję obiekty tworzą deszcz meteorów,lecz działa on tylko na pierwszy element z tablicy Enemys. Jak zrobić, aby każdy element z Enemys został trafiany?

Trafiony obiekt przekazuje true do funkcji  w której element rain zostaje wyrzucony z tablicy.

Oto kod z problemem :

RainPower.prototype.hits = function(){
 if(this.x > Enemys[i].x && this.x < Enemys[i].x + Enemys[i].w && this.y + this.h > Enemys[i].y ){
   console.log('ssss')
return true
}else{
  return false
}
}

 

Answer 1

Ale Ty przecież metodą hist nie iterujesz po żadnej tablicy? Czym jest ta zmienna "i"? Gdzie wywołujesz tę metodę? I nie baw się w taki sposób z prototypami bo się trochę w tym chyba zagubiłeś, proponuję abyś zrobił to z duchem czasu po prostu jako klasę i w niej odpowiednie metody i pola, to wtedy myślę, że sam zobaczysz gdzie tkwi błąd.

Comments

odnosisz się też to zmiennej "x" z tego obiektu, ale czym jest zmienna Enemys? to też pole tego obiektu czy chcesz się odnieść do czegoś globalnego czy co?

---

@Tomek Sochacki, wytnę większą ilość kodu bo może rzeczywiście jest mało dokładnie. 

function RainPower (){
  this.x = rand(300,cw-300);
  this.y = rand(-1000,-50);
  this.w = 20;
  this.h = 50;
  this.hitting = false;
  this.toDelete = false;
  this.hide = function(){
    this.toDelete = true;
  }
  this.speedY = 2;
  this.gravity = function(){
    this.y +=this.speedY;
  }

};

 RainPower.prototype.draw = function(){ ctx.fillStyle = 'black' ctx.drawImage('jakies tam zdjecie',this.x,this.y,this.w,this.h) this.gravity() }


RainPower.prototype.hits = function(){
 if(this.x > Enemys[i].x && this.x < Enemys[i].x + Enemys[i].w && this.y + this.h > Enemys[i].y ){
   console.log('ssss')
return true
}else{
  return false
}
}

//enemy pojawiają się co 10 sekund
for(var i =0; i<Enemys.length;i++){
  Enemys[i] = new Enemy();
}


var rain = []
for(var i =0;i<30;i++){
 rain[i] = new RainPower()
}
// deszcz meteorow zaczyna się tworzyć, gdy nacisnę przycisk na ekranie, lecz wyciełem to bo jest tego sporo



if(rainpower.start == true){
    for(var i=0; i<rain.length;i++){
  rain[i].draw();
  rain[i].hits();

    for(var j =0;j<Enemys.length;j++){
    if(rain[i].hits(Enemys[j])){
      rain[i].hide()
      if(rain[i].toDelete == true){
        rain.splice(i,1);
      }
    }
    }
      }
    }

 

---

Dodam, że robię wszystko w czystym JS z racji tego iż chce się więcej nauczyć i odrozniać JS od bibliotek.

---

Ale klasa jest elementem js w standardzie es6. Jeśli się uczysz jsa ucz się aktualnych standardow