Would anybody be knowledg-able to fix this?

[Brian Lillard]

Honestly I doubt anybody can find the loogical flaw in this source code

[Paul Anscombe]

Brian Lillard:

Honestly I doubt anybody can find the loogical flaw in this source code

Is it a guessing game?  

post code using the source button </>   not pictures

define the problem

[William Roeder]

1. Why did you post your MT4 question in the MT5 Indicators section instead of the MQL4 section, (bottom of the Root page)?
             General rules and best pratices of the Forum. - General - MQL5 programming forum? (2017)
   Next time, post in the correct place. The moderators will likely move this thread there soon.

2. Don't post pictures of code, they are generally too hard to read.

   Please edit your (original) post and use the CODE button (or Alt+S)! (For large amounts of code, attach it.)
         General rules and best pratices of the Forum. - General - MQL5 programming forum #25 (2019)
             Messages Editor
         Forum rules and recommendations - General - MQL5 programming forum (2023)

3. multiplier = 1 / _Point;

4. Brian Lillard: Honestly I doubt anybody can find the loogical flaw in this source code

   Do not use series index in object names, as they are not unique. As soon as a new bar starts, you will be trying to create a new name (e.g., “name0”), same, existing, previous, name (e.g., “name0” now on bar one.)

   Use time (as int) or a non-series index:

   #define  SERIES(I)   (Bars - 1 - I) // As-series to non-series or back.

[Brian Lillard]

If the high to open is larger than the open to the low then the label above candle is bigger than the label below vice versa

Not saying it should run in tester yet, or that it doesn't format the price to pips accurately, it's just font sizes 6 and 4 init'd.

The integer was double

[Yohana Parmi]

Brian Lillard:

Honestly I doubt anybody can find the loogical flaw in this source code

Hi,

I just read the picture.

The logic that seems to need to be refined is what I found in the process load.
- Every time the price moves 1 tick, you force it to reprocess 2500 bars.

We can imagine if the price tick moves quickly during an economic news event.
Maybe the processing load would be lighter if the process was done once for each new bar.

Regards.

[Brian Lillard]

Try the code out for yourself