Przekazanie array z modelu do widoku

Question

Cześć,

Mam problem z przekazaniem wyniku z modelu do widoku. Przy testowaniu model wypluwa prawidłową tablicę ale widok nic z tym nie robi. Po wpisaniu tablicy z palca do widoku działa.

var model = {
  numCreatures: 4,
  numPictures: 8,
  numLetters: 2, 
  chooseCreatures: function(numCreatures){
      var arrNumb = []
      while(arrNumb.length < this.numCreatures){
      var randomnumber = Math.ceil(Math.random()*this.numPictures)
      if(arrNumb.indexOf(randomnumber) > -1) continue;
      arrNumb[arrNumb.length] = randomnumber;
      }
      view.introductionCreature(arrNumb);
  }

 

 

var view = {
introductionCreature: function(numbPic){
    var imgs = document.querySelectorAll('.character');
    for (let i=0; i<numbPic.length; i++){
    var num=numbPic[i];
    imgs[i].src='img/postac' + num + '.jpg';
    }
  }

 

Answer 1

Nie wiem za bardzo o co chodzi, pomijając fakt, że w przesłanym przykładzie pozjadałeś zamknięcia nawiasów, to wszystko działa...

https://jsfiddle.net/et4hswar/

Comments

Uruchamiając w konsoli  model.chooseCreatures(model.numCreatures); wyświetla mi prawidłowy łańcuch liczb. Ale view.introductionCreature(arrNumb); się nie wykonuje.

Wpisując w konsoli view.introductionCreature([2,4,1,7]); dostaje to to chcę.

Sorry za nawiasy, skopiowałem tylko część kodu.

---

Nadal nie wiem co chcesz osiągnąć, w moim przykładzie wszystkie pętle się wykonują. Jedyną różnicą jest zakomentowanie  

//imgs[i].src='img/postac' + num + '.jpg';

ze względu, że nie mam obrazków. Możesz dać błąd jaki dostajesz w consoli? Na chwile obecną jedyne co przychodzi do głowy to, to co napisał Alwox.

edit: zobacz jeszcze co ci zwraca: var imgs = document.querySelectorAll('.character');

może w tym tkwi problem.

---

Nie mam błędu. Jest to bardzo dziwne, bo nawet w twoim przykładzie na jsfiddle nie mam wyniku. Na chromie i FF. 

Answer 2

Dane przekazywane są poprawne ale pętla w view nie sprawdza ilości obiektów w imgs. Obstawiam więc, że z model przychodzi więcej liczb, niż tych elementów co skutkuje błędem przy próbie przypisania wartości do nieistniejącego obiektu. Gdy wpisujesz tablicę na sztywno prawdopodobnie piszesz ich wystarczająco mało.

Comments

No właśnie nie, bo testuję to w konsoli wklejając wypluwane dane z modelu i działa.

---

var model = {
  numCreatures: 4,
  numPictures: 8,
  numLetters: 2,
  chooseCreatures: function(numCreatures){
      var arrNumb = []
      while(arrNumb.length < this.numCreatures){
      var randomnumber = Math.ceil(Math.random()*this.numPictures)
      if(arrNumb.indexOf(randomnumber) > -1) continue;
      arrNumb[arrNumb.length] = randomnumber;
      }
      view.introductionCreature(arrNumb);
			console.log(arrNumb)
  }}

var view = {
introductionCreature: function(numbPic){
	var imgs = [
		{img1:1},
		{img2:2},
		{img3:3},
		{img4:4}
	]

    for (let i=0; i<numbPic.length; i++){
    var num=numbPic[i];
		imgs[i].src='img/postac' + num + '.jpg';
    }
		console.log(imgs);
  }
}

model.chooseCreatures(model.numCreatures)

zmieniłem tylko tablicę imgs (bo nie mam dokumentu html), dodałem 2 console logi i wszystko działa więc problem jest raczej gdzieś indziej.

---

No właśnie, wkleiłem twój kod do jsfiddle a tam pusto. Na chromie i FF. ??

---

jsfiddle nigdy nie używałem, czemu nie użyjesz po prostu konsoli Chroma?

---

Używam teraz co się do do sprawdzenia. konsole chroma też.