major code cleanup

JSLegendDev · Jan 4, 2024 · 85add3d · 85add3d
1 parent 3e5a4bc
commit 85add3d
Show file tree

Hide file tree

Showing 7 changed files with 32 additions and 21 deletions.
diff --git a/entities/character.js b/entities/character.js
@@ -13,10 +13,16 @@ export function makeCharacter(p) {
     height: 32,
 
     setAnim(name) {
-      this.currentAnim = name;
-      this.currentFrame = 0;
-      this.animationTimer = 0;
-      this.previousTime = 0;
+      if (this.currentAnim !== name) {
+        this.currentAnim = name;
+        this.currentFrame = 0;
+        this.animationTimer = 0;
+        this.previousTime = 0;
+      }
+    },
+
+    setDirection(direction) {
+      if (this.direction !== direction) this.direction = direction;
     },
 
     setAnimFrame(animData) {

diff --git a/entities/dialogBox.js b/entities/dialogBox.js
@@ -61,7 +61,6 @@ export function makeDialogBox(p, x, y) {
     },
     draw() {
       if (!this.isVisible) return;
-      p.noSmooth();
       p.image(this.spriteRef, this.x, this.y);
       p.fill("black");
       p.textSize(24);

diff --git a/entities/npc.js b/entities/npc.js
@@ -28,6 +28,11 @@ export function makeNPC(p, x, y) {
       };
     },
 
+    setup() {
+      this.prepareAnims();
+      this.setAnim("idle-down");
+    },
+
     update() {
       this.previousTime = this.animationTimer;
       this.animationTimer += p.deltaTime;
@@ -39,7 +44,6 @@ export function makeNPC(p, x, y) {
       this.screenX = this.x + camera.x;
       this.screenY = this.y + camera.y;
 
-      p.noSmooth();
       debugMode.drawHitbox(p, this);
       drawTile(
         p,

diff --git a/entities/player.js b/entities/player.js
@@ -5,7 +5,6 @@ import { drawTile, getFramesPos, isMaxOneKeyDown } from "../utils.js";
 export function makePlayer(p, x, y) {
   return {
     ...makeCharacter(p),
-    direction: "down",
     speed: 200,
     x,
     y,
@@ -35,30 +34,36 @@ export function makePlayer(p, x, y) {
       if (!isMaxOneKeyDown(p) || this.freeze) return;
 
       if (p.keyIsDown(p.RIGHT_ARROW)) {
-        if (this.direction !== "right") this.direction = "right";
-        if (this.currentAnim !== "run-side") this.setAnim("run-side");
+        this.setDirection("right");
+        this.setAnim("run-side");
         this.x += moveBy;
       }
 
       if (p.keyIsDown(p.LEFT_ARROW)) {
-        if (this.direction !== "left") this.direction = "left";
-        if (this.currentAnim !== "run-side") this.setAnim("run-side");
+        this.setDirection("left");
+        this.setAnim("run-side");
         this.x -= moveBy;
       }
 
       if (p.keyIsDown(p.UP_ARROW)) {
-        if (this.direction !== "up") this.direction = "up";
-        if (this.currentAnim !== "run-up") this.setAnim("run-up");
+        this.setDirection("up");
+        this.setAnim("run-up");
         this.y -= moveBy;
       }
 
       if (p.keyIsDown(p.DOWN_ARROW)) {
-        if (this.direction !== "down") this.direction = "down";
-        if (this.currentAnim !== "run-down") this.setAnim("run-down");
+        this.setDirection("down");
+        this.setAnim("run-down");
         this.y += moveBy;
       }
     },
 
+    setup() {
+      this.prepareAnims();
+      this.direction = "down";
+      this.setAnim("idle-down");
+    },
+
     update() {
       this.previousTime = this.animationTimer;
       this.animationTimer += p.deltaTime;
@@ -79,7 +84,6 @@ export function makePlayer(p, x, y) {
         p.scale(-1, 1);
         p.translate(-2 * this.screenX - this.tileWidth, 0);
       }
-      p.noSmooth();
       debugMode.drawHitbox(p, this);
       drawTile(
         p,

diff --git a/main.js b/main.js
@@ -31,6 +31,7 @@ new p5((p) => {
     canvasEl.canvas.style = "";
 
     p.textFont(font);
+    p.noSmooth(); // for pixels to not become blurry
 
     world.setup();
     battle.setup();

diff --git a/scenes/menu.js b/scenes/menu.js
@@ -21,7 +21,6 @@ export function makeMenu(p) {
     },
     draw() {
       p.clear();
-      p.noSmooth();
       p.image(this.startScreenImgRef, 0, 0);
       p.tint(255, this.alpha);
       p.image(this.startTextImgRef, 0, 320);

diff --git a/scenes/world.js b/scenes/world.js
@@ -37,12 +37,10 @@ export function makeWorld(p, setScene) {
           default:
         }
       }
-      this.player.prepareAnims();
-      this.player.setAnim("idle-down");
+      this.player.setup();
       this.camera.attachTo(this.player);
 
-      this.npc.prepareAnims();
-      this.npc.setAnim("idle-down");
+      this.npc.setup();
     },
 
     update() {