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;
> }

Reply via email to