string does not name a type

[Nelo87]

I already searched in google on how to fix this problem, but I still get the error.

I was using "using namespace std;" at first. I changed that for the following, because it seemed that people were getting results. Nonetheless, that didn't work for me.

This is the code:

ETA: I get the error in line 20.

#include <iostream>
#include <string>

using std::cout;
using std::cin;
using std::endl;
using std::string;

//Class Definition
class DayOfYear {
private:
        static string month;
        int day;
public:
	//Constructor
	DayOfYear(int a) { day = a;}
	void print();
}//end of class 

string DayOfYear::month;
//Function Member Definitions
////****************************************************
void DayOfYear::print()
{

if (day >= 1 || day <= 31)
{
month = "January";
cout<<month<<" "<<day;
cout<<endl;
}

else if (day >=32 || day <= 59)
{
month = "February";
cout<<month<<" "<<day;
cout<<endl;
}

else if (day >=60 || day <= 90)
{
month = "March";
cout<<month<<" "<<day;
cout<<endl;
}

else if (day >=91 || day <= 120)
{
month = "April";
cout<<month<<" "<<day;
cout<<endl;
}

else if (day >=121 || day <= 151)
{
month = "May";
cout<<month<<" "<<day;
cout<<endl;
}

else if (day >=152 || day <= 181)
{
month = "June";
cout<<month<<" "<<day;
cout<<endl;
}

else if (day >=182 || day <= 212)
{
month = "July";
cout<<month<<" "<<day;
cout<<endl;
}

else if (day >=213 || day <= 243)
{
month = "August";
cout<<month<<" "<<day;
cout<<endl;
}

else if (day >=244 || day <= 273)
{
month = "September";
cout<<month<<" "<<day;
cout<<endl;
}

else if (day >=274 || day <= 304)
{
month = "October";
cout<<month<<" "<<day;
cout<<endl;
}

else if (day >=305 || day <= 334)
{
month = "November";
cout<<month<<" "<<day;
cout<<endl;
}

else if (day >=335 || day <= 365)
{
month = "December";
cout<<month<<" "<<day;
cout<<endl;
}

}//end of print()




//*************************MAIN FUNCTION****************************************
int main ()
{

string DayOfYear::month = " ";
int dDay = 0;


cout<<"This program helps you find a day of the year."<<endl;

cout<<"Please enter the number of the year that you want me to show:";
cin>>dDay;
cout<<endl;

//Object Creation
DayOfClass object1(dDay);
object1.print();


system("pause");

return 0;

}//end of main function 

[smilodon]

if you want to use the keyword using
using namespace std;

a better idea would be to do the following
std::string myString;

Also I have heard, but am not entirely certain as to why, but don't use system for pausing, rather use
std::cin>>aString;

[firedraco]

Try just DayOfYear::month.

Anyway, using namespace std; is worse then what he is doing here (only using what he is using).

And yes, don't use system()...read here for how to pause without it:
http://www.cplusplus.com/forum/articles/7312/

[smilodon]

for statics I thought you had to include the type of the variable?

[firedraco]

Yeah, but he is already declaring it in the class what type it is...(I don't use statics much though so I could be wrong).

[Duthomhas]

The reason you are getting the error on line 20 is that there is no semicolon terminating your class declaration ending on line 18.

Next, you are trying to do something to a private member variable in your main function on line 119. Don't do that. If you want to initialize it to something (like " ") then do that on line 20.

Finally, on line 130 you mistyped "DayOfYear" as "DayOfClass".


Oh yeah, if you want to use system() -- which I don't recommend; see the link that firedraco posted for you -- then you must also #include <cstdlib>.

Your using statements are fine.

Hope this helps.

[Nelo87]

Thank you everybody.

I went to sleep early yesterday as I was very tired. Now, I can see the mistakes clearly.

Have a good day!