Oda/managing-apps
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Refactor TradingBotBase to streamline recovery logic for recently canceled positions. Removed redundant recovery call and added comments for clarity in position management.

Browse Source
This commit is contained in:
cryptooda
2025-11-12 00:50:16 +07:00
parent 2057c233e5
commit 8d97fce41c
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/Managing.Application/Bots/TradingBotBase.cs
View File

@@ -372,6 +372,9 @@ public class TradingBotBase : ITradingBot
private async Task ManagePositions() private async Task ManagePositions()
{ {
// Recovery Logic: Check for recently canceled positions that might need recovery
await RecoverRecentlyCanceledPositions();
// Early exit optimization - skip if no positions to manage // Early exit optimization - skip if no positions to manage
// Optimized: Use for loop to avoid multiple iterations // Optimized: Use for loop to avoid multiple iterations
bool hasOpenPositions = false; bool hasOpenPositions = false;
@@ -400,8 +403,7 @@ public class TradingBotBase : ITradingBot
if (!hasOpenPositions && !hasWaitingSignals) if (!hasOpenPositions && !hasWaitingSignals)
return; return;
// Recovery Logic: Check for recently canceled positions that might need recovery
await RecoverRecentlyCanceledPositions();
// First, process all existing positions that are not finished // First, process all existing positions that are not finished
// Optimized: Inline the filter to avoid LINQ overhead // Optimized: Inline the filter to avoid LINQ overhead