From 1383e82470aa60a2db36b103b5d6d49b83fadb26 Mon Sep 17 00:00:00 2001 From: Abrynos <6608231+Abrynos@users.noreply.github.com> Date: Wed, 22 May 2024 21:17:49 +0200 Subject: [PATCH] Apply feedback --- .../TradeStatistics.cs | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/ArchiSteamFarm.OfficialPlugins.Monitoring/TradeStatistics.cs b/ArchiSteamFarm.OfficialPlugins.Monitoring/TradeStatistics.cs index 35ae2a9f7a157..fbb2206835c1a 100644 --- a/ArchiSteamFarm.OfficialPlugins.Monitoring/TradeStatistics.cs +++ b/ArchiSteamFarm.OfficialPlugins.Monitoring/TradeStatistics.cs @@ -47,12 +47,13 @@ internal void Include(ParseTradeResult result) { ArgumentNullException.ThrowIfNull(result); lock (Lock) { - if (result is { Result: ParseTradeResult.EResult.Accepted, Confirmed: true }) { - ItemsGiven += (uint) (result.ItemsToGive?.Count ?? 0); - ItemsReceived += (uint) (result.ItemsToReceive?.Count ?? 0); - } - switch (result.Result) { + case ParseTradeResult.EResult.Accepted when result.Confirmed: + ++ConfirmedOffers; + ItemsGiven += (uint) (result.ItemsToGive?.Count ?? 0); + ItemsReceived += (uint) (result.ItemsToReceive?.Count ?? 0); + + goto case ParseTradeResult.EResult.Accepted; case ParseTradeResult.EResult.Accepted: ++AcceptedOffers; @@ -70,10 +71,6 @@ internal void Include(ParseTradeResult result) { break; } - - if (result.Confirmed) { - ++ConfirmedOffers; - } } } }