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