Go to file
Francesco Allara 1108215d5e Initial commit 2024-03-11 15:42:20 +01:00
bin Initial commit 2024-03-11 15:42:20 +01:00
config Initial commit 2024-03-11 15:42:20 +01:00
migrations Initial commit 2024-03-11 15:42:20 +01:00
public Initial commit 2024-03-11 15:42:20 +01:00
src Initial commit 2024-03-11 15:42:20 +01:00
tests Initial commit 2024-03-11 15:42:20 +01:00
.env Initial commit 2024-03-11 15:42:20 +01:00
.env.test Initial commit 2024-03-11 15:42:20 +01:00
.gitignore Initial commit 2024-03-11 15:42:20 +01:00
Makefile Initial commit 2024-03-11 15:42:20 +01:00
README.md Initial commit 2024-03-11 15:42:20 +01:00
composer.json Initial commit 2024-03-11 15:42:20 +01:00
composer.lock Initial commit 2024-03-11 15:42:20 +01:00
docker-compose.yml Initial commit 2024-03-11 15:42:20 +01:00
phpunit.xml.dist Initial commit 2024-03-11 15:42:20 +01:00
symfony.lock Initial commit 2024-03-11 15:42:20 +01:00

README.md

Ma questa e' una possibile implementazione della soluzione al test Ho preferito tenerlo semplice senza richieste HTTP, quindi non c'e' stato bisogno di aggiungere chiamate a cURL

Tutta la business logic del gioco e' nel modello Game, quindi tutta la logica e' in unit tests che e' possibile vedere in tests/Service/GameServiceTest.php ed e' possibile eseguire con make phpunit

Per installare le dipendenze basta eseguire make composer_install

La ragione per la mancanza di un'interfaccia completa HTTP e' che in ogni caso non c'e' molto da testare poiche' io tendenzialmente uso controller slim e un service layer cosicche' sia possibile riutilizzare la logica fra diversi client, nel caso ci fosse una interfaccia CLI, http etc.

Per la mancanza di commenti, spero di aver utilizzato nomi di metodi, classi e variabili abbastanza chiari da non rendere necessario l'aggiunta di commenti io reputo i commenti un code smell poiche' di solito per me sono segno di codice non chiaro, e anche perche' i commenti rappresentano codice, pero' a differenza del codice non e' validato da nessun test, quindi nel caso il codice cambi e ci si dimentichi di aggiornare i commenti, sarebbero da considerare poi non veritieri

Comunque spero di avere la possibilita' di spiegare ogni decisione poi nel prossimo colloquio nel caso la soluzione sia considerata accettabile

A presto (spero)