Function call not working

Why isn't this function call working?
time = CalculateTime(service);

Code:

---

//******************************************************************//
//Program Queue client file                                        //
//                              //
//  //
//  //
//                                                              //
//                                                                  //
//                                                                  //
//******************************************************************//

#include<iostream>
#include<iomanip>
#include<fstream>
#include <cctype>  // defines isalpha() function
#include<string>
#include <time.h>
#include <cstdlib>



using namespace std;

#include"QueueImple.cpp"

int CalculateTime(int service);


int main()
{
        queueList Lines;
    sortedList Words;
        ifstream Infile;
        ofstream Outfile;
        Infile.open("inputQ.txt");
        Outfile.open("OUTPUTQ.TXT");
        int y = rand();
        int x = rand();
        int minutes = 720;
        int Seed, arrive, service;
        int customer = 0;
int CustomerNumber=0;
int runningtime;
int time=0;

srand(Seed);
y = rand();
x = rand();
Infile>>customer;

while(customer < 5)
{


srand(customer);
arrive = rand()%100;  //First cutsomer arrival time
service = rand()%100;    //First customer service time
CustomerNumber++;
Outfile<<"Customer "<<CustomerNumber<<endl;
Outfile<<"Customer "<<CustomerNumber<<" arrival time "<<arrive<<endl;
Outfile<<"Customer "<<CustomerNumber<<" service time "<<service<<endl;

time = CalculateTime(service);

Outfile<<time;
Infile>>customer;
}
/*srand((unsigned)time(NULL)); //Casts time's return to unsigned
  int d=rand()%12;  //Should be more random now 
  Outfile<<d;
*/

    return 0;

}

int CalculateTime(int service)
{
int minutes= 720;
int time;
{
        while(minutes > 0)

        service++;
minutes--;
}
time = service;
return time;
}

---

Use indentation, so others will know what you mean.
I think that problem is in curly brackets {} in CalculateTime() function. Anyway, that function seems strange. Why to use a loop, if you can do all the work by single statement

Code:

---

return (service + 720);

---

? Even why to use a function? (maybe for further development :))

Code:

---

int CalculateTime(int service)
{
int minutes= 720;
int time;
{ //why do you have this brace here?
        while(minutes > 0)

        service++;
minutes--;
}
time = service;
return time;
}

---

something like this will work:

Code:

---

int CalculateTime(int service)
{
        int minutes= 720;
     
        while(minutes > 0){
              service++;
              minutes--;
        }

        return service;
}

---

I don't know why you would do that thought, since minutes = 720, what your while loop is doing is adding 720 to service.

axon

here are some better ways (because I feel like writing some code):

your code

Code:

---

int CalculateTime(int service)
{
int minutes= 720;
int time;
{
        while(minutes > 0)

        service++;
minutes--;
}
time = service;
return time;
}

---

for loop

Code:

---

int CalculateTime(int service)
{
  int time;

      for(int minutes=720;minutes>0;minutes--)
          service++;

  time = service;
  return time;
}

---

without a loop

Code:

---

int CalculateTime(int service)
{
  int time;

  service+=720;

  time = service;
  return time;
}

---

if you must use a function

Code:

---

int CalculateTime(int service) {return service+720;}

---

what would make the most sense is to just not use a function...