Ich würde hier an den entsprechenden Stellen mal Klammern setzen.
Code:
for (int i=0; i<=OrdersTotal()-1; i++)
if(OrderSelect(i,SELECT_BY_POS,MODE_TRADES))
if(OrderSymbol()==_Symbol)
if(OrderType()==OP_BUY && useCloseSignalChange==true && signal=="verkaufen") {bool CLOSE_OP_BUY =OrderClose(OrderTicket(),OrderLots(),Bid,10,clrNONE);}
if(OrderType()==OP_SELL && useCloseSignalChange==true && signal=="kaufen") {bool CLOSE_OP_SELL =OrderClose(OrderTicket(),OrderLots(),Ask,10,clrNONE);}
if(OrderType()==OP_BUY && useBreakEvenStop==true && OrderStopLoss()<OrderOpenPrice() && Ask>OrderOpenPrice()+BreakEvenSignal*_Point) {bool TRAIL_OP_BUY =OrderModify(OrderTicket(),OrderOpenPrice(),OrderOpenPrice()+BreakEvenStop*_Point,OrderTakeProfit(),0,clrNONE);return;}
if(OrderType()==OP_SELL && useBreakEvenStop==true && OrderStopLoss()>OrderOpenPrice() && Bid<OrderOpenPrice()+BreakEvenSignal*_Point) {bool TRAIL_OP_SELL =OrderModify(OrderTicket(),OrderOpenPrice(),OrderOpenPrice()-BreakEvenStop*_Point,OrderTakeProfit(),0,clrNONE);return;}
Da ist u.a. schlechter Programmierstil.
traderdoc