Multi Currency Signal EA code error

To add comments, please log in or register
Jim Contant
318
Jim Contant  

I have learned enough programming skills to build my own working EA'S. With this one though I need a little help.

The way it works is, the signal is based off of the alignment of 3 different currency pairs and the RSI indicator on

3 timeframes of each. Also there is supposed to be a trendline break of the 1 min. chart on the currency being traded.

This part I know is really hard to program. Right now though I would be happy to just have it work with the 9 RSI

alignments. I have the 1 min and 5 min coded now but am getting 1 error about parentis's and some warnings. I know

that these might not even be the problem but there IS something wrong in the code. Would appreciate some help on

this because I would like to get this up and testing.

Thanks,

JimTrader1

Files:
Simon Gniadkowski
17660
Simon Gniadkowski  

iMA and iRSI both ask for strings as the first parameter passed to them, https://docs.mql4.com/indicators/iMA https://docs.mql4.com/indicators/iRSI you are passing a double

   double EURUSD;

   double EURGBP;

   double GBPUSD;


   rsiegs=iRSI( EURGBP ,1,4,PRICE_CLOSE,1);
   magd=iMA( GBPUSD ,1,MovingPeriod,MovingShift,MODE_LWMA,PRICE_WEIGHTED,0);
Jim Contant
318
Jim Contant  
RaptorUK:

iMA and iRSI both ask for strings as the first parameter passed to them, https://docs.mql4.com/indicators/iMA https://docs.mql4.com/indicators/iRSI you are passing a double


Hi RaptorUK, Thanks for looking at my code problem.I am going to resend the entire code because some of it was missing.The Orders part of it etc. I will

look into the string part you sent me.It just looks to me like what I allready have in there. I still have a bunch to learn yet.Anyway that's why I want you to be able to see the entire code.Don't know what happened as to why it didn't all send?

This is a one of kind method for trading. The way it works is, when the 3 currencys align with RSI in the same direction, the volatility increases. We only shoot for

5 pips and use a 14 pip stop.It uses a 1m 5m and 15m timeframe. You only trade the EURUSD.This involves 9 charts on the screen with only the 1m on each currency showing. RSI on the others.Sort of like a stack of solataire cards.I know the stop and target are kind of reverse but because this method can actually approach 100% accuracy,is ok. I feel this is really worth working on turning into an EA because just sitting and waiting for an alignment is not too enticing. Also you are more than welcome to have this should I get it done and tested.

This time I sent the code as a txt file.

Files:
watoea.txt 9 kb
Simon Gniadkowski
17660
Simon Gniadkowski  
jimtrader1:


Hi RaptorUK, Thanks for looking at my code problem.I am going to resend the entire code because some of it was missing.The Orders part of it etc. I will

look into the string part you sent me.It just looks to me like what I allready have in there. I still have a bunch to learn yet.Anyway that's why I want you to be able to see the entire code.Don't know what happened as to why it didn't all send?

You have defined some variables, e.g.

double GBPUSD;

. . . what is it's value ?

Your code doesn't compile . . .

Jim Contant
318
Jim Contant  
RaptorUK:

You have defined some variables, e.g.

. . . what is it's value ?

Your code doesn't compile . . .


It sure doesn't compile.That's why I need help. The current value of the GBPUSD is 1.65041 if that's what you want. Do I have the currency symbols in correctly?

This is what I have never done before.Always used the currency symbol on the active chart or NULL.Here's one. Also how do we put this on the chart? We are only

trading the EURUSD. Does the EURUSD need to be on the chart and the program recognizes the other ones as variables? I guess this is the only way this can work.

Simon Gniadkowski
17660
Simon Gniadkowski  
jimtrader1:


It sure doesn't compile.That's why I need help. The current value of the GBPUSD is 1.65041 if that's what you want. Do I have the currency symbols in correctly?

This is what I have never done before.Always used the currency symbol on the active chart or NULL.Here's one. Also how do we put this on the chart? We are only

trading the EURUSD. Does the EURUSD need to be on the chart and the program recognizes the other ones as variables? I guess this is the only way this can work.

You have defined a variable as a Double . . . but you don't seem to understand Data Types, read here: https://docs.mql4.com/basis/types

iRSI and iMA need a string data type as their first input, you have used EURGBP and GBPUSD in the examples I already posted, you have declared these ad doubles, iRSI and iMA want strings . . .e.g. "GBPUSD" or "EURGBP"

Simon Gniadkowski
17660
Simon Gniadkowski  
jimtrader1:

I have the 1 min and 5 min coded now but am getting 1 error about parentis's and some warnings.

For info: https://en.wikipedia.org/wiki/Bracket otherwise known as "Parentheses" . . you have to have matched pairs . . .
Jim Contant
318
Jim Contant  
RaptorUK:
For info: https://en.wikipedia.org/wiki/Bracket otherwise known as "Parentheses" . . you have to have matched pairs . . .


Ok.I will read up on this and go from there.I understand what you are saying.I do know what a Double means. I think one thing that

throws me off a lot being new is, there are different ways of placing things like the Start function and I have been known to put the Double in there

more than necessary. Looks like you have probably hit on most of the code problem though.

Thanks for correcting me on my spelling of "Parentheses.Was in a hurry and can't spell as well as I used to.

To add comments, please log in or register