Freezer

It's a speed tester.
I wrote this program with some of information I gathered from here. Please place the attached file on your C: partition and compile it. tell me your idea about the code, bugs and if possible your score and machine type.
Thank you.

Code:

---

#include<math.h>
#include<iostream.h>
#include<fstream.h>
#include<string.h>
#include<windows.h>
#include<conio.h>

//prototypes
void func(int);
int getrange(int minimum, int Maximum);
void showdata();

void func(int a)
{
        a++;
        a--;
        a+=34;
        a-=32;
        a/=3;
}

void showdata()
{
        ifstream dat_if;
        int readn;
        char readc[20];

        dat_if.open("C:\\data.txt",ios::in | ios::nocreate);
        if(!dat_if.good())
        {
                cout <<"Error opening data file!"<<endl;
                dat_if.close();
                return;
        }

       

        dat_if>>readc>>readn;
        while(!dat_if.eof()&&dat_if.good())
        {
                cout<<readc<<" : "<<readn <<endl;
                dat_if>>readc>>readn;
               
        }
       
        dat_if.close();
}

int getrange(int m, int M)
{
        int inpv; //input value
       
        cout <<"A number from "<< m<<" to " << M<<": ";
        cin >>inpv;
        if(inpv >3 || inpv <1)
        {
                cout<<"Not in range!"<<endl;
                cin.clear();
                cin.ignore(300,'\n');
                inpv=getrange(m,M);  //<=Look at this.
               
        }
        else
        {
                //cin.clear(); It seems not required
                cout<<endl;
        }
        return inpv;
}

int main()    //Salem forced me to use int instead of void for compatibility
{
        cout<<"Define the program's precision:"<<endl;
       
        //This function gets input in specified range;
        int pric =getrange(1,3);
       
        //////////////////////// Selfish function:
        SetPriorityClass( GetCurrentProcess(), REALTIME_PRIORITY_CLASS );
        ////////////////////////
       
        //Initializings
        int x=0;
        char ch[1000]="uv";
        char ch2[]="j";
        double doa[100];
        double doa2[100];
        double doub=33.3433;
        DWORD TimeFrame = 1000 * 5 * pric;
       
        cout<< "Freezing..."  <<endl;
       
        /*This function returns
        miliseconds passed since Windows started.*/
        DWORD StartTime = GetTickCount();
       
       
        /*In this loop some simple operations execute to make CPU work
        harder. Actually I didn't spend enough time to optimize this block.
        Tell me your suggestions. */
       
        while((GetTickCount() - StartTime) < TimeFrame)
        {
                (x<100) ? doa[x]=doa2[x]:doa[32]=doa2[23];
                if(x<500)  strcat(ch,ch2); else func(x);
                doub+=3433.52234;
               
                doub-=4334.3;
                doub/=3.34;
                doub*=4.544;
                doub=sqrt(doub);
                doub=x-(x*x*x)+19*x/3*x;
                doub=doub/2-322343;
                x++;

        }
       
        cout<<"Your score is: "<< int(x/(10000*pric))<<endl;
       
        //This function reads some recent score records from a file.
        showdata();
       
        //This function sends the string as a command to system.
        system("pause");//waits till user hits any key
       
       
        return 0;       

}

---

All windows compilers should compile it, shouldn't they?

You do realise that calling a function "func" is an offence whereby you're taken out the back and shot at dawn, don't you?

Quote:

---

Originally Posted by siavoshkc

It's a speed tester.
All windows compilers should compile it, shouldn't they?

---

No, VC++7.1 and later (Visual Studio.Net 2003) won't compile it, because you used the old, outdated headers. They were removed in 7.1.

On a funny side note, I do believe that enabling all optimization options would cause the compiler to simply remove most of your "make CPU work" stuff.

What if I add x to all of them?

Quote:

---

Originally Posted by ahluka

You do realise that calling a function "func" is an offence whereby you're taken out the back and shot at dawn, don't you?

---

What do you mean? I mean I understood your goal. But your sentences seems heavy to me.

The name of a function should be descriptive. When a project gets larger using names like func make it difficult too see whats going on.

OK rename it to worker_sample().

Take a look to new version

Code:

---


/* Freezer ver 1.8
* - New CPU work stuff
* - Security hole patched (no shell command anymore)
* - Now searches for data file. If not found asks user for path
* - Better search for data file(NEW)
* - Tests for priority(NEW)
* - Very modular design(NEW)
* - Goes to high priority only in needed time(NEW)
*/

int cpuburner(int);
void testfunc(int);
int getrange(int minimum, int Maximum);
void showdata(char *);
void getpath(char* );

