rowinruizendaal / multiplayer-snake

🐍 Multiplayer snake game 🐍

Hi there 👋

Tech and Tools

Current favourites

Other tools I use

"The best error message is the one that never shows up."

multiplayer-snake's Issues

Lost instantly, not enough time to move

It seems like the speed is very high, I couldn't even move before already losing the game.

Maybe consider showing the snakes on the countdown, so the user knows where their snake is.

firebase session on begin screen not implemented

Internal server error after picking username

leaderboard display block bug

Shoutout

Can I have a shoutout, please?

Data flowchart niet volledig

Het lijkt er naar mij op dat de flow in je chart niet helemaal correct is. Er wordt bijvoorbeeld vanuit de server een nieuwe positie gereturned, alleen lijkt het er op deze manier op dat deze nooit bij de client terecht komt

Peer review

Peer review

Repository

• Repository heeft nu nog de standaard RTW naam, als je een naam voor je project hebt kun je die aanpassen in de settings
• Er is een beschrijving van het concept aanwezig die erg goed is
• Live link is aanwezig
• Ik zou code meer bundelen in een /src folder. Nu staan er losse .js bestanden in de root
• Er is een dataflow aanwezig, maar voor de rest nog geen API beschrijving, real-time events or install notes

Concept

• Ik begrijp wat de app doet vanuit de beschrijving
• De app staat live en werkt ook al voor een best groot deel. Ik kon zelf het spel niet in mijn eentje spelen, maar ik kon wel al in een room en zag op 1 scherm dat ik won en op het andere scherm dat ik verloor
• Ik denk als je dit goed werkend krijgt dat er weinig meer aan toegevoegt hoeft te worden, het zou zo al super vet zijn!

Data life cycle

Data life cycle is uitgebreid, ik zie alleen niet gelijk het verschil tussen API calls & websockets verbindingen. Dit zou nog beter kunnen met kleuren

Real-time events

Nog niet aanwezig.

API info

Nog niet aanwezig

Code comments

• Ik vind het heel netjes opgezet met duidelijk onderscheid tussen normale variables en de constantes van de game (in ALL CAPS)
• Er zijn wel veel (vind ik) onnodige enters in de code, maar ik begrijp dat dit per persoon verschilt
• Misschien zou je nog kunnen kijken naar Babel om ES6 import en export bijvoorbeeld te gebruiken

Authors

May I also have a shoutout? :)
https://github.com/InjuMichorius

Snakes dont move through walls

When playing the original snake game, you're able to enter a wall and exit on the other side.

Rematch didn't work

When both players clicked rematch, nothing happened, also no feedback that it went wrong.

Consider moving some functions out of server.js

It may be a personal thing, but I think a server becomes easier to read/understand when you can quickly see what is called when. Functionality behind it, the functions, can be outside the main js file.

Remove course

Consider removing the folder "course", so your application looks more like your own project.

Countdown before game begins

game should have a countdown before the game starts, now the game starts when the second user joins

readMe mist beschrijving

App beschrijving mist en eigenlijk vrijwel alle andere beschrijvingen, zoals externa data, real time events, etc.

Wiki contains not significant information

I think the wiki contains information about the course HCD

Change your application name

Consider changing your application name. real-time-web = mulitplayer-snake

copy gamecode button not fully clickable

Only the bottom part of the button is clickable for me

README on snake branch too scanty

I'm a nerd, so I know what snake is. If i were to go to my friends, they don't really know what snake nor the goal is. Add a description that explains how the game works!

Can't join a game by visiting the 'copy game code' url

Lets me pick a username and then i still have to manually add the code

README can be more organised

I'm a bit confused on where to find information about the application. The first week where we implemented a live chat on our team page would fit better on your wiki, since it has little to do with the current project :)

set pointer-events: none to nav when collapsed

The hidden nav blocks me from clicking the button and selecting the game code