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

display de erros de forma gráfica durante inicialização da aplicação #4

Open
lucasgpulcinelli opened this issue Feb 4, 2024 · 2 comments · May be fixed by #8
Open

display de erros de forma gráfica durante inicialização da aplicação #4

lucasgpulcinelli opened this issue Feb 4, 2024 · 2 comments · May be fixed by #8
Labels
feature-request Request for a new feature

Comments

@lucasgpulcinelli
Copy link
Owner

No momento, quando há um erro durante a inicialização da aplicação, o erro é colocado via log.Printf, entretanto isso não é ideal caso o usuário não esteja usando a linha de comando (mesmo que os erros sejam apenas gerados por conta de argumentos da linha de comando, ainda acredito que o goICMCsim deve agir como uma aplicação gráfica o máximo de tempo possível e ter I/O de terminal apenas como facilitador para alguns usos).

O @ISS2718 comentou que talvez a melhor solução que tenhamos seja criar uma janela vazia, um popup com o erro, e quando o popup for fechado o programa termine. Isso é uma possibilidade via o recurso de popup do fyne, mas deixo aberto a discussão se existe uma alternativa melhor, e aberto a voluntários para a implementação.

@ISS2718
Copy link
Collaborator

ISS2718 commented Feb 5, 2024

Caso ocorram esses erros o simulador deve continuar a inicialização?

@lucasgpulcinelli
Copy link
Owner Author

Isso é questão de opinião na realidade. Eu considero que não, pois erros na inicialização de um charmap podem ser bem confusos, imagine a situação: você passa como argumento inicial um charmap, ele falha na leitura, mas você ignora e roda o seu programa normalmente. Da forma que o código está agora, todos os caracteres seriam completamente vazios, então toda a aplicação falharia em seu display. A causa pode não ser óbvio para alguém que está começando a usar o simulador agora

@ISS2718 ISS2718 added feature-request Request for a new feature macOS test Still to be tested on macOS and removed macOS test Still to be tested on macOS labels Feb 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request for a new feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants