need help figuring something out.

so i have made an EA for a System i have been working on for a while.

the system consists of three different indicators, which you can see in the picture as three seperate lines of dot.

the way the system works is when you get a candle where all three indicators are the same colour (Red or Blue) and position is opened.

my problem is this:

the way i currently have my EA programmed is when it opens a position( the green line on the left side that says "buy" under it) and then hits the Take Profit(the red line with the star above it and "Takeprofit" below it) if at the time of the Take profit being hit all three indicators are still the same colour, a new position will be opened right away.

this would be at the same red line as the Takeprofit in the picture i posted.

what i want to happen is this:

when the TP is hit, i dont want anything to happen until one of these two things happens.

1) one or two of the indicators turns the opposite color which is red in the example, and then turns back to blue so that all three indicators are all blue. when all three are blue again a position is opened again. this happening is indicated in my picutre i posted by having the yellow box around it and it says "Reset area" underneath it.

or all three indicators turn the opposite colour of the first position.



how can i do this?

string SignalTaken = "None";

start()

if(Line1 = blue && Line2 = blue && Line3 = blue && SignalTaken = "None") {
SignalTaken = "SignalTaken"
OrderSend()...
}

if(Line1 = red || line2 = red || line3 = red) SignalTaken = "None";

//Do reverse for sell

Hey Ronald Raygun, thank you for the reply, i really appreciate it.

i just implemented what you recommended, and i think i did it correctly, but now the EA isn't making any trades?

and it is only supposed to trade at the beginning of a new bar, but right in the middle i got an error 129, which is invalid price, but i am just buying at bid and ask.

and i let it keep running and now it wont even give an error.

if anyone wants to fiddle around with it here it is:

it is pretty simple.

The whole concept of the code is to set a variable like an on/off switch.

When all the colors are the same the switch is on
When the colors are different the switch is off

That code I put checks to see if the switch is on/off and trades with that additional filter.

right, i realize that.

there must just be somewhere i screwed up.


should there be a Signal taken variable for longs, and a seperate one for shorts?