From f4007665fa084dc2a51d7b5b1bf9466e5b50e591 Mon Sep 17 00:00:00 2001 From: brendonparker Date: Mon, 25 Jul 2022 18:59:58 -0400 Subject: [PATCH] Fix divide by zero error. --- src/CromulentBisgetti.ContainerPacking/PackingService.cs | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/CromulentBisgetti.ContainerPacking/PackingService.cs b/src/CromulentBisgetti.ContainerPacking/PackingService.cs index e7fae8b..f5df54c 100644 --- a/src/CromulentBisgetti.ContainerPacking/PackingService.cs +++ b/src/CromulentBisgetti.ContainerPacking/PackingService.cs @@ -55,7 +55,8 @@ public static List Pack(List containers, List decimal itemVolumeUnpacked = algorithmResult.UnpackedItems.Sum(i => i.Volume); algorithmResult.PercentContainerVolumePacked = Math.Round(itemVolumePacked / containerVolume * 100, 2); - algorithmResult.PercentItemVolumePacked = Math.Round(itemVolumePacked / (itemVolumePacked + itemVolumeUnpacked) * 100, 2); + var totalPackedVolume = itemVolumePacked + itemVolumeUnpacked; + algorithmResult.PercentItemVolumePacked = totalPackedVolume == 0 ? 0 : Math.Round(itemVolumePacked / totalPackedVolume * 100, 2); lock (sync) {