EA Code

In this case, you can create 2 more global variables. But they will be reset to zero after restarting MT4.



double PrevSLBuyDynamic=0,PrevSLSellDynamic=0; // Global variables 

void VirtualTrail(){
  double low2=iLow(NULL,0,iLowest(NULL,0,MODE_LOW,Nr_of_Bars_for_HiLo,0));
  double hi=iHigh(NULL,0,iHighest(NULL,0,MODE_HIGH,Nr_of_Bars_for_HiLo,0));

  double SLBuyDynamic=(low2-(MarketInfo(NULL,MODE_STOPLEVEL))*(MarketInfo(NULL,MODE_POINT)));

  if(PrevSLBuyDynamic>SLBuyDynamic){
      SLBuyDynamic=PrevSLBuyDynamic;
    }
  else{
      PrevSLBuyDynamic=SLBuyDynamic;
    }

  double SLSellDynamic=(hi+(MarketInfo(NULL,MODE_STOPLEVEL))*(MarketInfo(NULL,MODE_POINT)));

  if(PrevSLSellDynamic>0 && PrevSLSellDynamic<SLSellDynamic){
      SLSellDynamic=PrevSLSellDynamic;
    }
  else{
      PrevSLSellDynamic=SLSellDynamic;
    }
//...
// your code
//...
}

And I am back with another coding question.

I managed to write an EA which takes trades based on the Advanced Currency Meter.

The attached code is just a small script I wrote to try and exclude certain symbols when the EA runs. (The code will obviously be incorporated in the EA, I just wrote a script to test)


extern string Symbols_To_Exclude = "GBPUSD,AUDUSD,USDJPY,GBPCHF";
enum currencies {Top6=5,TopRow=13,All=27};
input currencies Symbols_To_Trade = Top6;
string ccy;
string dir;

void OnStart()
 {  
  for (int c=0;c<=Symbols_To_Trade;c++)
   {
    ccy=ObjectGetString(0,"currency"+IntegerToString(c,1),OBJPROP_TEXT);//Symbol

    int d = (c+40);

    dir=ObjectGetString(0,"currency"+IntegerToString(d,1),OBJPROP_TEXT);//Direction

    for (int i=0; i<=70; i+=7)
     {
      string exclude =StringSubstr(Symbols_To_Exclude,i,StringLen(ccy));

      if (ccy== exclude)break;

      MessageBox ("Not excluded " +ccy);
     }   
   }
 }
 //+------------------------------------------------------------------+


As you can see I have a drop down menu for which symbol groups to trade, now I want to exclude certain symbols too.





It looks as if it works, but I excluded GBPUSD, AUDUSD and USDJPY as a test, when the message box pops up, it doesn't show GBPUSD, but it still shows USDJPY once, and AUDUSD twice, I understand it is because the loop runs, but if it is in an EA, it will actually take a trade on the excluded pairs, (or am I wrong, because in my mind, the OrderSend func will be where the MessageBox func is now, (after all the other expressions it had to pass) so the OrderSend function will be executed?)

Thanks again.

Here it is

I tested the script and it doesn't work for me as it did for you. It just alerts for EURUSD over and over many times. No other symbols are excluded. I cannot test the rest of the code since the indicator doesn't work for me. For example, I can't see if the Symbols and Directions are being recognized correctly.

One thing that stands out to me though is that I would probably change "if (ccy== exclude)break;" to "if (ccy== exclude)continue;". So if an excluded symbol is reached then it will ignore that symbol but not stop the process entirely. So it can go on to check other valid symbols.

The indicator you just attached looks exactly the same as the previous indicator. It also doesn't work just like the last one too. I tried 2 different platforms. Please double check you are uploading the same one you are using the script with.

I cannot see what you are saying without testing with your indicator because the script reads objects created by the indicator. All the script did for me was give be a message for EURUSD over and over.

Why do I get an "improper enumerator cannot be used" error?

This is the enumerator code :


// Indi Inputs & variable
extern int FastMAperiod=20;

enum mamethod {Simple=0,Exponential=1,Smoothed=2,Linear=3};

input mamethod FastMA_method=Simple;

extern int MidMAperiod=40;

input mamethod MidMA_method=Simple;

extern int SlowMAperiod=80;

input mamethod SlowMA_method=Simple;


And this is where they are called:


void OnTick()
  {
//Indicators
   FastMA = iMA(NULL,0,FastMAperiod,0,FastMA_method,0,1);

   MidMA = iMA(NULL,0,MidMAperiod,0,MidMA_method,0,1);

   SlowMA = iMA(NULL,0,SlowMAperiod,0,SlowMA_method,0,1);

Thanks
Slowly but surely I am learning