Как провести рефакторинг: разделить класс на два, которые взаимодействуют через интерфейс

Шаги малыша.

Шаг 1. Внесите небольшое изменение, используя превосходный рефакторинг Resharper.
Шаг 2. Протестируйте (желательно модульный), чтобы убедиться, что код все еще работает. Шаг 3. Используйте систему контроля версий

Повторите эти 3 шага много раз.

Возможно, это не сработает, если вы выполняете один из тех рефакторингов, которые «должно быть большим шагом сразу». В этом случае сделайте как можно больше мелких рефакторингов, прежде чем делать большой шаг.

Называйте меня старомодным, но лично я использую описанные вами автоматизированные функции только для технических задач (таких как переименование и т. д.)

Все остальные виды рефакторинга, такие как извлечение интерфейсов и тому подобное, я предпочитаю выполнять вручную. Вы обнаружите, что вы можете провести еще больше рефакторинга, и обычно код получается чище.

Сначала краткий ответ, я собираюсь написать об этом в блоге, спасибо за идею!

Итак, предположим, что у нас есть это:

class PresenterAndController
  {
    public void Control()
    {
      Present();
    }

    public void Present()
    {
      // present something
    }
  }

Сначала нам нужно преобразовать использование всех прямых членов в косвенные, то есть квалифицировать их с помощью поля. Для этого используйте рефакторинг суперкласса extract для извлечения части Presenter в суперкласс:

  internal class Presenter
  {
    public void Present()
    {
      // present something
    }
  }

  class PresenterAndController : Presenter
  {
    public void Control()
    {
      Present();
    }
  }

Создайте поле в PresenterAndController, которое будет его делегировать, и используйте Generate (Alt-Ins) для создания делегирующих членов. Они будут скрывать базовые методы, поскольку они одинаковы:

  internal class Presenter
  {
    public void Present()
    {
      // present something
    }
  }

  class PresenterAndController : Presenter
  {
    private Presenter myPresenter;

    public void Present() // hides base method
    {
      myPresenter.Present();
    }

    public void Control()
    {
      Present(); // it now references method from this type
    }
  }

Удалите Presenter из списка наследования, и вы закончите с разделением классов. Теперь вы можете извлекать интерфейсы, использовать базовые типы, где это возможно, и иным образом приводить их в порядок.

Я никогда не использовал VisualStudio, но считаю автоматический рефакторинг в Eclipse довольно надежным - по крайней мере, таким же надежным, как если бы я выполнял ту же работу.

У меня есть только один важный совет - убедитесь, что вы можете вернуться к состоянию до начала рефакторинга, ничего не потеряв. Тогда просто погрузитесь и сделайте это. Возможно, вам придется остановиться и начать заново, но этого не стоит бояться (если вы прислушались к моему единственному совету). Вы многому научитесь на этом.

прежде, чем вы это сделаете, инвестируйте в систему контроля версий: изучите Git. Рефакторинг без SCM похож на вождение с завязанными глазами.