2 questions

First, how to make sure only trades in profit (loss) are closed?

Is this correct?

for(open_trades){
if(TradeIsProfit){
Lots = TradeLots;
exitTrade();
}



Second, ... hmm, first is most important.

Use TradeProfit > 0. TradeIsProfit only indicates that the profit target is hit.