Hi.
anonymous wrote :
| But in some weird case where my validation require some kind of twisted
logic using some properties from the Person entity and all the other entities
already persisted, is there a clean way to add this logic in the model? Or this
is conceptually wrong?
|
For me, putting validation logic in your model is nasty. And it's conceptually
wrong. If you want to validate something related to one or several entities,
and you don't have another way to do it (according your requirements) then you
should perfom such validation in your controller (SLSB or SFSB).
The example I gave you it was recently adopted by me for validate duplicates
for an entity (of course when the properties are not the @Ids). Time ago I was
doing this validation in a session bean, something as follows:
| public void persis(Person entity){
|
| try {
| em.createQuery("select p from P where p.lastName=:lastName and
| p.firstName = :firstName).setParameter("lastName",
| entity.lastName).setParameter("firstName",
| entity.firstName).getSingleResult();
|
| //user exists, then throw some AlreadyExistException.
| } catch (NoResultException e) {
| em.persist(entity);
| }
|
| }
|
But the above code involves two SQL's:
| select from person ...where lastname....
| insert into person...... (if the above sql does not return value)
|
But delegating this validation to DB layer (@UniqueConstraint) the only SQL
will always be:
| insert into person....
|
So, in order to reduce the DB round trip I decided to use DB approach for this
case. Of course, if you are free to make changes in the database schema. I
think this is total portable, because I don't know some database server that
does not allow to use "UNIQUE" constraints.
anonymous wrote : The validation you gave me works great. The only complain is
that a get some error logs about:
Yes, this is annoying, I ask the same time ago, see below link (I din't try it
yet myself):
http://www.jboss.com/index.html?module=bb&op=viewtopic&t=98560
anonymous wrote :
| Hey, i'm your neighbor! From Paraguay :D
|
Nice to hear it.
Cheers.
View the original post :
http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4049075#4049075
Reply to the post :
http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4049075
_______________________________________________
jboss-user mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/jboss-user
