problem with function

[xxlt3xx]

I'm trying to convert the numerical value of months to the name using a void function but I cant get it to compile it give me a strange error that is too long to put in the post. I appreciate any help as I've been working on this for a few days. here is a snippet of the area that's giving me a headache.

#include <iostream>
#include <string>
#include <iomanip>
#include <cmath>
using namespace std;



void monthName(int num);//this is the function protocol for the problem function

int main()
   {

       char response;
       
       int monthCalc;
       
       int convert_month;
       
       int subscribe_info;
       
       int order_date;

       int monthNumber;
       
       int currentYear;
       
       int begin_date;

       int currentMonthNumber;
       
      int currentMonthName;
       
       int expire_month;

       double expire_year;
       
       double buyYear;
       
       double buy_year;
       
       int subsciptMonthName; 
                 
       string firstName;
       
       string lastName;

       cout <<"Enter first letter of the current month: ";

       currentMonthNumber = months();
       
  

       cout <<"Enter current year ";

       currentYear = yearReading(begin_date);

     do
      {
       cout << endl << endl;
       
       cin.ignore();
   
       cout << "Enter subscriber first name: ";

       getline (cin, firstName);
              
       cout << "Enter subscriber last name: ";

       getline (cin, lastName);
       
       cout << endl;

       cout << "Enter first letter of the subscription month: ";

       monthNumber = months();
       
       buy_year = yearReading (buyYear);
       
       subscribe_info = subscriber_month (order_date);

       cout << fixed << showpoint << setprecision(2) << endl;

       cout << endl << endl << endl;
       
       

       cout <<"Current Date: "<<  monthName(currentMonthNumber)<< " " << currentYear <<endl;//this is where I'm having the problem 

       cout <<"Subscriber: " << lastName<<", "<< firstName << endl << endl;
       
       cout <<"Start Date: " << monthName(monthNumber) << " " ;//same here

       cout << static_cast<int>(buyYear) <<endl;

       if(monthNumber == 1)
        {
       
           expire_month = monthNumber + 11;
       
        }
       else
        {
           expire_month = monthNumber - 1;
       
           expire_year = buyYear + subscribe_info;
              
           cout <<"Expiration Date: " << monthName(expire_month)<<" " <<static_cast<int> (expire_year)<< endl;// and here
           
           cout << "Account Status: " << "Current " << endl;
           
           monthCalc =  ((subscribe_info * 12) - expire_month - 12);
           
           cout <<setw(18)<< monthCalc << "months remaining on subsciption " << endl;
        }

          cout << endl << endl << endl;
        
          cout << "Would you like to run the program again (Y/N)? " ;
          
          cin >> response;
          
          response = toupper(response);
    }     
        
       while (response =='Y'); 
            
          cout << endl;
          
          return 0;
  }







void monthName(int number) // this is the function thats giving me problems 
  {
   
   
      switch (number)  
        {
              case 1:
              cout << "January" ;
              break;  
          
              case 2:
              cout << "February" ;
              break;  
          
              case 3:
              cout << "March" ;
              break;  
          
              case 4:
              cout << "April" ;
              break; 
          
              case 5:
              cout << "May" ;
              break; 
          
              case 6:
              cout << "June" ;
              break;  
          
              case 7:
              cout << "July" ;
              break;  
          
              case 8:
              cout << "August" ;
              break;  
          
              case 9:
              cout << "September" ;
              break;  
          
              case 10:
              cout << "October" ;
              break;  
          
              case 11:
              cout << "November" ;
              break;  
          
              case 12:
              cout << "December" ;
              break;  
         
        }
    
         
     }     
          
  

[Lv0f92yv]

Can you tell us the line to which the error points? I have found that looking at and just before the problematic line (the one to which the error points) can yield useful results hen debugging. Give this a shot, if you haven't already.

Sometimes the error is actually far before the flagged line, but the compiler can't detect it. They're not perfect after all.

[xxlt3xx]

I added comments to the code to show where the problem is happening also this is the error I'm getting : 105 ")) << monthName(currentMonthNumber)' note candidates are: std::basic_ostream<_CharT, _Traits>& std::basic_ostream<_CharT, _Traits>::operator<<(std::basic_ostream<_CharT, _Traits>&(*)(std::basic_ostream<_CharT, _Traits>&)) [with _CharT = char, _Traits = std::char_traits<char>] ......

[Lv0f92yv]

I believe one problem is that the function you are using in cout, "monthName(...)", does not return a value (void). You cannot use a function that returns void in a print statement, nor any other statement in which a value is required. Try letting monthName return some value.

Since your current function prints out a value itself, perhaps you can use the call to it as a seperate statement. Assuming the rest of your program is error free, I think this would get the results you're looking for. Alternatively, build a string instead of coutting it in your monthName function, and return it to the caller. This way, you should be able to use it in cout.

[m4ster r0shi]

Lookup tables are your friends:

//...

const char * month_names[]={"January","February","March","April","May",
"June","July","August","September","October","November","December"}

//...

int main()
{
     //...
     
     int month_num;
     cout << "enter month (1-12): ";
     cin >> month_num;
     cout << "you picked: " << month_names[month_num-1] << endl;

     //...
}