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.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
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…
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
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.
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…