Need Help with C programming - I am new to C/C++

[akshay19]

I have specified code below and I am looking for more efficient code. Can someone help me out? Appreciated all your help. Thank you.

Contact information: akshay_shah@ameritech.net

#include <stdio.h>
#include <math.h>
#include <stdlib.h>

main()
{
	double Myval = 0.0;
	Myval = BusyWork(3.0, "qwertyuiopasdfghjklzxcvbnm");
	printf("Myvalue: %d16.2\n", Myval);
}


BusyWork(double dInit, char* pszKey)
{
int i;
short j, k;
double dVal = dInit;
double dTemp;

	if (dInit <=0.0)
	return 0.0;
	for (i = 0; i<1000000; i++)
	{
	 for (j=0; j<10; j++)
	 {
	  dTemp = (double)(i*4);
	  if (j%2)
 	 	dVal += dVal * 0.1;
	  else
	  	dVal -= dVal * 0.1;
	  dVal /= dInit;
	  dVal -= ((double)j * 2.2) + 1.0;
	  dVal += ((double)j * 1.5) + 1.0;
	  for (k = 0; k<(short)strlen(pszKey); k++)
	  {
	   dVal += (double)(pszKey[k] - (char)k);
	   dTemp += 0.1;
	  }
	   dVal += dTemp;
	   dVal += sin((double)(j % 4) * 90.0);
	 }
	}
dVal += BusyWork(dInit - 1.0, pszKey);
return dVal;
}

[destin]

Does this program compile for you?

[akshay19]

Yes it does compile and works for me, but it is runs for 2 to 3 min to complete. I want to make it more efficient so that it runs quickly. Appreciate if you come up with something better. Thank you.

[redhead]

Just add a predifinition of the BussyWork() and it will compile.
Now to the real question, I've tried a few things, but since you have that 1000000 times loop it dosn't seem like theres much to improve, altho the reverse calling of the function is taking some extra time. But what exactly is it supposed to do ? if that was any cleare there might be some shortcuts elsewhere, than just all this number crunching.

[akshay19]

Good Question. I was only give psuedocode, but don't know what they are trying to acheive with this short program. Well I am going to work on some alternate logic and verify start time and end time of execution, which ever is shorter I will pick on and post here. Meanwhile if you or anyone finds alternate way of doing then please go ahead and post it here. Thank you for all your help.