C / C++ / MFC

12-Aug-04 5:54

I have an enum that I'm trying to declare in my App, that will be accessed by the other dialogs and such in the project. However I keep getting the following error message: error C2061: syntax error :identifier 'MODE'

I've tried to do this before and ended up doing a workaround. I'd rather do it the right way and have never seemed to get the hang of it. Here's what I have in my class definition, with the line it specifies in bold:

class CBugReporterApp : public CWinApp
{
public:
	CBugReporterApp();
	enum _MODE {NEWMODE,EDITMODE,BROWSEMODE} ;

	_MODE MODE;


// Overrides
public:
	virtual BOOL InitInstance();

// Implementation
	afx_msg void OnAppAbout();
	DECLARE_MESSAGE_MAP()
	void SetAppState(MODE enumState);	MODE GetAppState(void);
private:
	MODE m_enumState;
};

I know it's something very stupid, but I can't figure out what I'm doing wrong. Any help would be appreciated. Thanks in advance

[insert witty comment here]

bdiamond

David Crow12-Aug-04 6:07

12-Aug-04 6:07

bdiamond wrote:
I know it's something very stupid...

No, just something you may not have thought of. The enum keyword is not like typedef in that you also have to declare an actual variable of the new type before it can be used. So, just remove the _MODE MODE; statement, and change the three references to MODE to be _MODE instead. Make sense?

"When I was born I was so surprised that I didn't talk for a year and a half." - Gracie Allen

BlackDice12-Aug-04 6:14

12-Aug-04 6:14

I tried that first with just 'MODE' as the name of the enum like this:

class CBugReporterApp : public CWinApp
{
public:
	CBugReporterApp();
	enum MODE {NEWMODE,EDITMODE,BROWSEMODE} ;

	


// Overrides
public:
	virtual BOOL InitInstance();

// Implementation
	afx_msg void OnAppAbout();
	DECLARE_MESSAGE_MAP()
	void SetAppState(MODE enumState);
	MODE GetAppState(void);
private:
	MODE m_enumState;
};

but I get this errormessage:
error C2248: 'CBugReporterApp::GetAppState' : cannot access protected member declared in class 'CBugReporterApp'

[insert witty comment here]

bdiamond

Navin12-Aug-04 6:34

Navin

12-Aug-04 6:34

error C2248: 'CBugReporterApp::GetAppState' : cannot access protected member declared in class 'CBugReporterApp'

What line generates that error? That seems to be something else - this code fragment looks much closer to being correct than the first one you posted.

An expert is somebody who learns more and more about less and less, until he knows absolutely everything about nothing.

BlackDice12-Aug-04 6:49

12-Aug-04 6:49

please see post below in response to DavidCrow's question. Thanks!!

[insert witty comment here]

bdiamond

David Crow12-Aug-04 6:36

12-Aug-04 6:36

You'll need to provide the relevant code for the GetAppState() method.

"When I was born I was so surprised that I didn't talk for a year and a half." - Gracie Allen

BlackDice12-Aug-04 6:42

12-Aug-04 6:42

in looking at the above message where I just declare the enum as 'MODE', here's what I have:

MODE CBugReporterApp::GetAppState(void)
{
	return m_enumState;
}

and here's the line it's now crashing on in my UI handler for a menu button: pCmdUI->Enable(theApp.GetAppState() == CBugReporterApp::BROWSEMODE);

[insert witty comment here]

bdiamond

David Crow12-Aug-04 7:28

12-Aug-04 7:28

At the time the compiler encounters the MODE type, it does not know "who" it belongs to since the GetAppState() method is defined outside of the class declaration. So, you can either put the definition of the GetAppState() method inside of the class declaration, or you can qualify the MODE type by prefacing it with CBugReporterApp. Make sense?

"When I was born I was so surprised that I didn't talk for a year and a half." - Gracie Allen

BlackDice12-Aug-04 7:41

12-Aug-04 7:41

Yes, that makes sense and I changed it and it works for that problem. However I'm still getting this message for the UI update function that I showed earlier:

error C2248: 'CBugReporterApp::GetAppState' : cannot access protected member declared in class 'CBugReporterApp'

As you can see from my class definition, the function is declared as public, so what the heck is it talking about? I don't mean to be a nuisance, but I've gone too long making band-aids for certain problems or just getting something to work and not knowing why.

[insert witty comment here]

bdiamond

David Crow12-Aug-04 8:19

12-Aug-04 8:19

bdiamond wrote:
However I'm still getting this message for the UI update function that I showed earlier:

Show me the whole function and I'll give it a look.

"When I was born I was so surprised that I didn't talk for a year and a half." - Gracie Allen

BlackDice12-Aug-04 9:01

12-Aug-04 9:01

here's the function:

void CMainFrame::OnUpdateMaintenance(CCmdUI *pCmdUI)
{
	pCmdUI->Enable();
	pCmdUI->Enable(theApp.GetAppState() == theApp.BROWSEMODE);
}

I've also tried it with
int i = (int)theApp.GetAppState();

and I still get the same error

[insert witty comment here]

bdiamond Sleepy | :zzz:

David Crow12-Aug-04 9:25

12-Aug-04 9:25

I think you would be better served my adding three "status" methods to your CBugReporterApp class. Something like:

class CBugReporterApp : public CWinApp
{
public:
    bool IsNewMode( void )    { return NEWMODE == m_enumState; }
    bool IsEditMode( void )   { return EDITMODE == m_enumState; }
    bool IsBrowseMode( void ) { return BROWSEMODE == m_enumState; }
};

Does this help?

"When I was born I was so surprised that I didn't talk for a year and a half." - Gracie Allen

BlackDice12-Aug-04 6:56

12-Aug-04 6:56

Even when I comment out the UI handler code I get the following errors:

error C2143: syntax error :missing ';'before 'CBugReporterApp::GetAppState'

error C2556: 'int CBugReporterApp::GetAppState(void)' : overloaded function differs only by return type from 'CBugReporterApp::MODE CBugReporterApp::GetAppState(void)'

 error C2501: 'MODE' : missing storage-class or type specifiers

 error C2371: 'CBugReporterApp::GetAppState' : redefinition; different basic types

[insert witty comment here]

bdiamond