Need help with an array

[julzhef]

Hello everyone,
I am trying to make the number of days in a month display when the user enters the month (1-12). For example, if the user enters the number 2, then they would see that there are 28 days in that month. Can someone tell me what I am doing wrong???? It will display the information as long as the number is 1-12, but I cannot get my "invalid entry" to work.

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

//declare array
int days[12] = {31,28,31,30,31,30,31,31,30,31,30,31};

//declare variables
int monthNumber = 0;
char continueYN = ' ';

int main ()
{
//Prompt user
	cout << "Do you want to begin? " ;
	cin >> continueYN ;
	continueYN = toupper(continueYN);

	while (continueYN == 'Y')
	for (; continueYN != 'N';)	
	{
	
	cout << "Please Enter A Month of The Year (1-12): " ;
	cin >> monthNumber;
	monthNumber --;


	if ((monthNumber > '0') || (monthNumber < '12'))
	{
		cout << "Days of this month equal: " << days[monthNumber] << endl ;
	}
	else 
	{
			cout << "INVALID ENTRY...Please enter a value betweent 1 & 12 " << endl;
	}

	cout << "Do you want to continue? " ;
	cin >> continueYN ; 
	continueYN = toupper(continueYN);
	}
	
//end switch

system ("pause");
return 0;
}
//end of main function

[DSLq5Di1]

	cin >> monthNumber;
	//monthNumber --;


	if ((monthNumber > 0) && (monthNumber <= 12))
	{
		cout << "Days of this month equal: " << days[monthNumber-1] << endl ;
	}

- For clarity, remove monthNumber-- and use days[monthNumber-1] on output instead.
- Comparing integer to char literal, remove the single quotes around your numbers.
- monthNumber should be greater than 0 AND less than or equal to 12, you are using the OR operator.

Your program also spits the dummy if you enter something other than what is expected, try handling those cases too. ^^

[creekist]

Here's how I'd do it:

int main() {
	
	int month = 0;
	cout << "Enter your month biatch: ";
	cin >> month;
	cout << endl;

	switch (month) {
		
		case 1:
		cout << "Number of days in January is 31." << endl;
		break;
	
		case 2:
		cout << "Number of days in February is 28, for Leap Year, it's 29." << endl;
		break;
		
		//blah blah etc.

		}

	return 0;}

Or if you wanted it to loop this is how I'd do it:

#include<iostream>
using namespace std;

int main(){
	int month = 0;
	char answer;

	cout << "Do you want to enter the months? Type y for yes, and n for no, to quit at any time enter 0." << endl;
	cin >> answer;

	if (answer == 'y') {
		cout << "Enter your month." << endl;
		
		while (true) {
			if (cin >> month) {
				if ((month == 1) || (month == 3) /*blah blah etc.*/){
					cout << "The number of days in this month is 31." << endl;
					cout << "Enter your month." << endl;}
				
				if (month == 2){
					cout << "The number of days in this month is 28, or for Leap Years: 29." << endl;
					cout << "Enter your month." << endl;}
				
				if (/*30 Day Months blah blah blah etc.*/) {/*blah blah you get it*/}
				
				if (month == 0) { break;}}}}


	if (answer == 'n'){}

	
	return 0;}

May be wrong though.