Hi all,
Maybe i am out of topic but i'm sure that this is the place where i can find an answer.
I try to test Indicator Fisher with EA.
The EA is: if fisher bar2 is red, the fisher bar1 is green and the fisher bar0 becomes grater than zero open buy (immediately when the bar0 becomes greater than zero). vice versa close buy. The opposite condition for open\close sell.
I'm testing with eurusd 4 hour.

The Ea works correctly but sometimes don't close the order (for example today at 16.00 gmt time the sell order was to be closed but the EA not closed the order).
Also the strategy tester sometimes close the order in a wrong time (for example today at 16.00).

Instead I download Fisher indicator from internet and maybe there are some errors.
I don't have feeling with indicator code. Can you explain me better your indication and how to correct the indicator code?

This is the analisys part of the EA.
If the first if is true the EA set the variable TipoOrdDaEseg=0 (Open Buy) and call BuySeel() function that open buy.
Instead If the second if is true the EA set the variable TipoOrdDaEseg=1 (Open Sell) and call BuySeel() function that open sell.

In the first time, sorry for my bad english. I made a simple EA with this indicator too and I saw one thing. When you make a Visual Backtest you can see in the end how the indicator draw and if you add the indicator new and compare you can see the diference.

2) Where is V[0] = a*newValue +(1-a) V[1]??? ---> I cannot find this in the source code that was uploaded.

Original Code. Where does Value1 come from? The previous bar's Value. Where does Fish1 come from? The previous ExtBuffer0. But since it is counting up, previous means newer. Using newer bar's data for creating older ones is the definition of repainting. Can't be used for trading, just like you can't open a order at a previous bar's price.

Original Code. Where does Value1 come from? The previous bar's Value. Where does Fish1 come from? The previous ExtBuffer0. But since it is counting up, previous means newer. Using newer bar's data for creating older ones is the definition of repainting. Can't be used for trading, just like you can't open a order at a previous bar's price.

Original Code. Where does Value1 come from? The previous bar's Value. Where does Fish1 come from? The previous ExtBuffer0. But since it is counting up, previous means newer. Using newer bar's data for creating older ones is the definition of repainting. Can't be used for trading, just like you can't open a order at a previous bar's price.

Count down, create an Buffer and remember Value1 for next bar

Don't magnify roundoff

If you introduce new array Buffer then who assign the value for Buffer initially. Nothing so it will have all zeros, right?

Then,

Value1 = Buffer[i+1];

is not necessary because all the value of Buffer[i+1] is zero when i = 0, 1, 2, 3, 4 ...

I don't understand what you are trying to do. If you want to do previous one, then it should be i-1, isn't it?

Please DO correct me if I am wrong. This "FOR-LOOP" is incrementing not decrementing.