エキスパート: MarketPredictor - ページ 2

[Mustafa Seyyid Sahin]

Ryan L Johnson #:

私が読んだところによると、即時約定注文のすべての形式では、type_filling フィールドが設定されている 必要があるようです。

取引要求構造 - データ構造 - 定数、列挙および構造 - MQL5リファレンス - MetaTrader 5のアルゴリズム/自動取引言語に関するリファレンス

注文プロパティ - 取引定数 - 定数、列挙、および構造体 - MQL5リファレンス - MetaTrader 5のアルゴリズム/自動取引言語に関するリファレンス

ヒントをありがとうございます。コードは5日間進行中で、取引が設定されない問題は解決しました。）

[Conor Mcnamara]

新しい反復を行う必要がある

    // ボラティリティ（ATR）に基づいてアルファを調整する。
    double atr = iATR(_Symbol, PERIOD_CURRENT, period); // ATRを計算する
    if(atr > 0.0)
        alpha = atr * 0.1; // アルファ値をボラティリティに比例させる
    else
        alpha = inputAlpha; // ATRが利用できない場合、入力値にフォールバックする。

このコードではATRを正しく計算できません。

https://www.mql5.com/ja/docs/indicators/iatr

戻り値

指定したテクニカル指標のハンドルを 返します。

コードであるハンドルを返し、ATRの値は返しません。

[Sergey Panin]

1. バグ修正：

- FFTにおいて：偶数配列と奇数配列に対する再帰的FFT呼び出しは、配列サイズが2の次数でない場合、無限再帰につながる可能性がある。

配列サイズが2の次数であることを確認する必要があります。これは現在のコードではチェックされていません。

- CalculateFractalComponentFFT: 関数では、FFTを使用しますが、Nが2の次数であることはチェックしません。

また、FFTの後、最初のN/2要素のみを使用しており、これは正しいのですが、FFTコードでは結合時のインデックス付けに誤りがあります。
2.改善点:

- ExecuteTrade 関数で、PositionSelect(_Symbol) を使ってオープンポジションをチェックするのは正しくない。

なぜなら、この関数はシンボルにポジションがあれば真を返すが、現在オープンしているとは限らないからである。

すべてのポジションをループして、マジックナンバーとシンボルをチェックする方がよいでしょう。

- また、ExecuteTrade では、すでにオープンしているポジションがあるかどうかをチェックしないので、複数のポジションをオープンすることができます。

オープンするポジションを1つに限定する必要があります（または、ポジションを識別するためにマジックナンバーを使用します）。

- OptimiseParameters: 関数では、移動平均の計算を組み込みの iMA 関数で置き換えることができます。

- SimulatePrice関数では、MathRand()を使用することはモンテカルロ法には適していません。