パーフェクトオーダーとMACDを組み合わせたインジケーターのサンプルコードをそのままコピペしてコンパイルすると、そのままではエラーになります。原因は、下降シグナル側の条件式が上昇シグナル側のコピペのまま直し忘れられていて、しかもif文の括弧の数が合っていないためです。この記事では、上昇パーフェクトオーダー時にMACDがゴールデンクロスしたら上矢印、下降パーフェクトオーダー時にMACDがデッドクロスしたら下矢印を表示するインジケーターを、実際にコンパイルして使える状態まで解説します。
パーフェクトオーダーとMACDを組み合わせたインジケーター
パーフェクトオーダーとMACDを組み合わせたインジケーターを作成したいと思います。
- 上昇パーフェクトオーダーでMACDがゴールデンクロスしている時に上矢印
- 下降パーフェクトオーダーでMACDがデッドクロスしている時に下矢印
メタエディタ(MetaEditor)を立ち上げる
メタエディタ(MetaEditor)を立ち上げましょう。操作は以下に記載しています。

今回は名前を「PerfectOrder-MACD」で作成します。
プロパティを記述する
プロパティを記述しましょう。今回は、
- 上矢印
- 下矢印
の2つになります。
#property indicator_buffers 2//バッファ2つ
バッファを定義する
バッファを定義しましょう。上矢印と下矢印用を作ります。
double UP_Arrow[],DOWN_Arrow[];//上矢印と下矢印
パラメーターを記述する
パラメーターを記述しましょう。移動平均線の短期・中期・長期それぞれの期間と、MACDの期間を指定します。
input int MA_Short_Period = 25;//短期移動平均線の期間 input int MA_Middle_Period = 50;//中期移動平均線の期間 input int MA_Long_Period = 200;//長期移動平均線の期間 input int fast_ema_period = 12;//MACDのfast_emaの期間 input int slow_ema_period = 26;//MACDのslow_emaの期間 input int signal_period = 9;//MACDのSignalの期間
OnInit()関数を記述する
OnInit()関数を記述しましょう。インデックスに紐付け、矢印の見た目を設定します。
SetIndexBuffer(0, UP_Arrow);//インデックス0にUP_Arrowを格納 SetIndexBuffer(1, DOWN_Arrow);//インデックス1にDOWN_Arrowを格納 SetIndexStyle(0,DRAW_ARROW,STYLE_SOLID,2,Red);//インデックス0を上矢印設定 SetIndexStyle(1,DRAW_ARROW,STYLE_SOLID,2,Yellow);//インデックス1を下矢印設定 SetIndexArrow(0,233);//矢印の表示設定 SetIndexArrow(1,234);//矢印の表示設定
OnCalculate()関数を記述する
OnCalculate()関数を記述しましょう。まず変数と、未計算バー数を用意します。
double MA_Short,MA_Middle,MA_Long;//MA変数 double MACDMain,MACDSignal;//MACD変数 double MACDMainPre,MACDSignalPre;//MACD過去変数 int limit = Bars-IndicatorCounted();//未だ計算されていないバー数(チャート上のバー数 - 計算済みバー数)
ループの中で、それぞれの値を格納します。
MA_Short = iMA(_Symbol,_Period,MA_Short_Period,0,MODE_SMA,PRICE_CLOSE,i);//MA_Short MA_Middle = iMA(_Symbol,_Period,MA_Middle_Period,0,MODE_SMA,PRICE_CLOSE,i);//MA_Middle MA_Long = iMA(_Symbol,_Period,MA_Long_Period,0,MODE_SMA,PRICE_CLOSE,i);//MA_Long MACDMain = iMACD(_Symbol,_Period, fast_ema_period, slow_ema_period, signal_period, PRICE_CLOSE, MODE_MAIN, i);//MACDMain MACDSignal = iMACD(_Symbol,_Period, fast_ema_period, slow_ema_period, signal_period, PRICE_CLOSE, MODE_SIGNAL, i);//MACDSignal MACDMainPre = iMACD(_Symbol, _Period, fast_ema_period, slow_ema_period, signal_period, PRICE_CLOSE, MODE_MAIN, i+1);//MACDMain過去 MACDSignalPre = iMACD(_Symbol, _Period, fast_ema_period, slow_ema_period, signal_period, PRICE_CLOSE, MODE_SIGNAL, i+1);//MACDSignal過去 UP_Arrow[i] = EMPTY_VALUE;//配列を空にする DOWN_Arrow[i] = EMPTY_VALUE;//配列を空にする
上昇パーフェクトオーダー時にMACDがゴールデンクロスしていたら上矢印を表示します。
if(MA_Short > MA_Middle && MA_Middle > MA_Long && Close[i] > MA_Short && MACDMain > MACDSignal && MACDMainPre < MACDSignalPre)
{
UP_Arrow[i] = Low[i] - 20 * Point;//上矢印表示
}
下矢印が表示されない問題を修正する
この記事の元になったコードでは、下降シグナルの条件式が次のようになっていました。
if((MA_Short > MA_Middle && MA_Middle > MA_Long && Close[i] > MA_Short && MACDMain > MACDSignal && MACDMainPre < MACDSignalPre) //下降パーフェクトオーダー時にMACDがデッドクロス
{
DOWN_Arrow[i] = High[i] + 20 * Point;//下矢印表示
}
コメントは「下降パーフェクトオーダー時にMACDがデッドクロス」となっていますが、条件式そのものは1つ上の上昇シグナルの条件式(MA_Short > MA_Middle、MACDMain > MACDSignalなど、すべて「上向き」の不等号)をそのままコピーしたものです。上昇の条件をコピーして下降用に直し忘れているため、このままでは下矢印は上矢印と同じタイミングでしか出ず、下降シグナルとして機能しません。
さらに、if((MA_Short ...のように開き括弧が2つあるのに対し、条件式の末尾に閉じ括弧が1つしかなく、if文の括弧の数が合っていません。この状態のままではMetaEditorでコンパイルするとエラーになります。
下降パーフェクトオーダーは「短期MAが中期MAより下、中期MAが長期MAより下、終値が短期MAより下」、MACDのデッドクロスは「MACDMainがMACDSignalより下で、1本前はMACDMainの方が上だった」という、上昇側とは逆向きの不等号で判定します。正しくは次のようになります。
if(MA_Short < MA_Middle && MA_Middle < MA_Long && Close[i] < MA_Short && MACDMain < MACDSignal && MACDMainPre > MACDSignalPre)
{
DOWN_Arrow[i] = High[i] + 20 * Point;//下矢印表示
}
不等号をすべて反転させ、括弧も開き1つ・閉じ1つに揃えています。これで上矢印・下矢印の両方が意図通りのタイミングで表示されます。
コンパイルする
コンパイルしてシグナルが表示されていたら完了です。

ただ移動平均線の角度などは決めていないので角度などを組み合わせるとまたさらに精度が良くなります。
ソースコード全体
ここまでの修正をすべて反映した、PerfectOrder-MACDのソースコード全体は以下に記載しています。
//+------------------------------------------------------------------+
//| PerfectOrder-MACD.mq4 |
//| Copyright 2020, FX-EA System Project Creator |
//| https://creator.fx-ea-system-project.com/ |
//+------------------------------------------------------------------+
#property copyright "Copyright 2020, FX-EA System Project Creator"
#property link "https://creator.fx-ea-system-project.com/"
#property version "1.01"
#property strict
/*プロパティ*/
#property indicator_chart_window
#property indicator_buffers 2//バッファ2つ
/*バッファ*/
double UP_Arrow[],DOWN_Arrow[];//上矢印と下矢印
/*パラメーター*/
input int MA_Short_Period = 25;//短期移動平均線の期間
input int MA_Middle_Period = 50;//中期移動平均線の期間
input int MA_Long_Period = 200;//長期移動平均線の期間
input int fast_ema_period = 12;//MACDのfast_emaの期間
input int slow_ema_period = 26;//MACDのslow_emaの期間
input int signal_period = 9;//MACDのSignalの期間
//+------------------------------------------------------------------+
//| Custom indicator initialization function |
//+------------------------------------------------------------------+
int OnInit()
{
//--- indicator buffers mapping
SetIndexBuffer(0, UP_Arrow);//インデックス0にUP_Arrowを格納
SetIndexBuffer(1, DOWN_Arrow);//インデックス1にDOWN_Arrowを格納
SetIndexStyle(0,DRAW_ARROW,STYLE_SOLID,2,Red);//インデックス0を上矢印設定
SetIndexStyle(1,DRAW_ARROW,STYLE_SOLID,2,Yellow);//インデックス1を下矢印設定
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 MA_Short,MA_Middle,MA_Long;//MA変数
double MACDMain,MACDSignal;//MACD変数
double MACDMainPre,MACDSignalPre;//MACD過去変数
int limit = Bars-IndicatorCounted();//未だ計算されていないバー数(チャート上のバー数 - 計算済みバー数)
for(int i = limit-1; i >= 0; i--)
{
MA_Short = iMA(_Symbol,_Period,MA_Short_Period,0,MODE_SMA,PRICE_CLOSE,i);//MA_Short
MA_Middle = iMA(_Symbol,_Period,MA_Middle_Period,0,MODE_SMA,PRICE_CLOSE,i);//MA_Middle
MA_Long = iMA(_Symbol,_Period,MA_Long_Period,0,MODE_SMA,PRICE_CLOSE,i);//MA_Long
MACDMain = iMACD(_Symbol,_Period, fast_ema_period, slow_ema_period, signal_period, PRICE_CLOSE, MODE_MAIN, i);//MACDMain
MACDSignal = iMACD(_Symbol,_Period, fast_ema_period, slow_ema_period, signal_period, PRICE_CLOSE, MODE_SIGNAL, i);//MACDSignal
MACDMainPre = iMACD(_Symbol, _Period, fast_ema_period, slow_ema_period, signal_period, PRICE_CLOSE, MODE_MAIN, i+1);//MACDMain過去
MACDSignalPre = iMACD(_Symbol, _Period, fast_ema_period, slow_ema_period, signal_period, PRICE_CLOSE, MODE_SIGNAL, i+1);//MACDSignal過去
UP_Arrow[i] = EMPTY_VALUE;//配列を空にする
DOWN_Arrow[i] = EMPTY_VALUE;//配列を空にする
if(MA_Short > MA_Middle && MA_Middle > MA_Long && Close[i] > MA_Short && MACDMain > MACDSignal && MACDMainPre < MACDSignalPre)//上昇パーフェクトオーダー時にMACDがゴールデンクロス
{
UP_Arrow[i] = Low[i] - 20 * Point;//上矢印表示
}
if(MA_Short < MA_Middle && MA_Middle < MA_Long && Close[i] < MA_Short && MACDMain < MACDSignal && MACDMainPre > MACDSignalPre)//下降パーフェクトオーダー時にMACDがデッドクロス
{
DOWN_Arrow[i] = High[i] + 20 * Point;//下矢印表示
}
}
//--- return value of prev_calculated for next call
return(rates_total);
}
//+------------------------------------------------------------------+
まとめ
知識を応用するだけでパーフェクトオーダーとMACDを組み合わせたインジケーターを作ることができましたね。下降シグナルの条件式は上昇シグナルの単純なコピペでは機能しないため、不等号の向きをすべて反転させる点が今回のポイントです。
これぐらいできると基本的なインジケーターはできてしまいます。
コメント