Implementado funcionalidade para maquinda de cafe #20
Conversation
| @@ -0,0 +1,11 @@ | |||
| <?xml version="1.0" encoding="UTF-8"?> | |||
There was a problem hiding this comment.
Os arquivos da IDE (a pasta .idea e o arquivo .iml) podem ser adicionados no .gitignore.
Esses arquivos geralmente não são versionados.
There was a problem hiding this comment.
criado arquivo .gitignore para desconsiderar pasta e arquivo
| return itemGratis; | ||
| } | ||
|
|
||
| private double getValorTotal() { |
There was a problem hiding this comment.
Não sei se vale muito a pena esses métodos que tem uma linha só...
No fim das contas eles só repassam a chamada de outro método, não tem nenhuma regra e também não me parece melhorar legibilidade em geral.
There was a problem hiding this comment.
Removido a classe comprarController foi criado um command para gerar o processo de pedido
| else isPagamentoRealizado = pagamentoService.realizarPagamento(getValorTotal()); | ||
|
|
||
|
|
||
| if (isPagamentoRealizado) |
There was a problem hiding this comment.
Pra if/else em Java é geralmente conveção sempre usar as chaves ({ e }) mesmo quando é uma linha só. Isso também melhora a legibilidade em geral.
Não sei se é o caso aqui, mas só o prepararBebida está no if. O atualizarEstoque e o que está pra frente não está.
Mas ele deveria atualizar o estoque se eu não preparei a bebida?
There was a problem hiding this comment.
Alterado somente é atualizado o estoque apos finalizar a retirada da bebida pronta
|
|
||
| public class ComprarController { | ||
|
|
||
| EstoqueService estoqueService = new EstoqueService(); |
There was a problem hiding this comment.
A estrutura controller/service para separa as responsabilidades é uma boa ideia.
There was a problem hiding this comment.
Verdade realizei algumas altereções nao sendo necessario mais a classe compraController removida!
|
|
||
| } | ||
|
|
||
| private boolean isItemGratis(int opcao, int qtdCopos) { |
There was a problem hiding this comment.
Esse método parece fazer mais coisas do que só verificar se é grátis. Talvez a regra de ser grátis também se encaixe melhor em outro lugar, tipo na própria bebida/item da máquina.
There was a problem hiding this comment.
hehe...nossa bem observado nao ficou bom nao hein... alterado!! agora passo a gerenciar se a bebida do metodo isNotGratis atraves de interface onde é possivel determina para cada bebida se é a mesma é gratis ou não
| import model.Color; | ||
| import service.AbastecimentoService; | ||
|
|
||
| public class ControlerReabastecer { |
There was a problem hiding this comment.
Acho que vale a pena seguir o mesmo padrão de nomenclatura de antes. Tipo AbastecimentoController.
There was a problem hiding this comment.
Verdade fora do padrao, porem removi a classe agora passo a gerencia o abastecimento da maquina apenas nas classe AbastecimentoController e AbastecimentoService
| public class ControlerReabastecer { | ||
| private final AbastecimentoService abastecimentoService = new AbastecimentoService(); | ||
|
|
||
| public void consultarEstoque() { |
There was a problem hiding this comment.
Esse método faz mais do que consultar. Talvez um outro nome ou dividir as responsabilidades aqui.
There was a problem hiding this comment.
Removido a classe não é mais necessaria!
| if (necessidade != 0) { | ||
| abastecimentoService.reabastecerAMaquina(); | ||
| } else { | ||
| System.out.println(Color.GREEN.getColorCode() + "\nMaquina se encontra abastecida!" + Color.RESET.getColorCode()); |
There was a problem hiding this comment.
Criar um service pra impressão das mensagens pode ser uma boa.
There was a problem hiding this comment.
Separei as mensagem fixa, dos fluxo acredito que ainda de para melhorar!
src/menu/MenuAbastecer.java
Outdated
| public class MenuAbastecer { | ||
| private static final ControlerReabastecer controlerReabastecer = new ControlerReabastecer(); | ||
|
|
||
| public static void abastecer(){ |
There was a problem hiding this comment.
Esse método também é um pouco estranho. Ele chama abastecer, mas dentro dele é uma chamada pra um método de nome consultar.
There was a problem hiding this comment.
Modificado implementado opcao para confirmar ou nao o abastecimento.
No description provided.