Email alert needed for this sweet indicator?

Just wondering if anyone smart enough can add a simple alert to send email only once for each signal generated. I have worked on this for 4 days straight and am about to call it quits.

Can't get around the fact that each tick sends me emails, resulting in more than I would like. It shows in the code that I have an alert() set instead of sendmail(), this is just for testing purposes. Can someone help please. Don't know if I am even close with how it should be implimented.

Just want 1 email sent per signal if conditions are met.

Can someone give it a try, anyone?

Source code below.

Does this indicator work otherwise? I just tried it in my demo terminal and it didn't show up as a custom..

Hi england33, it should work, does on my demo.

Here is a pic of the indy. These are not signals to put a trade on, they are warnings that potential trades are setting up for me. This alert will save a lot of time sitting in front of the computer screen. Hope someone can help.

Does not seem like it would be that difficult to do, can't get around getting the alert to stop after a signal. It alerts every tick after the signal is given.

Any ideas?

What a pain in the ass trying to configure what should be a simple alert. I thought about trying to wait for a new bar to form before shooting the alert off again but no luck yet.

Nope still learning mql4 myself and don't have the time to dedicate to it but I would like to help if I could. The only thing I can think of is in your code your alert is in the wrong place.

//+------------------------------------------------------------------+
//| Strength Weakness Alerts.mq4 |
//| Copyright 2009, MetaQuotes Software Corp. |
//| http://www.metaquotes.net |
//+------------------------------------------------------------------+
#property copyright ""
#property link ""

#property indicator_chart_window
#property indicator_buffers 4
#property indicator_color1 Blue
#property indicator_color2 Pink
#property indicator_color3 LimeGreen
#property indicator_color4 Red

extern bool showVSA = true;
extern bool showStochBB = true;
extern int NumberOfBars = 2000;
double Offset;
double Offset2;
extern int SignalSize = 0.5;
int ExtCountedBars=0;
double MA[];
double Spread[];
double Range;
double SprMovAvg = 4;
int c=0;
string spreadtype, spreadtype1, volumetype, volumetype1 ,updnbar, updnbar1, closebar, closebar1, VSAType, newVSA;

//extern int SpreadPeriod=30, VolumePeriod=30;

//extern double MAVol = 100;
int MAPeriod = 4;

