Время жизни переменных, не понимаю одного момента Опции

[Andrew Selivanov]

А вот auto_ptr с массивом использовать не надо - убъёть!

Litkevich Yuriy Проще всего переделать код на использование vector<int>.
С ним твоя исходная функция будет выглядеть так:

vector<int> func (foo foovar)
{
  vector<int> foodim;
  ...
  return foodim;
}

А использовать её будешь так:

vector<int> pntr;
int  b;
pntr = func(foovar);
b=pntr[2];

Написано об этом очень хорошо у Страуструпа в самом начале "Язык программирования С++ 3ед." - книжка, которую каждый уважающий себя программист на С++ должен прочитать и понять.

Мммм... вариант, но мне кажется, что так будет несколько неудобно обрабатывать ошибку (т.е. нулевое значение при возврате), кроме того ( поправьте если ошибаюсь), из временного внутреннего вектора будет лишнее копирование. Мне кажется, корректней будет так:

int func(int foovar, std::vector<int>& data)
{
    if(shit_happens)
   {
      //неуспешное завершение
      return -1;
    }
    data.push_back(val1);
    data.push_back(val2);
    data.push_back(val3);
    //успешное завершение
    return 0;
}

//использование
std::vector<int> data;
int retval = func(3,data);
if(retval)
{
    //что то не так...
}

Ну и typedef не забыть если это конкретный тип, или сделать сразу шаблончик из функции

И еще вариант: можно вернуть итератор...

[Tonal]

Если это действительно ошибка - кидай исключение.
Если - одно из возможных значений - можно обойтись возвратом пустого вектора.

Насчёт копирования - вменяемые компиляторы С++ применяют оптимизацию возвращаемого значения, так что в итоге, скомпилированный код будет очень похож на код с передачей ссылки.
Хотя если не очень доверяешь компилятору - можно и руками пооптимизировать, но мне кажеться, что делать это надо только когда понятно что именно.

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

Сообщение отредактировал Tonal - 5.2.2008, 8:37

[Litkevich Yuriy]

видно что размерчик ему известен

Да известен, это буфер для работы с UART'ом, собственно, некоторые функции кладут в этот буфер в определенное место свои данные, потом буфер уезжает к внешнему девайсу, максимальный размер буфера в котором ЦЕЛЫХ 28 байт (для девайса это много).

[AD]

А вот auto_ptr с массивом использовать не надо - убъёть!

Litkevich Yuriy Проще всего переделать код на использование vector<int>.
С ним твоя исходная функция будет выглядеть так:

vector<int> func (foo foovar)
{
  vector<int> foodim;
  ...
  return foodim;
}

А использовать её будешь так:

vector<int> pntr;
int  b;
pntr = func(foovar);
b=pntr[2];

Написано об этом очень хорошо у Страуструпа в самом начале "Язык программирования С++ 3ед." - книжка, которую каждый уважающий себя программист на С++ должен прочитать и понять.

Полностью согласен с данным автором. На мой взгляд, если и делать подобное, то, как указано выше. А вообще не советовал бы возвращать указатель на локальный массив. Лучше бы завести его глобально или через namespace.