Re: Problem with sharing sessions/ multithreading

[Ralph Goers]

Kumar, Kiran wrote:
here's what I did. and changing all the code to use this.
==========================================
	private static final String LOCK = "Lock";
	
	public Document getDocument(Map objectModel,String attrname)
	throws ParserConfigurationException
	{
        Request request = ObjectModelHelper.getRequest(objectModel);
        Session session = request.getSession(true);
        Document doc = (Document)session.getAttribute(attrname);
        if   (doc == null)
        {
             // It is possible for more than one thread to get here at 
			  // the same time with doc == null
             synchronized(LOCK)
             {     
                   // So check again. Only the first caller should 
					//	actually create the
document.
                   doc = (Document)session.getAttribute(attrname);
                   if (doc == null)
                   {     
                       DocumentBuilderFactory dbf  = 
	
DocumentBuilderFactory.newInstance();                                 
                       doc = dbf.newDocumentBuilder().newDocument();
                       session.setAttribute(attrname, doc);
                    }           
              }
        }
        return doc;
	}
=====================================

but why do we need static method??  please guide me
 

As Alan Holub pointed out, the JVM does not guarantee that the 
constructor for the Document returned by the call to newDocument() may 
not have been run at the time the Document object refrerence is assigned 
to the session attribute.  Simply putting the call to newDocument() 
inside its own synchronized block should address that problem.

Ralph