using enum says invalid conversion from 'int' to 'type'

Question

In my class I defined an enum like this:

class myClass 
{
 public:
    enum access {
      forL,
      forM,
      forA
    };
    typedef access AccessType;
    AccessType aType;
};

Later in defined an object like this:

myClass ob;
ob->aType = 0;

However I get this error:

error: invalid conversion from 'int' to 'myClass::AccessType {aka myClass::access}' [-fpermissive]

Don't enum fields map to integers?

Possible duplicate: [Cast int to Enum in C#](http://stackoverflow.com/questions/29482/cast-int-to-enum-in-c-sharp) — dani herrera, Jan 02 '12 at 17:59

score 25 · Accepted Answer · answered Jan 02 '12 at 17:53

25

No, they are stored as integers but they are distinct types (e.g. you can even overload based on the enum type). You must convert explicitly:

myClass ob;
ob->aType = (myClass::AccessType)0;

or ever better write the corresponding named value of the enum:

myClass ob;
ob->aType = myClass::forL;

Or perhaps if you want to use the enum just as a set of integer constants, change the type of the field:

class myClass 
{
 public:
    enum {
      forL,
      forM,
      forA
    };
    int aType; // just stores numbers
};

Conversion from enum to int is implicit.

answered Jan 02 '12 at 17:53

Yakov Galka

70,775
16
139
220

I wrote `ob->aType=myClass::access::forL;` but it says 'myClass::access' is not a class or namespace – mahmood Jan 02 '12 at 18:01
4

I'd suggest the proper C++ cast for this: `auto exampleEnum = static_cast(exampleInt);` – underscore_d Dec 31 '15 at 17:29

bobbymcr · Answer 2 · 2012-01-02T18:05:29.037

13

Enumeration members are backed by integer values but there is no implicit conversion from an integer to an enum type. You need to use an explicit cast if you really want to write it like this:

ob->aType = static_cast<myClass::access>(0);

edited Jan 02 '12 at 18:05

answered Jan 02 '12 at 17:54

bobbymcr

23,769
3
56
67

1

Is this a portable solution (I mean compiler warnings in future)? – mahmood Jan 02 '12 at 17:56
3

As long as the specified integer is in the valid range of the enum, any standard-compliant compiler should allow this as per 7.2.10 of the C++ spec: "An expression of arithmetic or enumeration type can be converted to an enumeration type explicitly. The value is unchanged if it is in the range of enumeration values of the enumeration type; otherwise the resulting enumeration value is unspecified." – bobbymcr Jan 02 '12 at 18:15

score 4 · Answer 3 · answered Jan 02 '12 at 17:54

4

You can't do an implicit cast from int -> enum, since at compile time there is no way to know that the cast is valid.

You can do implicit casts the other way, so you could (if you wanted to) do:

int foo = forL;

answered Jan 02 '12 at 17:54

rejj

1,216
7
13

score 1 · Answer 4 · answered Jul 11 '17 at 16:15

I just had same issue. i have to initialize an object from what i read in an xml file and for sure, i have no control over what could happen to that file.

Constructor has an enum as argument :

enum status_t { NOT_STARTED, STARTED, DONE };
MyObject::MyObject(int id, status_t status) : m_id(id), m_status(status){}

So when parsing the Xml i have to cast it. I prefered then to handle the cast in the constructor so the other classes do not have to know which is the valid enums.

MyObject::MyObject(int id, int status) : m_id(id){
    m_status = status_t(status);
}

But no way to be sure the value coming from xml will be in the correct range.

Here is the solution i came with :

MyObject::MyObject(int id, int status) : m_id(id){
    switch(status){
        case NOT_STARTED:
        case STARTED:
        case DONE:
            m_status=status_t(status);
            break;
        default:
            m_status=NOT_STARTED;
            break;
    }
}

This is an implementation choice, to force a default value in case of non consistent data. One could prefer throwing out an exception, in my case it will do this way.

using enum says invalid conversion from 'int' to 'type'

4 Answers4

Linked