I would like to point out that this is a REALLY bad idiom. You should use a static initializer.
private static Map usersMap = new HashMap(); Also, since this is a static field in a very small class, there is very little reason to use a getter. No need for 7 lines of code when one will do. On 12/31/07 1:17 AM, "helena21" <[EMAIL PROTECTED]> wrote: > private static Map usersMap=null; > > public static Map getUsersMap(){ > if(usersMap==null){ > usersMap=new HashMap(); > } > return usersMap; > }