parameters

Hi,
I have something wrong with some code that I'm trying to write.
The error message said "function does not take 1 parameters"
I'm using Visual C++ 6.0 and the MSDN Library states that I'm calling with incorrect number of parameters. The message occurs on the second line in the Switch case #2. It has something to do with the Sec_To_HMSTime prototype. (I think) I would appreciate it greatly if someone could help me locate the problem. Another set of eyes usually always helps! Thanks, mcorn.

Code:

---

#include<iostream.h>
#include<stdlib.h>

void GetTime(int*,int*,int*);
int CalcSeconds(int,int,int);
bool CheckInput(int*,int*);  //new function protocol
void GetSeconds(int*);    // "      "        "
int Sec_To_HMSTime(int,int*,int*,int*); //"      "        "

int main()
{

        int hr,min,sec,seconds;
        //char another;
        int total_sec, tot_sec, choice;

        cout<<"\n\nWelcome to a time-conversion program! \n\n";


        do
        {

                cout<<"\nPlease pick a choice:"
                        <<"\n1) Convert H:M:S to seconds."
                        <<"\n2) Convert seconds to H:M:S."
                        <<"\n3) Exit." << endl << endl;
                cin>>choice;

                switch(choice)
                {
                       
                  case 1:
                    GetTime(&hr,&min,&sec);
                    total_sec=CalcSeconds(hr,min,sec);

                    cout<<"\n\nYou Entered " << total_sec << " seconds ";
                    cout<<"\nIt is " << total_sec << " seconds "<<endl;
            break;

                  case 2:
                          GetSeconds(&seconds);
                          tot_sec=Sec_To_HMSTime(seconds);

                          cout<<"\nYou Entered" << tot_sec;
                          cout<<"\nIt is"<<hr<<":"<<min<<":"<<sec;
                          break;
                  case 3:
                          cout<<"\nTest3"<< endl << endl;
                          break;
                 
               
                  default:
                      cout<<"\nThat's not a valid choice!";
                }

        }while(choice !=3);
       
        cout<<"\nBye!"<<endl<<endl;


       
       
       

        cout<<"\nThanks for playing! \n";
        return 0;
}

void GetTime(int *h_ptr,int *m_ptr, int *s_ptr)
{

        int ask_them_again = 1;
        char colon;

        while(ask_them_again == 1)
        {

                cout<< "\n Please enter the time in hours minutes seconds "
                        << "\n Such as 4:15:34 (Note: Minutes and Seconds < 60 ) \n\n==>";
                cin>> *h_ptr >> colon >> *m_ptr >> colon >> *s_ptr;

                ask_them_again = 0;
                if(*m_ptr > 59 || *s_ptr > 59)
                {

                        cout << "\n WHOA!!! Invalid Time!!! ";
                        ask_them_again = 1;
                }
        }
}

int CalcSeconds(int hr, int min, int sec)
{

        int total_sec;
        total_sec = hr*3600 + min*60 + sec;
        return total_sec;
}

int Sec_To_HMSTime(int h_m_s, int *hr, int *min, int *sec)
{

        int total_secs;
        *hr = h_m_s/3600;
        h_m_s = h_m_s%3600;
        *min = h_m_s/60;
        *sec = h_m_s%60;
        return total_secs;
}

---

tot_sec=Sec_To_HMSTime(seconds);

It means exactly what it says. You've declared the function as -

int Sec_To_HMSTime(int,int*,int*,int*);

You need to pass in the addresses of your integers hr,min,sec as the further three parameters. So it'd look something like -

tot_sec=Sec_To_HMSTime(seconds, &hr,&min,&sec);

If you say you are going to pass (int,*int,*int,*int) and not giving default arguments, you cant simply get away with just passing a single int

Try this;

Code:

---

#include<iostream.h>
#include<stdlib.h>

void GetTime(int*,int*,int*);
int CalcSeconds(int,int,int);
bool CheckInput(int*,int*);  //new function protocol
void GetSeconds(int*);    // "      "        "
void Sec_To_HMSTime(int,int*,int*,int*); //"      "        "

int main()
{

        int hr,min,sec,seconds;
        //char another;
        int total_sec, choice;

        cout << endl << "Welcome to a time-conversion program!" << endl << endl;


        do{

                cout << endl << "Please pick a choice:"
                << endl << "1) Convert H:M:S to seconds."
                << endl << "2) Convert seconds to H:M:S."
                << endl << "3) Exit." << endl << endl;
                cin>>choice;

                switch(choice){
                case 1:
                        GetTime(&hr,&min,&sec);
                        total_sec=CalcSeconds(hr,min,sec);

                        cout << endl << "You Entered " << hr << ":" << min << ":" << sec;
                        cout << endl << "It is " << total_sec << " seconds "<< endl;
                        break;

                case 2:
                    GetSeconds(&seconds);
                    Sec_To_HMSTime(seconds,&hr,&min,&sec);
 
                    cout << endl << "You Entered " << seconds;
                    cout << endl << "It is "<< hr <<":"<< min << ":" << sec ;
                    break;
                case 3:
                    cout << endl << "Test3"<< endl << endl;
                    break;
 

                default:
                    cout<<"\nThat's not a valid choice!";
                }

        }while(choice !=3);

        cout << endl << "Bye!" << endl << endl;
        cout << endl << "Thanks for playing!"<< endl;
        return 0;
}

void GetTime(int *h_ptr,int *m_ptr, int *s_ptr)
{
        char colon;

        while(1)
        {

                cout << endl << "Please enter the time in hours minutes seconds "
                << endl <<"Such as 4:15:34 (Note: Minutes and Seconds < 60 )"
                << endl << endl << "==>";
                cin >> *h_ptr >> colon >> *m_ptr >> colon >> *s_ptr;

               
                if(*m_ptr > 59 || *s_ptr > 59)
                {

                        cout << endl << "WHOA!!! Invalid Time!!! ";
                }
                else break;
        }
}

void GetSeconds(int *sec)
{
                cout<< endl << "Please enter the number of seconds" << endl;
               
                cin >> *sec ;

}

int CalcSeconds(int hr, int min, int sec)
{
        return hr*3600 + min*60 + sec;
}

void Sec_To_HMSTime(int h_m_s, int *hr, int *min, int *sec)
{

        *hr = h_m_s / 3600;
        h_m_s = h_m_s % 3600;
        *min = h_m_s / 60;
        h_m_s = h_m_s % 60;
        *sec = h_m_s;
}

---

<edit>..beaten to it ! </edit>