I am getting an error that says:ISO C++ forbids comparison between pointer and i

Question

I am getting an error that says:ISO C++ forbids comparison between pointer and integer [-fpermissive] in this area:

case 2:
   if (isALeapYear == true)
   monthLength = 29;
   else
   monthLength = 28;

Please tell me what I am doing wrong and how to fix this.

---------------------------------------------------------------------------------------------------------------------

#include <iostream>
#include <iomanip>

using namespace std;

int numberOfDays();
bool dateIsValid();
bool isALeapYear();
int dayOfTheYear();

int main()
{
int year, month, day;
int sum;
char c;
bool validInput = false;
while ((cin) && !validInput)
{
cout << "Enter a date in the form YYYY-MM-DD: " << flush;
cin >> year >> c >> month >> c >> day;
}
bool dateIsValid();
int numberOfDaysInMonth();
void dayOfYear();
// Then add the day number to that sum
int dayNum = sum + day;
cout << setw(2) << setfill('0') << month
<< "/" << setw(2) << setfill('0') << day << "/"
<< setw(4) << year;
cout << " is day #" << dayNum << " of that year." << endl;
return 0;
}

//-------------------------------------------------------------------
// Check to see if this is a valid date
// The Gregorian calendar began On Oct 15, 1582. Earlier dates
// are invalid.
//-------------------------------------------------------------------
bool dateIsValid()
{
int validInput;
int year;
int month;
int day;
if (year < 1582)
   validInput= false;
else if (year == 1582 && month < 10)
   validInput = false;
else if (year == 1582 && month == 10 && day < 15)
   validInput = false;
// Months must be in the range 1..12
else if (month < 1 || month > 12)
   validInput = false;
}

//--------------------------------------------------------------------
// Days must be in the range 1..K where K is the number of
// days in that month.
//--------------------------------------------------------------------
int numberOfDaysInMonth()
   {
int month;
int day;
int validInput;
   int numberOfDaysInMonth = 0;
   switch (month)
   {
   case 1:
   case 3:
   case 5:
   case 7:
   case 8:
   case 10:
   case 12:
   numberOfDaysInMonth = 31;
   break;

   case 4:
   case 6:
   case 9:
   case 11:
   numberOfDaysInMonth = 30;
   break;

case 2:
if (isALeapYear == true)
numberOfDaysInMonth = 29;
else
numberOfDaysInMonth = 28;
   }

if (day < 1 || day > numberOfDaysInMonth)
   validInput = false;
else
   validInput = true;

if (!validInput)
   {
   cout << "Sorry, that is not a valid date" << endl;
   string garbage;
   getline (cin, garbage); // discard the rest of the input line
   }
if (!cin)
{
cout << "Could not obtain valid input." << endl;
return -1;
}
   }

//--------------------------------------------------------
// Input is valid - compute the day number
// Add up the number of days in all earlier months
// of this year
//--------------------------------------------------------
void dayOfTheYear()
{
int sum = 0;
int month;
for (int m = 1; m < month; ++m)
{
int monthLength = 0;
switch (m)
   {
   case 1:
   case 3:
   case 5:
   case 7:
   case 8:
   case 10:
   case 12:
   monthLength = 31;
   break;

   case 4:
   case 6:
   case 9:
   case 11:
   monthLength = 30;
   break;

   case 2:
   if (isALeapYear == true)
   monthLength = 29;
   else
   monthLength = 28;
   }

sum += monthLength;
}
}

bool isALeapYear(int)
{
int year;
if (((year % 4 == 0) && (year % 100 != 0))
       || (year % 400 == 0))
       return true;
else
return false;
}

Explanation / Answer

Here actually u are trying to compare the values of integer obtained with another integer. Since you have dynamically allocated the space for the function or the variable. you are getting an error..

Try putting single or double quote in "isALeapYear(int)" function and the case 2 code. i.e;

if (isALeapYear == "true")
   monthLength = 29;
   else
   monthLength = 28;

then the error is rectified.

Related Questions

Navigate

• Browse (All)
• Browse I
• Subjects

Integrity-first tutoring: explanations and feedback only — we do not complete graded work. Learn more.