Take profit candle pips
To get the Candle "pips" you should be subtracting the "Low" from the "High" of the candle surely.... Also, you need to be dividing that value with Point (not multiplying)... that is why you are getting such a small take profit value...
double takeprofitbuy = MathAbs(NormalizeDouble(iHigh(NULL,0,1)-iLow(NULL,0,1),5)/Point);
You also only need one "takeprofit" variable... as you will always be subtracting the Low from the High...
Hope that works and I am understanding your question correctly...
To get the Candle "pips" you should be subtracting the "Low" from the "High" of the candle surely.... Also, you need to be dividing that value with Point (not multiplying)... that is why you are getting such a small take profit value ...
You also only need one "takeprofit" variable... as you will always be subtracting the Low from the High...
Hope that works and I am understanding your question correctly...
I tried, but it does not work. Take profit becomes an absurd value. Strangling, removing * point and digits, seems to work properly (although sometimes the value of sl seems to me less than the tp)
//+------------------------------------------------------------------+ //| engulfingh4test.mq4 | //| Copyright 2018, MetaQuotes Software Corp. | //| https://www.mql5.com | //+------------------------------------------------------------------+ #property copyright "Copyright 2018, MetaQuotes Software Corp." #property link "https://www.mql5.com" #property version "1.00" #property strict extern double Lots=0.1; double stoploss = NormalizeDouble((1000*Point),Digits); double takeprofit = MathAbs(iHigh(Symbol(),0,1)-iLow(Symbol(),0,1)); datetime NewTime=0; //+------------------------------------------------------------------+ //| Expert initialization function | //+------------------------------------------------------------------+ int OnInit() { //--- //--- return(INIT_SUCCEEDED); } //+------------------------------------------------------------------+ //| Expert deinitialization function | //+------------------------------------------------------------------+ void OnDeinit(const int reason) { //--- } //+------------------------------------------------------------------+ //| Expert tick function | //+------------------------------------------------------------------+ void OnTick() { if(NewTime!=Time[0]) // Run only on new bars { NewTime=Time[0]; double media = iMA(Symbol(),0,30,0,MODE_EMA,PRICE_CLOSE,1); bool orderOpended=false; if ( Ask>media && iOpen(NULL,0,2)>iClose(NULL,0,2) && // Place Buy logic in if statemet iHigh(NULL,0,1)>iHigh(NULL,0,2) && // Buy logic iLow(NULL,0,1)<iLow(NULL,0,2)) // Buy logic orderOpended=OrderSend(Symbol(),OP_BUY,Lots,Ask,3,iLow(NULL,0,1),Ask+takeprofit,NULL,0,0,Blue); else if(Bid<media && iOpen(NULL,0,2)<iClose(NULL,0,2) && // Place Sell logic in if satement iHigh(NULL,0,1)>iHigh(NULL,0,2) && // Sell logic iLow(NULL,0,1)<iLow(NULL,0,2)) // Sell logic orderOpended=OrderSend(Symbol(),OP_SELL,Lots,Bid,3,iHigh(NULL,0,1),Bid-takeprofit,NULL,0,0,Red); } } //+------------------------------------------------------------------+
Sorry... yes the "takeprofit" value shoud be multiplied by Point as you originally had it... so, you need to put that back in...
double takeprofit = MathAbs(iHigh(Symbol(),0,1)-iLow(Symbol(),0,1)) * Point;
...that will then give you the correct "takeprofit" point size.... so your Take Profit levels should now work ok...
I tried, but it does not work. Take profit becomes an absurd value. Strangling, removing * point and digits, seems to work properly (although sometimes the value of sl seems to me less than the tp)
no, the strange thing is that no point works ... it is with the point that gives me the take profit at the same price of the entry level ... I do not know why
I am confusing you Roberto.... There should not be any "Point" in the calculation because the "High" minus the "Low" gives you the difference in "Points".... which is what you need as your "takeprofit" value...
double takeprofit = MathAbs(iHigh(Symbol(),0,1)-iLow(Symbol(),0,1));
I am confusing you Roberto.... There should not be any "Point" in the calculation because the "High" minus the "Low" gives you the difference in "Points".... which is what you need as your "takeprofit" value...
We would miss Mike, you are very kind to help me. Now I would like to set the breakeven function when the price comes in half from the take profit ... I entered the function, but does not seem to go, no operation is closed in breakeven. Maybe I'm wrong method to get half of my take profit?
//+------------------------------------------------------------------+ //| engulfingh4test.mq4 | //| Copyright 2018, MetaQuotes Software Corp. | //| https://www.mql5.com | //+------------------------------------------------------------------+ #property copyright "Copyright 2018, MetaQuotes Software Corp." #property link "https://www.mql5.com" #property version "1.00" #property strict extern double Lots=0.1; double stoploss = NormalizeDouble((1000*Point),Digits); double takeprofit = MathAbs(iHigh(Symbol(),0,1)-iLow(Symbol(),0,1)); datetime NewTime=0; double Breakeven = MathAbs(takeprofit/2); input double Magic = 280456; //+------------------------------------------------------------------+ //| Expert initialization function | //+------------------------------------------------------------------+ int OnInit() { //--- //--- return(INIT_SUCCEEDED); } //+------------------------------------------------------------------+ //| Expert deinitialization function | //+------------------------------------------------------------------+ void OnDeinit(const int reason) { //--- } //+------------------------------------------------------------------+ //| Expert tick function | //+------------------------------------------------------------------+ void OnTick() { if(NewTime!=Time[0]) { NewTime=Time[0]; double media = iMA(Symbol(),0,30,0,MODE_EMA,PRICE_CLOSE,1); bool orderOpended=false; if ( Ask>media && iOpen(NULL,0,2)>iClose(NULL,0,2) && // Place Buy logic in if statemet iHigh(NULL,0,1)>iHigh(NULL,0,2) && // Buy logic iLow(NULL,0,1)<iLow(NULL,0,2)) // Buy logic orderOpended=OrderSend(Symbol(),OP_BUY,Lots,Ask,3,Ask-takeprofit,Ask+takeprofit,NULL,Magic,0,Blue); else if(Bid<media && iOpen(NULL,0,2)<iClose(NULL,0,2) && // Place Sell logic in if satement iHigh(NULL,0,1)>iHigh(NULL,0,2) && // Sell logic iLow(NULL,0,1)<iLow(NULL,0,2)) // Sell logic orderOpended=OrderSend(Symbol(),OP_SELL,Lots,Bid,3,Bid+takeprofit,Bid-takeprofit,NULL,Magic,0,Red); } } //+------------------------------------------------------------------+ void BreakEven() { double MyPoint=Point; if(Digits==3 || Digits==5) MyPoint=Point*10; for(int i=OrdersTotal()-1; i>=0;i--) { if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES)) if(OrderSymbol()==Symbol()) if(OrderMagicNumber()==Magic) if(OrderType()==OP_BUY) { if(Bid-OrderOpenPrice()>Breakeven*MyPoint) if(OrderOpenPrice()>OrderStopLoss()) int BBM=OrderModify(OrderTicket(),OrderOpenPrice(),OrderOpenPrice(),OrderTakeProfit(),0,clrNONE); } else if(OrderType()==OP_SELL) { if(OrderOpenPrice()-Ask>Breakeven*MyPoint) if(OrderOpenPrice()<OrderStopLoss()) int BSM=OrderModify(OrderTicket(),OrderOpenPrice(),OrderOpenPrice(),OrderTakeProfit(),0,clrNONE); } } }
Hi Roberto... why have you forgotten about your "takeprofit" and Breakeven" variables...
// For Long positions int BBM=OrderModify(OrderTicket(),OrderOpenPrice(),OrderOpenPrice()+Breakeven,OrderOpenPrice()+takeprofit,0,clrNONE); // For Short positions int BBM=OrderModify(OrderTicket(),OrderOpenPrice(),OrderOpenPrice()-Breakeven,OrderOpenPrice()-takeprofit,0,clrNONE);
Also,... it looks like you are still having issues with calculating the Point value of your variables... and your BreakEven() function is a bit of a mess... Try this...
//+-------------------------+ //| Break Even Function | //+-------------------------+ void BreakEven() { double MyPoint=Point; if(Digits==3 || Digits==5) MyPoint=Point*10; for(int i=OrdersTotal()-1; i>=0;i--) { if( OrderSelect(i,SELECT_BY_POS,MODE_TRADES) == true && OrderSymbol()==Symbol() && OrderMagicNumber()==Magic) { if( OrderType()==OP_BUY && Ask-OrderOpenPrice()>Breakeven) { int BBM=OrderModify(OrderTicket(),OrderOpenPrice(),OrderOpenPrice()+Breakeven,OrderOpenPrice()+takeprofit,0,clrNONE); } if( OrderType()==OP_SELL && OrderOpenPrice()-Bid>Breakeven) { int BBM=OrderModify(OrderTicket(),OrderOpenPrice(),OrderOpenPrice()-Breakeven,OrderOpenPrice()-takeprofit,0,clrNONE); } } } }
See if that works....
Don't forget to "Call" the Breakeven function....
//+------------------------------------------------------------------+ //| Expert tick function | //+------------------------------------------------------------------+ void OnTick() { BreakEven();
Hi Roberto... why have you forgotten about your "takeprofit" and Breakeven" variables...
Also,... it looks like you are still having issues with calculating the Point value of your variables... and your BreakEven() function is a bit of a mess... Try this...
See if that works....
Don't forget to "Call" the Breakeven function....
Thanks Mike for helping me. I forgot to put the fuction in OnTick, and now everything works fine.
Sure... No Problem....good luck... go for it... and don't be afraid to try things out
If I can give you some advice before you start your next MT4/MT5 project.... spend some time reading up about "How to Structure EA's" and anything on "Good Coding Practice".... there is lots of good stuff out there... As your code becomes longer and more complicated the more important it is to have it all organised. If you can get into good habits NOW... then you will save yourself a whole lot of time and heartache later on....
Sure... No Problem....good luck... go for it... and don't be afraid to try things out
If I can give you some advice before you start your next MT4/MT5 project.... spend some time reading up about "How to Structure EA's" and anything on "Good Coding Practice".... there is lots of good stuff out there... As your code becomes longer and more complicated the more important it is to have it all organised. If you can get into good habits NOW... then you will save yourself a whole lot of time and heartache later on....
thank you, I will treasure your advice
- Free trading apps
- Over 8,000 signals for copying
- Economic news for exploring financial markets
You agree to website policy and terms of use
Hi everyone, I'm a beginner and I'm trying to learn. I have a problem, in this practice expert, I insert the stop loss exactly on the minimum of the candle and I would like the RR ratio to be 1 to 1. I'm trying to calculate the difference in pips between the maximum and the minimum and then add it to my entry price, but it includes a take profit too close.