From 0b91250c3d2e8e39fd4ef70a0292718faffeaef8 Mon Sep 17 00:00:00 2001 From: Chomp Date: Fri, 8 Aug 2025 15:57:30 +0100 Subject: [PATCH] Used single `TryGetValue` instead of ContainsKey + direct access --- Libraries/SPTarkov.Server.Core/Services/FenceService.cs | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Libraries/SPTarkov.Server.Core/Services/FenceService.cs b/Libraries/SPTarkov.Server.Core/Services/FenceService.cs index 03b67fcf..56e0f202 100644 --- a/Libraries/SPTarkov.Server.Core/Services/FenceService.cs +++ b/Libraries/SPTarkov.Server.Core/Services/FenceService.cs @@ -707,6 +707,7 @@ public class FenceService( ) { var priceLimits = traderConfig.Fence.ItemCategoryRoublePriceLimit; + var assortRootItems = baseFenceAssortClone .Items.Where(item => string.Equals(item.ParentId, "hideout", StringComparison.OrdinalIgnoreCase) && item.Upd?.SptPresetId == null @@ -720,6 +721,7 @@ public class FenceService( return; } + // Create new assorts until we've fulfilled the count requirement for (var i = 0; i < assortCount; i++) { if (!assortRootItems.Any()) @@ -758,7 +760,7 @@ public class FenceService( } var itemDbDetails = itemHelper.GetItem(chosenBaseAssortRoot.Template).Value; - if (priceLimits.ContainsKey(itemDbDetails.Parent) && price > priceLimits[itemDbDetails.Parent]) + if (priceLimits.TryGetValue(itemDbDetails.Parent, out var priceLimit) && price > priceLimit) { // Too expensive for fence, try another item i--;