Making the alert() function work | Forex Factory

Options

Permalinks
Print Thread

Similar Threads

Horizontal line alert function 14 replies

required to function call function 2 replies

function for calculating profit doesn't work... 1 reply

stop loss function and edit time function for Breakout EA 1 reply

How to install an alert function? 0 replies

Making the alert() function work

  • #1
  • First Post: Dec 26, 2010 6:29pm Dec 26, 2010 6:29pm
I'm having some problems with making the alert function work. Iwant it to only give one signal at the close of the bar. What it s doing is giving a signal every time it checks my signal function resulting in the popup box constantly appearing. This is very irritating.

Can someone please help a novice coder.

Thanks,

Mark
Mark ... Everything you have been told is a LIE
  • #2
  • Dec 26, 2010 6:40pm Dec 26, 2010 6:40pm
Here's the code snippet.

PHP Code
if (isBuy(i))
{
 Up_Arrow_Buffer[i] = Low[i] - (ArrowDistance * Point * PipFactor);
        Down_Arrow_Buffer[i] = EMPTY_VALUE;
        if (i==0 && lastSignalTime < Time[0])
        {
            Alert("BUY :", Symbol());
            lastSignalTime = Time[0];
        }
}
Mark ... Everything you have been told is a LIE
  • #3
  • Dec 26, 2010 7:01pm Dec 26, 2010 7:01pm
I haven't tested this but the code should produce only 1 alert per bar - except I don't see where you have declared the lastSignalTime?

Try this :-

Inserted Code
[color=#000000][color=#0000bb][/color][font=Courier New][color=#007700]if ([/color][color=#0000bb]isBuy[/color][color=#007700]([/color][color=#0000bb]i[/color][/font][font=Courier New][color=#007700]))
{[/color][/font][/color]
[color=#000000][font=Courier New][color=#007700] [b][color=red]static datetime lastSignalTime;[/color][/b][/color][/font][/color][color=#000000][font=Courier New][color=#007700]
 [/color][color=#0000bb]Up_Arrow_Buffer[/color][color=#007700][[/color][color=#0000bb]i[/color][color=#007700]] = [/color][color=#0000bb]Low[/color][color=#007700][[/color][color=#0000bb]i[/color][color=#007700]] - ([/color][color=#0000bb]ArrowDistance [/color][color=#007700]* [/color][color=#0000bb]Point [/color][color=#007700]* [/color][color=#0000bb]PipFactor[/color][/font][font=Courier New][color=#007700]);
        [/color][color=#0000bb]Down_Arrow_Buffer[/color][color=#007700][[/color][color=#0000bb]i[/color][color=#007700]] = [/color][color=#0000bb]EMPTY_VALUE[/color][/font][font=Courier New][color=#007700];
        if ([/color][color=#0000bb]i[/color][color=#007700]==[/color][color=#0000bb]0 [/color][color=#007700]&& [/color][color=#0000bb]lastSignalTime [/color][color=#007700]< [/color][color=#0000bb]Time[/color][color=#007700][[/color][color=#0000bb]0[/color][/font][font=Courier New][color=#007700]])
        {
            [/color][color=#0000bb]Alert[/color][color=#007700]([/color][color=#dd0000]"BUY :"[/color][color=#007700], [/color][color=#0000bb]Symbol[/color][/font][font=Courier New][color=#007700]());
            [/color][color=#0000bb]lastSignalTime [/color][color=#007700]= [/color][color=#0000bb]Time[/color][color=#007700][[/color][color=#0000bb]0[/color][/font][color=#007700][font=Courier New]];
        }
} [/font][/color][/color]
  • #4
  • Last Post: Dec 26, 2010 7:13pm Dec 26, 2010 7:13pm
I added staticto the code and that seems to have solved the problem.

Much thanks
Mark ... Everything you have been told is a LIE
0 traders viewing now