Error C2664, C2228

[slippingslider]

I'm trying to learn how to set ranges by using classes and I get Error c2664 and C2228 when I build. I saw the question in my practice texrbook and I cant figure out what I'm missing.

It says "ui" should have a class on the left of "ui' but it already does.

Its declared right after the Range. Can you help

#include <iostream>
#include <iomanip>
#include <string>
#include <cctype>
using namespace std;
 
class IntRange
{
private:
    int input;
    int lower;
    int upper;
public:
    IntRange(int, int);
    int    getInt();
};
IntRange::IntRange(int lower, int upper)
{
    lower = 50;
    upper = 70;
}
 
int IntRange::getInt()
{
    cin.get(input);
    cin.ignore();
 
    while (input < lower || input > upper)
    {    cout <<" The input number is out of range.\n";
        cout <<" Please enter a value between " << lower;
        cout <<" and " << upper << ".\n";
        cin.get(input);
        cin.ignore();
    }
    return input;
}
int main()
{
 
    IntRange Range(55, 65);
    int ui;
 
    cout << "Enter a number between 55 and 65.\n\n";
    cout << "57 is an exit number, entering 57 would stop the program.\n\n";
    ui = Range.getInt();
    while (ui != 57)
    {
        cout <<"You entered " << ui << endl.
        ui = Range.getInt();
    }
    return 0;
}

[arne]

I'm trying to learn how to set ranges by using classes and I get Error c2664 and C2228 when I build. I saw the question in my practice texrbook and I cant figure out what I'm missing.

It says "ui" should have a class on the left of "ui' but it already does.

Its declared right after the Range. Can you help

#include <iostream>
#include <iomanip>
#include <string>
#include <cctype>
using namespace std;

class IntRange
{
private:
int input;
int lower;
int upper;
public:
IntRange(int, int);
int getInt();
};
IntRange::IntRange(int lower, int upper)
{
lower = 50;
upper = 70;
}

int IntRange::getInt()
{
cin.get(input);
cin.ignore();

while (input < lower || input > upper)
{ cout <<" The input number is out of range.\n";
cout <<" Please enter a value between " << lower;
cout <<" and " << upper << ".\n";
cin.get(input);
cin.ignore();
}
return input;
}
int main()
{

IntRange Range(55, 65);
int ui;

cout << "Enter a number between 55 and 65.\n\n";
cout << "57 is an exit number, entering 57 would stop the program.\n\n";
ui = Range.getInt();
while (ui != 57)
{
cout <<"You entered " << ui << endl.
ui = Range.getInt();
}
return 0;
}

Your line

                cout <<"You entered " << ui << endl.
 

should not end with a '.', but with a ';'. Is that the line the compiler complains about?

[slippingslider]

yeah, u r right but that's for the error c2228 [It should have been a semicolon]

For the error c2664 on line 26 and 33. If I took away the "input" in the cin.get(input), the error would go away but it doesnt yet do what I'd like it to do...

[arne]

yeah, u r right but that's for the error c2228 [It should have been a semicolon]

For the error c2664 on line 26 and 33. If I took away the "input" in the cin.get(input), the error would go away but it doesnt yet do what I'd like it to do...

What about

cin >> input;
 

?

[slippingslider]

What about

cin >> input;
 

?

I tried that already so it builds but the problem is there is something wrong with the input, it doesn't redisplay the entered number. It just shows a 0 even if the number entered is 58.

Thats why I was thinking it was the way I was reading the input so I tried using cin.get

[arne]

I tried that already so it builds but the problem is there is something wrong with the input, it doesn't redisplay the entered number. It just shows a 0 even if the number entered is 58.

Thats why I was thinking it was the way I was reading the input so I tried using cin.get

You may change your constructor to

IntRange::IntRange( int l, int u )
{
    lower = l;
    upper = u;
}
 

and add

    cout << input << endl;
 

after the cin to echo your input.