code style - Как я могу проверить целое число против множества возможных значений?

Question

Я пишу небольшую обертку вокруг OpenGL, чтобы использовать ее объектно-ориентированным способом. Я создал класс, называемый буфером, и в его конструкторе я хотел бы, чтобы он проверял тип буфера, который был передан против множества типов GLenum. Вот соответствующая часть кода:

В этом коде switch заявление вмешается к соответствующему значению, и пройти через другие допустимые значения, пока не вырвались из switch заявления. Все остальные значения будут входить в предложение default и выдавать исключение. Я мог бы, однако, также использовать if заявление:

if (type != GL_ARRAY_BUFFER && 
    type != GL_ATOMIC_COUNTER_BUFFER && 
    type != GL_COPY_READ_BUFFER && 
    type != GL_COPY_WRITE_BUFFER && 
    // *** Lots more valid values here... *** 
    ) { 
     throw std::logic_error("Invalid GLenum 'type' in Buffer constructor."); 
    } 

    // ... 
} 

Однако, это использует много повторения type != и не обязательно яснее. Есть ли принятое соглашение при этом, для ясность кода и для скорость? Должен ли я вообще проверять мои входы, когда это не может привести к неопределенному поведению?

Answer 1

Я хотел бы использовать std::set, если есть много значений:

static std::set<int> invalidValues = 
    { GL_ARRAY_BUFFER 
    , GL_ATOMIC_COUNTER_BUFFER 
    , GL_COPY_READ_BUFFER 
    , GL_COPY_WRITE_BUFFER   
    // ... 
    }; 

if(invalidValues.find(type) != invalidValues.end()) { 
    throw std::logic_error("Invalid GLenum 'type' in Buffer constructor."); 
} 

Это будет служить для скорости и лучшей читаемости.

Следует ли вообще проверять мои входы, если это не может привести к неопределенному поведению?

Вы никогда не хотите undefined поведение в вашей программе. Вы не можете полагаться, что это приведет к исключению (сбою) или такому.