Optimization

[kallzz]

Iam dealing with a project that involves code optimization and have a question
on static variables


This is a function that calculates realtime information based on real time data.So this function will
be called as long as we get real time data and the nof times called may be enormous.
Does making these double variables static, improve CPU Performance?
void calculateData()
{


static double dDtScaleMin=m_TimeScaleAttr.GetDataScaleMin();
static double dDtScaleMax=m_TimeScaleAttr.GetDataScaleMax();
static long dtBoundMin=m_TimeScaleAttr.GetDataBoundMin();
static long dtBoundMax=m_TimeScaleAttr.GetDataBoundMax();
static double dtDiffScaleMinMax=dDtScaleMax-dDtScaleMin;



Do some other calculations with the above mentioned double values.

}

[CornedBee]

Do these values change? If so, this initialization method is bad because the assignment is only done on the first function call.

The static variables mean that the app doesn't have to allocate stack space every time it enters. Whether this operation (it's one add and one sub instruction per call) noticeably slow down your app is another question. But if the call count is enormous it might well do it.

The only real solution is, write both and measure.

[kallzz]

ya the variables are assigned with different values each time it enters the function

[Hamid Mushtaq]

You could declare static variables and assign them seperately.
By this I mean, you could write your code in the following way : -


void calculateData()
{
static double dDtScaleMin;
static double dDtScaleMax;
static long dtBoundMin;
static long dtBoundMax;
static double dtDiffScaleMinMax;

dDtScaleMin = m_TimeScaleAttr.GetDataScaleMin();
dDtScaleMax = m _TimeScaleAttr.GetDataScaleMax();
dtBoundMin = m_TimeScaleAttr.GetDataBoundMin();
dtBoundMax = m_TimeScaleAttr.GetDataBoundMax();
dtDiffScaleMinMax = dDtScaleMax-dDtScaleMin;

// other stuff
...
}

[nsh123]

Originally posted by kallzz
ya the variables are assigned with different values each time it enters the function

with the code you posted, i dont think they get initialized.

Also looking at the variable it seems that they are to be initialized once and for all.
And thus in that sense it does improve speed, since the reinitialization is avoided.

regards

[kallzz]

for each 100 msec the m_TimeScaleAttr object changes

void calculateData()
{


static double dDtScaleMin;
dDtScaleMin=m_TimeScaleAttr.GetDataScaleMin();
static double dDtScaleMax;
dDtScaleMax=m_TimeScaleAttr.GetDataScaleMax();
static long dtBoundMin;
dtBoundMin=m_TimeScaleAttr.GetDataBoundMin();
static long dtBoundMax;
dtBoundMax=m_TimeScaleAttr.GetDataBoundMax();
static double dtDiffScaleMinMax;
dtDiffScaleMinMax=dDtScaleMax-dDtScaleMin;



Do some other calculations with the above mentioned double values.

}