Changeset 794

Timestamp:

Aug 10, 2010, 4:17:13 PM (10 years ago)

Author:

keesvb

Message:

thread safe implementation of Identity increment

File:

• trunk/grails-app/domain/dbnp/studycapturing/Identity.groovy (modified) (1 diff)

trunk/grails-app/domain/dbnp/studycapturing/Identity.groovy

@@ -59 +59 @@
                 if (!identifier) {
                         // no, increment the iterator
-                        identifier = iterator++
-
-                        // has the iterator become too large?
-                        if (iterator >= maximumIdentity) {
-                                // yes, reset it back to 0 as this iterator
-                                // works for the complete application, not
-                                // only the user session. We don't want it
-                                // to go too high :)
-                                iterator = 0
-                        }
+                        incrementIterator()
                 }
 
                 println ".instantiating [" + super.getClass() + "] ("+ identifier + ")"
+        }
+
+        // This method is synchronized to assure a thread-safe increment of the iterator variable
+        synchronized private void incrementIterator() {
+                // Increment the iterator variable
+                identifier = iterator++
+
+                // has the iterator become too large?
+                if (iterator >= maximumIdentity) {
+                        // yes, reset it back to 0 as this iterator
+                        // works for the complete application, not
+                        // only the user session. We don't want it
+                        // to go too high :)
+                        iterator = 0
+                }
+
         }