You will have to debug this yourself to see how and why your strategy behaves as it does. Based on the code above, its working as expected which is to return out of the OnBarUpdate() method, if an order is working, it will utlimately cancel itself out.
Announcement
Collapse
No announcement yet.
Partner 728x90
Collapse
NinjaTrader
Ending Ninjascript strategy
Collapse
X
-
ending ninjascript
Would coding like this be correct, and possibly solve this?
if (Position.MarketPosition == MarketPosition.Long && Position.GetProfitLoss(GetCurrentBid(), PerformanceUnit.Currency) + Performance.RealtimeTrades.TradesPerformance.Curre ncy.CumProfit > 200)
{
ExitLong();
return;
}
Comment
-
-
And it would only get done once (assuming the ExitLong happens before the the next bar starts; or if the exit don't happen right away it could get done on multiple bars until the position actually exited.)
You would probably still want the code from post #44 (placed after the code from post #47) to cause the strategy to continue to exit without doing anything on subsequent bars.
Comment
Latest Posts
Collapse
| Topics | Statistics | Last Post | ||
|---|---|---|---|---|
|
Started by Geovanny Suaza, 02-11-2026, 06:32 PM
|
0 responses
558 views
0 likes
|
Last Post
|
||
|
Started by Geovanny Suaza, 02-11-2026, 05:51 PM
|
0 responses
324 views
1 like
|
Last Post
|
||
|
Started by Mindset, 02-09-2026, 11:44 AM
|
0 responses
101 views
0 likes
|
Last Post
by Mindset
02-09-2026, 11:44 AM
|
||
|
Started by Geovanny Suaza, 02-02-2026, 12:30 PM
|
0 responses
545 views
1 like
|
Last Post
|
||
|
Started by RFrosty, 01-28-2026, 06:49 PM
|
0 responses
547 views
1 like
|
Last Post
by RFrosty
01-28-2026, 06:49 PM
|

Comment