You can try adding the alerts yourself using this simple tutorial:
https://www.earnforex.com/blog/how-to-add-alerts-to-metatrader-indicators/
You do not need to learn coding to do it.
https://www.earnforex.com/blog/how-to-add-alerts-to-metatrader-indicators/
You do not need to learn coding to do it.
Great! It means that you know some coding basics at least. Then you will have no problem fixing the indicator.
To prevent alerts from repeating indefinitely, you can use something like:
to remember the last time alert triggered and add it to the condition check like this:
To prevent alerts from repeating indefinitely, you can use something like:
MQL4:
LastAlertTime = Time[0];
MQL4:
if ((val1[0]!=EMPTY_VALUE) && (val1[0]!=0) && (SoundBuy) && (LastAlertTime != Time[0]))
I comment there in coding. It is giving repeated alert and current one is no alert at all. I tried all possible ways. No luck. Can you pls make it work and post the file...
Sorry, I am not here to code for you. You can apply the suggested changes if you want. If the current code does not give alerts at all, then you need remove the current alert code and to uncomment the lines that produced the alerts. Apply the LastAlertTime check to that code to prevent multiple alert calls of the same signal.
That's a not a runtime error, that's a compilation error. To fix it, you need to declare LastAlertTime as a global variable in your code. You can do it like this, just below your line 25:
You also need to add:
to each of the alert-calling scopes, otherwise it will not work.
MQL4:
double b4plusdi,nowplusdi,b4minusdi,nowminusdi; datetime LastAlertTime = D'';
You also need to add:
MQL4:
LastAlertTime = Time[0];
Your conditions are checking whether the last alert time is not on the current bar, but your code never assigns the last alert time to any bar's time.
As I have said, you need to add this line to do that:
In your current code, I would add it after line 98 and line 90.
As I have said, you need to add this line to do that:
MQL4:
LastAlertTime = Time[0];
That's because you have put your arrow-setting code into dependence from the last alert time for unknown reason. Like on the line 75 for example.
I have provided rather clear instructions for adding alerts to the file attached in your original post. Looking at your latest file, I am no longer sure what you are trying to achieve here.
Similar threads
