svn commit: r14524 - trunk/src/argouml-app/src/org/argouml/cognitive/ItemUID.java

Author: bobtarling
Date: 2008-04-29 16:56:03-0700
New Revision: 14524

Modified:
   trunk/src/argouml-app/src/org/argouml/cognitive/ItemUID.java

Log:
Highlight design flaw

Modified: trunk/src/argouml-app/src/org/argouml/cognitive/ItemUID.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/cognitive/ItemUID.java?view=diff&rev=14524&p1=trunk/src/argouml-app/src/org/argouml/cognitive/ItemUID.java&p2=trunk/src/argouml-app/src/org/argouml/cognitive/ItemUID.java&r1=14523&r2=14524
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/cognitive/ItemUID.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/cognitive/ItemUID.java	2008-04-29 16:56:03-0700
@@ -68,6 +68,9 @@
  * could be used to remove the dependancy to the model component here, which
  * I find a bit unaesthetic. So far, not enough to write it (though it is not
  * much work).
+ * 
+ * TODO: This class seems to expect arguments to be guaranteed to implement
+ * some specific method. So why doesn't it expect an interface to ensure that?
  *
  * @author Michael Stockman
  */
@@ -121,6 +124,7 @@
      * Generates a new unique ID and returns it as a String. The contents
      * of the String is supposed to be unique with respect to all Strings
      * generated by other instances of this class.
+     * TODO: We should consider using java.util.UUID for this instead.
      *
      * @return	A String with unique content.
      */