オーダーデリートによる2重の処理

Question

前提・実現したいこと

現在、FXトレーダー実践プログラミング　現代の錬金術シリーズという本で勉強しておりその中でちょっとわからない処理があり教えてほしい。

発生している問題・エラーメッセージ

下記のコードは本からそのまま書いたものです。
僕の中では、for文でオーダーの選択を行いその後オーダーデリート関数とチケット番号を取得する関数で待機注文をキャンセル。
そのあとif文で成行買いでも成行売りでもなければ再びオーダーデリート関数とチケット番号を取得する関数で待機注文をキャンセル
するという解釈になっているのですがどうして2回のデリート処理が必要なのでしょうか？

該当のソースコード

for(int i=0;i<OrdersTotal();i++)
{
if(OrderSelect(i,SELECT_BY_POS)==false)break;
if(OrderSymbol()!=Symbol() || OrderMagicNumber()!=magic)continue;
OrderDelete(OrderTicket());　　　←ここ
if(OrderType()!=OP_BUY&&OrderType()!=OP_SELL)
}
OrderDelete(OrderTicket());　　　←ここ
break;
}
}

試したこと

ここに問題に対して試したことを記載してください。

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

ここにより詳細な情報を記載してください。

Answer 1

mahさんの言うとおり誤植だと思います、１つ目のOrderDeleteは不要ですね。
ちなみに、このfor文の記述では正常に動作しないケースがあるので、
OrderSelect()に関係するfor文は昇順ではなく降順にした方が良いと思います。
http://metatradermania.blog24.fc2.com/blog-entry-56.html

Comments

[nisei2]

for文は昇順、降順を気にした方がいいんですね、ありがとうございます！

Answer 2

Google booksでプレビューが見れたので確認してみましたが、誤植ではないでしょうか。
直前の説明に書いてあるとおり、OP_BUYとOP_SELL以外でOrderDeleteしたかったのだと思います。

Comments

[nisei2]

そうだったのですか、これですっきりしました。ありがとうございます。