Entrar Ainda não tem acesso?

Importante

Você está vendo a versão anterior da nova experiência da Alura que estamos preparando para você. Em breve, ela ganha uma identidade visual novinha totalmente pensada em potencializar seus estudos!

1
resposta

validando senha

Referente ao curso Praticando Java: encapsulamento, no capítulo Encapsulamento e atividade Validação de login

por Felipe Bernardo Esteves
| 24.8k xp | 7 posts
public class Principal {
    public static void main(String[] args) {

        Usuario usuario1=new Usuario("felipe2","coconut");
        usuario1.validarSenha();

import java.util.Scanner;

public class Usuario {
    private String login;
    private String senha;

    public Usuario(String login, String senha) {
        this.login = login;
        this.senha = senha;
    }

    public void validarSenha() {
        Scanner scanner = new Scanner(System.in);
        String senha="";
        String login="";
        for (int i = 3; i >= 1; i--) {
            System.out.println("digite o login:");
            login=scanner.nextLine();

            System.out.println("digite a senha:");
            senha=scanner.nextLine();
            int restantes=i-1;
          if(senha.equals(this.senha) && login.equals(this.login)){
              System.out.println("Login bem sucedido!");
              return;
          }
          if(restantes>0) {
              System.out.println("acesso negado!");
              System.out.printf("voce tem %d tentativas restantes\n ", restantes);
          }

        }
        System.out.println("voce excedeu o limite de tentativas");
    }
}
1 resposta
por Victor Costa Santos
| 3604.3k xp | 7549 posts
Alura Scuba Team

Oii Felipe, tudo bem?

Seu código está bem estruturado! A lógica do loop com contador de tentativas e a verificação de login e senha juntos ficou bem resolvida.

Tem um ponto de atenção que vale você revisar: dentro de validarSenha(), você declarou as variáveis locais String senha="" e String login="". O problema é que esses nomes criam uma "sombra" sobre os atributos da classe (this.senha e this.login). Dentro do if, você até usou this.senha e this.login corretamente, então o código funciona. Mas a variável local senha e o atributo this.senha coexistem com o mesmo nome no mesmo escopo, o que pode gerar confusão na leitura e manutenção.

Uma convenção comum é usar nomes distintos para as variáveis locais do método, como senhaDigitada e loginDigitado:

String senhaDigitada = "";
String loginDigitado = "";

E aí no if fica mais legível:

if (senhaDigitada.equals(this.senha) && loginDigitado.equals(this.login)) {

Além disso, repara que quando o usuário erra na última tentativa (i = 1, ou seja, restantes = 0), o código não imprime "acesso negado!" porque o if(restantes>0) bloqueia essa mensagem. O usuário vai direto para "voce excedeu o limite de tentativas" sem saber que errou nessa última tentativa. Faz sentido para o que você quer que o código faça? Vale pensar se essa mensagem de negação deveria aparecer sempre que errar, ou só quando ainda tiver tentativas.

Qualquer dúvida, é só trazer aqui!

Conte com a Alura para evoluir seus estudos. Em caso de dúvidas, fico à disposição.

Bons estudos!

Sucesso ✨

Imagem da comunidade

Tópicos relacionados

Mostrar mais tópicos Mostrar menos tópicos

Conteúdos Alura com o tema

Trilhas por carreira

Carreiras de IA

Carreiras de Dados

Carreiras de Cyber

Carreiras de DevOps & Cloud

Carreiras de UX & UI

Carreiras de Mobile & Front-End

Carreiras de Back-End

Carreiras de Negócios

Cursos universitários FIAP

Graduação | Pós-graduação | MBA