MQL4 アラートのタイミングを、条件が満たされたタイミングにしたい

Question

前提・実現したいこと

MQL4を用いて、エントリーポイントとなる条件を満たすとシグナル（矢印）とアラートが鳴るというインジケータを作っています。現状は、特にエラーはなく、シグナルもアラートも実現できていますが、そのどちらもタイミングが、そのローソク足が完成したタイミングで表示されてしまいます。そうではなく、条件を満たしたタイミングで、シグナルを表示し、アラートが鳴るようにしたいです。具体的には、シグナルは条件を満たしている間ずっと表示、アラートはローソク足一つに対し1回だけ発生というようにしたいです。
どこをどのように変更すれば実現できますか。
よろしくお願いします。

該当のソースコード

//+------------------------------------------------------------------+
//|                                                         test.mq4 |
//|                        Copyright 2021, MetaQuotes Software Corp. |
//|                                             https://www.mql5.com |
//+------------------------------------------------------------------+
#property copyright "Copyright 2021, MetaQuotes Software Corp."
#property link      "https://www.mql5.com"
#property version   "1.00"
#property strict
#property indicator_chart_window

#property indicator_buffers 2

#property indicator_color1 clrAntiqueWhite 
#property indicator_color2 clrAntiqueWhite
#property indicator_width1 2 
#property indicator_width2 2

double arrEntUp[], arrEntDown[];
int maxlimit = 10000;
datetime lastAlertTime;
double MathAbs;

//+------------------------------------------------------------------+
//| Custom indicator initialization function                         |
//+------------------------------------------------------------------+
int OnInit()
  {
//--- indicator buffers mapping
  IndicatorBuffers(2);
  SetIndexBuffer(0, arrEntUp);
  SetIndexBuffer(1, arrEntDown);

  SetIndexStyle(0, DRAW_ARROW, STYLE_SOLID);
  SetIndexStyle(1, DRAW_ARROW, STYLE_SOLID); 
  
  SetIndexArrow(0, 233);//上矢印 
  SetIndexArrow(1, 234);//下矢印
  
//---
   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[])
  {
//---
   double dForceNow,dMFINow,dMANow1,dMANow2,dOpen1,dOpen2,dLow1,dLow2,dHigh,dClose1,dClose2;
   
   
   for(int i=maxlimit-1; i>=0; i--){
      dForceNow = iForce(NULL,0,13,MODE_SMA,PRICE_CLOSE,i);
      dMFINow = iMFI(NULL,0,14,i);
      dMANow1 = iMA(NULL,0,20,0,MODE_SMA,PRICE_CLOSE,i);//今のma
      dMANow2 = iMA(NULL,0,20,0,MODE_SMA,PRICE_CLOSE,i+1);//1本前のma
      dOpen1 = iOpen(NULL,0,i);
      dOpen2 = iOpen(NULL,0,i+1);
      dLow1 = iLow(NULL,0,i);
      dLow2 = iLow(NULL,0,i+1);
      dHigh = iHigh(NULL,0,i+1);
      dClose1 = iClose(NULL,0,i);//今の終値(現在のレート)
      dClose2 = iClose(NULL,0,i+1);//1本前の終値
      double gapOfNow = MathAbs(dClose1 - dMANow1);//現在レートとmaの差
      double gapOfStart = MathAbs(dOpen1 - dMANow1);//始値とmaの差
      
      if(Digits==2 || Digits==3){
         gapOfNow *= 100;
         gapOfStart *= 100;
      }else if(Digits==4 || Digits ==5){
         gapOfNow *= 10000;
         gapOfStart *= 10000;
      }
      double gapOfGap = MathAbs(gapOfNow - gapOfStart);
   
    arrEntUp[i] = EMPTY_VALUE;
    arrEntDown[i] = EMPTY_VALUE;
    //  陽線　　　　　　　　　　　　　　/　forceが0以上　　　　　　/　　MFIが50以上　　　　　/　　　差の差が3pips以下　/　　1本前の足の終値がma以下　　/ 1本前の足の始値ma以下（1本前が陰線だった場合）/実体がmaまたぐ
    if(dOpen1 < dClose1 && dForceNow >= 0 && dMFINow > 50 && gapOfGap <= 3 && dMANow2 > dClose2 && dMANow2 > dOpen2 && dOpen1 < dMANow1 && dMANow1 < dHigh){
      arrEntUp[i] = Low[i];
      
      if(lastAlertTime != Time[0]){
         Alert(Symbol()+"/n"+"ハイエントリー");
         lastAlertTime = Time[0];
      }
         //   陰線　　　　　　　　　　　　　/ forceが0以下       / MFIが50以下　　　　/ 差の差が3pips以下 / 1本前の足の終値がma以上  /  1本前の足の始値がma以上（1本前が陽線だった場合）/実体がmaまたぐ
    }else if(dOpen1 > dClose1 && dForceNow <= 0 && dMFINow < 50 && gapOfGap <= 3 && dMANow2 < dClose2 && dMANow2 < dOpen2 && dLow1 < dMANow1 && dMANow1 < dOpen1){
      arrEntDown[i] = High[i];
      
      if(lastAlertTime != Time[0]){
         Alert(Symbol()+"\n"+"ローエントリー");
         lastAlertTime = Time[0];
      }
    }
       
    }
   
   
   
//--- return value of prev_calculated for next call
   return(rates_total);
  }
