Thread: error C2065 Help me please

I'm coding my homework Calendar program but it have error
(error C2065: 'JANUARY' : undeclared identifier) Help me please.

Date.h

Code:

#ifndef _Date_
#define _Date_
using namespace std;
class Date{
public:
    enum Month{JANUARY=1,FEBRUARY,MARCH,APRIL,MAY,JUNE,JULY,AUGUST,SEPTEMBER,OCTOBER,NOVEMBER,DECEMBER};
    Date(void);
    Date(int,Date::Month,int);
    Date(Date::Month,int,int);
    int getYear(void);
    Date::Month getMonth(void);
    int getDay(void);
    int daysInMonth(Date::Month,int);
    bool isLeapYear(int);
    string toString(void);
    void setDay(int);
    void setMonth(int);
    void setYear(int);
    string monthToString(Date::Month);
private:
    int d,y;
    Month m; 
};
#endif

Date.cpp

Code:

#include<iostream>
#include<string>
#include"Date.h"
#include<sstream>
using namespace std;
Date::Date(void){
    d = 1;
    m = JANUARY;
    y = 2000;
}
Date::Date(int dd,Date::Month mm,int yy){
    d = dd;
    m = mm;
    y = yy;
}
Date::Date(Date::Month mm,int dd,int yy){
    d = dd;
    m = mm;
    y = yy;
}
void main(){
    Date aaa(1,JANUARY,1998);
    
}

in main function Date aaa(1,JANUARY,1998);
error C2065: 'JANUARY' : undeclared identifier

Outside the class, you need to fully qualify the name.

Code:

int main(){
    Date aaa(1,Date::JANUARY,1998);
     
}

Three more things.
1. main returns int, not void.

2. Putting "using namespace" inside a header file is poor style.
If you were writing a library of some sort, and your library user had several namespaces to deal with, they might not appreciate the wholesale inclusion of ::std into their project.

3. There is no need for all the Date:: scope resolutions within your Date class.

Originally Posted by Salem

2. Putting "using namespace" inside a header file is poor style.
If you were writing a library of some sort, and your library user had several namespaces to deal with, they might not appreciate the wholesale inclusion of ::std into their project.

If you want to put "using namespace" inside a header file, why not do it in the following way?

Code:

#ifndef FOO_H
#define FOO_H
#include "bar.h"

namespace foo
{
using namespace bar;
   // ...
   int foovar = 123;
}

#endif // FOO_H

In this case, anyone who includes "foo.h" does not automatically bring in the bar namespace to their source file.

Then move the enum out of the class.

Also some good advice: SourceForge.net: Do not remove parameter names - cpwiki