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

Tratamento de exceções #84

Open
henriquesimoes opened this issue Jul 24, 2020 · 3 comments
Open

Tratamento de exceções #84

henriquesimoes opened this issue Jul 24, 2020 · 3 comments
Labels
invalid This doesn't seem right

Comments

@henriquesimoes
Copy link
Owner

Embora tenhamos utilizado poucas exceções, em algumas partes incluímos o lançamento, mas não realizamos o tratamento em local algum. Em alguns casos, o código é esperado funcionar sem ocorrer a exceção, mas poderia ocorrer o lançamento. Um exemplo é o estado inválido do mapa ao estar cheio e se tentar inserir uma nova armadilha ou walker. Acho que deveríamos tratar isso mesmo que seja impossível de ocorrer com a configuração atual.

@henriquesimoes henriquesimoes added the invalid This doesn't seem right label Jul 24, 2020
@renanffernando
Copy link
Collaborator

Então, no caso da armadilha até dá pra simplesmente não inserir né?

Mas no caso do Walker fica difícil né? Se o Walker for justamente o jogador, não dá pra não inserir ele. Tu consegue imaginar como tratar isso?

@henriquesimoes
Copy link
Owner Author

No caso do Walker complica realmente. Talvez fazer alguma de inserir alguma possível coordenada numa exceção funcionasse. Mas daí teria que ser necessariamente de algo que pode ser removido do mapa, como um monstro.

Só que bate a dúvida se realmente aconteceria isso de ser o jogador. Porque agora o mapa é populado só quando o usuário deu play e já selecionou o personagem dele.

@renanffernando
Copy link
Collaborator

Pois é, seria meio impossível de acontecer. Talvez uma opção é deixar morrer o processo se for um jogador e dizer que ta tudo bem se for um monstro.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

2 participants