stefanalexlupu / secret-santa Goto Github PK

Salut, am vazut pe Linkedin ca ai cerut niste indrumari in legatura cu proiectul, sper sa-ti fie de folos parerile mele 😃

Ceva precizari mai generale:

1. Cum ti-a mai sugerat cineva, MVC e alegere destul de populara. Mie personal imi place sa folosesc peste Express un framework cum e NestJS pentru ca iti impune o arhitectura mai organizata.
2. E de preferat sa ai consitenta in raspunsurile API-ului, vad ca unele rute raspund cu HTML, altele cu JSON si nu e foarte bine separat.
3. Cred ca e un pic over-inginereala ce zic la punctul asta, dar privind in perspectiva, ar fi util sa abstractizezi cum salvezi/interactionezi cu obiectele pentru camere si userzi. Poate in viitor o sa fie necesar sa folosesti un server de cache mai serios, nu doar variabile in memory

Ceva precizari mai specifice despre cod:

1. As evita sa folosesc null pe cat se poate de mult, declararea sa conincida cu intializarea unui state valid daca se poate
2. In ES2015 s-au introdus string templates in Javascript, sunt o varianta mai eleganta decat "string" + "string"
3. Cred ca poti sa te folosesti de ID-ul socket-ului ca sa identifici un user (ar trebui sa fie unice) si asa scapi si de for-ul asta .
4. In codul din client folosesti var si Promises in loc de const/let si async/await
5. Aici cred ar fi bine de evitat orice display de date intre <script>. Poate e doar paranoia personala :)))