double Strength[],Weakness[],S[],W[];
//+------------------------------------------------------------------+
//| Custom indicator initialization function |
//+------------------------------------------------------------------+
int init()
{
//---- indicators


IndicatorShortName("Strength Weakness Alerts");

SetIndexBuffer(0,Strength);
SetIndexStyle(0,DRAW_ARROW,0,SignalSize);
SetIndexArrow(0, 120);
SetIndexLabel(0,"Strength");

SetIndexBuffer(1,Weakness);
SetIndexStyle(1,DRAW_ARROW,0,SignalSize);
SetIndexArrow(1, 120);
SetIndexLabel(1,"Weakness");

SetIndexBuffer(2,S);
SetIndexStyle(2,DRAW_ARROW,0,SignalSize);
SetIndexArrow(2,251);
SetIndexLabel(2,"Demand");

SetIndexBuffer(3,W);
SetIndexStyle(3,DRAW_ARROW,0,SignalSize);
SetIndexArrow(3, 251);
SetIndexLabel(3,"Supply");

//----
return(0);
}
//+------------------------------------------------------------------+
//| Custom indicator deinitialization function |
//+------------------------------------------------------------------+
int deinit()
{
//----

//----
return(0);
}
//+------------------------------------------------------------------+
//| Custom indicator iteration function |
//+------------------------------------------------------------------+
int start()
{

//----
switch(Period()){
case PERIOD_M1: Offset=3;Offset2=1;break;
case PERIOD_M5: Offset=4;Offset2=2;break;
case PERIOD_M15: Offset=8;Offset2=4;break;
case PERIOD_M30: Offset=12;Offset2=6;break;
case PERIOD_H1: Offset=16;Offset2=8;break;
case PERIOD_H4: Offset=26;Offset2=12;break;
case PERIOD_D1: Offset=80;Offset2=40;break;
case PERIOD_W1: Offset=120;Offset2=50;break;
case PERIOD_MN1: Offset=180;Offset2=70;break;
}
Offset = Offset*Point;
Offset2 = Offset2*Point;

for(c=0;c<=NumberOfBars;c++)
{
//--------
if(Close[c+1]>Close[c+2])
{
updnbar="UP BAR";
}
if(Close[c+1]<Close[c+2])
{
updnbar="DOWN BAR";
}
//--------
if(Close[c+1]<=(((High[c+1]-Low[c+1])*0.30)+Low[c+1]))
{
closebar="DOWN CLOSE";
}
else if(Close[c+1]>=(High[c+1]-((High[c+1]-Low[c+1])*0.30)))
{
closebar="UP CLOSE";
}
else
{
closebar="MID CLOSE";
}

//-------------Copy------------------------------------------+

if(Close[c+2]>Close[c+3])
{
updnbar1="UP BAR";
}
if(Close[c+2]<Close[c+3])
{
updnbar1="DOWN BAR";
}
//--------
if(Close[c+2]<=(((High[c+2]-Low[c+2])*0.30)+Low[c+2]))
{
closebar1="DOWN CLOSE";
}
else if(Close[c+2]>=(High[c+2]-((High[c+2]-Low[c+2])*0.30)))
{
closebar1="UP CLOSE";
}
else
{
closebar1="MID CLOSE";
}

//--------Calculation------------------------------------------+

double main = iStochastic(NULL,0,14,3,3,MODE_SMA,0,MODE_MAIN,c+1);
double signal = iStochastic(NULL,0,14,3,3,MODE_SMA,0,MODE_SIGNAL,c+1);
double upper = iBands(NULL,0,20,2,2,PRICE_HIGH,MODE_UPPER,c+1);
double lower = iBands(NULL,0,20,2,2,PRICE_LOW,MODE_LOWER,c+1);
double point1 = iMA(NULL,0,50,0,MODE_EMA,PRICE_CLOSE,c+1);
double point2 = iMA(NULL,0,50,0,MODE_EMA,PRICE_CLOSE,c+3);
double point3 = iMA(NULL,0,50,0,MODE_EMA,PRICE_CLOSE,c+6);
double main2 = iStochastic(NULL,0,5,3,3,MODE_SMA,0,MODE_MAIN,c+1);
double signal2 = iStochastic(NULL,0,5,3,3,MODE_SMA,0,MODE_SIGNAL,c+1);

int bar=Bars;

//if(newbar==1)

if(showVSA==true){
//-------------Stoch BollingerBand condtion-------------------------
if((Low[c+2]<= Open[c+1] && Open[c+2]<=Close[c+1]
&& main > signal && (main <=30) && (Open[c+1]<=lower))

//--------Strength Weakness condition--------------------------------
|| (updnbar1=="DOWN BAR" && closebar1=="DOWN CLOSE"
|| closebar1=="MID CLOSE" && (updnbar=="UP BAR" && closebar=="UP CLOSE")
&& Low[c+1]< Open[c] && Open[c+1]<Close[c] && (main2 <=30)))
{
Strength[c+1] = Low[c+1] - Offset;
//newbar = 1;
}

else
//-------------Stoch Bollinger Bands condition----------------------------
if((High[c+2]>=Open[c+1] && Open[c+2]>=Close[c+1]
&& main < signal && (main >=70) && (Open[c+1]>=upper))

//-------------Strength Weakness condition--------------------------------
|| (updnbar1=="UP BAR" && closebar1=="UP CLOSE"
|| closebar1=="MID CLOSE" && (updnbar=="DOWN BAR" && closebar=="DOWN CLOSE")
&& High[c+1]>Open[c] && Open[c+1]>Close[c] && (main2 >=70)))
{
Weakness[c+1] = High[c+1] + Offset;
//newbar = 1;
}
}

//--------Oversold/Overbought peircing Bollinger Band---------------------------------
if(showStochBB==true){
double ma = iMA(NULL,0,5,-1,MODE_EMA,PRICE_CLOSE,c+1);
double ma2 = iMA(NULL,0,5,-1,MODE_EMA,PRICE_CLOSE,c+2);


if((main <=20 && Low[c+1]<lower && Low[c+2]<Low[c+1] && ma>ma2)
|| (main <=20 && point1>point2 && point2>point3 && updnbar=="UP BAR" && (closebar=="MID CLOSE" || closebar=="UP CLOSE")))
{
S[c+1] = Low[c+1] - Offset2;
//newbar = 1;
}

else

if((main >=80 && High[c+1]>upper && High[c+2]>High[c+1] && ma<ma2)
|| (main >=80 && point1<point2 && point2<point3 && updnbar=="DOWN BAR" && (closebar=="MID CLOSE" || closebar=="DOWN CLOSE")))
{
W[c+1] = High[c+1] + Offset2;
//newbar = 1;
}
}
}
//----
return(0);
}
//+------------------------------------------------------------------+



