It is not a good idea to use Global Variables of the client terminal in this way. If the EA is running on more than 1 chart they will interfere with each other.
There is no check in your code to see if the trade was closed by SL or TP. The variable is only reset if the trade is closed by the EA. Check OrderCloseTime() and if it !=0, reset the variable.
To add comments, please log in or register