RSI no repaint and fix arrow in the canldestick

Back to topics list To post a new topic, please log in or register
avatar
86
fly7680 2016.05.11 12:08
 

Hello everyone, I would like to fix the arrow in the candle just there but the signal must not disappear if the condition disappears.

thank you

#include <stdlib.mqh>
#include <stderror.mqh>

//--- indicator settings
#property indicator_chart_window
#property indicator_buffers 1

#property indicator_type1 DRAW_ARROW
#property indicator_width1 1
#property indicator_color1 0xFFAA00
#property indicator_label1 "Buy"

//--- indicator buffers
double Buffer1[];

datetime time_alert; //used when sending alert
bool Audible_Alerts = true;
double myPoint; //initialized in OnInit

void myAlert(string type, string message)
  {
   if(type == "print")
      Print(message);
   else if(type == "error")
     {
      Print(type+" | RSInoRepaint @ "+Symbol()+","+Period()+" | "+message);
     }
   else if(type == "order")
     {
     }
   else if(type == "modify")
     {
     }
   else if(type == "indicator")
     {
      if(Audible_Alerts) Alert(type+" | RSInoRepaint @ "+Symbol()+","+Period()+" | "+message);
     }
  }

//+------------------------------------------------------------------+
//| Custom indicator initialization function                         |
//+------------------------------------------------------------------+
int OnInit()
  {   
   IndicatorBuffers(1);
   SetIndexBuffer(0, Buffer1);
   SetIndexEmptyValue(0, 0);
   SetIndexArrow(0, 242);
   //initialize myPoint
   myPoint = Point();
   if(Digits() == 5 || Digits() == 3)
     {
      myPoint *= 10;
     }
   return(INIT_SUCCEEDED);
  }

//+------------------------------------------------------------------+
//| Custom indicator iteration function                              |
//+------------------------------------------------------------------+
int OnCalculate(const int rates_total,
                const int prev_calculated,
                const datetime& time[],
                const double& open[],
                const double& high[],
                const double& low[],
                const double& close[],
                const long& tick_volume[],
                const long& volume[],
                const int& spread[])
  {
   int limit = rates_total - prev_calculated;
   //--- counting from 0 to rates_total
   ArraySetAsSeries(Buffer1, true);
   //--- initial zero
   if(prev_calculated < 1)
     {
      ArrayInitialize(Buffer1, 0);
     }
   else
      limit++;
   
   //--- main loop
   for(int i = limit-1; i >= 0; i--)
     {
      if (i >= MathMin(5000-1, rates_total-1-50)) continue; //omit some old rates to prevent "Array out of range" or slow calculation   
      //Indicator Buffer 1
      if(iRSI(NULL, PERIOD_CURRENT, 2, PRICE_CLOSE, i) > 50 + 3 * myPoint //Relative Strength Index > fixed value + fixed value
      )
        {
         Buffer1[i] = High[i]; //Set indicator value at Candlestick High
         if(i == 0 && Time[0] != time_alert) { myAlert("indicator", "Buy"); time_alert = Time[0]; } //Instant alert, only once per bar
        }
      else
        {
         Buffer1[i] = 0;
        }
     }
   return(rates_total);
  }
//+------------------------------------------------------------------+
avatar
Moderator
3427
GumRai 2016.05.11 12:53 #
 
fly7680:

Hello everyone, I would like to fix the arrow in the candle just there but the signal must not disappear if the condition disappears.

thank you

Then why do you code

      else
        {
         Buffer1[i] = 0;
        }

?


avatar
86
fly7680 2016.05.11 13:20 #
 
I am a new programmer and do not know well all the code .
How could I change that line of code ?
Also I can delay the signal a few seconds by inserting a different statement ?
avatar
Moderator
3427
GumRai 2016.05.11 14:12 #
 
fly7680:
I am a new programmer and do not know well all the code .
How could I change that line of code ?

You don't change the section of code that I asked about, you remove it.

If you don't understand that, then you are trying to work with code that is too complicated for you.

You have to be able to fully understand exactly what the code is doing.

Back to topics list  

To add comments, please log in or register