From: Matthew G. <mat...@gm...> - 2007-07-12 16:57:23
|
On Wednesday 11 July 2007, Fabien Ch=C3=A9reau wrote: > Hi Matthew, > > I think you should keep things simple in the AstroImage class. I.e. you > shouldn't include in it the notion of angular size and rotation which is > quite undefined for a polygon... > I thik AstroImage should only contain methods like: > //! Rotate the image with respect to it center > //! @param angle the rotation angle in radian > void rotate(double angle); > > //! Scale the image with respect to its center > //! @param factor the scaling factor. The image is shrinked if <1 > void scale(double factor); > > Those methods would only modify the polygon points with respect to their > previous position. Each method would compute a transformation matrix and > apply it to the vectors of the polygon. OK, I shall move them out of you wish. > Your plug-in itself will have to know what the absolute rotation and > scaling are for one nebula, but these definitions are nebula specific. Would you prefer that the Nebula class provides a method to get and set the= =20 ConvexPolygon, and the manipulation of the Nebula position is done outside= =20 the Nebula class, or for the Nebula class to have these functions? Matthew |