L'arrêt d'une itération d'une liste de tableaux en Java

Je suis itération d'une ArrayList de clients nommée clientList qui contient des clients de la classe Client (user,pass)

ArrayList<Client> clientList= new ArrayList<Client>();

Voici l'itération. Je veux arrêter l'itération s'il fonde un utilisateur (user) et si le mot de passe (pass) correspond à:

            for (Client c : clientList) {
                userA = c.getUser();
                if (userA.equals(user)) {
                    passA = c.getPassword();
                    if (passA.equals(pass)) {
                        loginOK = true;
                        found= true;
                    }

J'ai essayé le suivant tandis que (trouve == false) mais c'est s'accrochent si elle ne marche pas à trouver un utilisateur sur la liste de tableaux:

        while (found == false) { /
            for (Client c : clientList) {
                userA = c.getUser();
                if (userA.equals(user)) {
                    passA = c.getPassword();
                    if (passA.equals(pass)) {
                        loginOK = true;
                        found= true;
                    }
                }
            }
        }
  • utilisation de mots clés: break;
  • Vous devriez également envisager d'utiliser un HashMap au lieu d'un List. La clé de la user et la valeur de la pass. Ensuite, votre for-boucle s'évanouit; vous obtenez plutôt if (hashMap.get(user).equals(pass)). Mais vous devriez sérieusement modifier votre code pour que... le Client classe ne serait pas nécessaire.
InformationsquelleAutor Peter | 2011-06-09
  1. 1

    Allais-je écrire de cette façon:

    while (!found) { 
    for (Cliente c : clientList) {
        userA = c.getUser();
        if (userA.equals(user)) {
            passA = c.getPassword();
            if (passA.equals(pass)) {
                loginOK = true;
                found= true;
                break;
            }
        }
    }
}

    Ma conjecture est que vous n'avez pas remplacer equals et hashCode dans votre Cliente classe ou il n'est pas correct.

    • avec break, found n'est pas plus nécessaire
    • Je n'aurais pas accepté ma réponse. Voyant que les autres ont les upvotes, je pense que c'est à la justice d'accepter les uns les autres.
    InformationsquelleAutor duffymo
  2. 8

    Vous devriez break sortir de la boucle quand la valeur est trouvée.

    for (something) {
   if (anotherThingIsFound) {
      break;
   }
}
//Execution will continue here when you break...

    Noter qu'il est également possible de sortir de boucles imbriquées, avec l'aide d'étiquettes. E. g.

    outer:
while (someCondition) {
   for (criteria) {
      if (somethingHappened) {
         break outer; 
      }
      if (anotherThingHashHappened) {
         break;
      }
   }
   //Execution will continue here if we execute the "normal" break.
}
//Execution will continue here when we execute "break outer;"

    continue ne travaillons également avec des étiquettes.

    InformationsquelleAutor Kaj
  3. 5

    Pourquoi ne pas simplement break?

    for (Client c : clientList) {
    userA = c.getUser();
    if (userA.equals(user)) {
        passA = c.getPassword();
        if (passA.equals(pass)) {
            loginOK = true;
            found = true;
            break;
        }
    }
}

    (Je suppose que vous devez dire à la différence entre arriver à la fin et trouver quelqu'un et arriver à la fin et pas de trouver quelqu'un. Vous avez probablement besoin d'une seule variable si, au lieu de deux...)

    Avec votre while boucle tentative, vous allez effectuer une itération sur l'ensemble de la liste pour toujours si l'utilisateur n'est pas trouvé, et même si l'utilisateur est trouvé, il sera en boucle sur l'ensemble de la liste une fois - parce que votre boucle for est à l'intérieur de de la boucle while. La condition du while est seulement vérifié une fois par itération de la boucle while.

    InformationsquelleAutor Jon Skeet
  4. 3

    Vous devez utiliser le break mot-clé:

            for (Client c : clientList) {
            userA = c.getUser();
            if (userA.equals(user)) {
                passA = c.getPassword();
                if (passA.equals(pass)) {
                    loginOK = true;
                    found = true;
                    break;
                }

    Voir Java mot-clé break de la documentation.

    InformationsquelleAutor MGwynne
  5. 2

    si vous souhaitez utiliser votre found d'un attribut, d'introduire un break et de supprimer la boucle while:

    for (Cliente c : clientList) {
    userA = c.getUser();
    if (userA.equals(user)) {
        passA = c.getPassword();
        if (passA.equals(pass)) {
        loginOK = true;
        found= true;
        }
    }

    if (found)
        break;
}

    De cette façon, vous n'avez pas besoin d'utiliser le while boucle:

    • +1 - c'est la bonne réponse.
    InformationsquelleAutor Buhake Sindi
  6. 1

    Sauté null de la sécurité et de la classe de fonte de sécurité pour des raisons de simplicité

    public class Cliente {
    public boolean equals(Object other){
         Cliente cOther = (Cliente) other;
         return this.getUser().equals(other.getUser()) &&
                this.getPassword().equals(other.getPassword())
    }
    public int hashCode(){
        return this.getUser().hashCode()*this.getPassword().hashCode();
    }
}

...
Cliente c = new Cliente();
c.setPassword(pass);
c.setUser(user);
boolean found = clientList.contains(c);
    InformationsquelleAutor Marcin Michalski
  7. 1

    À l'aide de break fonctionne très bien, mais si vous voulais le faire avec une boucle while au lieu de cela, alors vous pourriez le faire comme ceci:

        Iterator<Client> it = clientList.iterator();
    while (it.hasNext() && !found) {
        Client c = it.next();
        userA = c.getUser();
        if (userA.equals(user)) {
            passA = c.getPassword();
            if (passA.equals(pass)) {
                loginOK = true;
                found = true;
            }
        }
    }
    InformationsquelleAutor Anon