OrderComment() as an identifier?

[Erman Ardianto]

Hello,

Usually we use OrderMagicNumber() to identify whether the active order opened by given EA or not. Can we always use always OrderComment() for this purpose?

Thanks

[Ray]

System modifies the comment on some positions when closed

[Erman Ardianto]

I see, so we still can use it on active order?

[LesioS]

In my opinion yes... According to phy's info you should comment content tests provide using https://docs.mql4.com/strings/StringFind function.

[ukt]

devilian1899:
I see, so we still can use it on active order?

Beware... if you take partial take profit where lots value is less than OrderLots() the terminal will 100% close your order --> reopen new order with new ticket# for the remaining lots.
Magic# gets assigned to this new order - but your comment will not contain what you put there on OrderSend()

Note that I'm only expanding on what phy rightly stated originally...

fred GBPUSD,H1: deinit() called, "done"=1
fred GBPUSD,H1: <EAprint> after final Close of remaining Lots: OrderLots()=1, and expect EMPTY ticket#: getTicket(magic#123)=-1, and expect zero orders in terminal: OrdersTotal()=0
fred GBPUSD,H1: close #3 buy 1.00 GBPUSD at 2.0086 sl: 2.0061 tp: 2.0111 at price 2.0083
fred GBPUSD,H1: <EAprint> after 2*1Lots PartialTakeProfitCloses, Ticket=#3, OrderComment()="split from #2", Magic=123, OrdersTotal()=1
fred GBPUSD,H1: <EAprint> getTicket(123) OrdersTotal()=1, Ticket=#3, Magic=123
fred GBPUSD,H1: close #2 buy 1.00 GBPUSD at 2.0086 sl: 2.0061 tp: 2.0111 at price 2.0083
fred GBPUSD,H1: <EAprint> after 1*1Lot PartialTakeProfitCloses, Ticket=#2, OrderComment()="split from #1", Magic=123, OrdersTotal()=1
fred GBPUSD,H1: <EAprint> getTicket(123) OrdersTotal()=1, Ticket=#2, Magic=123
fred GBPUSD,H1: close #1 buy 1.00 GBPUSD at 2.0086 sl: 2.0061 tp: 2.0111 at price 2.0083
fred GBPUSD,H1: <EAprint> origOrderTicket #1, OrderComment()="OrderSend Opens 3Lots", Magic=123, OrdersTotal()=1
fred GBPUSD,H1: open #1 buy 3.00 GBPUSD at 2.0086 sl: 2.0061 tp: 2.0111 ok
fred GBPUSD,H1: start() called, "done"=0
fred GBPUSD,H1: init() called, "done"=0
fred started for testing
fred GBPUSD,H1: loaded successfully

[Ray]

There you have it.