Apply feedback

JustArchiNET · May 22, 2024 · 1383e82 · 1383e82
1 parent c82463f
commit 1383e82
Showing 1 changed file with 6 additions and 9 deletions.
diff --git 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;
-			}
 		}
 	}
 }