Desafio atualizado #4

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

No description provided.

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

.idea/.gitignore

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


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

.idea

out/

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

src/cafes/Acucar.java

+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

src/cafes/Bebida.java

+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

src/cafes/Bebida.java

+    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

src/cafes/Bebida.java

+    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

src/cafes/MaquinaDeCafe.java

+    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

src/cafes/MaquinaDeCafe.java

+
+    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

src/cafes/MaquinaDeCafe.java

+
+        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

src/cafes/MaquinaDeCafe.java

+
+    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

src/cafes/Receitas.java

@@ -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

src/cafes/Acucar.java

@@ -12,4 +13,17 @@ private Acucar(String quantidade){
    public String getQuantidadeDeAcucar() {
        return this.quantidadeDeAcucar;
    }
+
+    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

src/cafes/Bebida.java

@@ -4,15 +4,15 @@
 public class Bebida {
    private int id;
    private String nome;
-    Receitas receita;
+    Receita receita;


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

src/cafes/MaquinaDeCafe.java

+
+    }
+
+    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

src/cafes/MaquinaDeCafe.java

+    }
+
+    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

src/cafes/MaquinaDeCafe.java

+
+        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

src/cafes/MaquinaDeCafe.java

+
+            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

src/cafes/MaquinaDeCafe.java

+                    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

src/cafes/MaquinaDeCafe.java

+                    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

src/cafes/MaquinaDeCafe.java

+
+                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

src/cafes/MaquinaDeCafe.java

+        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

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Desafio atualizado #4

Desafio atualizado #4

lilianjaf commented Sep 14, 2021

gcestaro Sep 14, 2021

gcestaro Sep 14, 2021

gcestaro Sep 14, 2021

gcestaro Sep 14, 2021

gcestaro Sep 14, 2021

gcestaro Sep 14, 2021

gcestaro Sep 14, 2021

gcestaro Sep 14, 2021

gcestaro Sep 14, 2021

gcestaro Sep 14, 2021

gcestaro left a comment

gcestaro Sep 16, 2021

gcestaro Sep 16, 2021

gcestaro Sep 16, 2021

gcestaro Sep 16, 2021

gcestaro Sep 16, 2021

gcestaro Sep 16, 2021

gcestaro Sep 16, 2021

gcestaro Sep 16, 2021

gcestaro Sep 16, 2021

gcestaro Sep 16, 2021


		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++) {

Desafio atualizado #4

Are you sure you want to change the base?

Desafio atualizado #4

Conversation

lilianjaf commented Sep 14, 2021

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

gcestaro left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment