Menu

iMouseCursor Interface

2010-04-17
2013-04-24
  • Joël Langlois

    Joël Langlois - 2010-04-17

    I wanted to point out that you have repeated the exact same functionality from iMouseCursor within the MouseCursor class, creating overhead and redundant code.

    The MouseCursor class doesn't even need to derive the iMouseCursor interface in order to work.

     
  • Bidji

    Bidji - 2010-10-14

    It's worth than that : it's not redundant code, it's useless code cause all accessors contained in iMouseCursor are overridden in MouseCursor.
    Using an auto-implemented property is not the same than using a accessor fully implemented: in the first case, there is no need of an attribute, and in the second one, you have to add an attribute (private) in the class.

    So this interface must be deleted cause in the best case it's useless and in the worst it's dangerous…

     

Log in to post a comment.

MongoDB Logo MongoDB