ChartEvent returns nothing

void OnChartEvent(const int id, const long& lparam, const double& dparam, const string& sparam)
  {
   if(id==CHARTEVENT_OBJECT_CLICK)
     {
      Print ("Now inside OnChartEvent");
      if(sparam=="PauseButton")
        {
         PauseEA=1;
         Print("The mouse has been clicked on the object with name '"+sparam+"'");      
        }
     }
  }

Hi honest knave,

I tried your code but still don't get PauseEA to show 1 en clicked.

It shouldn't matter where in my EA's code this part is put, should it?

What else could I check?

Ah, I had a weird problem with the arrays, when I used void OnTick arrays wouldn't load any values at all. Only after eliminating the OnTick function would arrays work properly.

Thanks for help and advice 

I always like to try and simplify things when I can't figure out what is going wrong.

Here is a bare-bones version of your code:

#property strict
#property indicator_chart_window

int OnInit()
  {
   CreatePauseButton();
   return(INIT_SUCCEEDED);
  }

void OnDeinit(const int reason)
  {
   ObjectDelete("PauseButton");
  }

void CreatePauseButton ()
   {
   ObjectCreate(0,"PauseButton",OBJ_BUTTON,0,0,0);
   ObjectSetInteger(0,"PauseButton",OBJPROP_XDISTANCE,530);
   ObjectSetInteger(0,"PauseButton",OBJPROP_YDISTANCE,55);
   ObjectSetInteger(0,"PauseButton",OBJPROP_XSIZE,100);
   ObjectSetInteger(0,"PauseButton",OBJPROP_YSIZE,50);
   
   ObjectSetString(0,"PauseButton",OBJPROP_TEXT,"Pause EA");
   }


void OnChartEvent(const int id, const long& lparam, const double& dparam, const string& sparam)
  {
   if(id==CHARTEVENT_OBJECT_CLICK)
     {
      Print ("Now inside OnChartEvent");
      if(sparam=="PauseButton")
        {
         Print("The mouse has been clicked on the object with name '"+sparam+"'");      
        }
     }
  }

int OnCalculate(const int rates_total,
                const int prev_calculated,
                const datetime &time[],
                const double &open[],
                const double &high[],
                const double &low[],
                const double &close[],
                const long &tick_volume[],
                const long &volume[],
                const int &spread[])
  {
   return(rates_total);
  }

When applied to a chart, the button is drawn and the print statements work as expected:

Hi again :)

when I add

#property indicator_chart_window

to my EA it goes berserk :)

my displayed text and the button never even get created. it seems the EA doesn't even enter the function(s) to do so. So I stopped editing for now. I got to find out first what went wrong in my code that screws up things, like the array problem when using ontick... 

it was so much work and I just hope i didn't screw the whole thing up.

i added the EA as a attached file, maybe you could have a quick peek at it. I would thank you in advance for your time and effort and kindness.

please know that i'm a '67 novice mql4 coder, so expect old style commodore basic style coding :)

Two things come to my mind:

1. You need to enable the EventHandler in the OnInit() function
2. Use StringCompare rather than name == "Pausebutton"

ChartSetInteger(0, CHART_EVENT_MOUSE_MOVE, true);  // example enables mouse move events

The code I wrote was an indicator (sorry, I didn't see you were working with an EA). However, the principle is exactly the same. This in uncompiled as I'm on the move, but it should work as an EA:

#property strict

int OnInit()
  {
   CreatePauseButton();
   return(INIT_SUCCEEDED);
  }

void OnDeinit(const int reason)
  {
   ObjectDelete("PauseButton");
  }

void CreatePauseButton ()
   {
   ObjectCreate(0,"PauseButton",OBJ_BUTTON,0,0,0);
   ObjectSetInteger(0,"PauseButton",OBJPROP_XDISTANCE,530);
   ObjectSetInteger(0,"PauseButton",OBJPROP_YDISTANCE,55);
   ObjectSetInteger(0,"PauseButton",OBJPROP_XSIZE,100);
   ObjectSetInteger(0,"PauseButton",OBJPROP_YSIZE,50);
   
   ObjectSetString(0,"PauseButton",OBJPROP_TEXT,"Pause EA");
   }


void OnChartEvent(const int id, const long& lparam, const double& dparam, const string& sparam)
  {
   if(id==CHARTEVENT_OBJECT_CLICK)
     {
      Print ("Now inside OnChartEvent");
      if(sparam=="PauseButton")
        {
         Print("The mouse has been clicked on the object with name '"+sparam+"'");      
        }
     }
  }

void OnTick()
  {
  }

 I can't take a proper look at your EA at the moment (cutting and pasting is about as advanced as I can go on the move!)

Hi honest knave,

i tried a bit more but didn't get any results.

i just don't want to mess up my existing code...

if you don't mind i'll wait till u get the time to have a look. but i really don't want to take up to much of your time so just let me know if i should wait... thanks

Hello rme0108,

I compiled your indicator "as is" and applied it to a chart. The OnChartEvent code is working fine:

I apologise if I'm telling you something you already know, but Print statements don't popup on the screen. They are only written to the Expert log. Perhaps that is why you thought it isn't working?

As an aside, you may want to consider using the strict directive as it will help keep your code on the straight'n'arrow:

#property strict

OOOps,

so sorry, I never tried on a life chart. My programming and testing was done only on the Strategy Tester and using backtesting.

I had no idea that chart events only work on forward testing when attached to  a chart.

sorry

thanks for the help 

No problem, I'm glad you got it sorted.

I'm going back a while so it may have changed, but my workaround in backtesting was this:

The buttons still click in backtesting.

So each tick, I had to check the state of the button using

ObjectGetInteger(0,ObjName,OBJPROP_STATE);