Changeset 8844

Timestamp:

2004-12-17T14:51:50+13:00 (19 years ago)

Author:

mdewsnip

Message:

Tidied up getElementAt() a little and added debugging statement.

File:

• trunk/gli/src/org/greenstone/gatherer/cdm/DOMProxyListModel.java (modified) (1 diff)

trunk/gli/src/org/greenstone/gatherer/cdm/DOMProxyListModel.java

@@ -192 +192 @@
     }
 
-    public synchronized Object getElementAt(int index) {
+    public synchronized Object getElementAt(int index)
+    {
     Object object = cache.get(new Integer(index));
-    if(object == null) {
-        // Retrieve the required element
-        Element element = (Element) children.item(index);
-        // Now wrap it in the object of the users choice
-        object = class_type.create(element);
-        cache.put(new Integer(index), object);
-    }
+    if (object != null) {
+        return object;
+    }
+
+    // Retrieve the required element
+    Element element = (Element) children.item(index);
+    DebugStream.println("Element at index " + index + " not in cache: " + element);
+
+    // Now wrap it in the object of the users choice
+    object = class_type.create(element);
+    cache.put(new Integer(index), object);
     return object;
     }