Господа Админы

Давайте создадим форум по вопросам рефакторинга. А то как-то мне кажется, что многие об этом не знают , а хорошо было бы, чтобы узнали. Как ни как, а очень важный раздел программирования.

А что это?

Рефакторинг - это процес улучшения существующего кода ("наведения порядка в коде").
Оновние задачи рефакторинга:

1. Оптимизация (не по скорости, а по понятности) кода для более лёгкого дальнейшего расширения.
2. Повторное использование кода.
3. Инкапсуляция особого поведения классов в подклассы.
4. Саме главное - отделение визуальной части от функциональности. т.е. всё что есть на форме - создано ТОЛЬКО для того, чтобы "на него сотреть" и не должно нести за собой НИКАКОЙ функциональности.

и т.п.

самый простейший например (с вашего позволения буду использовать синтаксис Java):
Начальный файл

Код

public class MovieCostPrinter{  public static void main(String[] args) {    MovieCostPrinter costPrinter = new MovieCostPrinter();        Movie movie1 = new Movie(Movie.NEW);    Movie movie2 = new Movie(Movie.OLD);    Movie movie3 = new Movie(Movie.REGULAR);        costPrinter.printMovieCost(movie1);    costPrinter.printMovieCost(movie2);    costPrinter.printMovieCost(movie3);  }    public void printMovieCost(Movie movie) {    double cost = 0;    switch (movie.getMovieType()) {      case Movie.REGULAR:        cost = movie.getCost();        break;      case Movie.NEW:        cost = movie.getCost()*1.1; //так прийдётся делать каждый раз        break;      case Movie.OLD:        cost = movie.getCost()*0.65;    } //Не сильно удачное решение.    System.out.println(cost);  } } class Movie {  public static final int REGULAR = 1;  public static final int NEW = 2;  public static final int OLD = 3;  private int movieType;  private final double cost = 20;  Movie(int movieType) {    this.movieType = movieType;  }  public int getMovieType() {    return movieType;  }  public double getCost() {    return cost;  } }

Теперь если нам потребуется изменить что-то, добавить новый тип и т.п. - это будет очень сложно.
Проведём рефакторинг:

1. Переместим вычисление стоимости в подклассы (создадим их) и заменяем конструктор фабрчным методом:

Код

public class MovieCostPrinter {  public static void main(String[] args) {    MovieCostPrinter costPrinter = new MovieCostPrinter();    Movie movie1 = Movie.createMovie(Movie.NEW);    Movie movie2 = Movie.createMovie(Movie.OLD);    Movie movie3 = Movie.createMovie(Movie.REGULAR);    costPrinter.printMovieCost(movie1);    costPrinter.printMovieCost(movie2);    costPrinter.printMovieCost(movie3);  }  public void printMovieCost(Movie movie) {    System.out.println(movie.getCost()); //Просто вызываем метод абстрактного класса.  } } abstract class Movie {  public static final int REGULAR = 1;  public static final int NEW = 2;  public static final int OLD = 3;  protected final double cost = 20;  public abstract double getCost();  static Movie createMovie(int movieType) { // Заменяем конструктор фабричным методом    switch (movieType) {      case REGULAR:        return new RegularMovie();      case NEW:        return new NewMovie();      case OLD        return new OldMovie();      default:        throw new RuntimeException("Invalid movie type"); //Вызываем Exception если чё не то.    }  } } class RegularMovie extends Movie {  public double getCost() {    return cost; //перенос особого поведения в наследника  } } class NewMovie extends Movie {  public double getCost() {    return cost * 1.1;  //перенос особого поведения в наследника  } } class OldMovie extends Movie {  public double getCost() {    return cost *0.65; //перенос особого поведения в наследника  } }

Вроде как ничего в поведении не поменялось. Однако есть некоторые преймущества перед старым кодом:
1. Если нам понадобится где-то ещё вычислить стоимость, то не потребуется switch.
2. На всю работу с классами Movie у нас всего один switch. А это уж поверь - очень большой плюс.
3. Всё особое поведение инкапсулировано в отдельные подклассы, следовательно если добавиться ещё один тип, то надо изменить всего один switch и написать соответсвующий подкласс.

На самом деле это очень простой пример, на котором не возможно понять выгоды проделанного. Для того чтобы понять выгоду, нужно разобрать очень большой пример. И рефакторинги, которые при этом осущиствляются - очень сложные. Этот пример просто не может отразить полной сути.
В принципе, полиморфизм - общеизвестный факт. Тут нет ничего нового. Важно научиться его правильно использовать. К примеру у кого не разу не было такого, чтобы свой объект наследовался от како-го либо другого, при этом он использовалось всего несколько методов родительского объёкта. Для этого надо выполнять замену наследования делегирование. И ситуации наоборот, что свой объект представляет собой только делегирующие методы, тогда нужно выполнять обратную операцию.
Вооооообщем всё так не опишешь, можно только запутать....

Ну я думаю все смысл поймали.

По этому поводу я могу посоветовать книгу (это не реклама, просто книга действительно очень хорошая ) Рефакторинг, Мартин Фаулер. В ней методы рефакторинга очень хорошо описаны.

Ещё в тему есть понятие шаблонов проектирования. Тоже близкое по своему назначению.

Это сообщение отредактировал(а) altarasjuk - 20.7.2003, 14:00

это излишне....

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

altarasjuk да я не имею не чего против, но просто подумай как много людей будут заходить в этот раздел, он просто вымрет....

Step прав..... Хотя идея бесспорно очень интересна!

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

Admin, ну на счёт вымрет - не вымрет, не знаю, но то что о рефакторинге и вообще о правилах написания программ никто не знает (сужу по разделу Delphi). Уж в самой Делфе так повелось, что.... ну не умеют люди отделить интерфейс от реализации . А всё почему? да потому что каждая книга по Delphi начинается с "Создадим форму. Поцепим на неё кнопку. И УРААА!!!! мы изучили программирование!!!!", один только подход чего стоит?

Идея очень интересная, мне нравиться. Но вот чем заполнять такой раздел я совершенно не представляю.
Публиковать большие объемы кода (иначе какой рефакторинг?), а потом вместе обсуждать что получиться: боюсь у нас опять будет на флейм похоже.
Может сделать иначе: тема рефакторинг и грамотное написание кода, но по смыслу создание OpenSource? Скажем разрабатывается несколько проектов, постоянно висит последняя редакция кода, каждый может взглянуть, обсудить и предложить что-то свое (или преслать новый кусок кода). А админы и ответственные товарищи хороший кусок кода добавляют в разрабатываемый проект. Заодно и проблемы написания переносимого кода обсудим.
Этакая технология написания Linux - каждый может сделать то, что у него хорошо получается.

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

Как такая идея?

А раздел "статьи" уже работает?

altarasjuk
Можешь начать постить о рефакторинге в раздел "Технологии и алгоритмы", ибо считаю, что рефакторинг - технология, достойная того, чтобы о ней знали. Если тебе много есть, чего сказать, то сделаем тему, куда ты будешь постить свои мысли, а обсуждение можно забить в соседнюю тему.

Цитата

Уж в самой Делфе так повелось, что.... ну

это точно....

Цитата

Если тебе много есть, чего сказать

podval, ну тук как сказать, допустим я просто физически не потяну вести такую тему один (работы просто валйом), но допустим переодически какие-то мысли, напутствия, а может даже статейки, буду кидать с удовольствием. Хорохо если все кто в этом разбираются что-то дополняли, что-то своё вносили.... Ну я думаю мысль уловил.....

Это сообщение отредактировал(а) altarasjuk - 22.7.2003, 23:44

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

podval, я завтра еду в отпуск на неделю, как раз за это время чё-то напишу, и в понедельник или вторник - опубликую. Я думаю начать с самого больного вопроса - разделение реализации и интерфейса.