int main(int argc , char *argv[])
{

        cout<<"Define the program's precision:"<<endl;
        int pric =getrange(1,3);
       
        cout<< "Freezing..."  ;
        SetPriorityClass( GetCurrentProcess(), REALTIME_PRIORITY_CLASS );
        //cout<< "Priority: "<< GetPriorityClass(GetCurrentProcess())  <<endl;
        if(200>GetPriorityClass(GetCurrentProcess()))
                cout<<" (Could not swicth to highest priority)"<<endl;
        else
                cout<<endl;
       
        cout<<"Your score is: "<< cpuburner(pric)<<endl;

        SetPriorityClass( GetCurrentProcess(), NORMAL_PRIORITY_CLASS );
               
        char pat[100];
        strncpy(pat,argv[0],99);
       
        strcpy(strrchr(pat,92),"\\freez.dat");
        showdata(pat);
        cout<<  "Press a key to exit...";
        cout.flush();
        getch();
        return 0;
       
}


void getpath(char *pat)
{
       
        cout<<"Enter path of freez.dat: " ;
        cin.ignore(1,'\n'); //Tell me why this should be here
        cin.getline(pat,30,'\n');
       
}

void showdata(char *pat)
{
       
       
        ifstream dat_if;
        int readn;
        char readc[10];
       
        dat_if.open(pat,ios::in | ios::nocreate);
        if(!dat_if.good())
        {
               
                dat_if.clear();
                getpath(pat);
                strcat(pat, "\\freez.dat");
               
                dat_if.open(pat,ios::in | ios::nocreate);
               
                if(!dat_if.good())
                {
                        cerr <<"Error opening data file from path: "<<pat<<endl;
               
                        return;
                }
        }

       

        dat_if>>readc>>readn;
        while(!dat_if.eof()&&dat_if.good())
        {
               
                cout<<readc<<" : "<<readn <<endl;
                dat_if>>readc>>readn;
               
        }
       
        dat_if.close();
}

int getrange(int m, int M)
{
        int inpv;
       
        cout <<"A number from "<< m<<" to " << M<<": ";
        cin >>inpv;
        if(inpv>3 || inpv<1)
        {
                cout<<"Not in range!"<<endl;
                cin.clear();
                cin.ignore(300,'\n');
                inpv=getrange(m,M);
               
        }
        else
        {
               
                cout<<endl;
        }
        return inpv;
}


int cpuburner(int pric)
{
        int x=0;
        int sigma(int);
        int fac(int);
        char ch[1000]="sa";
        char ch2[]="j";
        double doa[100];
        int c;
        int n=GetTickCount()%10000;
        int seed=120+100*cos(n);
       
        double doa2[100];
        double doub=33.3433;
        DWORD TimeFrame = 1000 * 5 * pric;
        DWORD StartTime = GetTickCount();
       

        while((GetTickCount() - StartTime) < TimeFrame)
        {
                x<100 ? doa[x]=doa2[x]:doa[32]=doa2[23];
                if(x<500)  strcat(ch,ch2); else testfunc(x);
                doub+=3433.52234;
               
                doub-=4334.3+x;
                doub/=3.34+x;
                doub*=4.544;
                testfunc(doub);
                doub=sqrt(doub);
                doub=x-(x*x*x)+19*x/3*x;
                doub=doub/2-322343/(x+1);
               
       
                /*ofstream ofile;
                *ofile.open("C:\\randnum.txt");
                *for(int n=0;n<6000;n++){
                *This is a random number generator*/

                c=(fac(n+3)*seed-(floor((pow(60*n,3)*cos(n)*seed)-sigma(n+seed)))+100*seed/
                        (n-pow(floor(log(n)),3)));
                c=abs((int)(c+10*cos(n))%10000+(doub));

                doub-=4334.2323+x;
                doub/=3.342+x;
                doub*=43.54423223;
                doub=sqrt(doub)-1*x;
                doub=x-(x*x*x+12/(x+2))+19*x/3*x*x;
                doub=doub/2-322343*x;

                //ofile<<c<<endl;

                //cout<<c<<endl; //Enable it for wathing random nums.
                //}
                x++;
               
        }
        return (x/(10*pric));
}


void testfunc(int a)
{
        a++;
        a--;
        a+=35564;
        a-=320;

        a/=3;
}

int sigma(int n)
{
        double dub; //It is random so no need to be initialized
       
        for(int i=1;i<=n;i++)
        {
                dub+=(i*i)/(i+i*2);
        }
        return floor(dub);
}

int fac(int n)
{
        int f=1;
        for(int i=1;i<=n;i++)
        {
                f*=i;
        }
        return f;
}

---