возврат значения через reference параметр - C/C++

Есть функция которая по ряду причин может иметь возвращаемый тип только void
правильно ли будет так возвращать vector из функции

void create(vector<Points>& points)
{
vector<Points> v;
points = v;
}

main()
{
vector<Points> a;
create(a);
}

не будет ли какого нибудь лишнего копирования или чего доброго не прибьет ли выход из функции create созданную на стеке временную переменную v?

Здравствуйте, Vegost, Вы писали:

V>Есть функция которая по ряду причин может иметь возвращаемый тип только void
V>правильно ли будет так возвращать vector из функции

V> void create(vector<Points>& points)
V> {
V> vector<Points> v;
V> points = v;
V> }
V>main()
V>{
V> vector<Points> a;
V> create(a);
V>}

V>не будет ли какого нибудь лишнего копирования или чего доброго не прибьет ли выход из функции create созданную на стеке временную переменную v?

Лишнего не будет, но копирование один раз будет вызвано в месте где points = v;

поэтому я бы так не делал, а делал бы сразу

void create(vector<Point>& points) {
    points.push_back(Point(1,2));
    ...
}

Здравствуйте, Vegost, Вы писали:

V>не будет ли какого нибудь лишнего копирования или чего доброго не прибьет ли выход из функции create созданную на стеке временную переменную v?

все будет хорошо, но, если локальный вектор тебе уже не нужен, ты можешь лишнее копирование убрать:

void foo(std::vector<int>& r) {
  std::vector<int> tmp;
  ...
  r = std::move(tmp);
}

Здравствуйте, Dair, Вы писали:

V>>не будет ли какого нибудь лишнего копирования или чего доброго не прибьет ли выход из функции create созданную на стеке временную переменную v?

D>Лишнего не будет, но копирование один раз будет вызвано в месте где points = v;
D>поэтому я бы так не делал, а делал бы сразу

D>

D>void create(vector<Point>& points) {
D>    points.push_back(Point(1,2));
D>    ...
D>}
D>

Мы не знаем, как формируется внутренний массив и какие требования к целостности данных.
Если таки необходимо строить данные в локальном массиве, то я бы сделал вместо

points = v;

по старинке

points.swap(v); 
// или по-современному 
points = std::move(v);

Здравствуйте, Dair, Вы писали:

D>Лишнего не будет, но копирование один раз будет вызвано в месте где points = v;
D>поэтому я бы так не делал, а делал бы сразу
D>

D>void create(vector<Point>& points) {
D>    points.push_back(Point(1,2));
D>    ...
D>}
D>

На самом деле функция выглядит как то так:

vector<Points> getPoints()
{
vector<Points> v;
...
return v;
}

void create(vector<Points>& points)
{
...
points = getPoints();
...
}

Здравствуйте, Vegost, Вы писали:

V>Есть функция которая по ряду причин может иметь возвращаемый тип только void

иногда дефолтный конструктор дорог или просто отсутствует

    void create(std::optional<vector<Points>>& out_points)
    {
        vector<Points> v;
        out_points.emplace(std::move(v)); //или любой другой конструктор
    }


main()
{
    std::optional<vector<Points>> a;
    create(a);
}

	От:	Vegost
	Дата:	15.02.18 10:26
	Оценка:

От:	Dair	https://dair.spb.ru
Дата:	15.02.18 10:30
Оценка:

	От:	Анатолий Широков
	Дата:	15.02.18 10:31
	Оценка:	+4

	От:	Stanislav V. Zudin
	Дата:	15.02.18 10:35
	Оценка:	+2

	От:	Vegost
	Дата:	15.02.18 10:48
	Оценка:

От:	Caracrist	https://1pwd.org/
Дата:	07.04.18 19:46
Оценка: