Форум программистов [Powered by Invision Power Board]

Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > .NET для новичков > Повторяющий код

Автор: Dino99rus 26.9.2008, 19:35

Код


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

Код



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";
  }
 ...
}

?

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

Автор: mihryak 26.9.2008, 20:02

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

т.к. код весьма страшен, и я не думаю, что он будет применяться в реальном приложении, то соглашусь полностью с dive и посоветую почитать про ООП и паттерны
например, http://dofactory.com/patterns/PatternTemplate.aspx#_self1

Автор: Dino99rus 26.9.2008, 20:38

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

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