observação 1

Ao tentar rodar o programa recebi um erro: Exception in thread "main" java.lang.ExceptionInInitializerError at org.ravin.App.mainMenu(App.java:34) at org.ravin.App.main(App.java:21) Caused by: java.lang.NullPointerException at java.base/java.util.Objects.requireNonNull(Objects.java:209) at org.ravin.views.View.(View.java:8)

Acredito que seja pela modificação do local da logo do ravin: View.java:8, como a imagem não está no mesmo nível de diretório da View.java, é preciso ir voltando o diretório através da adição de ''../' até que que ele encontre a imagem. Usar "../../../images/logo.png" funcionou pra mim.

Beleza, nesse basta arrumar o path.

Observação 2

Percebi que a estrutura de pastas foi modificada em algum momento em commits anteriores, talvez até em branchs anteriores, pois já esta assim na dev:

Essa interface tinha acabado sobrando em algum commit mesmo, mas foi deletado nessa Issue

observação 3

App.java: exceptions devem ser tratadas dentro do método que chamar outros métodos que podem as gerar, nunca na main, pois nesse caso, se ocorrerem irão parar a execução do programa. Sei que há casos de exemplificação que aparerecem dessa forma, mas acredito que sejam apenas para ser mais didático e poupar mais linhas de código para o leitor.

Pois é, nesse caso eu tava me atrapalhando com a IDE, pois declarei uma Exception nova para uma mesa cheia, mas não tinha alocado onde ela seria tradada. Clicando em sugestões da IDE a Exception acabou subindo até a main. Oops!

observação 4

Ao tentar cadastrar uma mesa recebi o segfuinte erro: java.lang.NullPointerException: Cannot invoke "org.ravin.models.Funcionario.getNome()" because the return value of "org.ravin.models.Mesa.getFuncionario()" is null at org.ravin.views.mesa.SubmenuMesa.imprimeMesa(SubmenuMesa.java:56) at org.ravin.views.mesa.CadastrarMesaView.cadastrarMesa(CadastrarMesaView.java:31) at org.ravin.views.mesa.SubmenuMesa.menuMesa(SubmenuMesa.java:27) at org.ravin.App.processaOpcaoMenu(App.java:60) at org.ravin.App.mainMenu(App.java:39) at org.ravin.App.main(App.java:21)

Acho que colocando um teste na linha SubmenuMesa.java:56 pra checar se há um funcionário e em caso contrário omitir a informação, já resolve;

Pois então! Eu tinha deixado a alocação de Funcionário para um momento posterior, separando a criação da mesa de um sistema de alocação de funcionário. O problema é que faltou implementar esses sistema. Vou criar uma Issue pra isso

observação 5

Um problema semelhante a observação 4 ocorre ao selecionar a opção listar mesas e entre a lista de mesas existir uma sem funcionário:

Exception in thread "main" java.lang.NullPointerException: Cannot invoke "org.ravin.models.Funcionario.getNome()" because the return value of "org.ravin.models.Mesa.getFuncionario()" is null at org.ravin.views.mesa.RecuperarMesaView.listarMesas(RecuperarMesaView.java:31) at org.ravin.views.mesa.SubmenuMesa.menuMesa(SubmenuMesa.java:30) at org.ravin.App.processaOpcaoMenu(App.java:60) at org.ravin.App.mainMenu(App.java:39) at org.ravin.App.main(App.java:21)

Talvez nesse caso, o retorno poderia ser nulo, já que nem toda mesa precisa ter um funcionário alocado. Isso é tranquilo de corrigir.

observação 6

Em que momento se insere um funcionário a uma mesa?

Existe a possibilidade de trocar o funcionário de uma mesa?

O mesmo que o comentário da observação 4.

observação 7

Pouco relevante, mas na CadastrarReservaView linha 52, no caso de um cliente não ser encontrado o ideal é exibir a mensagem retornada pela controller.

Relevante sim! Já que não temos retornos HTTP, a essência dos controllers é servirem para disparar erros informativos.

observação 8

Ao tentar inserir uma reserva obtive um erro que encerrou a aplicação:

Esse último bug...meu deus kkkk

Obrigado pelo review detalhado!