Ah sorry, I didn't grasp the fact that you were bothered by profitable ones. I thought you had issues because I read that you didn't have all your unprofitable trades closed. In this case, I wonder whether your second following line is the cause of ...

Close Only Unprofitable positions

Another point about the definition of the profit of a trade : The real profit of a trade is double real_profit_value = OrderProfit()-OrderCommission()-OrderSwap(); If you have 50$ of OrderProfit() but 20$ of OrderSwap() and 35$ of OrderCommission() ...

Close Only Unprofitable positions

Hello ndasilva, I would suggest you to use a buffer variable in a first place. For example, like that : int total_nb_orders = OrdersTotal(); and place total_nb_orders in your for statement. Another thing I would do is to use the real number of ...

Close Only Unprofitable positions