The main thing to remember is that there really is no "right" or "wrong" although I doubt that getPrice() is an appropriate name for any of the responsibilities you normally assign to an Order class. It should probably be getTotalAmount() or simply getTotal(). getPrice() sounds like a responsibility you'd give to an Item class.
What you really should ask yourself is whether it make sense to assign a responsibility to a certain object. Use Craig Larman's GRASP
patterns discussed in his book "Applying Patterns and UML", the
Law of Demeter and other OO design principles to guide you on what
you should.
Junilu
[ June 02, 2002: Message edited by: Junilu Lacar ]