Bugfixing please analyse let me know

Question

There is code in the xxx-class
It has the following code which takes in a List, and converts it to a Map. The List can contain multiple entries with the same eventId, so this code would just say "the last one in wins." This makes it dependent on the order items are contained in argEventList (which is a dangerous practice.) The progression is the List is converted to a Map (which eliminates duplicates), then converted to a List, passed to the jsp and converted to a Vector. The code should be fixed to only use a List. In the setter, it could use a temporary Map to eliminate duplicates (logic could be put in place to add the item if the eventId doesn't already exist in the Map, and if the eventId key does exist, only overwrite it if the User Type is 1 (optional). )
public void setEventList(List argEventList)
     this.personalNotificationEvents = new HashMap();
     if (argEventList == null)
          return;
     ListIterator iter = argEventList.listIterator();
     PersonalNotificationEvent pne = null;
     while (iter.hasNext())
          pne = (PersonalNotificationEvent) iter.next();
          if (pne != null)
               this.personalNotificationEvents.put(pne.getEventId(), pne);
}

Answer 1

You would need to contact your local store for product availability. You can also check to see if the item is available for in-store pickup on the item page linked in your original post.
Alex M. | Best Buy Mobile - State College, PA (Store 369)
Any opinions expressed in this post are those of
the author and do not represent Best Buy Co., Inc.