VPF::преобразование к QString при возврате из функции

exelim

Дата 5.8.2013, 21:09 (ссылка)

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

Загрузка ...

Новичок

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

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

Добрый день, имеется такая функция

Код:

Код


QString const &
BasicBlockReporter::getResource( std::string const & _resourcePath ) const
{
return m_resources->getString( _resourcePath ).c_str();

}

метод getString( std::string const & _str ) возвращает const char * . Супервайзер говорит, что тут будут проблемы с памятью ( собственно на этой функции падает приложение). Что не так и что надо исправить? помогите пожалуйста.

bsa

Дата 5.8.2013, 21:15 (ссылка)

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

Загрузка ...

Эксперт

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

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

exelim, функция возвращает указатель. А ты пытаешься возвратить временный объект (c_str() создает C-строку, из которой конструктор QString создает временный объект QString). Для решения проблемы возвращай не ссылку, а объект QString. Или перепиши m_resources->getString таким образом, чтобы она возвращала константную ссылку на QString.

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

Правильно заданный вопрос - половина ответа
Отзывы о работе модератора.

exelim

Дата 5.8.2013, 21:31 (ссылка)

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

Загрузка ...

Новичок

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

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

переписать функцию getString нет возможности, работать надо только с этой функцией. А что если создать объект типа QString а потом вернуть его ?

QString const &
BasicBlockReporter::getResource( std::string const & _resourcePath ) const
{
QString temp = m_resources->getString( _resourcePath ).c_str();
return temp;
}

так получится ? а насчет того, чтобы возвращать не ссылку,а объект - тоже не вариант, надо именно ссылка.

Это сообщение отредактировал(а) exelim - 5.8.2013, 21:32

math64

Дата 6.8.2013, 07:19 (ссылка)

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

Загрузка ...

Эксперт

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

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

Так тоже не получится. Хотя компилятор может не заметить ошибку, tmp создаётся на стеке и будет уничтожена при возврате из функции.
Нужно возвращать QString - этот класс спректирован с учётом того, что часто приходится возвращать QString.
Или сделать tmp полем класса - но тогда tmp будет удалена только при удалениии объекта BasicBlockReporter.

bsa

Дата 6.8.2013, 11:21 (ссылка)

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

Загрузка ...

Эксперт

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

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

Цитата(math64 @ 6.8.2013, 08:19

)

Или сделать tmp полем класса - но тогда tmp будет удалена только при удалениии объекта BasicBlockReporter.

без mutable или const_cast тут не обойтись. Лучше один из моих вариантов.

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

Правильно заданный вопрос - половина ответа
Отзывы о работе модератора.

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