X-Git-Url: https://git.auder.net/assets/current/gitweb.css?a=blobdiff_plain;f=client%2Fsrc%2Fcomponents%2FComputerGame.vue;h=2745e4c14ac7355bcc5617ef4235de073bab491e;hb=7e355d68b5962aad106e28a9d669a47f3cbec43a;hp=f5d462371d21c2de94a81f74299a9ccec51672e3;hpb=93d1d7a7e07e7bad94318d3cb4cbe906d85bd811;p=vchess.git

diff --git a/client/src/components/ComputerGame.vue b/client/src/components/ComputerGame.vue
index f5d46237..2745e4c1 100644
--- a/client/src/components/ComputerGame.vue
+++ b/client/src/components/ComputerGame.vue
@@ -8,10 +8,10 @@
 <script>
 import BaseGame from "@/components/BaseGame.vue";
 import { store } from "@/store";
-import Worker from 'worker-loader!@/playCompMove';
+import Worker from "worker-loader!@/playCompMove";
 
 export default {
-  name: 'my-computer-game',
+  name: "my-computer-game",
   components: {
     BaseGame,
   },
@@ -21,47 +21,57 @@ export default {
   data: function() {
     return {
       st: store.state,
-      game: { },
+      game: {},
       vr: null,
       // Web worker to play computer moves without freezing interface:
       timeStart: undefined, //time when computer starts thinking
-      lockCompThink: false, //to avoid some ghost moves
+      compThink: false, //avoid asking a new move while one is being searched
       compWorker: null,
     };
   },
   watch: {
     "gameInfo.fen": function() {
-      // (Security) No effect if a computer move is in progress:
-      if (this.lockCompThink)
-        return this.$emit("computer-think");
       this.launchGame();
     },
-    "gameInfo.userStop": function() {
-      if (this.gameInfo.userStop)
+    "gameInfo.score": function(newScore) {
+      if (newScore != "*")
       {
-        // User stopped the game: unknown result
+        this.game.score = newScore; //user action
         this.game.mode = "analyze";
+        if (!this.compThink)
+          this.$emit("game-stopped"); //otherwise wait for comp
       }
     },
   },
   // Modal end of game, and then sub-components
   created: function() {
-    // Computer moves web worker logic: (TODO: also for observers in HH games ?)
-    this.compWorker = new Worker(); //'/javascripts/playCompMove.js'),
+    // Computer moves web worker logic:
+    this.compWorker = new Worker();
     this.compWorker.onmessage = e => {
-      this.lockCompThink = true; //to avoid some ghost moves
       let compMove = e.data;
+      if (!compMove)
+        return; //after game ends, no more moves, nothing to do
       if (!Array.isArray(compMove))
         compMove = [compMove]; //to deal with MarseilleRules
       // Small delay for the bot to appear "more human"
       const delay = Math.max(500-(Date.now()-this.timeStart), 0);
       setTimeout(() => {
+        // NOTE: Dark and 2-moves are incompatible
         const animate = (this.gameInfo.vname != "Dark");
-        this.$refs.basegame.play(compMove[0], animate);
-        if (compMove.length == 2)
-          setTimeout( () => { this.$refs.basegame.play(compMove[1], animate); }, 750);
-        else //250 == length of animation (TODO: should be a constant somewhere)
-          setTimeout( () => this.lockCompThink = false, 250);
+        const animDelay = (animate ? 250 : 0);
+        let moveIdx = 0;
+        let self = this;
+        (function executeMove() {
+          self.$refs.basegame.play(compMove[moveIdx++], animate);
+          if (moveIdx >= compMove.length)
+          {
+            self.compThink = false;
+            if (self.game.score != "*") //user action
+              self.$emit("game-stopped");
+          }
+          else
+            setTimeout(executeMove, 500 + animDelay);
+        })();
       }, delay);
     }
     if (!!this.gameInfo.fen)
@@ -77,16 +87,17 @@ export default {
       this.compWorker.postMessage(["init",this.gameInfo.fen]);
       this.vr = new V(this.gameInfo.fen);
       const mycolor = (Math.random() < 0.5 ? "w" : "b");
-      let players = ["Myself","Computer"];
+      let players = [{name:"Myself"},{name:"Computer"}];
       if (mycolor == "b")
         players = players.reverse();
-      // NOTE: (TODO?) fen and fenStart are redundant in game object
+      // NOTE: fen and fenStart are redundant in game object
       this.game = Object.assign({},
         this.gameInfo,
         {
           fenStart: this.gameInfo.fen,
           players: players,
           mycolor: mycolor,
+          score: "*",
         });
       this.compWorker.postMessage(["init",this.gameInfo.fen]);
       if (mycolor != "w" || this.gameInfo.mode == "auto")
@@ -94,9 +105,9 @@ export default {
     },
     playComputerMove: function() {
       this.timeStart = Date.now();
+      this.compThink = true;
       this.compWorker.postMessage(["askmove"]);
     },
-    // TODO: do not process if game is over (check score ?)
     processMove: function(move) {
       // Send the move to web worker (including his own moves)
       this.compWorker.postMessage(["newmove",move]);
@@ -108,8 +119,9 @@ export default {
       }
     },
     gameOver: function(score) {
-      // Just switch to analyze mode: no user action can set score
+      this.game.score = score;
       this.game.mode = "analyze";
+      this.$emit("game-over", score); //bubble up to Rules.vue
     },
   },
 };