Achitectural question - what is the best object model for this?

I am deliberating about design issues on a project right now, and I'd be interested in hearing how someone else might tackle this.  Just as background, I am using EJB3, Hibernate, and JSF at the moment.  I am going to offer a somewhat abstract example that is actually pretty close to what I am trying to do.

I need to accommodate some object that represent different shapes.  They are defined as a series of points on a grid.  For example, there could be a square, a circle, or a polygon with an arbitrary number of sides.

In my mind, each shape could be pretty easily defined as a list of pairs of coordinates.  (Although the circle would need a single point and a radius, or something similar.)  Aside from that on exception, it's just an ordinal list of coordinate pairs.

Going forward - I am going to need to have a mechanism for each shape to be checked against a given point, to see if that point falls within any of the shapes on the grid.

My initial thoughts had been to have an abstract shape class to define the basic operations (get id, set id, getCollectionOfPoints, setCollectionOfPoints, etc.) and have each shape extend that class and provide it's own method to see if a set of provided coordinates fall within it's bounds.  A simple true or false response will be a sufficient answer, and the equation to check each shape type will never change.  The check method on the entity would then just be called from a session bean.

There is a part of me that doesn't like this idea though, because it feels like I am putting some degree of logic into an entity.  Is there a good way to accommodate these different types and still keep the actual work in a session bean?  (Please assume that entirely new shapes could be added to the system in the future, so rather than just adding a new entity with it's own method to check, the session bean would probably also need to be modified, which would be nice to avoid if possible.)

Does anyone have any thoughts or a better solution?
Who is Participating?

[Product update] Infrastructure Analysis Tool is now available with Business Accounts.Learn More

I wear a lot of hats...

"The solutions and answers provided on Experts Exchange have been extremely helpful to me over the last few years. I wear a lot of hats - Developer, Database Administrator, Help Desk, etc., so I know a lot of things but not a lot about one thing. Experts Exchange gives me answers from people who do know a lot about one thing, in a easy to use platform." -Todd S.

I am not very familiar with Java but the class organization that you are planning on looks good to me.

>Please assume that entirely new shapes could be added to the system in the future, so rather than just adding a
>new entity with it's own method to check, the session bean would probably also need to be modified, which would be
>nice to avoid if possible.
Again due to my unfamiliarity with Java I dont know what goes in a bean but if you are using run time polymorphism, there would be no code changes required apart from addition of the new shape class.
I suggest one single Shape class. BTW, are you looking to Polygon?

It looks like it has all your requirements. (contains(double x, double y) etc)
In order to specify the best practise for your project, we first need to answer a few questions.

a) Is the logic of the "contains" method common for multiple objects.
b) Is it likely to have the method "contains" change in the future?
c) Is there any possibility that you might want to change the logic of "contains" during runtime?

If you answer the above questions it will get clearer to you if it is really a bad practise to "couple" the contains logic to your entities.

If you decide to decouple those two, then you could choose the appropriate design pattern (maybe facade or decorator).


Experts Exchange Solution brought to you by

Your issues matter to us.

Facing a tech roadblock? Get the help and guidance you need from experienced professionals who care. Ask your question anytime, anywhere, with no hassle.

Start your 7-day free trial
PerplexityAuthor Commented:
Thanks...I just needed a little help to think about it critically.
It's more than this solution.Get answers and train to solve all your tech problems - anytime, anywhere.Try it for free Edge Out The Competitionfor your dream job with proven skills and certifications.Get started today Stand Outas the employee with proven skills.Start learning today for free Move Your Career Forwardwith certification training in the latest technologies.Start your trial today
Java EE

From novice to tech pro — start learning today.