5

我有一个看起来像这样的方法:

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

有什么方法可以巩固这一点,因为唯一不同的是属性名称?

4

3 回答 3

12

制作一个单独的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;
}
于 2010-04-25T16:40:27.250 回答
9
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).

于 2010-04-25T16:40:23.100 回答
0

假设parser.Parse()返回相同的类Servings[],您可以创建该类型的空对象,其 X 和 Y 都为零。然后你可以有一个函数,它返回的第一个元素Servings[],如果它存在,或者new FoodDescriptionParser.Parser(Description),如果Description存在,或者,最后,那个空对象。并根据需要收集 X 或 Y。

于 2010-04-25T16:44:00.657 回答