Save Order Highest Profit and Lowest Profit values
BUMP!
I would highly recommend that you complete a c++ tutorial if you want to learn mql. Please take no offense but it's rather apparent by your code that you are used to higher level langs that do all the memory management for you, which is going to cause havoc on MT due to memory leaks. I'm not sure what you were hoping to accomplish with your example above... just like any other language you'd have to add your objects to a collection, but there is none here. So what's happening is you're orphaning objects on the heap every time you assign a new object to an existing pointer. The only way to can do that here is if you assign the object also to something that will handle the memory for you like an MQL collection for example. Here's an example of creating dynamic objects and working with them using pointer collections.
#property strict #property indicator_chart_window #include <Arrays\ArrayObj.mqh> class ProfitTracker; ProfitTracker *g_tracker; //+------------------------------------------------------------------+ int OnInit() { g_tracker = new ProfitTracker(); g_tracker.DisplayOnChart(true); return(INIT_SUCCEEDED); } //+------------------------------------------------------------------+ //| Custom indicator iteration function | //+------------------------------------------------------------------+ int start() { g_tracker.Refresh(); return 0; } //+------------------------------------------------------------------+ void OnDeinit(const int reason) { if(CheckPointer(g_tracker)) delete g_tracker; } //+------------------------------------------------------------------+ //| //+------------------------------------------------------------------+ template<typename T> class vector : public CArrayObj { public: T operator[](int i){return At(i);} bool Add(T element) { return CArrayObj::Add(element);} }; //+------------------------------------------------------------------+ //| //+------------------------------------------------------------------+ class ProfitTicket : public CObject { protected: double m_current; double m_highest; double m_lowest; int m_ticket; public: //constructor ProfitTicket(const int ticket); ~ProfitTicket(); //getters double Current() const { return m_current; } double Highest() const { return m_highest; } double Lowest() const { return m_lowest; } int Ticket() const { return m_ticket; } bool Refresh(); string ToString(); virtual int Compare(const CObject *node,const int mode=0)const override; }; //+------------------------------------------------------------------+ //| //+------------------------------------------------------------------+ ProfitTicket::ProfitTicket(const int ticket) { m_ticket = ticket; m_highest= DBL_MIN_10_EXP; m_lowest = DBL_MAX; } //+------------------------------------------------------------------+ ProfitTicket::~ProfitTicket() { printf("Removing ticket from tracking. Final status:: %s",ToString()); } //+------------------------------------------------------------------+ bool ProfitTicket::Refresh() { if(!OrderSelect(m_ticket,SELECT_BY_TICKET)) return false; double profit = OrderProfit()+OrderSwap()+OrderCommission(); if(profit > m_highest) m_highest = profit; if(profit < m_lowest) m_lowest = profit; m_current = profit; return true; } //+------------------------------------------------------------------+ int ProfitTicket::Compare(const CObject *node,const int mode=0)const override { const ProfitTicket *other=node; if(this.Ticket() > other.Ticket()) return 1; if(this.Ticket() < other.Ticket()) return -1; return 0; } //+------------------------------------------------------------------+ string ProfitTicket::ToString() { return StringFormat( "ticket: %d, highest: %.2f, lowest: %.2f", m_ticket, m_highest, m_lowest ); } //+------------------------------------------------------------------+ //| profit tracker src //+------------------------------------------------------------------+ class ProfitTracker : public CObject { protected: bool m_display_on_chart; vector<ProfitTicket*> m_list; public: ProfitTracker():m_display_on_chart(false){} ~ProfitTracker() { Comment("");} void DisplayOnChart(bool flag) { m_display_on_chart = flag;} void Refresh(); }; //+------------------------------------------------------------------+ void ProfitTracker::Refresh() { m_list.Sort(); for(int i =OrdersTotal()-1;i>=0;i--) { if(OrderSelect(i,SELECT_BY_POS) && OrderType() < 2) { ProfitTicket *ticket = new ProfitTicket(OrderTicket()); if(m_list.Search(ticket) < 0) m_list.InsertSort(ticket); else delete ticket; } } string comm = ""; for(int i = m_list.Total()-1;i>=0;i--) { if(!m_list[i].Refresh()) m_list.Delete(i); else if(m_display_on_chart) comm+= m_list[i].ToString()+"\r\n"; } if(m_display_on_chart) Comment(comm); }
So you can save OrderProfit() , and then overwrite the value every time a new value has been reached.
I will try to explain with pseudo code:
double profit_highest[ticket_number],profit_lowest[ticket_number]; for(int i=0;i<OrdersTotal();i++) { if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES)==true) { if(OrderSymbol()==Symbol() && OrderMagicNumber()==MAGICMA) { // create my data structures here if(OrderProfit()>profit_highest[ticket_number]) { profit_highest[ticket_number]=OrderProfit(); // Overwrite old value because Orderprofit() is higher then previous value... } if(OrderProfit()<profit_lowest[ticket_number]) { profit_lowest[ticket_number]=OrderProfit(); // Overwrite old value because OrderProfit() is lower then previous value... } } } }
This way you will end up with the higerst as well as the losest order profit value.
You only have to add some order tracking functionality.
I would highly recommend that you complete a c++ tutorial if you want to learn mql. Please take no offense but it's rather apparent by your code that you are used to higher level langs that do all the memory management for you, which is going to cause havoc on MT due to memory leaks. I'm not sure what you were hoping to accomplish with your example above... just like any other language you'd have to add your objects to a collection, but there is none here. So what's happening is you're orphaning objects on the heap every time you assign a new object to an existing pointer. The only way to can do that here is if you assign the object also to something that will handle the memory for you like an MQL collection for example. Here's an example of creating dynamic objects and working with them using pointer collections.
I am so sorry for my lack of understanding, you are 100% right. I am completely used to higher level languages, and have zero experience with C++. Nevertheless, I am here to learn, and have some understanding of programming ecommerce based systems, but, I am here to learn. Thank you very much for your example, and patience with me. I will try to structure my code better, according to your example, and post my progress here
Thank you very much,
Winst
I am so sorry for my lack of understanding, you are 100% right. I am completely used to higher level languages, and have zero experience with C++. Nevertheless, I am here to learn, and have some understanding of programming ecommerce based systems, but, I am here to learn. Thank you very much for your example, and patience with me. I will try to structure my code better, according to your example, and post my progress here
Thank you very much,
Winst
No worries, bro, and if you know of any good django tutorials - send 'em my way.
So you can save OrderProfit() , and then overwrite the value every time a new value has been reached.
I will try to explain with pseudo code:
This way you will end up with the higerst as well as the losest order profit value.
You only have to add some order tracking functionality.
Thank you! Thank you so much! I will digest these replies, including yours, and post back with my progress.
My motive for writing this code is, that i want to manage orders, apart from using a stoploss, i wanted to keep track of what orders experience with profit and loss, based on current orderprofit, highets profit, and lowest profit, + market conditions, perhaps close these orders.
Thank you again for your valuable advice, I'm here to learn
Winst
- Free trading apps
- Over 8,000 signals for copying
- Economic news for exploring financial markets
You agree to website policy and terms of use
Hello,
This is my first post here, although I have been scouring these forums, to the point where I can see the usual suspects and their valuable contribution.
To these senior posters, I wanted to take a quick second to say thank you, for your valuable advice and input
My problem is:
I have a method which is supposed to monitor open trades, save their highest profit, and lowest profit points in a data type. I have tried, simple variables, arrays, structs, and finally class objects
For example.
I have multiple open orders of any type (BUY or SELL). These orders naturally, go through a history, where their current profit is X or Y (negative or positive). I wanted to understand the effort an order goes through, until it reaches it's take profit, or stop loss
I thought I could accomplish this with simple variables, but the naming of those variables, became a problem for me.
I create all of these data types within a function that cycles through all open orders like this:
I thought I would use multi dimensional arrays, but I could not make that work either.
I then tried using class objects, since, in my day job I write PHP and Python for a living, but admittedly MQL4 is still very new to me.
My code now looks like:
The problem I have with my code, is that I can't seem to set data to each OrderTicket instance, the effect is, that order 1 is correct, but order 2 has the values of order 1
I think this post is getting a bit long, so I'm going to stop it here.
Thank you for your advice, and assistance