How to get a entrance via enum type in a constructor in c++

Question

1.00/5 (1 vote)

See more:

Hi...
I have a class like so :

C++

class ERROR {
public :
	enum ENUM_ERROR{
		ERROR1001,
		ERROR1002,
		ERROR1004
	};
	std::string ERROR(std::string*);
};

And i want to use the enum as a entry :

C++

std::string ERROR::ERROR(ERROR::ENUM_ERROR error) {
	std::ostringstream ERROR1002 (std::ostringstream::ate);
	ERROR1002 << "\033[01;31mERROR 1002 : You must insert a value between 001 and " <<  MAX_NODE << "!\033[0m" << "\n\033[01;33mYou have inserted : ";
	switch(error) {
	case ERROR::ERROR1001:
	{
		return "\033[01;31mERROR 1001 : You must insert an integer number!\033[0m";
	}
	case ERROR::ERROR1002:
	{
		return ERROR1002;
	}
	case ERROR::ERROR1004:
	{
		return "\033[01;31mERROR 1004 : Both numbers must not be equal\033[0m";
	}
	}
}

but compiler gives me this three errors :

C++

definition.h:13:13: error: ‘ERROR’ has not been declared
 std::string ERROR::ERROR(ERROR::ENUM_ERROR error) {
             ^
definition.h:13:26: error: ‘ERROR’ is not a class or namespace
 std::string ERROR::ERROR(ERROR::ENUM_ERROR error) {
                          ^
definition.h:13:51: error: expected ‘,’ or ‘;’ before ‘{’ token
 std::string ERROR::ERROR(ERROR::ENUM_ERROR error) {
                                                   ^

What am i doing wrong?

Posted 9-Sep-15 21:28pm

JENOVA_SHINRA

Add a Solution

Comments

Sergey Alexandrovich Kryukov 10-Sep-15 3:38am

What did you mean by
std::string ERROR(std::string*);?
—SA

JENOVA_SHINRA 10-Sep-15 3:52am

This is how NS2 creates it :
int command(int, const char *const *);
and this is how it uses it :
AODV::command(int argc, const char*const* argv) {

JENOVA_SHINRA 10-Sep-15 3:44am

I just wanted to create a entry like i saw in NS2...
I saw in NS2 that so many variables has been created like that.

3 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

CPallini · Accepted Answer · 2015-09-09T21:53:00

Quote:
std::string ERROR(std::string*);

Wrong: if you give the name of the class to an ordinary method, then the compiler considers it a constructor, but constructors must NOT have return values.

Quote:
td::string ERROR::ERROR(ERROR::ENUM_ERROR error)

Wrong: there is no declaration for this method (because the signature doesn't match). Moreover, the same argument of the previous mistake applies: the name of the class is reserved for constructors.

Possibly you meant:

C++

class ERROR {
public :
    enum ENUM_ERROR
    {
        ERROR1001,
        ERROR1002,
        ERROR1004
    };
    static std::string Error(ENUM_ERROR ee);
};

C++

std::string ERROR::Error(ERROR::ENUM_ERROR ee) 
{
  std::ostringstream ERROR1002 (std::ostringstream::ate);
  //..
}

Sergey Alexandrovich Kryukov · Accepted Answer · 2015-09-09T21:36:00

Solution 1

This is not a valid C++ code. You need declare all types before using them. The purpose of the line std::string ERROR(std::string*); is not clear and does not seem to make any sense to me. I suggest you just remove it. Or create a constructor instead.

Perhaps you need to explain your goals, but the enumeration type can be perfectly used in the switch statement, pretty much the way you do it. Your compilation errors are totally unrelated to it.

—SA

Posted 9-Sep-15 21:36pm

Sergey Alexandrovich Kryukov

Comments

CPallini 10-Sep-15 4:10am

Have my 5.

Sergey Alexandrovich Kryukov 10-Sep-15 4:26am

Thank you, Carlo.
Believe or not, but I just posted another, bigger one, Solution 3, in response to the comment which reveals that the inquirer is not really getting it. :-)
—SA

Sergey Alexandrovich Kryukov · Accepted Answer · 2015-09-09T22:24:00

JENOVA_SHINRA wrote:

This is how NS2 creates it:

C++

int command(int, const char *const *);
and this is how it uses it :
AODV::command(int argc, const char*const* argv) {

Don't tell me that some software "creates" something which would not compile. :-)

I have no idea what makes you thinking why this code sample has anything to do with yours. Isn't it quite obvious? Your wanna-be-a-function has the name which is the same as the name of the class, which immediately makes it not a valid function. That's why I mentioned a constructor.

Let me tell you what I think: your whole activity is one big abuse: trying to write "code" having no clue of the use of the language you are trying to use. This is just a waste of time, yours and ours. Here, I spend time on answering your questions, but have little hope that you can use the answers, so it would be just the waste. Asking questions of certain type has its prerequisites. Don't start development before you read some C++ book at least once (later on, you can use it as a reference), of course, doing some simple exercises as you go would be very useful.

—SA