From what I can see through the murk, I am wondering why you are creating a new Graphics instance rather than using the one passed with the event. Is there a reason?
It would probably be better to use a function here, rather than a class.
Andy
P.S. If you need to use a class here:
1. make the method DrawRectangle() static and call via ::
2. create an instance and call DrawRectangle() via .