VPF::Ошибка в конструкторах? - Форум программистов

sergu

Дата 19.4.2008, 19:15 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 22
Регистрация: 19.9.2006

Репутация: нет
Всего: нет

Написал класс

Код


#include <string>
using namespace std;
class User
{
private:
    unsigned int user_id;
    unsigned int user_name_length;
    unsigned int password_length;
    unsigned int privilege;
    string user_name;
    string password;
public:
    User();
    User(const User &);
    User& operator = (const User& tmp);
    ~User();
    string& GetUserName() {return user_name;}
    string& GetPassword() {return password;}
    unsigned int GetUserId() { return user_id;}
    unsigned int GetPrivilege() { return privilege;}
    unsigned int GetUserNameLength() {return user_name_length;}
    unsigned int GetPasswordLength() {return password_length;}
    void SetUserId(unsigned int id){ user_id = id;}
    void SetUserName(string& new_user_name)
    {
        user_name = new_user_name; 
        user_name_length = user_name.length();
    }
    void SetUserName(const char *new_user_name )
    {
        user_name = new_user_name; 
        user_name_length = user_name.length();
    }
    void SetPassword(string& new_password){ password = new_password;}
    void SetPassword(const char * new_password){ password = new_password;}
    void SetPrivilege(short int new_privilege){ privilege = new_privilege;}
    void SetUserNameLength(unsigned int l = 0)
    {
        if (l == 0) user_name_length = user_name.length();
        else user_name_length = l;
    }
    void SetPasswordLength(unsigned int l = 0)
    {
        if (l == 0) password_length = password.length();
        else password_length = l;
    }

};
User::User()
{
    user_id = 0;
    privilege = 0;
}
//====================================================
User::User(const User &  cpy_user)
{
    user_id = cpy_user.GetUserId();
    user_name_length = cpy_user.GetUserNameLength();
    password_length = cpy_user.GetPasswordLength();
    privilege = cpy_user.GetPrivilege();
    user_name = cpy_user.GetUserName();
    user_name = cpy_user.GetPassword();
}
//====================================================
User& User:: operator = (User& tmp)
{
    if ( this == &tmp ) return *this;
    user_id = tmp.GetUserId();
    user_name_length = tmp.GetUserNameLength();
    password_length = tmp.GetPasswordLength();
    privilege = tmp.GetPrivilege();
    user_name = tmp.GetUserName();
    password = tmp.GetPassword();
}
User::~User()
{
    user_id = 0;
    privilege = 0;
//    user_name.clear();
//    password.clear();
}
void main()
{

}

При компиляции вылазиет ошибка
'GetUserId' : cannot convert 'this' pointer from 'const class User' to 'class User &'
в чем дело?

Daevaorn

Дата 19.4.2008, 19:20 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 2155
Регистрация: 29.11.2004
Где: Москва

Репутация: нет
Всего: 70

Внутри конструктора копий, вы вызываете несколько методов объекта cpy_user. Этот объект является константным, а методы нет. Решение: сделать методы которые возвращают данные, но не изменяют их, константными.

archimed7592

Дата 28.4.2008, 11:03 (ссылка)

(нет голосов)

Загрузка ...

Архимед

Профиль
Группа: Завсегдатай
Сообщений: 2531
Регистрация: 12.6.2004
Где: Moscow

Репутация: 6
Всего: 93

Ага, а делается это так:

Код


string& GetUserName() const {return user_name;}

... и далее по аналогии.

--------------------

If you have an apple and I have an apple and we exchange apples then you and I will still each have one apple. But if you have an idea and I have an idea and we exchange these ideas, then each of us will have two ideas.
© George Bernard Shaw

1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Для новичков \| Следующая тема »