Есть ли способ удалить дублирование в этом коде - PullRequest
Новая
       16

Есть ли способ удалить дублирование в этом коде

5 голосов
/ 25 апреля 2010

У меня есть метод, который выглядит следующим образом: 

   private double GetX()
    {
        if (Servings.Count > 0)
        {
            return Servings[0].X;
        }
        if (!string.IsNullOrEmpty(Description))
        {
            FoodDescriptionParser parser = new FoodDescriptionParser();
            return parser.Parse(Description).X;
        }
        return 0;
    }

и у меня есть другой метод, который выглядит следующим образом: 

  private double GetY()
    {
        if (Servings.Count > 0)
        {
            return Servings[0].Y;
        }
        if (!string.IsNullOrEmpty(Description))
        {
            FoodDescriptionParser parser = new FoodDescriptionParser();
            return parser.Parse(Description).Y;
        }
        return 0;
    }

Есть ли способ объединить это, поскольку единственное, что отличается, это имена свойств?

Ответы [ 3 ]

12 голосов
/ 25 апреля 2010

Создайте отдельный GetServing метод: 

private Serving GetServing() {
    if (Servings.Count > 0)
        return Servings[0];

    if (!string.IsNullOrEmpty(Description)) {
        FoodDescriptionParser parser = new FoodDescriptionParser();
        return parser.Parse(Description);
    }
    return null;
}

private double GetX() {
    Serving serving = GetServing();
    if (serving == null) return 0;
    return serving.X;
}

private double GetY() {
    Serving serving = GetServing();
    if (serving == null) return 0;
    return serving.Y;
}
9 голосов
/ 25 апреля 2010 
private double Get(Func<SomeType, double> valueProvider)
{
    if (Servings.Count > 0)
    {
        return valueProvider(Servings[0]);
    }
    if (!string.IsNullOrEmpty(Description))
    {
        FoodDescriptionParser parser = new FoodDescriptionParser();
        return valueProvider(parser.Parse(Description));
    }
    return 0;
}

Что можно использовать так: 

var x = Get(value => value.X);
var y = Get(value => value.Y);

Примечание: SomeType - это тип Servings[0], который, если я правильно понимаю, ваш код должен совпадать с типом parser.Parse(Description).

0 голосов
/ 25 апреля 2010

Предполагая, что parser.Parse() возвращает тот же класс, что и Servings[], вы можете создать нулевой объект этого типа, для которого оба X и Y равны нулю. Тогда у вас может быть функция, которая возвращает первый элемент Servings[], если он существует, или new FoodDescriptionParser.Parser(Description), если существует Description, или, наконец, этот нулевой объект. И соберите X или Y по мере необходимости.

...