• Loop over entryset of a hashmap, not the entities

• Concatenate strings using StringBuilder

• Use Findbugs - it will help expose patterns to be avoided.