VPF::Повторяющий код - Форум программистов

Dino99rus

Дата 26.9.2008, 19:35 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 83
Регистрация: 10.1.2007

Репутация: 1
Всего: 2

Код


private string getTextFile()
        {
            string result = "";
            if (_journalCharterRes.Count == 0)
                return "";
            result += this.NameJournal + "\r" + this.DateReleaseJournal + "\r" + "->\r";
            foreach (JournalCharter jc in _journalCharterRes)
            {
                result += jc.Name + "\r";
                foreach (string pathPic in jc.ArrayPathPic)
                {
                    result += pathPic + "\r";  
                }                
                result += "<-\r->\r"; // все из-за этого
            }
            return result;
        }

и вот

Код


private string getResFile()
        {
            string result = "";
            if (_journalCharterRes.Count == 0)
                return "";
            result += this.NameJournal + "\r" + this.DateReleaseJournal + "\r";
            foreach (JournalCharter jc in _journalCharterRes)
            {
                result += jc.Name + "\r";
                foreach (string pathPic in jc.ArrayPathPic)
                {
                    result += pathPic + "\r";
                }                
            }
            return result;
        }

Вот как такое обходить? Разбивать метод на на 3 или более методов это не выход.

dive

Дата 26.9.2008, 19:46 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 83
Регистрация: 25.10.2007
Где: Санкт-Петербург

Репутация: нет
Всего: 5

Код



private final int TEXT_FILE = 1;
private final int RES_FILE = 2;

private string getFile(int _type) {
 ...
 result += this.NameJournal + "\r" + this.DateReleaseJournal + "\r" + ( _type == TEXT_FILE ? "->\r" : "");
 ...
  if (_type == RES_FILE) {
   result += "<-\r->\r";
  }
 ...
}

?

если задачка посерьезнее, то паттернами, фабричными.

Это сообщение отредактировал(а) dive - 26.9.2008, 19:46

mihryak

Дата 26.9.2008, 20:02 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 731
Регистрация: 28.4.2007
Где: С-Пб

Репутация: 10
Всего: 36

dive, грустно без enum'ов smile

т.к. код весьма страшен, и я не думаю, что он будет применяться в реальном приложении, то соглашусь полностью с dive и посоветую почитать про ООП и паттерны
например, шаблонный метод

Dino99rus

Дата 26.9.2008, 20:38 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 83
Регистрация: 10.1.2007

Репутация: 1
Всего: 2

dive,
mihryak,
Спасибо за советы. smile

Я уже код переписал. И метод стал отличаться. Про паттерны... еще до них не дошел. Сейчас читаю рефакторинг (Мартин Фаулер).
Все еще в переди.

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| .NET для новичков \| Следующая тема »