<div dir="ltr">I had created two JIRAs for that :<div><br></div><div><a href="https://issues.jboss.org/browse/ROASTER-4">https://issues.jboss.org/browse/ROASTER-4</a><br></div><div><a href="https://issues.jboss.org/browse/ROASTER-5">https://issues.jboss.org/browse/ROASTER-5</a> (looks like this one is done)<br>
</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-04-21 21:22 GMT+02:00 Lincoln Baxter, III <span dir="ltr"><<a href="mailto:lincolnbaxter@gmail.com" target="_blank">lincolnbaxter@gmail.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hey Antonio,<div><br></div><div>I'm relatively certain that we don't support Generic types on method parameters yet, but we do support them on Types and Methods themselves via the GenericCapableSource interface.</div>
<div><br></div><div>Looks like we need to add this to Parameters. Could you add a feature request for this? Thanks!</div><div><br></div><div>~Lincoln </div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote"><div><div class="h5">On Sun, Mar 30, 2014 at 1:05 PM, Antonio Goncalves <span dir="ltr"><<a href="mailto:antonio.mailing@gmail.com" target="_blank">antonio.mailing@gmail.com</a>></span> wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr">Hi,<div><br></div><div>Still playing with Roaster so I could fix <a href="https://issues.jboss.org/browse/FORGE-1618" target="_blank">https://issues.jboss.org/browse/FORGE-1618</a> (Command constraint-new-validator to create a new validator implementation). I don't know how to do two things (nicely) :</div>
<div><br></div><div><b># Parameterize interface</b></div><div><br></div><div>For a constraint, I need to implement a parametrize interface like this :</div><div><br></div><div>public class MaxValidatorForString implements <font color="#ff0000">ConstraintValidator<Max, Number></font> {<br>
</div><div><br></div><div><br></div><div><pre style="margin-top:0px;margin-bottom:0px;padding:0px;max-height:30em;overflow:auto;white-space:pre-wrap;word-wrap:normal;color:rgb(51,51,51);font-size:12px;line-height:16px;background-color:rgb(245,245,245)">
<span style="background-color:rgb(255,255,255);font-family:arial;font-size:small;line-height:normal;color:rgb(34,34,34)">The only way I could find is passing a String :</span></pre></div><div>
<p><span>final </span>JavaClassSource javaClass = Roaster.<i>create</i>(JavaClassSource.<span>class</span>);<br>
javaClass.setName(<span>"MaxValidatorForString"</span>);<br>
javaClass.addImport(ConstraintValidator.<span>class</span>);<br>
javaClass.addInterface(<span>"<font color="#ff0000">ConstraintValidator<Max, Number></font>"</span>);<br>
</p></div><div><br></div><div>It's a shame because with addInterface(Class), it does the import automatically and things are more typed. But I cannot add types :</div><div>
<p><span>final </span>JavaClassSource javaClass = Roaster.<i>create</i>(JavaClassSource.<span>class</span>);<br>
javaClass.setName(<span>"MaxValidatorForString"</span>);<br>
javaClass.addInterface(<font color="#ff0000">ConstraintValidator.<span>class</span></font>);<br>
</p></div><div><br></div><div>It would be good to have something like :</div><div><div class="gmail_extra"><div><p>javaClass.addInterface(ConstraintValidator.<span>class</span>).<font color="#ff0000">addType(Max.class).addType(Number.class)</font>;<br>
</p></div><div><br></div><div><b># Typed parameters</b></div><div><br></div><div>My isValid method takes two parameters. And the only way to add two parameters seems to be by a String</div><div><br></div><div>
<p>javaClass.addMethod().setPublic().setName(<span>"isValid"</span>).setReturnType(<span>"boolean"</span>).setParameters(<span><font color="#ff0000">"Number value, ConstraintValidatorContext context"</font></span>).setBody(<span>"return false;"</span>).addAnnotation(<span>Override</span>.<span>class</span>);<br>
</p></div><div><br></div><div>Again, it would be nice to have typed parameters so the import is implicit, and things are a bit more typed :</div><div><br></div><div><div><p>javaClass.addMethod().setPublic().setName(<span>"isValid"</span>).setReturnType(<span>"boolean"</span>).<font color="#ff0000">addParameter(<span>Number.class, "value").addParameter(ConstraintValidatorContext.class, "context"</span>)</font>.setBody(<span>"return false;"</span>).addAnnotation(<span>Override</span>.<span>class</span>);<br>
</p></div></div><div><br></div><div><br></div><div>So I'm just wondering if I'm missing something or this is not implemented yet on Roaster</div><span><font color="#888888"><div><br></div>-- <br>Antonio Goncalves <br>
Software architect and Java Champion<br>
<br><a href="http://www.antoniogoncalves.org/" target="_blank">Web site</a> | <a href="http://twitter.com/agoncal" target="_blank">Twitter</a> | <a href="http://www.linkedin.com/in/agoncal" target="_blank">LinkedIn</a> | <a href="http://www.parisjug.org/" target="_blank">Paris JUG</a> | <a href="http://www.devoxx.fr/" target="_blank">Devoxx France</a>
</font></span></div></div></div>
<br></div></div>_______________________________________________<br>
forge-dev mailing list<br>
<a href="mailto:forge-dev@lists.jboss.org" target="_blank">forge-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/forge-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/forge-dev</a><span class="HOEnZb"><font color="#888888"><br></font></span></blockquote></div><span class="HOEnZb"><font color="#888888"><br>
<br clear="all"><div><br></div>-- <br>Lincoln Baxter, III<br><a href="http://ocpsoft.org" target="_blank">http://ocpsoft.org</a><br>
"Simpler is better."
</font></span></div>
<br>_______________________________________________<br>
forge-dev mailing list<br>
<a href="mailto:forge-dev@lists.jboss.org">forge-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/forge-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/forge-dev</a><br></blockquote></div><br><br clear="all"><div><br></div>-- <br>Antonio Goncalves <br>Software architect and Java Champion<br>
<br><a href="http://www.antoniogoncalves.org/" target="_blank">Web site</a> | <a href="http://twitter.com/agoncal" target="_blank">Twitter</a> | <a href="http://www.linkedin.com/in/agoncal" target="_blank">LinkedIn</a> | <a href="http://www.parisjug.org/" target="_blank">Paris JUG</a> | <a href="http://www.devoxx.fr/" target="_blank">Devoxx France</a>
</div>