Reuse same tween for monster shaking
authorMegaBrutal <code+git@megabrutal.com>
Fri, 29 Dec 2017 23:25:00 +0000 (00:25 +0100)
committerMegaBrutal <code+git@megabrutal.com>
Fri, 29 Dec 2017 23:25:00 +0000 (00:25 +0100)
Now one tween is created and reused to shake a monster when it is
damaged. As a side-effect, the amplitude of shaking is now constant
while it was random in the previous commit.

I decided to use one tween for a monster to avoid creating a memory
leak by leaving unused tweens in memory.

modified:   ld40.js

ld40.js

diff --git a/ld40.js b/ld40.js
index 53e41f8..7bf03d7 100644 (file)
--- a/ld40.js
+++ b/ld40.js
@@ -32,6 +32,9 @@
             }
         }, false);
 
+        function sign(n) {
+            if (n >= 0) { return 1 } else { return -1 };
+        }
 
         class Player extends Phaser.Text {
             constructor(x, y) {
                 this.weapon = new JusticeBlaster(game, this);
                 this.weapon.bulletSpeed = SPEED_MPROJECTILE;
                 this.weapon.trackSprite(this);
+                this.shaketween = game.add.tween(this).to({ x: this.x + (sign(Math.random() - 0.5)) * 5 }, 10, Phaser.Easing.Sinusoidal.InOut, false, 0, 8, true);
             }
 
             enablePhysics() {
 
             damage(amount) {
                 super.damage(amount);
-                if (this.alive) { game.add.tween(this).to({ x: this.x + (Math.random() - 0.5) * 10 }, 10, Phaser.Easing.Sinusoidal.InOut, true, 0, 8, true); }
+                if (this.alive) { this.shaketween.start(); }
             }
 
             kill() {