protected override void OnExecutionUpdate(Execution execution, string executionId, double price, int quantity, MarketPosition marketPosition, string orderId, DateTime time) { if (execution.Order.Name.StartsWith("EnterLong")) { if (orderState == OrderState.Filled || orderState == OrderState.PartFilled) { //long position open } } else if (execution.Order.Name.StartsWith("ExitLong")) { if (orderState == OrderState.Filled || orderState == OrderState.PartFilled) { //long position closed } } else if (execution.Order.Name.StartsWith("ExitLongLimit")) { if (orderState == OrderState.Accepted) { //ExitLongLimit accepted } else if (orderState == OrderState.Cancelled) { //ExtLongLimit cancelled e.g. CancelOrder() } else if (orderState == OrderState.Filled || orderState == OrderState.PartFilled) { //ExitLongLimit filled, long position closed } } }
Announcement
Collapse
No announcement yet.
Partner 728x90
Collapse
NinjaTrader
OnExecutionUpdate
Collapse
X
-
OnExecutionUpdate
could someone check that I have this right ..
Code:Last edited by delTik; 04-27-2016, 10:13 AM.Tags: None
-
Hello delTik,
I would not be able to say if this is correct as the code that places the entry orders is not included. The names you are looking for would need to match the signal names used for the order.
Using string.StartsWith is fine to use.
You can also use IOrders to match orders.
For the most part this looks fine.
The question is, when you test the code does it do what you are expecting?Chelsea B.NinjaTrader Customer Service
Latest Posts
Collapse
Topics | Statistics | Last Post | ||
---|---|---|---|---|
Started by Mestor, Today, 03:15 AM
|
0 responses
1 view
0 likes
|
Last Post
by Mestor
Today, 03:15 AM
|
||
Started by Mestor, 03-10-2023, 01:50 AM
|
11 responses
305 views
0 likes
|
Last Post
by z.franck
Today, 02:29 AM
|
||
Started by Padan, Today, 02:02 AM
|
0 responses
6 views
0 likes
|
Last Post
by Padan
Today, 02:02 AM
|
||
Started by bukowa, 11-09-2023, 06:07 PM
|
20 responses
407 views
0 likes
|
Last Post
by Adfra
Today, 01:36 AM
|
||
Started by yertle, Yesterday, 05:49 PM
|
2 responses
11 views
0 likes
|
Last Post
by yertle
Today, 12:12 AM
|
Comment