Logical Error!

[cher1201]

Hey guys! So I executed this program and for some reason, whenever I input valid values for areacode, exchange, etc. it returns as an invalid statement and asks the user to reenter it. Somewhere along the way there's a simple logical error that needs fixing. Can any of you wonderful people find what's wrong?

#include "phone.h"

Phone::Phone():msg(10){ //intialize const data member

        areacode = 999;

        exchange = 999;

        line = 9999;
}

Phone::Phone(int ac, int ex, int l, int mval):msg(mval){ //initialize const data member

        areacode = ac;

        exchange = ex;

        line = l;

}


bool Phone::setAreaCode(int ac){

        bool validate = false;

        if (ac >= 0 && ac <= 999){
                areacode = ac;
                bool validate = true;
        }

        return (validate);
}

bool  Phone::setExchange(int ex){

        bool validate = false;

        if (ex >= 0 && ex <= 999){
                exchange = ex;
                bool validate = true;
        }

        return (validate);
}

bool Phone::setLine(int l){

        bool validate = false;
        if (l >= 0 && l <= 9999){
                line = l;
                bool validate = true;
        }

        return (validate);
}

void Phone::input(){
        char slash;

        int ac, ex, l;
        do{
                cout << "Please input your phone number EX: (123) 456 - 7890." << endl;
                cin >> slash >> ac >> slash >> ex >> slash >> l;

        } while(!setAreaCode(ac) && !setExchange(ex) && !setLine(l));
}

void Phone::display (){

        cout << "The maximum number text messages you have are: " << msg << endl;
        cout << "(" << getAreaCode() << ") " << getExchange() << " - " << getLine() << endl;
}

int Phone::getAreaCode(){

        return areacode;
}

int Phone::getExchange(){

        return exchange;
}

int Phone::getLine(){

        return line;
}

[ne555]

        bool validate = false;

        if (ac >= 0 && ac <= 999){
                areacode = ac;
                bool validate = true; //here you declare _another_ variable
        }

[cher1201]

I'm sorry, but when you say to declare another variable, what do you mean?

[pearlyman]

bool Phone::setAreaCode(int ac){

        bool validate = false;             // (Declared as a bool type)

        if (ac >= 0 && ac <= 999){
                areacode = ac;
                bool validate = true;      // (Declared as a bool type)
        }

When you declare a variable, you name the type, bool.
after that you only need to name the variable. So it would become

bool Phone::setAreaCode(int ac){

        bool validate = false;             // (Declared as a bool type)

        if (ac >= 0 && ac <= 999){
                areacode = ac;
                validate = true;            //  reassigned variable (no declaration) 
        }

[cher1201]

Thank you so much guys! Appreciate all the help!