Getter, Function или просто установить его на конструкторе?

Question

Я не могу найти хороший способ и удобочитаемость, чтобы написать некоторые вычисления внутри класса. Представьте себе следующий класс и все способы, чтобы получить FinalPrice:

public class Order { 
    public Order(Product[] products) { 
     Items = products; 

вариант 1: Having объявление переменной для каждого свойства, которые хотят быть вычислен, ужасный читаемость

---

 var orderPrice = products.Sum(p => p.Price * p.Quantity); 
     var orderTaxes = products.Sum(p => p.Taxes * p.Quantity); 
     var orderDiscount = products.Sum(p => p.Price * p.Quantity * p.Discount); 

     OrderPrice = orderPrice; 
     OrderTaxes = orderTaxes; 
     OrderDiscount = orderDiscount; 

     FinalPrice = orderPrice + orderTaxes - orderDiscount; 

---

вариант 2: проблема с порядком в классе важна! Строка FinalPrice не может быть до других или она не будет работать, но не будет вызывать ошибку.

---

 OrderPrice = products.Sum(p => p.Price * p.Quantity); 
     OrderTaxes = products.Sum(p => p.Taxes * p.Quantity); 
     OrderDiscount = products.Sum(p=> p.Price * p.Quantity * p.Discount); 

     FinalPrice = OrderPrice + OrderTaxes - OrderDiscount; 

---

вариант 3: переписывание все формулы - плохо для manteinance. Скорее всего, позже введут различия в ценах.

---

 FinalPrice = products.Sum(p => p.Price * p.Quantity) + 
        products.Sum(p => p.Taxes * p.Quantity) - 
        products.Sum(p => p.Price * p.Quantity * p.Discount); 

    } 

---

вариант 4: с помощью методов получения. Это будет рассчитываться каждый раз, когда оно вызывается. Это простой расчет, но в значительной степени предполагайте что-то большее количество кода.

---

public decimal FinalPrice { get { 
     return OrderPrice + OrderTaxes - OrderDiscount; 
    } } 
} 

вариант 5: с помощью функции. Это хорошо или плохо?

---

public decimal CalculateFinalPrice() { 
     return OrderPrice + OrderTaxes - OrderDiscount; 
    } 

Answer 1

Я хотел бы сделать всю логику в добытчиках:

public decimal CalculateFinalPrice 
{ 
    get { return CalculateOrderPrice + CalculateOrderTaxes - CalculateOrderDiscount; } 
} 

public decimal CalculateOrderPrice 
{ 
    get { return products.Sum(p => p.Price*p.Quantity); } 
} 

public decimal CalculateOrderTaxes 
{ 
    get { return products.Sum(p => p.Taxes*p.Quantity); } 
} 

public decimal CalculateOrderDiscount 
{ 
    get { return products.Sum(p => p.Price*p.Quantity*p.Discount); } 
} 

Если это происходит медленно в вашем secenario, вы можете кэшировать свойства.

private decimal? _calculateOrderPrice; 
public decimal CalculateOrderPrice 
{ 
    get 
    { 
     if (_calculateOrderPrice == null) 
     { 
      _calculateOrderPrice = products.Sum(p => p.Price*p.Quantity; 
     } 
     return _calculateOrderPrice.Value; 
    } 
} 

Если вы перейдете к определению свойства, вы сразу увидите, как оно рассчитывается. Кроме того, вам не важно, какие вычисления необходимо выполнить в первую очередь.

Answer 2

Я хотел бы создать методы CalculateFinalPrice, CalculateOrderPrice, CalculateOrderTaxes и CalculateOrderDiscount, как это:

public decimal CalculateFinalPrice() { 
    return CalculateOrderPrice() + CalculateOrderTaxes() - CalculateOrderDiscount(); 
} 

public decimal CalculateOrderPrice() 
{ 
    // Logic here to calculate order price 
    return theOrderPrice; 
} 

public decimal CalculateOrderTaxes() 
{ 
    // Logic here to calculate order taxes 
    return theOrderTaxes; 
} 

public decimal CalculateOrderDiscount() 
{ 
    // Logic here to calcuate order discount 
    return theOrderDiscount; 
} 

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