//+------------------------------------------------------------------+

補足情報（FW/ツールのバージョンなど）

mt4はFXCM社、PCのOSはWindows11です。
参考にしたサイト
https://blog.codinal-systems.com/alert-per-bar/

Answer 1

OnCalculateで、現在の足は[0]だと思います。

従って、
for(int i=maxlimit; i>=1; i--){ではなく
for(int i=maxlimit-1; i>=0; i--){とするべきだと思います。

Comments

[tera-ryo]

ご回答ありがとうございます。土日が明けてから確かめたいと思います。

[tera-ryo]

シグナルについては、条件を満たしているときだけ表示するようにできました！アラートですが、補足情報に記載しているサイトを参考にコーディングしたのですが、毎ローソク足が完成したタイミング（5分足だと5分ごとに）でアラートが鳴ってしまいます。条件を満たしたローソク足に、最初の1度だけシグナルとともになるようにするにはどうすればよいですか。

[yu-ima]

スタティックな変数を用意し、初期値としてfalseを指定するなり、OnInitでfalseを代入しておいて、
アラートが発生した時、その変数が、falseならアラートを発生させて、その変数をtrueにします。
次アラートが発生しても　その変数がtrueならアラート処理を行わなければよいと思います。

また、アラート条件が無くなった時、変数をfalseにすれば、　次にアラート条件が発生した時、アラートを鳴らすことが出来ます。

[tera-ryo]

ありがとうございます。すみませんが、具体的にはどこをどのように変更すればよいですか。

[yu-ima]

.
.
double MathAbs;
int Alart_flag = 0; <<<<<< 追加
.
.
int OnInit()
{
　　 Alart_flag = 0; <<<<<< 追加
　　.
　　.
　　.
if(dOpen1 < dClose1 && dForceNow >= 0 && dMFINow > 50 && gapOfGap <= 3 && dMANow2 > dClose2 && dMANow2 > dOpen2 && dOpen1 < dMANow1 && dMANow1 < dHigh){
if ( Alart_flag == 0 ) { <<<<<< 追加
Alart_flag = 1 ; <<<<<< 追加
arrEntUp[i] = Low[i];
if(lastAlertTime != Time[0]){
Alert(Symbol()+"/n"+"ハイエントリー");
lastAlertTime = Time[0];
}
} <<<<<< 追加
}else if(dOpen1 > dClose1 && dForceNow <= 0 && dMFINow < 50 && gapOfGap <= 3 && dMANow2 < dClose2 && dMANow2 < dOpen2 && dLow1 < dMANow1 && dMANow1 < dOpen1){
if ( Alart_flag == 0 ) { <<<<<< 追加
Alart_flag = 1 ; <<<<<< 追加
arrEntDown[i] = High[i];

if(lastAlertTime != Time[0]){
Alert(Symbol()+"\n"+"ローエントリー");
lastAlertTime = Time[0];
}
} <<<<<< 追加

のようにすれば、アラームは１度だけなります。

アラーム解除したい場合は、
解除条件が成立した時　Alart_flag = 0 ;を実行する条件文を書けば可能です。

[tera-ryo]

ご丁寧な対応ありがとうございました。解決できました。