<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>Hardy,</div><div>I think this feature is important for the alpha.</div><div>I did half of the job (see <a href="http://opensource.atlassian.com/projects/hibernate/browse/BVAL-99)">http://opensource.atlassian.com/projects/hibernate/browse/BVAL-99)</a></div><div><br></div><div>I think the rest should be fairly straightforward but it will probably take a good day of work. It's all fresh in my mind so don't hesitate to ask questions on how to finish.</div><div><br></div><div><div>On Jan 23, 2009, at 15:13, Emmanuel Bernard wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>Today, ConstraintValidator accepts Object and the implementation is responsible for handling the casting and raise an exception if the type is not supported.</div><div><br></div><div>The idea is to get type-safe validator and a discovery mechanism to associate a validator to a given runtime type being validated. The resolution is not 100% complete, you might end up with:</div><div> - no valid validator</div><div> - no way to decide between two or more validators</div><div>and get exceptions at runtime.</div><div><br></div><div>I would really like feedback on it as we could arrange some of these rules to not fail (at least in the latter case):</div><div> - do we want this type-safe + auto resolution algorithm strategy or should we stick with the untyped solution</div><div> - do we want to fail in case of ambiguity or rather choose the first compatible provider?</div><div><br></div><div>Personally I find the feature quite elegant and the metadata can be used by tools to warn a user if a constraint does not match the static type it is applied on.</div><div><br></div><div><b>API and declaration<br></b><div><br></div><div>A more typesafe proposal would be:</div><div><br></div><div>public interface ConstraintValidator<A extends Annotation, T> {</div><div> void initialize(A annotation);</div><div> boolean isValid(T object, ConstraintValidationContext context);</div><div>}</div><div><br></div><div>public class StringSizeValidator implements ConstraintValidator<Size, String> {</div><div> ...</div><div>}</div><div><br></div><div><div>public class CollectionSizeValidator implements ConstraintValidator<Size, Collection> {</div><div> ...</div><div>}</div><div><br></div><div><div>public class MapSizeValidator implements ConstraintValidator<Size, Map> {</div><div> ...</div><div>}</div></div><div><br></div><div><div>public class ArraySizeValidator implements ConstraintValidator<Size, Object[]> {</div><div> ...</div><div>}</div><br></div></div><div><br></div><div>@Constraint(validatedBy={</div><div> StringSizeValidator.class, CollectionSizeValidator.class, </div><div> MapSizeValidator.class, ArraySizeValidator.class} )</div><div>public @interface Size { ... }</div><div><br></div><div>We then need to decide at runtime, which validator needs to be executed.</div><div>Here is a proposed algorithm heavily based on the Java Language Specification (boy it's hard to read it).</div><div><br></div><div><b>Resolution algorithm</b></div><div>The type T of a ConstraintValidator must not make direct use of generics itself.</div><div><br></div><div><i>This is because the algorithm compare the actual type at runtime and such notion would be lost. So you can do ConstraintValidator<Size, Collection> but not ConstraintValidator<Size, Collection<String>>.</i></div><div><br></div><div>For a given runtime object t of type T about to be validated for a given constraint:</div><div>if t is null the first validator in the array of available validators is used.</div><div>otherwise</div><div><br></div><div>A validator accepting U is said to be compliant with the type T if T is a subtype of U according to the JLS chapter 4.10.</div><div><br></div><div>If no compliant validator is found for T, a UnexpectedTypeForConstraint exception is raised.</div><div><i>If a validator is taking the responsibility to dispatch types (ie ConstraintValidator<Size,Object>), it must use the same exception in the same circumstances.</i></div><div><br></div><div>If one compliant validator is found, it is used.</div><div><br></div><div>If more than one compliant validator is found, the one with the most specific parameter is used. The informal intuition is that one parameter is more specific than another if any assignment handled by the first parameter could be passed on to the other one without a compile-time type error.</div><div><br></div><div>Of a given type T, a compliant validator accepting U is considered maximally specific if no other T compliant validator accepting V is such as V is a strict subtype of U. String subtype is defined by JSL chapter 4.10.</div><div><br></div><div>If more than one maximally specific validator is found, a AmbiguousValidatorException is raised.</div><div><br></div><div>WDYT?</div></div></div>_______________________________________________<br>hibernate-dev mailing list<br><a href="mailto:hibernate-dev@lists.jboss.org">hibernate-dev@lists.jboss.org</a><br>https://lists.jboss.org/mailman/listinfo/hibernate-dev<br></blockquote></div><br></body></html>