[BEANUTILS-326] Iterating by key/value pairs in Map in BeanUtilsBean.populate(...) method

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.7.0
• Fix Version/s: 1.8.0
• Component/s: Bean / Property Utils
• Assignee: Niall Pemberton
• Reporter: Vladimir Orlov
• Created: 14/Aug/08 10:20
• Updated: 26/Jul/09 07:04
• Resolved: 19/Aug/08 18:20

Description

I think that iterating by key/value pairs in Map should be done in more appropriate way than it is done currently: it takes set of keys, iterates by it and on each iteration cycle searchches for the value via Map.get(<key>) API. Consequently it does the search by key in map on every step and it should make the processign slower.

An alternative to such approach is to use Map.entrySet() API which returns the Set of key/value pairs (as instances of Map.Entry class). Using it eliminates the need to do a search by key (Map.get(...) API) in the map on each step of iteration and should work faster.