Как разумно получить доступ к приватной переменной

Tolias28 · Регистрация: 18.08.2010

Author24 — интернет-сервис помощи студентам

Есть в классе приватная переменная clientName. Мне нужно защитить ее так, чтобы для внешних классов она была только для чтения. Не получается. Например, следующий код не спасает:
Попытка №1

C++

class Client
{
private:
    wchar_t clientName[20];
 
public:
    wchar_t* getClientName()
    {
        return clientName;
    }
};

Потому что функция getClientName() возвращает указатель, которому если присвоить другой массив, тогда в переменной clientName класса будет содержаться уже совсем другая строка. То есть, уже нету защиты от записи извне.

C++

Client* c = new Client(L"test_client");
wchar_t* clName = c->getClientName();
wcscpy(clName, L"overrode client name");
wchar_t* checkClientName = c->getClientName(); //здесь функция возвратит уже совсем другую строку "overrode client name", а не "test_client"

Попытка №2
в функции getClientName() создавать новую строку, копировать в нее строку clientName и возвращать.

C++

wchar_t* getClientName()
{
    wchar_t tempClientName[20];
    wcscpy_s(tempClientName, clientName);
    return tempClientName;
}

На этот раз все вроде работает. Но компилятор предупреждает:

warning C4172: returning address of local variable or temporary

И правильно предупреждает. В моей практике уже было такое, что при возвращении адресса локальной переменной в этом адресе впоследствии содержалась не строка, а муссор. Значит этот способ тоже отпадает.

Попытка №3
Все то же самое, что во втором способе, только выделять память динамически

C#

wchar_t* getClientName()
{
    wchar_t* tempClientName = new wchar_t[20];
    wcscpy_s(tempClientName, 20, clientName);
    return tempClientName;
}

Этот способ пашет. НО! после каждого вызова функции getClientName() придется самостоятельно очищать память:

C++

1
2
3

wchar_t* clName = c->getClientName();
std::wcout << "Client name: " << clName;
delete(clName);

А это чрезвычайно неудобно и к тому же невозможно например в этом случае:

C++

1	std::wcout << "Client name: " << c->getClientName();

Ну и вот как здесь очистить память?

Простите меня новичка в C++, если я чего нибудь не знаю. Прошу поделиться, как реализовать доступ только чтения к переменной класса так, чтобы отсутствовали все выше перечисленные недостатки. (Раньше кодил на C# - там таких проблем и близко нет. Можешь полостью сосредоточиться на логике программы. А здесь в плюсах приходится тратить уйму времени на сам код....)

Байт · 05.03.2013, 14:01

C++

wchar_t* getClientName()
{
    static wchar_t tempClientName[20];
    wcscpy_s(tempClientName, clientName);
    return tempClientName;
}

static wchar_t tempClientName[20];

Tolias28 · 05.03.2013, 14:04 **[ТС]**

Байт, спасибо большое! Я сколько сидел и голову ломал, как это сделать, а как все просто, оказывается

@Andrew_Lvov · 05.03.2013, 14:05

Код

const wchar_t* getClientName() const;

@gray_fox · 05.03.2013, 14:05

Указатель на константу?

C++

wchar_t const* getClientName() const
{
    return clientName;
}

@Andrew_Lvov · 05.03.2013, 14:06

Не по теме:

gray_fox, что ж вы такой медлительный ? До вас уже ответили! )))

@ForEveR · 05.03.2013, 14:08

Использовать string, если уж на С++ пишете. И возвращать по значению/const ссылке.

@ITcrusader · 05.03.2013, 14:13

При всем уважении к варианту выше, не лучше ли так?

C++

    const wchar_t* getClientName() const
    {
        return clientName;
    }

Никаких копий и вечно живущих до конца программы массивов. Никаких накладных расходов на копирование.
Функция вернет указатель, позволяющий только лишь читать данные по адресу. Мне кажется - самый изысканный способ решить проблему.
P.S. Тьфу! Опять опоздал!

@gray_fox · 05.03.2013, 14:15

Не по теме:

Сообщение от Andrew_Lvov

gray_fox, что ж вы такой медлительный ? До вас уже ответили! )))

Ok (
)

Tolias28 213 / 137 / 8 Регистрация: 18.08.2010 Сообщений: 1,018
	05.03.2013, 14:04 [ТС]	3
	Байт, спасибо большое! Я сколько сидел и голову ломал, как это сделать, а как все просто, оказывается 0

@Andrew_Lvov 261 / 191 / 10 Регистрация: 19.08.2010 Сообщений: 760 Записей в блоге: 1
	05.03.2013, 14:05	4
	Код const wchar_t* getClientName() const; 1

@Andrew_Lvov
	05.03.2013, 14:06 #6
	Не по теме: gray_fox, что ж вы такой медлительный ? До вас уже ответили! ))) 0

@ForEveR В астрале 8049 / 4806 / 655 Регистрация: 24.06.2010 Сообщений: 10,562
	05.03.2013, 14:08	7
	Использовать string, если уж на С++ пишете. И возвращать по значению/const ссылке. 1

@gray_fox
	05.03.2013, 14:15 Как разумно получить доступ к приватной переменной #9
	Не по теме: Сообщение от Andrew_Lvov gray_fox, что ж вы такой медлительный ? До вас уже ответили! ))) Ok ( ) 0