こんにちは～。
MQLをマスターしたくて寝る時は加湿器を付ける（無意味）りょうです。
今回は前回のリファクタリングの続きを行います！
りょう愚かなコードを見ていただきます
不要なif文に要注意！ 深い入れ子は自分を苦しめる！
今回リファクタリングするコードはこちら↓。決済をする部分のコードです。
if( OrdersTotal() > 0 ) {
for( int i = OrdersTotal() - 1; i >= 0; i--) {
if( OrderSelect ( i, SELECT_BY_POS )) {
//ロングの時、パーフェクトオーダー終了で決済する
if( OrderType() == OP_BUY ) {
if( PerfectOrderIncrease ( smashort, smamiddle, smalong ) == false ) {
int result = OrderClose( OrderTicket(), OrderLots(), OrderClosePrice(), Slippage, clrRed );
if(result < 0) {
Print("注文に失敗しました");
} else {
bool orderSelectResult = OrderSelect (result, SELECT_BY_TICKET);
Print ("注文に成功しました。決済価格は " + (string)OrderClosePrice() + " です。");
}
}
//ショートの時、パーフェクトオーダー終了で決済する
} else if (OrderType() == OP_SELL) {
if( PerfectOrderFalling ( smashort, smamiddle, smalong ) == false ) {
int result = OrderClose( OrderTicket(), OrderLots(), OrderClosePrice(), Slippage, clrRed );
if(result < 0) {
Print("注文に失敗しました");
} else {
bool orderSelectResult = OrderSelect(result, SELECT_BY_TICKET);
Print("注文に成功しました。決済価格は " + (string)OrderClosePrice() + "です。");
}
}
}
}
}
}プログラミングスキルが高い人が見たら一目で無駄なところが多いと思うかもしれません。
にゃんぽこ下の６連続の『 } 』が気になるのう・・・
まず１行目と２行目に関してです。
１行目の『 if( OrdersTotal() > 0 )』は『１つでもエントリー状態であれば決済判定』という意味です。
２行目の『for( int i = OrdersTotal() – 1; i >= 0; i–)』は『 OrdersTotal() が０になるまでループし続ける』という意味で、つまり『 OrdersTotal() が０であれば実行されない』という意味です。
なので、１行目の『 if( OrdersTotal() > 0 ) 』は必要ないのです。
りょうつまり・・・僕は・・・
にゃんぽこそう・・・必要ないコードをせっせと考えて作っていたんだぜ
りょうガーン
続いて５，６行目です。
if( OrderType() == OP_BUY ) {
if( PerfectOrderIncrease ( smashort, smamiddle, smalong ) == false ) {
↑この２行のコードなのですが、もしif文の形式が
if( 条件１ ) {
if( 条件２ ) {
}else{
}
}
↑のようにif文の中にさらに分岐があればこの形式で良いのですが、今回の場合は
if( 条件１ ) {
if( 条件２ ) {
}
}
↑このように分岐がありません。なのでif文の中にif文をつくる必要がなく
if( 条件１ && 条件２ ) {
}
↑これでよいのです。なのでまとめます。
りょうあーそうだった・・・
ムゥ初心者はifで囲いがちにゃ
にゃんぽこほんっとにイフイフイフイフうるさいわよあーた
りょううぅ・・・すいません・・・
また、１６，１７行目の『//ショートの時、パーフェクトオーダー終了で決済する』の下にある
} else if (OrderType() == OP_SELL) {
if( PerfectOrderFalling ( smashort, smamiddle, smalong ) == false ) {
↑こちらに関しても同じ内容なのでまとめておきます。
このようにif文の中にif文を入れることを『入れ子（英語ではネスト）』と言います。
入れ子のし過ぎはコードがとても見づらくなり、のちの自分をとても苦しめます。注意しましょう！
にゃんぽこおいお前ちょっと待て！入れ子やってたな！
りょうそんな振り込め詐欺の一員みたいに言わないでください！
出し子じゃないですよ！
ムゥタイホにゃ！
りょう待ってください！僕は詐欺なんかしてません！入れ子をしたんです！
ただプログラミングが超絶下手くそなだけなんです！
DRY『重複は悪』
つづいて、それ以降のコードです↓。
int result = OrderClose( OrderTicket(), OrderLots(), OrderClosePrice(), Slippage, clrRed );
if(result < 0) {
Print("注文に失敗しました");
} else {
bool orderSelectResult = OrderSelect (result, SELECT_BY_TICKET);
Print ("注文に成功しました。決済価格は " + (string)OrderClosePrice() + " です。");
}
}
//ショートの時、パーフェクトオーダー終了で決済する
} else if (OrderType() == OP_SELL && PerfectOrderFalling ( smashort, smamiddle, smalong ) == false ) {
int result = OrderClose( OrderTicket(), OrderLots(), OrderClosePrice(), Slippage, clrRed );
if(result < 0) {
Print("注文に失敗しました");
} else {
bool orderSelectResult = OrderSelect(result, SELECT_BY_TICKET);
Print("注文に成功しました。決済価格は " + (string)OrderClosePrice() + "です。");
この中で、先ほど修正した１０行目以外は内容が完全に重複しています。
プログラミングの世界には『DRY（Don’t repeat yourself）』という言葉があります。
日本語では『重複は悪』と言います。
プログラミングでは同じ命令のコードを重ねて書くことはバグの原因となり、また人間が見直す時も非常に見づらく、百害あって一利なしなのです。
にゃんぽこ重複していたら「ダメだっ！」と思うのだ！
今回の場合は実行内容が全く一緒なので、
if( 条件１ || 条件２ ){
実行
}
↑このようなシンプルな形にできます。
//利益確定の条件（パーフェクトオーダー終了で決済）
for( int i = OrdersTotal() - 1; i >= 0; i--) {
if( OrderSelect ( i, SELECT_BY_POS )) {
if(( OrderType() == OP_BUY && PerfectOrderIncrease ( smashort, smamiddle, smalong ) == false ) //買いの場合
|| (OrderType() == OP_SELL && PerfectOrderFalling ( smashort, smamiddle, smalong ) == false )){ //売りの場合
int result = OrderClose( OrderTicket(), OrderLots(), OrderClosePrice(), Slippage, clrRed );
if(result < 0) {
Print("注文に失敗しました");
} else {
bool orderSelectResult = OrderSelect (result, SELECT_BY_TICKET);
Print ("注文に成功しました。決済価格は " + (string)OrderClosePrice() + " です。");
}
}
}
}
ここまで簡潔な形にできました！！
ムゥとてもスッキリしたにゃ～
最初と比べてかなりスッキリしましたが、もう少し簡潔に書いていきます。
仕上げ ＆ まとめ
if( 条件１ == false ){
}
↑この形は
if( !条件１ ){
}
↑このような形で書くことができます。
にゃんぽこ条件の前に『 ! 』を付けたら『falseの時』という意味になるんだよ
にゃんぽこなにもつけないと『trueの時』っていう意味だよ
//利益確定の条件（パーフェクトオーダー終了で決済）
for( int i = OrdersTotal() - 1; i >= 0; i--) {
if( OrderSelect ( i, SELECT_BY_POS )) {
if(( OrderType() == OP_BUY && !PerfectOrderIncrease ( smashort, smamiddle, smalong ) ) //買いの場合
|| (OrderType() == OP_SELL && !PerfectOrderFalling ( smashort, smamiddle, smalong ))){ //売りの場合
int result = OrderClose( OrderTicket(), OrderLots(), OrderClosePrice(), Slippage, clrRed );
if(result < 0) {
Print("注文に失敗しました");
} else {
bool orderSelectResult = OrderSelect (result, SELECT_BY_TICKET);
Print ("注文に成功しました。決済価格は " + (string)OrderClosePrice() + " です。");
}
}
}
}
↑このようになりました。
『 ! 』に関しては人によっては「『 == false 』のほうが見やすくて好き！」という人がいるかもしれません。
ムゥ自分が使いやすい方を使うとよいにゃ～
ということで、リファクタリングでかなりコードをスッキリさせることができました！
今回のまとめは
①入れ子（ネスト）を深くしすぎるのは要注意！
②DRY （Don’t repeat yourself） 重複は悪！
というところでしょうか！
僕もまだまだ未熟なのでこれからも気づかないうちにたくさんミスをしてしまうと思います・・・。
作業を進めていくとどんどん視野が狭くなっていって
『作業１』が終わる
↓
『作業２』を作り始める
↓
『作業２』の事しか考えず『作業１とまとめられる』事を考えていない
といった状況になっていってしまいます・・・。
りょうそういった悪循環にならないように気づいたらすぐに改善していけるように頑張ります～
にゃんぽこ頑張るのだ～
