inativo Correção de Codigo JAVA

[thorum]

Boa tarde a todos, tudo bem?,  estudando JAVA, irei publicar o codigo aqui e caso alguém possa  corrigir  e ver o que está errado  e etc

 

Citar

public static void main(String[] args) {
        
       
        //variavel de controle quando digitado S continua 
        char repetir ='s';
        //Leitor para usuário inserir dados
       Scanner leitor = new Scanner(System.in);
        //Iniciador do objeto "" String, 0.0f valor produto, 0 quantidade
       produtos produto = new produtos("",0.0f,0);
       //repetidor  que verifica os dois tipos de (S)
       while(repetir == 'S' || repetir == 's'){
        
        System.out.println(" == CADASTRO PRODUTO ==\n ");
        System.out.println("Insira o nome do Produto");
        produto.setNomeProduto(leitor.nextLine());
        System.out.println("Produto:"+produto.getNomeProduto()+"\n");
        System.out.println("Insira o valor do Produto");
        produto.setValorProduto(leitor.nextFloat());
        System.out.println("Carrinho: ");
        produto.setQuantidade(leitor.nextInt());
        System.out.println("Compra: "+produto.getQuantidade());
        
        
        System.out.println("Valor total: "+produto.calculoCompra());
        
        System.out.println("DESEJA CONTINUAR  S- SIM // N - NAO");
         repetir = leitor.next().charAt(0);
        
      
        
    }

CLASS

 

Citar

package lojaDoReal;

public class produtos {
    public String nomeProduto;
    public Float valorProduto;
    public int quantidade;

    public produtos(String nomeProduto, Float valorProduto,int quantidade) {
        this.nomeProduto = nomeProduto;
        this.valorProduto = valorProduto;
        this.quantidade = quantidade;
        
    }
    
    
    
    public float calculoCompra(){
        
        return  this.valorProduto * this.quantidade;
    }

    public int getQuantidade() {
        return quantidade;
    }

    public void setQuantidade(int quantidade) {
        this.quantidade = quantidade;
    }
    
    
    
    public String getNomeProduto() {
        return nomeProduto;
    }

    public void setNomeProduto(String nomeProduto) {
        this.nomeProduto = nomeProduto;
    }

    public Float getValorProduto() {
        return valorProduto;
    }

    public void setValorProduto(Float valorProduto) {
        this.valorProduto = valorProduto;
    }
    
    
    
}
 

 

RESULTADO DO CONSOLE

 

== CADASTRO PRODUTO ==
 
Insira o nome do Produto
COCA-COLA
Produto:COCA-COLA

Insira o valor do Produto
11,50
Carrinho: 
2
Compra: 2
Valor total: 23.0
DESEJA CONTINUAR  S- SIM // N - NAO
 

 

OBS: EU  NÃO PUBLIQUEI IMAGEM POIS A POLITICA DO TAMANHO DE IMAGEM É MEIO TRISTE 

 

Agradeço pela sua atenção  :)

 

 

 

 

 

Editado Junho 14, 2022 por thorum
Adicionar

[rubenlousada11]

Oi.

 

Tente indentar o codigo melhor. 

Na classe as variaveis deviam ser do tipo privadas.

 

No  main voce nao precisa de criar o objecto no começo, pode pedir ao utilizador os valores e apos obter todos os valores, criar o objeto. Assim não precisaria dos setters da class main e nao estaria sempre a utilizar o mesmo objeto.

 

Voce deveria tambem verificar os inputs do utilizador (ver se sao corretos) e usar try catches para prevenir o programa de crashar.

[AffonsoGames]

TÓPICO INATIVO

O seu tópico foi dado como inativo por ultrapassar 7 dias sem novas respostas e/ou pela expiração do seu conteúdo. Entre em contato com a Equipe de Moderação para obter suporte sobre esta ação.

Atenciosamente,
Equipe de Moderação, Gamer's Board.