gianbelinche / taller-tp-final Goto Github PK

Buenas, les dejo las correcciones del Tp

Server:
Las configuraciones que estan en el run se pueden hacer en el init y tener esos valores como atributos de clase
Eliminar comentarios en ChatMessageParser
Evitar copias de string
ClientHandler :: getCredentials quedo muy larga, acortar, idem handleNewPlayer
Map::Map es muy largo, idem MasterFactory

Cliente:
Dividir el cliente en directorios
Falta organizar, excepciones, entidades, parte de sdl, ventana de inicio y musica
Tener un archivo de defines para el path de las imagenes, musica,variables de sdl, etc (como lo que esta en player)
main muy largo, modelizar
no subir ejecutables al repo

Falta documentar clases importantes
falta un common para queues, socket, errores, protocolo
Directorio media, para musica, json, imagenes, etc
código repetido del socket.