//--------
/*string sp(int s)
{
int spread,avgspread,cnt,collectspread;
for(int i=SpreadPeriod;i>=1;i--)
{
spread=(High[s+i]-Low[s+i])/Point;
collectspread+=spread;
}
avgspread=collectspread/(SpreadPeriod-1);
double curspread=(High[s+1]-Low[s+1])/Point; //Alert(avgspread);
//double HMtimespread=curspread/avgspread;

double narrowspread= avgspread* 0.8;
double widespread= avgspread* 1.8;
double ultraspread= avgspread* 2.0;

if(curspread>=widespread && curspread<ultraspread)
{
spreadtype="WIDE SPREAD";
}
else if(curspread>=ultraspread)
{
spreadtype="ULTRAWIDE SPREAD";
}
else if(curspread<=narrowspread)
{
spreadtype="NARROW SPREAD";
}
else spreadtype="AVERAGE SPREAD";
return(spreadtype);
}
//--------
string vt(int s)
{
double vol,collectvol,avgvol;
for(int i=VolumePeriod;i>=1;i--)
{
vol=Volume[s+i];
collectvol+=vol;
}
avgvol=collectvol/(VolumePeriod-1);

double lowvol= avgvol*0.7;
double highvol= avgvol*1.5;
double ultravol= avgvol*2.0;

if(Volume[s+2]>=highvol && Volume[s+2]<ultravol)
{
volumetype="HIGH VOLUME";
}
else
if(Volume[s+2]>=ultravol)
{
volumetype="ULTRA HIGH VOLUME";
}
else
if(Volume[s+2]<=lowvol)
{
volumetype="LOW VOLUME";
}
else volumetype="AVERAGE VOLUME";

return(volumetype);
}*/

//------------Second Version

/*if(updnbar1=="DOWN BAR" && closebar1=="DOWN CLOSE" || closebar1=="MID CLOSE" && (updnbar=="UP BAR" && closebar=="UP CLOSE")
&& Low[c+3]>=Low[c+1] || High[c+3]<=High[c+1]
&& main > signal && (main >10 && main <80))
{
Strength[c+1] = Low[c+1] - Offset;
}

else
if(updnbar1=="UP BAR" && closebar1=="UP CLOSE" || closebar1=="MID CLOSE" && (updnbar=="DOWN BAR" && closebar=="DOWN CLOSE")
&& High[c+3]<=High[c+1] || Low[c+3]<=Low[c+1]
&& main < signal && (main <=90 && main >20))
{
Weakness[c+1] = High[c+1] + Offset;
}*/

will fix later today if someone doesn't first.....h

ya, that would probably help, eh! Thanks england33!

That would be awesome hayseed! I haven't quit working on this problem for 4 days, on my 5th day now, can't get around this every tick thing.

Maybe I need to have a variable that is reset after a new bar has formed so it doesn't keep alerting me more than once per signal. Just not sure how to put it into the code. Could be totally off though.

So far I am liking the conditions that I programmed this indy to find. Could maybe use a bit more tweaking. Like I say though, they are not a automatic buy and sell, just alerting to possible trade setups. Will save tons of hours sitting in front of chart.

hey scottstone.... added the alert example for strength and weakness at the beginning of the start function....... wasn't sure just what all you needed.....

you'll probably want to edit the alert and email message to your liking.....h

just been testing want I come up with seems to be working so far. I will take a look at yours and see how you went about it. I don't know if the way I did it is as good as yours but so far so good. Thanks for your effort hayseed! Thanks for helping someone out when you didn't have to.

It came to me earlier when I attempted to help someone else to code an alert that the idea just popped in my head. Funny how things happen.

The good thing about it is I know exactly what I need to do if I ever want to add an alert to anything else.

Thanks for your help!

Here is my version of the alert.