Функция возвращает некорректные данные

@pavel2210057 · Регистрация: 28.09.2017

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

Привет! Сразу к делу: не понимаю, почему функция возвращает неверное значение.
Вот код метода:

C++
1
2
3
inline Vec2 <float> Collusion::get_nearest_party(void) const {
    return Vec2<float>(Scalar2<float>(0, 0), Scalar2<float>(0, 0)); //возвращаю нулевой вектор для примера
}

При этом в выводе в консоль корректно (в данном случае 0) выводится только первое число.
Вот вывод:

C++
1
2
    std::cout << vec2->vec_begin->x << "\t" << vec2->vec_begin->y << "\t" <<
                    vec2->vec_end->x << "\t" << vec2->vec_end->y << "\n";

Присваиваю значение так:

C++
1
    Vec2<float> *vec2 = &collusion.get_nearest_party(); //делаю физический движок, поэтому не думайте о названиях

При это вывод становится корректным при других способах инициализации вектора, например:

C++
1
    Vec2<float> *vec2 = new Vec2<float>(Scalar2<float>(0, 0), Scalar2<float>(0, 0));

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

).
Помогите разобраться, это дико тормозит работу.

MrGluck · 15.05.2018, 19:17

Вы собираетесь вернуть адрес локальной переменной из функции. А если эта переменная расположена на стеке, то при выходе из скоупа (в данном случае тела функции), её содержимое обращается в тыкву. А вы пытаетесь обратиться по адресу и прочитать оттуда данные. Размещение данных в куче позволяет избежать эту проблему (данные остаются на том же месте в выделенном участке памяти), однако просто так возвращать динамическую память не рекомендуется. Либо делайте [[nodiscard]] хотя всё равно остаётся вероятность, что данные не освободят после использования. Либо пользуйтесь смарт поинтерами. Хотя можете отказаться от указателей и работать с обычными переменными, тогда всё будет ОК. Благо, RVO (NRVO) позволяет нам делать это без потери производительности.

@pavel2210057 · 15.05.2018, 19:26 **[ТС]**

MrGluck, спасибо! При замене указателей на обычные переменные функция возвращает корректные данные. Печально одно - столько прог написал, много работая с указателями, думал, что уже знаю их в совершенстве, а тут бац

MrGluck · 15.05.2018, 19:29

Это известная проблема, многие на неё натыкаются
Распространенные ошибки

Новые блоги и статьи Все статьи Все блоги /
Отображение реквизитов в документе по условию и контроль их заполнения Maks 04.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеСпецтехники", разработанного в конфигурации КА2. Данный документ берёт данные из другого нетипового документа. . .	Фото всей Земли с борта корабля Orion миссии Artemis II kumehtar 04.04.2026 Это первое подобное фото сделанное человеком за 50 лет. Снимок называют новым вариантом легендарной фотографии «The Blue Marble» 1972 года, сделанной с борта корабля «Аполлон-17». Новое фото. . .	Вывод диалогового окна перед закрытием, если документ не проведён Maks 04.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: реализовать программный контроль на предмет проведения документа. . .	Программный контроль заполнения реквизита табличной части документа Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: реализовать контроль заполнения реквизита "ПричинаСписания". . .
wmic не является внутренней или внешней командой Maks 02.04.2026 Решение: DISM / Online / Add-Capability / CapabilityName:WMIC~~~~ Отсюда: https:/ / winitpro. ru/ index. php/ 2025/ 02/ 14/ komanda-wmic-ne-naydena/	Программная установка даты и запрет ее изменения Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: при создании документов установить период списания автоматически. . .	Вывод данных в справочнике через динамический список Maks 01.04.2026 Реализация из решения ниже выполнена на примере нетипового справочника "Спецтехника" разработанного в конфигурации КА2. Задача: вывести данные из ТЧ нетипового документа. . .	Программное заполнения текстового поля в реквизите формы документа Maks 01.04.2026 Алгоритм из решения ниже реализован на нетиповом документе "ВыдачаОборудованияНаСпецтехнику" разработанного в конфигурации КА2, в дополнении к предыдущему решению. На форме документа создается. . .

MrGluck Форумчанин 8216 / 5047 / 1437 Регистрация: 29.11.2010 Сообщений: 13,453
	15.05.2018, 19:17
	Вы собираетесь вернуть адрес локальной переменной из функции. А если эта переменная расположена на стеке, то при выходе из скоупа (в данном случае тела функции), её содержимое обращается в тыкву. А вы пытаетесь обратиться по адресу и прочитать оттуда данные. Размещение данных в куче позволяет избежать эту проблему (данные остаются на том же месте в выделенном участке памяти), однако просто так возвращать динамическую память не рекомендуется. Либо делайте [[nodiscard]] хотя всё равно остаётся вероятность, что данные не освободят после использования. Либо пользуйтесь смарт поинтерами. Хотя можете отказаться от указателей и работать с обычными переменными, тогда всё будет ОК. Благо, RVO (NRVO) позволяет нам делать это без потери производительности. 1

@pavel2210057 61 / 28 / 24 Регистрация: 28.09.2017 Сообщений: 399
	15.05.2018, 19:26 [ТС]
	MrGluck, спасибо! При замене указателей на обычные переменные функция возвращает корректные данные. Печально одно - столько прог написал, много работая с указателями, думал, что уже знаю их в совершенстве, а тут бац 0

MrGluck Форумчанин 8216 / 5047 / 1437 Регистрация: 29.11.2010 Сообщений: 13,453
	15.05.2018, 19:29
	Это известная проблема, многие на неё натыкаются Распространенные ошибки 0