[MQL4] ObjectDelete() をループ内で実行するときには decrement にする

特定の Object をまとめて消したい時、以下のようにループで各オブジェクトを消したくなります。

しかし、以下は Object が2個以上あるとうまくいきません。

// NG
for (int i = 0; i < ObjectsTotal(EMPTY); i++) {
    string objectName = ObjectName(i);
    ObjectDelete(objectName);
}

ループの中で ObjectDelete() を実行しているため、 ObjectsTotal() の値が減っていってしまうためです。

では、先に ObjectsTotal() を変数に退避しておけばよいかというと、それもうまくいきません。ループの後半になると、既に存在しないインデックスを参照してしまうため、ObjectName() が "" （空文字列）を返すからです。

この手の、「イテレーティブなオブジェクトを参照しながら編集してはいけない」というのは、プログラマが最初の方に覚えるノウハウです。しかし、「ついやってしまう系」の代表でもあります¹。

MQL4 の場合は以下のようにインデックスを decrement にするとうまくいきます。

// OK
for (int i = ObjectsTotal(EMPTY) - 1; i >=0 ; i--) {
    string objectName = ObjectName(i);
    ObjectDelete(objectName);
}

しかし、この方法は Objects 配列の内部実装によるところが多いです。今のところ、これでうまく動いていますが、100% 確実とは言えません。

たまたまかもしれませんし、将来、動作が変わる可能性もあります。

私は、「もしObjectName() が取れなかったらエラーが出て止まる」で良いというスタンスで上記を使用しています²。

リアルタイムのトレードサイン等のように、絶対間違って欲しくない時は、先に ObjectName のリストを保存しておき、それを用いて ObjectDelete() を実行した方が良いかもしれません。

自分でループするより、まとめてオブジェクトを消すための関数、ObjectsDelteAll() が使えないか検討するのも良いかと思います。

オブジェクトの種類や名前のプレフィックスで選択的に削除も出来ます³。

Strategy of C