Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correcciones primera entrega #11

Open
Elianagam opened this issue Jul 14, 2020 · 0 comments
Open

Correcciones primera entrega #11

Elianagam opened this issue Jul 14, 2020 · 0 comments

Comments

@Elianagam
Copy link

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant