WIP feat(collision-detection): move to web worker

.eslintignore

@@ -1,4 +1,5 @@
 dist/
 node_modules/
 lib/jquery/
+lib/threadify
 coverage/

.eslintrc

@@ -8,6 +8,7 @@
         "no-trailing-spaces": ["error", { "skipBlankLines": true }],
         "padded-blocks": ["error", "always"],
         "no-unused-vars": 0,
+        "new-cap": 0,
         "no-console": "error",
         "no-only-tests/no-only-tests": 2
     },

gulpfile.js

@@ -140,6 +140,8 @@ gulp.task('default', gulp.series(['clean', 'html', 'css', 'js', 'vendor']));
 
 gulp.task('test', gulp.series(['default'], function (done) {
 
+  process.env.NODE_ENV = 'test';
+
   return new KarmaServ({
     configFile: `${configuration.paths.test}/config/karma.unit.js`,
     singleRun: true

lib/canvas/canvas.js

@@ -220,63 +220,6 @@ class Canvas {
 
   }
 
-  /**
-   * checks collision between obstacles and bird object
-   */
-  collisionDetection () {
-
-    let birdShape = this.bird.shape;
-    let obstacles = this.obstacles;
-    let birdWidth = birdShape.x + birdShape.width;
-
-    const _hitAction = () => {
-
-      if (typeof this.onClear === 'function') {
-
-        this.onClear();
-
-      }
-
-    };
-
-    obstacles.forEach(obstacle => {
-
-      let obstacleShape = obstacle.shape;
-      let obstacleHeight = obstacleShape.y + obstacleShape.height;
-      let obstacleWidth = obstacleShape.x + obstacleShape.width;
-
-      if (
-        (birdShape.x >= obstacleShape.x && birdWidth <= obstacleWidth) ||
-        (birdWidth >= obstacleShape.x && birdWidth <= obstacleWidth) ||
-        (birdShape.x === obstacleWidth) ||
-        (birdShape.x === obstacleShape.x) ||
-        (birdWidth === obstacleWidth)
-      ) {
-
-        if (obstacleShape.y === 0) { // check upper obstacle
-
-          if (birdShape.y <= obstacleHeight) {
-
-            _hitAction();
-
-          }
-
-        } else { // check lower obstacle
-
-          if (birdShape.y + birdShape.height >= obstacleShape.y) {
-
-            _hitAction();
-
-          }
-
-        }
-
-      }
-
-    });
-
-  }
-
   /**
    * completely deletes canvas and all its shapes
    */

lib/collision-detection/collisionDetection.js

@@ -0,0 +1,87 @@
+
+/* global threadify */
+class CollisionDetection {
+
+  /**
+   *  @param {Bird} options.bird
+   *  @param {Array<Obstacle>} options.obstacle
+   */
+  static collisionDetection ({ bird, obstacles }) {
+
+    const thread = this;
+
+    let birdWidth = bird.x + bird.width;
+
+    obstacles.forEach(obstacle => {
+
+      let obstacleHeight = obstacle.y + obstacle.height;
+      let obstacleWidth = obstacle.x + obstacle.width;
+
+      if (
+        (bird.x >= obstacle.x && birdWidth <= obstacleWidth) ||
+                (birdWidth >= obstacle.x && birdWidth <= obstacleWidth) ||
+                (bird.x === obstacleWidth) ||
+                (bird.x === obstacle.x) ||
+                (birdWidth === obstacleWidth)
+      ) {
+
+        if (obstacle.y === 0) { // check upper obstacle
+
+          if (bird.y <= obstacleHeight) {
+
+            thread.return(true);
+
+          }
+
+        } else { // check lower obstacle
+
+          if (bird.y + bird.height >= obstacle.y) {
+
+            thread.return(true);
+
+          }
+
+        }
+
+      }
+
+    });
+
+    thread.return(false);
+
+  }
+
+  static wrapped (canvas) {
+
+    // cleanup objects for later use
+    const bird = {
+      x: canvas.bird.shape.x,
+      y: canvas.bird.shape.y,
+      width: canvas.bird.shape.width,
+      height: canvas.bird.shape.height
+    };
+
+    const obstacles = canvas.obstacles.map(o => {
+
+      return {
+        x: o.shape.x,
+        y: o.shape.y,
+        width: o.shape.width,
+        height: o.shape.height
+      };
+
+    });
+
+    const wrappedFn = threadify(this.collisionDetection);
+
+    return {
+      collisionDetection: wrappedFn,
+      collideCheckObject: {
+        bird,
+        obstacles
+      }
+    };
+
+  }
+
+}

lib/main/main.js

@@ -1,4 +1,4 @@
-/* global Canvas, Footer, KeyboardHandler */
+/* global Canvas, Footer, KeyboardHandler, CollisionDetection */
 let mrflap, canvas, bird;
 
 function appendFooter () {
@@ -93,7 +93,30 @@ function spawnObstacles () {
   setInterval(() => {
 
     canvas.moveObstacles();
-    canvas.collisionDetection();
+
+    if (_isTestMode()) {
+
+      return;
+
+    }
+
+    // execute collision detection inside web worker
+    const {
+      collisionDetection,
+      collideCheckObject
+    } = CollisionDetection.wrapped(canvas);
+
+    const job = collisionDetection(collideCheckObject);
+
+    job.done = function (isCollided) {
+
+      if (isCollided) {
+
+        canvas.onClear();
+
+      }
+
+    };
 
   }, 1000 / 30);
 
@@ -153,6 +176,12 @@ function _reset () {
 
 }
 
+function _isTestMode () {
+
+  return $('#test-mode').length > 0;
+
+}
+
 document.addEventListener('DOMContentLoaded', event => {
 
   _init();

test/canvas/canvasSpec.js

@@ -176,75 +176,4 @@ describe('Canvas', function () {
 
   });
 
-  describe('#collisionDetection', function () {
-
-    let canvas, bird, hitSpy;
-
-    beforeEach(function () {
-
-      hitSpy = sinon.spy();
-
-      canvas = new Canvas({
-        mrflapDiv: mrflapDiv,
-        onClear: hitSpy
-      });
-
-      bird = canvas.drawBird();
-
-    });
-
-    it('should hit lower obstacle', function () {
-
-      // given
-      const {
-        obstacleBottom
-      } = canvas.drawObstacle({
-        heightBottom: 50
-      });
-
-      // when
-      for (let i = 0; i < 110; i++) {
-
-        canvas.moveObstacles();
-
-      }
-
-      canvas.collisionDetection();
-
-      // then
-      expect(hitSpy).to.have.been.called;
-
-    });
-
-    it('should hit upper obstacle', function () {
-
-      // given
-      const {
-        obstacleTop
-      } = canvas.drawObstacle({
-        heightBottom: 50
-      });
-
-      bird.moveUp({
-        speed: 100
-      });
-
-      const originalX = obstacleTop.x;
-
-      // when
-      for (let i = 0; i < 110; i++) {
-
-        canvas.moveObstacles();
-
-      }
-
-      canvas.collisionDetection();
-
-      // then
-      expect(hitSpy).to.have.been.called;
-
-    });
-
-  });
-
 });