4

Estou tentando fazer um loop for, que em determinada ocasião retorna true, e sai do laço.

Exemplo:

public boolean autentica (Usuario usuario) {
    for(i=0; i<listaUsuarios.size ; i++){
        if(usuario.getSenha().equals(listaUsuarios.get(i).getSenha()){
            return true;
            break;
        }
    }
}

Ou seja, quando acontecer determinada coisa, retorne true e saia do laço, porque se não vai continuar testando a senha nos outros usuários.

Resumindo, quero retornar true e sair do laço. Como faço? Pois assim dá erro.

1
  • Mas assim que você retorna, ele sai da função e do laço.
    – mutlei
    28/01/2015 às 12:02

2 Respostas 2

6

Se você quer dar break e o return dentro do if, não faz sentido. Não dá para colocar nada depois de um return. Se o problema é apenas o erro que está ocorrendo, basta tirar o break porque o único problema é ter um comando depois da saída do método. Quando você sai do método, certamente você saiu do laço.

Mas não basta retirar o break. Você ainda terá um caminho de execução que não retornará nada. Você precisa retornar um booleano sempre. O código precisa garantir que em todas as situações ele retorne alguma coisa do mesmo tipo.

Este método está indicando se achou algum usuário que autenticou, acho que você quer que informe se não autenticou também se não achar nenhum usuário com senha compatível, então basta colocar um return false no final do método se passou por todo o laço sem cair no if. Assim:

class Program {
    static String[] listaUsuarios = new String[] { "1", "12", "123" };
    public static void main (String[] args) throws java.lang.Exception {
        System.out.println(autentica("123"));
        System.out.println(autentica("456"));
    }
    public static boolean autentica (String usuario) {
        for(int i = 0; i < listaUsuarios.length; i++) {
            if(usuario.equals(listaUsuarios[i])) {
                return true;
            }
        }
        //opcionalmente faz alguma coisa aqui.
        return false;
    }
}

Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.

Haviam outros erros que impediram o funcionamento. Claro que tive que fazer algumas adaptações para simplificar. O seu ficaria assim (acho, obviamente não testei):

public static boolean autentica (Usuario usuario) {
    for(int i = 0; i < listaUsuarios.length; i++) {
        if(usuario.getSenha().equals(listaUsuarios.get(i).getSenha()) {
            //opcionalmente faz alguma coisa aqui.
            return true;
        }
    }
    //opcionalmente faz alguma coisa aqui.
    return false;
}
12
  • Obrigado, dessa forma que queria o algoritmo, valeu.
    – War Lock
    28/01/2015 às 12:19
  • porque vc editou? dessa forma não sai do laço, a forma correta era a que voce postou antes
    – War Lock
    28/01/2015 às 12:24
  • Você tem certeza? Vou fazer um teste real então. Se for o caso eu volto ao que estava. Aguarde.
    – Maniero
    28/01/2015 às 12:25
  • voce nao entendeu o contexto, essa forma q vc postou era como eu tava fazendo antes, porem ficava errado porque se o 1 usuario da lista tiver a senha igual a digitada, vai retornar true, porem o laço continua, e logo em seguida vai retornar false, ou seja precisa do break, por isso perguntei como usar ele, e voce solucionou daquela forma.
    – War Lock
    28/01/2015 às 12:28
  • Não você não entendeu. Se ele retorna true, o laço termina. Leia o primeiro parágrafo da resposta. Você fez o teste desta forma? Não tem como ele retornar true e depois retornar false. Ou ele faz um OU faz outro.
    – Maniero
    28/01/2015 às 12:33
1

O return em qualquer lugar que esteja finaliza o método (e por consequência o loop). O erro que deves estar encontrando é que o break é código morto, nunca alcançado, pois o método sempre retornará antes de atingi-lo. Retire o break que deverá funcionar.

4
  • 1
    Na verdade só retirar o breakcontinuará dando erro. Veja minha resposta.
    – Maniero
    28/01/2015 às 12:20
  • se retirar o break vai rodar, porem eu quero usar o break
    – War Lock
    28/01/2015 às 12:31
  • @WarLock não é possível colocar código depois do return. Se em algum momento o método atingir o return, ele retorna e a execução acaba. Coloque o return true; dentro do if e o return false; no final do método. 28/01/2015 às 12:33
  • Se tu entendes inglês te sugiro esta leitura: docs.oracle.com/javase/tutorial/java/nutsandbolts/branch.html 28/01/2015 às 12:44

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .