Прототип для функции, которая выделяет память на "куче" (C/C++)

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

Вместо этого вы должны вернуть объект, который соответствующим образом управляет своим ресурсом. (Найдите RAII.) Укажите тот факт, что ресурс в этом случае представляет собой массив из char , либо std :: string , либо std :: vector ] кажутся лучшими:

int f(std::vector<char>& buffer);

std::vector<char> buffer;
int result = f(buffer);

В «правильном» C ++ вы должны вернуть объект, содержащий выделение памяти где-то внутри него. Что-то вроде std :: vector.

Почему бы не сделать то же самое, что и malloc () - void * malloc (size_t numberOfBytes)? Таким образом, количество байтов является входным параметром, а выделенный адрес блока - возвращаемым значением.

UPD: В комментариях вы говорите, что f () в основном выполняет некоторые действия помимо выделения памяти. В этом случае использование std :: vector - намного лучший способ.

void f( std::vector<char>& buffer )
{
    buffer.clear();
    // generate data and add it to the vector
}

вызывающий будет просто передать выделенный вектор:

 std::vector buffer;
 f( buffer );
 //f.size() now will return the number of elements to work with

Передайте указатель по ссылке ...

int f(char* &buffer)

Однако вы можете рассмотреть возможность использования указателей с подсчетом ссылок, таких как boost :: shared_array, для управления памятью, если вы просто начиная с этого.

например,

int f(boost::shared_array<char> &buffer)

Используйте шаблон проектирования RAII (получение ресурсов - инициализация).

http: //en.wikipedia.org/wiki/RAII Понимание значения термина и концепции - RAII (получение ресурсов - это инициализация)

Просто верните указатель:

char * f() {
   return new char[100];
}

Сказав, что , вам, вероятно, не нужно связываться с явным распределением, подобным этому - вместо массивов символов используйте std :: string или std :: vector .

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

class Cookie {
public:
   Cookie () : pointer (new char[100]) {};
   ~Cookie () {
      delete[] pointer;
   }
private:
   char * pointer;

   // Prevent copying. Otherwise we have to make these "smart" to prevent 
   // destruction issues.
   Cookie(const Cookie&);
   Cookie& operator=(const Cookie&);
};

Если все f () делает с буфером, чтобы вернуть его (и его длину), пусть он просто вернет длину, и вызовет new его. Если f () также что-то делает с буфером, то действуйте так, как предполагалось полиглотом.

Конечно, может быть лучший дизайн для проблемы, которую вы хотите решить,

Правильный стиль, вероятно, состоит в использовании не char *, а std :: vector или std :: string в зависимости от того, для чего вы используете char *.

О проблеме передача параметра, который необходимо изменить, вместо передачи указателя передайте ссылку. В вашем случае:

int f(char*&);

и если вы последуете первому совету:

int f(std::string&);

или

int f(std::vector<char>&);

Я думаю, вы пытаетесь выделить одномерный массив. Если это так, вам не нужно передавать указатель на указатель.

int f(char* &buffer)

должно быть достаточно. И сценарий использования будет следующим:

char* data;
int data_length = f(data);
// ...
delete[] data;

При условии, что f выполняет new [] для сопоставления, он будет работать, но это не очень идиоматично.

Предполагая, что заполняет данные, а не просто malloc () - похоже, вам лучше обернуть распределение как std :: vector

void f(std::vector<char> &buffer)
{
    // compute length
    int len = ...

    std::vector<char> data(len);

    // fill in data
    ...

    buffer.swap(data);
}

EDIT - удалить ложный * из подписи