Indicators run prematurely

[monpere]

Indicators start running just by bringing up the indicators dialog. Here's a test case: Using the indicator code below, open the output window, start a new chart, right click and select 'indicators...', once the indicators window appears, start watching the output window. Close the indicators window and bring it up again, watch the output window again. It seems the Initialize() method of indicators/strategies are being called every time the indicators window is opened.


#region Variables
System.Windows.Forms.Timer xtimer; //### System Timer
int counter=0;
#endregion

protected override void Initialize()
{
//### Setup Timer
if ( xtimer == null ) {
xtimer = new System.Windows.Forms.Timer();
xtimer.Interval = 1000; //### Default 1 sec interval
xtimer.Tick += new EventHandler(timer_wakeup);
xtimer.Enabled = true;
}

}

protected override void OnBarUpdate()
{
}

private void timer_wakeup(object sender, EventArgs e) {
Print("Timer wakeup " +(counter++));
}

public override void Dispose()
{
//### Remove timer
xtimer.Interval = 1000000;
xtimer.Stop();
xtimer.Enabled = false;
xtimer.Dispose();
xtimer = null;;

base.Dispose();
Print("Disposed on exit");
}

[NinjaTrader_Dierk]

>> Indicators start running just by bringing up the indicators dialog.
Incorrect. Only Initialize() is called. If you have code in Initialize() which should not be executing then best move this code to OnBarUpdate().

[monpere]

Yes. As I said, seems Initialize() is being called when the indicators dialog is displayed. My question is, why on earth would you run Iinitialize() on every indicator and every strategy when a dialog box is popped up? Is this by design? This is a very weird programming architecture to me. This means if I have 500 indicators with timers in the Inintialize(), just bringing up a dialog window will start all 500 timers? and if in my normal working with NT throughout the day, I bring up the dialog box 10 more times I will have 5000 timers running for indicators that I am not even using ???? Does this seem like a plausible architecture? There has to be a better/proper way to architect the indicator/dialog code !!!! No code should ever be run until I hit the 'Ok' button !!!!

>> Indicators start running just by bringing up the indicators dialog.
Incorrect. Only Initialize() is called. If you have code in Initialize() which should be executing then best move this code to OnBarUpdate().

[NinjaTrader_Dierk]

>> why on earth would you run Iinitialize() on every indicator
That's how indicators are initialized.