<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Jan 11, 2013, at 10:01 AM, Sebastien Blanc <<a href="mailto:scm.blanc@gmail.com">scm.blanc@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr">Hi All,<div style="">In the scope of <a href="https://issues.jboss.org/browse/AEROGEAR-803">https://issues.jboss.org/browse/AEROGEAR-803</a> , we have a method to retrieve an user (based on its id). If the user don't exist it returns null.</div>
<div style=""><br></div></div></blockquote><blockquote type="cite"><div dir="ltr"><div style="">There were some suggestions to change that into :</div><div style="">- throwing a runtime exception (seems a bit brutal)</div></div></blockquote><div><br></div><div>This does seem brutal, unless there is some security issues with not doing this</div><div><br></div><div>i think it should be the client dev's responsibility to check if the return value exists.</div><div><br></div><blockquote type="cite"><div dir="ltr"><div style="">- or returning an empty user (can be confusing)</div><div style=""><br></div></div></blockquote><blockquote type="cite"><div dir="ltr"><div style="">What is your opinion about this ? </div><div style=""><br></div><div style="">Seb</div><div style=""><br></div><div style="">ps: the method <a href="https://gist.github.com/4510879">https://gist.github.com/4510879</a></div>
<div style=""><br></div><div style=""><br></div></div>
_______________________________________________<br>aerogear-dev mailing list<br><a href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a><br>https://lists.jboss.org/mailman/listinfo/aerogear-dev<br></blockquote></div><br></body></html>