That's the shortest EA I ever seen in my life
How about parameters? Shouldn't there be an external input for parameters?
Then 'cause the NonLagMas are changing color - what's use to set a tp and s/l for them? I thought that the trade should last from color change to color change. There is a neutral color also, when there should be no new trade at all but the existing trade should be closed.

In other words that's a nice try, but kind of simplistic.