Правильный синтаксис для использования new для char * array

Question

Правильный синтаксис для использования new для char * array

0

У меня есть следующее

typedef struct
{
    int        titleCount; 
    char**     titles;
} myStruct;

А потом

...
struct1->titleCount = 2;
struct1->titles = (char**) malloc(sizeof(char *) * (str->titleCount + 1));
...

Какой был бы правильный синтаксис для использования new вместо malloc?

ontherocks 19 фев. 2014, в 22:44

Источник

5

Наиболее приемлемый способ - это std::vector<std::string> titles; и тогда вы все в порядке.
chris 19 фев. 2014, в 21:17
0

Не могу помочь Существующий код :(
ontherocks 19 фев. 2014, в 21:19
1

Почему существующий код выделяет память для 3 char * когда у вас есть 2 заголовка? Вы заполняете последнее место NULL и используете это как разделитель?
Praetorian 19 фев. 2014, в 21:21
0

@Praetorian Да, я думаю, это было бы причиной
ontherocks 19 фев. 2014, в 21:23
0

@ontherocks, тогда какова titleCount ? Я имею в виду, что я предпочел бы переменную над разделителем, но иметь и то и другое немного ... бессмысленно.
chris 19 фев. 2014, в 21:24
3

Если вы портируете с C на C ++, почему вы делаете полумеры. Можно также оставить malloc внутри, пока вы не сможете с этим справиться
Ed Heal 19 фев. 2014, в 21:26

Показать ещё 4 комментария

Теги:

c++

malloc

memory-management

3 ответа

2

Если вы портируете с C на C++, лучшим решением будет:

std::vector<std::string> myTitles;

Вся структура не нужна.

Zac Howland 19 фев. 2014, в 19:11

0

Прежде всего, ваш код выделения памяти неверен, потому что вы не можете писать

struct1->titleCount

если только вы не написали

struct struct1 *struct1 = malloc( sizeof( struct struct1 ) );

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

str->titleCount + 1

элементы типа char *

Соответственно, соответствующий код будет выглядеть так:

str->titleCount = 2;
str->titles = new char * [str->titleCount + 1];

В следующий раз, пожалуйста, покажите фрагмент кода, который можно скомпилировать. Из вашего кода неясно, что такое struct1 и str. Вы только путаете других.

Vlad from Moscow 19 фев. 2014, в 19:57

Ещё вопросы

Наиболее приемлемый способ - это std::vector<std::string> titles; и тогда вы все в порядке.
Не могу помочь Существующий код :(
Почему существующий код выделяет память для 3 char * когда у вас есть 2 заголовка? Вы заполняете последнее место NULL и используете это как разделитель?
@Praetorian Да, я думаю, это было бы причиной
@ontherocks, тогда какова titleCount ? Я имею в виду, что я предпочел бы переменную над разделителем, но иметь и то и другое немного ... бессмысленно.
Если вы портируете с C на C ++, почему вы делаете полумеры. Можно также оставить malloc внутри, пока вы не сможете с этим справиться

Tim Bergel · Accepted Answer · 2014-02-19T19-54-00.000Z

в этом примере заголовки указывают на массив указателей на символ или, скорее всего, на строки. Поэтому я бы ожидал чего-то вроде:

titles = new char*[str->titleCount]; // or maybe keep the +1

а затем цикл, чтобы выделить отдельные строки и поместить в них указатели в массив, на который указывают заголовки.

Это работает. Однако я могу присвоить titles 2 строки, даже не указав размер в [] . Почему так?
Я думаю, что если вы сделаете новый символ * [], вы получите один символ *. Возможность хранить 2 строковых указателя - это просто удача - у второго перезаписана память, которой не должно быть, что в лучшем случае приведет к неопределенному поведению, а в худшем - к аварийному завершению вашей программы.
Да, я думаю, и по этой причине даже new char* (без скобок) работает.
Абсолютно - хотя я не уверен, что новый char * [] является частью стандарта (вероятно, да), новый char * абсолютно такой же, как и способ выделения одного экземпляра чего-либо.