diff --git a/BlacksmithWorkshop/BlacksmithWorkshopBusinessLogic/BusinessLogics/ReportLogic.cs b/BlacksmithWorkshop/BlacksmithWorkshopBusinessLogic/BusinessLogics/ReportLogic.cs index 2001944..65ad99d 100644 --- a/BlacksmithWorkshop/BlacksmithWorkshopBusinessLogic/BusinessLogics/ReportLogic.cs +++ b/BlacksmithWorkshop/BlacksmithWorkshopBusinessLogic/BusinessLogics/ReportLogic.cs @@ -35,7 +35,6 @@ namespace BlacksmithWorkShopBusinessLogic.BusinessLogics /// public List GetManufactureComponent() { - var components = _componentStorage.GetFullList(); var manufactures = _manufactureStorage.GetFullList(); var list = new List(); foreach (var manufacture in manufactures) @@ -47,14 +46,12 @@ namespace BlacksmithWorkShopBusinessLogic.BusinessLogics TotalCount = 0, TotalSum = 0 }; - foreach (var component in components) + foreach (var component in manufacture.ManufactureComponents) { - if (manufacture.ManufactureComponents.ContainsKey(component.Id)) - { - record.Components.Add((component.ComponentName, manufacture.ManufactureComponents[component.Id].Item2, component.Cost * manufacture.ManufactureComponents[component.Id].Item2)); - record.TotalCount += manufacture.ManufactureComponents[component.Id].Item2; - record.TotalSum += component.Cost * manufacture.ManufactureComponents[component.Id].Item2; - } + double componentCost = _componentStorage.GetElement(new() { Id = component.Key })?.Cost ?? 0; + record.Components.Add(new(component.Value.Item1.ComponentName, component.Value.Item2, componentCost)); + record.TotalCount += component.Value.Item2; + record.TotalSum += component.Value.Item2 * componentCost; } list.Add(record); }