Exclude two names fields in value object comparison, improved javadoc for support base classes.

dddsample/tracking/core/src/main/java/se/citerus/dddsample/tracking/core/domain/shared/experimental/EntitySupport.java

@@ -1,7 +1,12 @@
 package se.citerus.dddsample.tracking.core.domain.shared.experimental;
 
 /**
- * Base class for entities.
+ * Supporting base class for entities.
+ *
+ * While the Entity interface makes the pattern properties explicit,
+ * this class is less general and is suited for this particular application.
+ * </p>
+ * For example, the private id field is meant for autogenerated, surrogate primary keys.
  */
 public abstract class EntitySupport<T extends Entity, ID> implements Entity<T, ID> {
 

dddsample/tracking/core/src/main/java/se/citerus/dddsample/tracking/core/domain/shared/experimental/ValueObjectSupport.java

@@ -4,15 +4,24 @@ import org.apache.commons.lang.builder.EqualsBuilder;
 import org.apache.commons.lang.builder.HashCodeBuilder;
 
 /**
- * Base class for value objects.
+ * Supporting base class for value objects.
+ *
+ * While the ValueObject interface makes the pattern properties explicit,
+ * this class is less general and is suited for this particular application.
+ * </p>
+ * For example, the private id field is meant for autogenerated, surrogate primary keys.
+ * Also, you may want more flexibility in selecting significant fields for comparision,
+ * or you may need to be able to calculate equals millions of times every second,
+ * in which case reflection might not be fast enough.
  *
  * @param <T>
  */
 public abstract class ValueObjectSupport<T extends ValueObject> implements ValueObject<T> {
 
-  private transient int cachedHashCode = 0;
   @SuppressWarnings("UnusedDeclaration")
   private final Long id = null;
+  private transient int cachedHashCode = 0;
+  private final static String[] EXCLUDED_FIELDS = {"id", "cachedHashCode"};
 
   /**
    * @param other The other value object.
@@ -20,7 +29,7 @@ public abstract class ValueObjectSupport<T extends ValueObject> implements Value
    */
   @Override
   public final boolean sameValueAs(final T other) {
-    return other != null && EqualsBuilder.reflectionEquals(this, other, false);
+    return other != null && EqualsBuilder.reflectionEquals(this, other, EXCLUDED_FIELDS);
   }
 
   /**