Taking address of temporary [-fpermissive]

Чем заменить , чтобы избежать ошибки?

Функция cv::imread(..) возвращает значание Mat.

0

1

Сообщение от _20_ Чем заменить

Какой-то бессмысленный код.
Если img[i] - это адрес, то в данном выражении он никогда не может быть null, т.к. imread возвращает объект по значению, а значит проверка не имеет смысла, даже если бы адрес временного объекта можно было взять.

Добавлено через 1 минуту
И в #2 проверка также не имеет смысла, т.к. new будет бросать исключение, в случае неудачи, а не возвращать nullptr.
Чего сделать-то надо?

Добавлено через 9 минут
Т.к. не известно зачем там понадобился указатель, и автор не хочет отвечать, пока предложу такой вариант:
учитывая то, что imread работает со значением, не понятно зачем вообще нужна эта лишняя косвенность в первоначальном коде и потенциальные грабли с временем жизни.

1

Сообщение от _20_ Чем заменить

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

1

DrOffset,
Спасибо Вам за пояснения про сравнение, но вопрос был не про это. Меня интересует, как действовать, если функция возвращает объект, который я хочу сохранить. Казалось бы взять адрес да и сохранить в массиве, ан, нет не получается. Вот про это вопрос.

nd2 предложил через оператор new. Но, по моему, это прокатит, если определён конструктор Mat(Mat) (могу ошибаться). А что делать, если нет?

DrOffset, Ваш вариант хорош, если объекты небольшие или скорость не требуется. Одномерные массивы должны быть непрерывны в памяти, и при добавлении следующего элемента места может нехватить и начнётся перенос всех элементов на новое место, что не есть хорошо. Поэтому я использовал массив указателей.
Могли бы Вы объяснить "работает со значанием" - что Вы имели в виду?

TheCalligrapher, Объект должен пережить данную функцию и помереть только с моего разрешения.

0

Сообщение от _20_ А что делать, если нет?

Что значит нет? Конструктор копирования для этого объекта запрещен? Ну тогда ты особо ничего с ним не сделаешь, т.к. если ты хочешь неограниченно "продлить" ему время жизни, то его придется скопировать в память с динамическим размещением.

Сообщение от _20_ Ваш вариант хорош, если объекты небольшие или скорость не требуется. Одномерные массивы должны быть непрерывны в памяти, и при добавлении следующего элемента места может нехватить и начнётся перенос всех элементов на новое место, что не есть хорошо.

Опять эти разговоры про мифическую скорость. Ну да ладно.
О какой нехватке места идет речь? Ты собираешься в этом случае вручную перераспределять память под элементы? Выделять новый массив указателей, копировать туда указатели из старого массива, а старый массив удалять? Не проще ли список тогда использовать?

Сообщение от _20_ Меня интересует, как действовать, если функция возвращает объект, который я хочу сохранить. Казалось бы взять адрес да и сохранить в массиве, ан, нет не получается.

Компилятор тебе бъет по рукам, за то, что пытаешься взять адрес временного объекта, который помрет в конце выражения. Этот адрес, даже если бы его можно было взять, бесполезен, т.к. будет указывать на мертвый объект в следующей строчке.
Продлить жизнь временного объекта (но только до конца области видимости) можно, если использовать ссылку:
Но не зная задачу, трудно сказать подойдет это или нет.

Если же хочется найти какое-то шаблонное решение "на все времена", то это мимо. Надо думать головой в каждой конкретной ситуации. Нет серебряных пуль. А если бы были, то профессия программиста давно выродилась бы, т.к. подверглась бы тотальной автоматизации.

1

Сообщение от _20_ если определён конструктор Mat(Mat) (могу ошибаться). А что делать, если нет?

Для cv::Mat определён: http://docs.opencv.org/2.4/mod... ml#mat-mat

1

Сообщение от DrOffset О какой нехватке места идет речь?

Ну вот например, нашёла программа свободную область памяти размером в 500 Мб. Загрузила первый элемент, второй, третий... а их оказывается 101 по 5Мб каждый. И полсле сотого элемента памяти не хватит и придётся всё перемещть на свободный участок. Лучше уже пусть элементы находятся в памяти где придётся, а в массиве будут ссылки на них.

Сообщение от DrOffset Компилятор тебе бъет по рукам

Компилятор, конечно, молодец, но мне - то как быть, если объект ещё нужен? Видимо придётся так: создать пустой объект оператором new прировнять его результату функции, (или скопировать, как nd2 показал), а потом забрать ссылку в массив. Велосипедом попахивает, но видимо это единственный вариант.

0

Сообщение от _20_ Загрузила первый элемент, второй, третий... а их оказывается 101 по 5Мб каждый.

Здесь подойдет список.

Сообщение от _20_ но мне - то как быть, если объект ещё нужен?

Нормальная ситуация скопировать результат, который вернула функция. Никакой это не велосипед.
В чем проблема?
Если функция всегда должна давать такой объект, временем жизни которого ты сам хочешь полностью управлять, то это изначально должен быть динамический объект. Вывод: в этом случае нужно изменить функцию.

Сообщение от _20_ Видимо придётся так: создать пустой объект оператором new прировнять его результату функции, (или скопировать, как nd2 показал), а потом забрать ссылку в массив. Велосипедом попахивает, но видимо это единственный вариант.

Если что-то кажется неестественным, значит надо менять подход к задаче, а не пытаться натягивать сову на глобус

1

nd2, Определён. Я там ещё доискался функции .empty(), чтоб с нулём не сравнивать. Вопрос более на будущее. Но ответ в принципе уже дан.

DrOffset, Зачем же список? Почему Вам таблица указателей так не нравится?
Доступа к функции нет, а другой подход к копированию переменных я не вижу. Но задача уже решена, большое спасибо всем отписавшимся.

0

Сообщение от _20_ Я там ещё доискался функции .empty(), чтоб с нулём не сравнивать.

Так сравнение с 0, в твоём коде, это не про то, про что empty():

Сообщение от DrOffset И в #2 проверка также не имеет смысла, т.к. new будет бросать исключение, в случае неудачи, а не возвращать nullptr.

1

Сообщение от _20_ Зачем же список? Почему Вам таблица указателей так не нравится?

Ты же сам написал:

Сообщение от _20_ Загрузила первый элемент, второй, третий... а их оказывается 101 по 5Мб каждый. И полсле сотого элемента памяти не хватит и придётся всё перемещть на свободный участок.

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

1

DrOffset,

Большое спасибо за ответ. Думаю, что перенос 100 адресов - это перенос 200 (или 400) байт, более, чем терпимо для моей задачи. Так, что я пока останусь при массивах, но если мне понадобиться вдруг бОльшая скорость создания массива, то я непременно воспользуюсь Вашим советом.

0