How do I force users to input integers only in my program?

[48T7M4Gy]

The core issue for me is the testing of cin to handle the three limitations of a single input which is a simple one-liner. Looping around until the input satisfies the contract was secondary to the OP. We don't have enough information to know what is right or wrong here primarily because such assessments miss the point of the OP.

[48T7M4Gy]

/*
 Checks that input is an integer and within the set limits
 */

#include <iostream>

int main()
{
    
    std::cout << "Please enter a number between 1 and 7: " << std::endl;
    
    bool keepGoing = true;
    
    int x = -1;
    
    while(keepGoing == true)
    {
        std::cin >> x;
        if( !std::cin || x < 1 || x > 7 )
        {
            std::cout << "OOPS\n";
        }
        else
        {
            std::cout << "Bingo " << x << '\n';
        }
        
        std::cin.clear(); //clears error flags
        std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');//clears buffer
    }
    
    return 0;
}

[Pratik K]

@duoas Thanks. I thought about it and yeah there is a way to do it with strcmp but it'd be much longer and involve too many if elses to remain efficient. ignore is the way to go.

[48T7M4Gy]

I can't see any need for strcmp and cin.ignore() is not complete without cin.clear(). Try it by commenting out my line 28. In short, if you don't use clear then the stream will be in a permanent error state.