Attachments: Help Needed to Update Existing EA

Help Needed to Update Existing EA

#1
First Post: Jan 1, 2010 4:02am Jan 1, 2010 4:02am

Now
| Joined Aug 2006 | Status: 'I should have taken the money' | 654 Posts

Dear Traders,

I need some help to update the existing attached EA:

What is missing is that when the conditions are met for a signal (Fast (5) and Slow (12) EMA crossing and RSI (period 21) +/- 50) the message-box does not appear. It only plays the alert sound.
I can see at the end of the code in MetaEditor that the Message Box alert has been programmed, but for some reason the message box does not appear:

Quote

Disliked

//----
if(Alerts && entry)
{
PlaySound("stops.wav");
if (sigPrevious==1)
{
MessageBox("Entry point: buy at "+entry_point+"!!", "Entry Point", MB_OK);
}
else if (sigPrevious==2)
{
MessageBox("Entry point: sell at "+entry_point+"!!", "Entry Point", MB_OK);
}
entry=true;
}
//----
return(0);
}
//+------------------------------------------------------------------+

Is it also possible to program it that the EA also sends an e-mail or TEXT message when the conditions are met?

EA attached, thanks in advance for helping.

Regards,
Now

Attached File(s)

sidus_bago.mq4 4 KB | 217 downloads

Being right is one, taking it from the table is two.

#2
Jan 2, 2010 5:20am Jan 2, 2010 5:20am

Now
| Joined Aug 2006 | Status: 'I should have taken the money' | 654 Posts

Anyone interested?
Am offering compensation for the work.

Being right is one, taking it from the table is two.

#3
Jan 2, 2010 7:19pm Jan 2, 2010 7:19pm

Now
| Joined Aug 2006 | Status: 'I should have taken the money' | 654 Posts

I have been able to get the message-box going, more or less by myself by looking into the code of other indicators/advisors that I have.
Will have to wait for the market to open to see it working on live data because I get both a sell and buy alert on the same pair.

Incorporating an e-mail, SMS sending service would still be a great addition.

Any help is welcome.

This is the code I used for the message-box:

Quote

Disliked

if(Alerts && entry)
{
PlaySound("stops.wav");
if (sigPrevious==1)
{
Alert("Buy - ", Symbol());

}
else if (sigPrevious==2)
{
Alert("Sell - ", Symbol());

}
entry=true;
}

See updated EA attached.

Attached File(s)

sidus_bago_alert.mq4 4 KB | 220 downloads

Being right is one, taking it from the table is two.

#4
Jan 2, 2010 7:26pm Jan 2, 2010 7:26pm

CodeMeister
Joined Sep 2009 | Status: Making Code While Making Pips | 1,672 Posts

Quoting Now

Disliked

Incorporating an e-mail, SMS sending service would still be a great addition.

Any help is welcome.

Ignored

I am impressed with your efforts and your kind offer of compensation. If you get really stuck, send me a PM. The email notification is trivial. Just use SendMail in the same manner as MessageBox or Alert.

The key to email is the setup configuration. I am not going to include the details as there are plenty of threads already. One useful hint - use GMX mail if your ISP's SMTP server doesn't work.

#5
Jan 2, 2010 7:40pm Jan 2, 2010 7:40pm

Now
| Joined Aug 2006 | Status: 'I should have taken the money' | 654 Posts

Quoting CodeMeister

Disliked

I am impressed with your efforts and your kind offer of compensation. If you get really stuck, send me a PM. The email notification is trivial. Just use SendMail in the same manner as MessageBox or Alert.

The key to email is the setup configuration. I am not going to include the details as there are plenty of threads already. One useful hint - use GMX mail if your ISP's SMTP server doesn't work.

Ignored

Hello CodeMeister, thank you for your offer to help me out! Will get in touch with you when i get really stuck. I am using this EA for some time and want to automate it a bit more (message box, e-mail and SMS). The final decision to enter or not enter the market will always be based on the trader, so no blind signal following.

I found some code for the e-mail but will have to look into the setup/config as you mentioned:

Quote

Disliked

//Long
if(.....)
SendMail("....",".....");
if(.....)
SendMail("RSI Cross Long","RSI crossed long 50");

//Shrt
if(.....)
SendMail("....",".....");
if(.....)
SendMail("RSI Cross Shrt","RSI crossed shrt 50");

But that is for later, I want to get the signal message-box running good first.

I have attached a screen-shot of this "double" "buy and sell" signal.
I think the code I have is correct so will wait for market to open.
If someone can have a look in the meantime would be great!

Attached Image (click to enlarge)

Click to Enlarge

Name: double_alert.gif
Size: 50 KB

Being right is one, taking it from the table is two.

#6
Last Post: Edited 4:27pm Jan 5, 2010 3:38pm | Edited 4:27pm

Now
| Joined Aug 2006 | Status: 'I should have taken the money' | 654 Posts

I got it tested with live date in the meantime.

Still I am getting both buy/sell signal at times.
I think that it has got to do with the fact that the signal rules are not cheked only on closed candlestick, so while still open whiles the two ema's are crossing both ways, so kissing eachother it will have two signals and will also repaint the arrow.

My reason for semi-automating this EA was for my attention to be drawn on the specific pair that the EA is loaded on since I have between 15 and 20 charts active.

This EA is absolutely NOT meant as a no brainer sell/buy signal.

Concerning the above and the fact that I have more or less got the modification that I was hoping to get, I changed the text for the alert box as followed:

Quote

Disliked

if(Alerts && entry)
{
PlaySound("stops.wav");
if (sigPrevious==1)
{
Alert("Check entry conditions on: ", Symbol()," timeframe ",Period()," min");

}
else if (sigPrevious==2)
{
Alert("Check entry conditions on: ", Symbol()," timeframe ",Period()," min");

}
entry=true;
}

I have added the timeframe in case I have a chart on different timeframes for the same pair.

With "Check entry conditions on: ... " I refer to the system, my other indicators and personal trading perception, so or to buy or to sell is open for personal decision making at that moment so that it becomes a "brainer". You can go the threads of Sidus and Bagovino in this forum if you need some more info on those systems. You can also send me a PM if you like to know how I personally use these trading methods on my live account. The screenshot is actually of my live account. But I advise to go these threads first:

http://www.forexfactory.com/showthread.php?t=7603

http://www.forexfactory.com/showthread.php?t=6137

http://www.forexfactory.com/showthread.php?t=13007

See also attached chart and final EA if you want to test it..

Good luck and thanks for all assistance!

Attached Image (click to enlarge)

Click to Enlarge

Name: entry_conditions.gif
Size: 61 KB

Attached File(s)

sidus_bago_alert.mq4 4 KB | 226 downloads

Being right is one, taking it from the table is two.

Platform Tech
/
Help Needed to Update Existing EA
Reply to Thread

0 traders viewing now

Options

Similar Threads

Help Needed to Update Existing EA