Some Danger of Refactoring using ReSharper

No doubt, ReSharper (2.5 specific) had helpped me a lot when coding in C#. Particularly I like its Reformat code feature which it removes redundancy in the code.

For example

private void DoSomething(object sender, EventArgs e)
{
   DoStuff(e);
}

private DoStuff(EventArgs e)
{
...
}

Resharper will suggest to remove the parameters "sender"

However, what if your function looks like the following:

private void DoSomething(object sender, EventArgs e)
{
    if (this.m_oLogGridView.InvokeRequired)
    {
      this.Invoke(new EventHandler(DoSomething), new object[] {this, new object()});
    }
    else
    {
      DoStuff(e);
    }
}

Resharper will suggest you to remove the parameter "sender". However, if you remove the sender parameter without fixing "this.Invoke(method target, parameters)" function, you will run into runtime error. That's because the number of parameter we are passing using the this.Invoke() doesn't match the method parameter.

So, be careful when use refactoring.