SebastiaanBuwalda / PhaserSnake

Phaser snake game
http://18977.hosts.ma-cloud.nl/snakegame/
0 stars 0 forks source link

Redelijk hard gekoppeld #5

Open BerendWeij opened 7 years ago

BerendWeij commented 7 years ago

Je hebt feedback gekregen van BerendWeij op:

cursors.left.isDown || listenToSwipes==true&&swipeFunctions.returnSwipeDirection()=="left" 

URL: https://github.com/SebastiaanBuwalda/PhaserSnake/blob/master/snakegame/Scripts/inputHandler.js

Feedback: Je koppelt nu vrij hard aan mogelijke inputs. Als je naar nog meer inputs wilt luisteren dan wordt dit wel een erg grote if. Dit zou je ook met losse objecten moeten kunnen oplossen.

Ik zou deze check ook uit de movement houden. De SRP van movement is het moven opzich. Niet het luisteren naar input.