Java: for Schleife bricht vor allen Iteration wegen Methodenaufruf

Question

Ich stehe vor einem bizarren Problem. Im folgenden Codefragment bricht die zweite for-Schleife nach einer Iteration ab, wenn ich userRoles.removeRole (strRole) aufruft. Es gibt 2 Elemente in der Liste. Die erste for-Schleife wird zweimal ausgeführt. Aber der zweite wird nur einmal ausgeführt. Der genannte Methodenaufruf gibt Boolean zurück. Könnte mir bitte jemand helfen, was in meinem Code falsch ist?

 if(userRoles != null) 
     { 
      List<String> roles = userRoles.getRoles(); 
      String strUserName = userRoles.getUserName(); 

      for(String strRole: roles) 
      { 
       System.out.println("role : " + strRole); 
      } 
      //for(String strRole: roles) 
      for(int count = 0; count < roles.size() ; count++) 
      { 
       String strRole = roles.get(count); 
       System.out.println("role before check: " + strRole); 
       if(ur.hasRoleForUser(strRole, strUserName)) 
       { 
        System.out.println("role after check: " + strRole); 
        userRoles.removeRole(strRole); 
       } 
      } 

      System.out.println("role length: " + userRoles.getRoles().size()); 
      if(userRoles.getRoles().size() > 0) 
      { 
       ur.addUserRoles(userRoles); 
      } 
      blnSuccess = true; 
     } 

Answer 1

Die Schleife bricht, weil Sie ein Element der Liste entfernen, die Sie durchqueren (Nach dem Entfernen ist die Größe der Liste 1, zählen so < roles.size() wird false)

In der Schleife sollten Sie zuerst die Elemente erfassen, die Sie nach der Schleife entfernen möchten.

Answer 2

Ihre for Schleife wertet vor jeder Iteration count < roles.size() aus.

Aufgrund der Tatsache, dass in der ersten Iteration Sie userRoles.removeRole(strRole) aufrufen, in der nächsten Zeit wird die Schleife roles.size() bewerten die zurückgegebene Wert 1 wird seit dem 1. nicht größer als 1 (der Wert von count an diesem Punkt) ist Die Schleife stoppt weitere Iterationen.

Da @kamehl23 angeboten wird, sollten Sie Listen-Iteratoren verwenden. Eine weitere Empfehlung lautet nicht, um die Elemente zu ändern, die Sie gerade durchlaufen. Wir sparen Ihnen viel Zeit.

Viel Glück.

Answer 3

Vielen Dank für Ihre Antworten. Ich habe es nicht richtig bemerkt ... Ich dachte, die Rollen listet als neues Objekt auf, das die Liste des Objekts, von dem jedes Element entfernt wird, nicht verweist. Hier ist mein aktualisierter Code.

 if(userRoles != null) 
     { 
      List<String> roles = userRoles.getRoles(); 
      String strUserName = userRoles.getUserName(); 

      for(int count = roles.size()-1; count >= 0 ; count--) 
      { 
       String strRole = roles.get(count); 
       if(ur.hasRoleForUser(strRole, strUserName)) 
       { 
        userRoles.removeRole(strRole); 
       } 
      } 

      if(userRoles.getRoles().size() > 0) 
      { 
       ur.addUserRoles(userRoles); 
      } 
      blnSuccess = true; 
     }