JavaScript losowanie liczby jeszcze raz jeżeli nie spełni warunku.

Question

 

function computerLogic(array) {

  function random() {

    var rand = Math.floor(Math.random() * 9);

    var result = (array[rand] === computerChoice) || (array[rand] === playerChoice) ?
      random(): (function() {
        array[rand] = computerChoice;
        grid.item(rand).innerHTML = computerChoice;
      })(rand);
  }

  random();


}

Moze i proste ale moja logika mnie zawodzi.  

To  gra w kółko i krzyżyk , computerLogic(array) jest wywoływana za każdym razem jak użytkownik kliknie i wtedy losowana jest liczba . Jak pod  indexem tablicy znajduje sie X albo O ( playerChoice) albo (userChoice) chce żeby wylosowało liczbe jeszcze raz. Do tej pory robiłem to jakby rekursywnie ale nie chce zeby za każdym razem wywoływało znowu funkcje random tylko poprostu znowu wylosowało..I jakoś wogóle nie mam pomysłu..

https://github.com/uRTLy/Tic-Tac-Toe tutaj kod aplikacji ale przed tym jak zmieniłem na ternary operator zamiast ifów.
 

edit
może też wracać do początku funkcji znowu albo poprostu niech znowu włączy rand ale niech ponownie sprawdzi czy nie wylosowało indexu z zajętym polem. Tylko niech nie wywołuje się jeszcze raz bo pod koniec w konsoli przeglądarki mam 10 randomów wywołanych i maximum call stack exceeded przy kółku i krzyżyku....

 

Answer 1

Osobiście nie szedłbym w rekurencję, a w pętlę do...while.

Comments

A pomożesz z warunkiem? Bo do...while też już próbowałem i za cholere nie moge wymyślić logicznego warunku, to takie irytujące...

do{

rand=Math.floor(Math.random() * );

}while((array[rand]!== playerChoice ) || (array[rand] =computerChoice))
nie wróci do góry funkcji i nie sprawdzi ponownie . A potrzebuje tego żeby wykonało uzupełnienie tablicy.

---

var random;

do {
    random = losuj();
} while ( board[ random ] === computersChoice || board[ random ] === playersChoice )

 

---

Dobra ale wtedy tylko wylosuje liczbe ponownie a nie wykona reszty instrukcji, które są ważne.Musi sprawdzić ponownie od początku funkcji czy warunek jest spełniony żeby wykonać ważne instrukcje.

---

No ok, ale samo losowanie będzie w pętli. Jak wykona się pętla, to przejdzie do reszty instrukcji.

---

no już mniej więcej cie czaje i zamieniłem to nawet na coś takiego :

 

function computerLogic(array) {
function randomize(){
  var x=Math.floor(Math.random() * 9);
  return x;
}

  do {
      random = randomize();

  } while ( array[ random ] !== computerChoice && array[ random ] !== playerChoice );

console.log(random);
if(( array[random] !== computerChoice) && (array[random] !== playerChoice )){
array[random]=computerChoice;
grid.item(random).innerHTML=computerChoice;

}

}

tylko teraz mi nie updatuje wyglądu :O. ta petla do while dalej jest cos nie tak.Tam przy If sprawdzam poraz kolejny czy array[random] !== X ani O ,
Ale nie wykonuje się instrukcja w srodku czyli pętla źle działa.

---

No bo Twój warunek w pętli jest wadliwy… Pętla ma się wykonywać tak długo, jak długo wylosowana liczba BĘDZIE zajęta przez gracza lub komputer. U Ciebie warunek jest odwrotny.

---

function computerLogic(array) {
var random;
var zero = 0;
while (array[random] === zero){
  random=(function (){
    var a= Math.floor(Math.random()*9);
    return a;
  })();
}
  console.log(random);

if(array[random] === zero ){
array[random]=computerChoice;
grid.item(random).innerHTML=computerChoice;
}
}

 

Ułatwiłem to sobie, jeżeli array[random] === 0 to znaczy że nie jest równy ani playerChoice ani computerChoice.

Tylko dalej jest coś nie tak, myślałem że to z porównaniem coś nie tak więc dalem 0 w zmienna. Hmm nie wiem co jest nie tak. Albo ta funkcja random jest zle zrobiona ale już próbowałem każdej.

---

Ehh… Przeczytaj jeszcze raz to:

Pętla ma się wykonywać tak długo, jak długo wylosowana liczba BĘDZIE zajęta przez gracza lub komputer.

A teraz pomyśl, co robi Twój kod ;) 

---

while (array[random] !== zero)

zapewne o to ci chodzi ale teraz while crashuje program,

---

Jak będę mieć chwilę wolnego, to obadam Twój projekt na GH.

---

Wiesz co zrobie od nowa całe kółko i krzyżyk całkiem inaczej bo mam teraz troche inny pomysł.