czy istnieje sposób, aby usunąć powielania w tym kodzie

Question

Mam metodę, która wygląda tak:

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

i mam inną metodę, która wygląda tak:

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

Czy jest jakiś sposób, skonsolidować to, ponieważ jedyne co innego to nazwy nieruchomości?

Answer 1

Zrób osobną GetServing metody:

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

Answer 2

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

które mogłyby zostać wykorzystane tak:

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

Uwaga: SomeType jest typem Servings[0] który jeśli dobrze rozumiem swój kod poprawnie powinno być takie samo jak typ parser.Parse(Description).

Answer 3

Zakładając parser.Parse() powraca do tej samej klasy, które Servings[] posiada, można utworzyć null object tego typu, dla których zarówno X & Y są równe zeru. Wtedy możesz mieć funkcję, która zwraca pierwszy element z Servings[], jeśli istnieje, lub new FoodDescriptionParser.Parser(Description), jeśli istnieje, lub, w końcu, ten obiekt zerowy. I zbieraj X lub Y w razie potrzeby.