VPF::Игра в С++ - Форум программистов

Fazil6

Дата 13.7.2007, 19:01 (ссылка)

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

Загрузка ...

Эксперт

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

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

имеем код

Код


const std::string & CheckMaximum(const std::string &x, const std::string &y)
{
// выбор более длиной строки
    if(x.length() > y.length()) return x;

    return y;
}

.........

const std::string a("0");
const std::string &s = CheckMaximum(a, "rrrrrrrrrrrrrr, I am Lion. rrrrrrrr");
const std::string s2 = CheckMaximum(a, s);
std::cout << s2;

Как Вы думаете, что выведется на экран? Кто что вообще думает о том, что тут написано?

Это сообщение отредактировал(а) Fazil6 - 13.7.2007, 20:16

archimed7592

Дата 13.7.2007, 19:11 (ссылка)

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

Загрузка ...

Архимед

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

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

Цитата(Fazil6 @ 13.7.2007, 19:01

)

Как Вы думаете, что выведется на экран?

UB, ибо временный объект сконструированный из сторки "rrrr..." уничтожится(может уничтожиться) в конце sequence point. Т.е до вызова второго CheckMaximum.

Цитата(Fazil6 @ 13.7.2007, 19:01

)

Кто что вообще думает о том, что тут написано?

Думаю, что не хватает комментария к ф-ции, что "несмотря на то, что аргументы представляют из себя константные ссылки, эти ссылки могут быть использованны вне ф-ции(после её завершения), т.о. не рекомендуется вызывать ф-цию с аргументами, ссылающимися на временные объекты".

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

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

bsa

Дата 13.7.2007, 19:17 (ссылка)

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

Загрузка ...

Эксперт

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

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

эх... если бы я знал, когда уничтожается временный объект, то ответил точно... А так есть подозрение, что может сегфолт вывалится при втором сравнении. smile

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

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

archimed7592

Дата 13.7.2007, 19:48 (ссылка)

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

Загрузка ...

Архимед

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

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

Точно. Не может, а именно должен удалится в конце выражения.

Цитата(IS-1.9/12 Program execution)

A full-expression is an expression that is not a subexpression of another expression. If a language construct is defined to produce an implicit call of a function, a use of the language construct is considered to be an expression for the purposes of this definition.

Цитата(IS-12.2/3 Temporary objects)

When an implementation introduces a temporary object of a class that has a non-trivial constructor (12.1), it shall ensure that a constructor is called for the temporary object. Similarly, the destructor shall be called for a temporary with a non-trivial destructor (12.4). Temporary objects are destroyed as the last step in evaluating the full-expression (1.9) that (lexically) contains the point where they were created. This is true even if that evaluation ends in throwing an exception.

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

Doc_d0s

Дата 15.7.2007, 21:30 (ссылка)

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

Загрузка ...

Бывалый

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

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

думаю стоит сделать копирование длинной строки в функции...

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

Админ- это вождь Apache'й :)

archimed7592

Дата 15.7.2007, 22:00 (ссылка)

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

Загрузка ...

Архимед

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

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

Doc_d0s, в вышеозвученном примере одного изменения недостаточно будет. Нужно либо изменить тип возвращаемого значения(убрать ссылку, дабы копировалось) и изменить "клиентский" код, чтобы сохранялась не ссылка, а полностью значение. Для ref-counted строк overhead'а по идее не будет.

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

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