Desafio atualizado by lilianjaf · Pull Request #4 · movimentocodar/iniciante-java-oo-2021

lilianjaf · 2021-09-14T03:02:14Z

No description provided.

gcestaro · 2021-09-14T20:14:00Z

@@ -0,0 +1,3 @@
+# Default ignored files
+/shelf/


É uma boa prática ignorar os arquivos/pastas:

.idea

out/

gcestaro · 2021-09-14T20:36:48Z

+package cafes;
+
+public enum Acucar {
+    Nivel0("nenhuma"),Nivel1("pouquíssima"),Nivel2("pouca"),Nivel3("normal"),Nivel4("muito"),Nivel5("bastante");


a descrição dos campos do enum poderia seguir o mesmo que vc fez com o campo quantidadeDeAcucar (que poderia chamar descrição ao invés de quantidade, quantidade me remete a um número, o que acha?)

Uma convenção de nomenclatura para campos de enum é seguir o padrão UPPER_CASE, então poderia ser, por exemplo, NIVEL_0 ou NENHUMA, e assim por diante

gcestaro · 2021-09-14T20:58:25Z

+public class Bebida {
+    private int id;
+    private String nome;
+    Receitas receita;


Uma bebida tem várias Receitas? O nome da classe Receitas ficou no plural errado ou a ideia foi deixar uma classe com todas as Receitas mesmo?

Pq a visibilidade é default e não private?

gcestaro · 2021-09-14T20:59:29Z

+    private int id;
+    private String nome;
+    Receitas receita;
+    private double preco;


legal usar a classe BigDecimal do Java para valores que precisam de precisão decimal, como monetários

gcestaro · 2021-09-14T21:12:55Z

+    private String nome;
+    Receitas receita;
+    private double preco;
+    private static ArrayList<Bebida> menu = new ArrayList<Bebida>();


Pq a bebida tem um Menu? Consegue pensar em outra classe que representaria melhor essa relação tem um menu?

gcestaro · 2021-09-14T21:18:04Z

+    public static void main(String[] args) {
+        Scanner cafeScanner = new Scanner(System.in);
+        Bebida BebidasMenu = new Bebida();
+        ReservatorioDeAgua reservaDeAguaAtual = new ReservatorioDeAgua();


O reservatório de água não é uma propriedade (atributo) da MaquinaDeCafe?

gcestaro · 2021-09-14T21:19:48Z

+
+    private static void Inicializacao(Scanner cafeScanner, int AtualQuantidadeDeAgua, Bebida BebidasMenu) {
+
+        String msgReservatorioDeAgua = AtualQuantidadeDeAgua > 50? "suficiente para " + (AtualQuantidadeDeAgua /50) + " bebidas" : "vazios, favor repor";


( AtualQuantidadeDeAgua > 50 ) essa condição se repete no código, poderia extrair para uma variável, assim se o reservatório de água mudar de tamanho, você só precisará alterar em um lugar

gcestaro · 2021-09-14T21:22:07Z

+
+        if(AtualQuantidadeDeAgua > 50){
+            for (int i = 0; i < BebidasMenu.getMenu().size(); i++) {
+                System.out.println(BebidasMenu.getMenu().get(i).getId() + " " + BebidasMenu.getMenu().get(i).getNome() + ", preço: " + (BebidasMenu.getMenu().get(i).getPreco() == 0? "grátis" : moedaEmReais(BebidasMenu.getMenu().get(i).getPreco())));


Como vc acessa BebidasMenu.getMenu().get(i) várias vezes, poderia extrair para uma variável

gcestaro · 2021-09-14T21:23:13Z

+
+    private static String moedaEmReais(double valor) {
+        BigDecimal b = new BigDecimal(valor);
+        return "R$ " + b.setScale(2, RoundingMode.HALF_EVEN);


gcestaro · 2021-09-14T21:23:49Z

@@ -0,0 +1,51 @@
+package cafes;
+
+public class Receitas {


A classe deve representar uma unidade de negócio

gcestaro

Boa! 👏

gcestaro · 2021-09-16T12:20:11Z

    }
+
+    public static ArrayList<Acucar> getNiveisDeAcucar(){
+        ArrayList<Acucar> Niveis = new ArrayList<Acucar>();


Aqui vc tbm pode usar a interface List como retorno, assim se amanhã vc precisar mudar a implementação de ArrayList para outro tipo de List, só precisará mudar em um lugar

gcestaro · 2021-09-16T12:20:25Z

    private int id;
    private String nome;
-    Receitas receita;
+    Receita receita;


gcestaro · 2021-09-16T12:21:28Z

+
+    }
+
+    private static void Inicializacao(Scanner cafeScanner, ReservatorioDeAgua reservaDeAguaAtual, Bebida BebidasMenu){


A convenção para nomear métodos e seus parâmetros é camelCase

gcestaro · 2021-09-16T12:22:59Z

+    }
+
+    private static void Inicializacao(Scanner cafeScanner, ReservatorioDeAgua reservaDeAguaAtual, Bebida BebidasMenu){
+        String msgReservatorioDeAgua = reservaDeAguaAtual.getQuantidadeDeAguaNoReservatorio() >= 50? "suficiente para " + (reservaDeAguaAtual.getQuantidadeDeAguaNoReservatorio() /50) + " bebidas." : "vazios.";


Quando a linha fica muito grande é legal quebrar, nesse caso poderia quebrar antes do "?" e do ":", por exemplo

gcestaro · 2021-09-16T12:23:12Z

+
+        System.out.println("-----Seja bem-vindo a máquina de café 2021!-----");
+        System.out.println("Atenção: essa máquina não devolve troco.");
+        System.out.println("Para sair, digite 'sair' a qualquer momento.");


a qualquer momento mesmo?

gcestaro · 2021-09-16T12:24:49Z

+
+            if(precoBebidaEscolhida == 0){
+                System.out.println("A bebida será de graça.");
+                BebidasMenu.getMenu().get(Integer.parseInt(bebidaEscolhida)).receita.Processo(Acucar.Nivel0, reservaDeAguaAtual);


Essa sentença BebidasMenu.getMenu().get(Integer.parseInt(bebidaEscolhida)) se repete bastante, é legal extrair para uma variável/constante

gcestaro · 2021-09-16T12:28:57Z

+                    metodoDePagamentoEscolhido = getScanner(cafeScanner, "Digite 1 para pagamento em dinheiro e 2 para pagamento em cartão de débito.");
+                } while(metodoDePagamento(metodoDePagamentoEscolhido) == -1);
+
+                if(String.valueOf(metodoDePagamentoEscolhido).equals("1")){


A variável metodoDePagamentoEscolhido já é uma String, pq precisou do String.valueOf?

Quando vc está comparando uma variável do código com uma constante, nesse caso "1", inverte a condição para "1".equals(metodoDePagamentoEscolhido), dessa forma nunca acontecerá um NullPointerException.

"1" representa o que? Isso é conhecido como número mágico, pois não dá pra saber o que é só de olhar aqui. Uma ideia é extrair para uma constante, outra é ter um Enum de métodos de pagamento e usá-lo aqui

gcestaro · 2021-09-16T12:30:29Z

+                    String sugestaoMoeda = "";
+                    do {
+                        sugestaoMoeda = getScanner(cafeScanner, "Digite 1 ou 2 para aceitar a sugestão ou 3 para adicionar qualquer nota ou moeda. Atenção: essa máquina não devolve troco.");
+                    } while(aceitarSugestaoMoeda(sugestaoMoeda) == -1);


O nome do método aceitarSugestaoMoeda sugere que ele retorna um boolean, aceita ou não aceita (true/false)

gcestaro · 2021-09-16T12:32:11Z

+
+                System.out.println("--Favor escolher um nível de açúcar para sua bebida--");
+
+                for (int i = 0; i < Acucar.getNiveisDeAcucar().size() ; i++) {


gcestaro · 2021-09-16T12:35:31Z

+        return SugestaoDeNotasEMoedas;
+    }
+
+    private static NotasEMoedas CalcularSugestoes(double Preco, NotasEMoedas NotasEMoedasReais, boolean SegundaSugestao){


camelCase

lilianjaf added 3 commits September 13, 2021 20:03

Teste

2c42dfc

Testando

c159a70

Desafio em andamento

5c5915b

gcestaro reviewed Sep 14, 2021

View reviewed changes

Desafio finalizado

9e3e75a

lilianjaf changed the title ~~Desafio em andamento~~ Desafio finalizado Sep 15, 2021

gcestaro reviewed Sep 16, 2021

View reviewed changes

Máquina de café com todas funcionalidades e refatorado

3de67d8

lilianjaf changed the title ~~Desafio finalizado~~ Desafio atualizado Nov 1, 2021

lilianjaf added 4 commits November 2, 2021 01:09

Atualizado com Jackson e cupom em base32

64b1369

Alterações classe Cupons

5f7aad2

Atualição do Desafio com testes e nova refatoração

5b99054

Atualização do Desafio - Test runner

766bb1a


		private static void Inicializacao(Scanner cafeScanner, int AtualQuantidadeDeAgua, Bebida BebidasMenu) {

		String msgReservatorioDeAgua = AtualQuantidadeDeAgua > 50? "suficiente para " + (AtualQuantidadeDeAgua /50) + " bebidas" : "vazios, favor repor";


		System.out.println("--Favor escolher um nível de açúcar para sua bebida--");

		for (int i = 0; i < Acucar.getNiveisDeAcucar().size() ; i++) {

Conversation

lilianjaf commented Sep 14, 2021

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

gcestaro left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants