集合父记录中循环的性能

Question

I want to make my classes a bit smarter. So I am adding to a sub-list which is a child of DocumentLines and I have Documents . Consider them to be like OrderHeader , OrderLines . The reason why I call them documents was that I am using status to tell what type of order it is.

I want to have a property in my class that will get the current items in the child list and sum the cost up so I have a property I can bind to in my form. Is below ok or could it become too exhaustive if I have to say 20 products on an order?

public List<DocumentLines> Items;
public decimal OrderTotal
{
    get
    {
        decimal price = 0.00m;
        foreach (var item in Items)
        {
            price += item.LinePrice;
        }                
        return price;
    }
}

Answer 1

It won't be too exhaustive. It will be fine. You could consider using the LINQ Sum() method .

public List<DocumentLines> Items;
public decimal OrderTotal
{
    get
    {
        return Items.Sum(item => item.LinePrice);
    }
}

With C# 6 you can use expression-bodied members (=>) to shorten it further.

public List<DocumentLines> Items;
public decimal OrderTotal => Items.Sum(item => item.LinePrice);