]> git.mxchange.org Git - addressbook-ejb.git/commitdiff
Please cherry-pick:
authorRoland Häder <roland@mxchange.org>
Wed, 1 Apr 2020 18:20:51 +0000 (20:20 +0200)
committerRoland Häder <roland@mxchange.org>
Sat, 18 Apr 2020 13:14:35 +0000 (15:14 +0200)
- rewrote to avoid local Iterator<Foo> instance

Signed-off-by: Roland Häder <roland@mxchange.org>
src/java/org/mxchange/jcontacts/model/contact/AddressbookContactSessionBean.java

index 628bd9164b8d51834b2f826517bb332abf51f5da..ff63176143c489e2051037edff8c7d5ce9ef4a07 100644 (file)
@@ -17,7 +17,6 @@
 package org.mxchange.jcontacts.model.contact;
 
 import java.text.MessageFormat;
-import java.util.Iterator;
 import java.util.List;
 import java.util.Objects;
 import javax.ejb.Stateless;
@@ -98,21 +97,15 @@ public class AddressbookContactSessionBean extends BaseAddressbookEnterpriseBean
                        return null;
                }
 
-               // Get iterator
-               final Iterator<Contact> iterator = contacts.iterator();
-
                // Loop through all
-               while (iterator.hasNext()) {
-                       // Get contact
-                       final Contact next = iterator.next();
-
+               for (final Contact currentContact : contacts) {
                        // Is same contact?
-                       if ((Objects.equals(contact, next)) || (Contacts.isSameContact(contact, next))) {
+                       if ((Objects.equals(contact, currentContact)) || (Contacts.isSameContact(contact, currentContact))) {
                                // Debug message
-                               this.getLoggerBeanLocal().logDebug(MessageFormat.format("{0}.isContactFound: Found same contact: contactId={1}", this.getClass().getSimpleName(), next.getContactId())); //NOI18N
+                               this.getLoggerBeanLocal().logDebug(MessageFormat.format("{0}.isContactFound: Found same contact: contactId={1}", this.getClass().getSimpleName(), currentContact.getContactId())); //NOI18N
 
                                // Found it
-                               foundContact = next;
+                               foundContact = currentContact;
                                break;
                        }
                }