Динамическое распределение памяти массив символов

Question

Я дал размер массива вручную, как показано ниже:

int main(int argc, char *argv[]) 
{ 
    char buffer[1024]; 
    strcpy(buffer,argv[1]); 
    ... 
} 

Но если данные, передаваемые в аргументе превышает этот размер, он может создаст проблемы.

Это правильный способ выделения памяти динамически?

int main(int argc, char *argv[]) 
{ 
    int length; 
    char *buffer; 
    length = sizeof(argv[1]); //or strlen(argv[1])? 
    buffer = (char*)malloc(length*sizeof(char *)); 
    ... 
} 

Answer 1

Вам нужно добавить дополнительные байты для хранения завершающего нулевого байта строки:

length=sizeof(argv[1]) + 1; 

Тогда это должно быть в порядке.

Answer 2

sizeof говорит вам размер char*. Вы хотите strlen вместо

if (argc < 2) { 
    printf("Error - insufficient arguments\n"); 
    return 1; 
} 
length=strlen(argv[1]); 
buffer = (char*)malloc(length+1); // cast required for C++ only 

Я предложил несколько других изменений здесь

• вам нужно добавить дополнительные байты buffer для нулевого терминатора
• вы должны проверить, что пользователь прошел в argv[1]
• sizeof(char *) неверно при расчете хранения требуемого для строки. AC строка представляет собой массив char с так что вам нужно sizeof(char), который гарантированно будет 1, так что вам не нужно умножать на него

В качестве альтернативы, если вы работаете на Posix-совместимой системе, вам может упростить и использовать strdup вместо:

buffer = strdup(argv[1]); 

Наконец, убедитесь, что free эту память, когда вы закончите с этим

free(buffer); 

Answer 3

length= strlen(argv[1]) //not sizeof(argv[1]); 

и

//extra byte of space is to store Null character.  
buffer = (char*)malloc((length+1) * sizeof(char)); 

С sizeof(char) всегда один, вы также можете использовать это:

buffer = (char*)malloc(length+1);      

Answer 4

Правильный путь заключается в использовании std::string и пусть C++ сделать работу для вас

#include <string> 

int main() 
{ 
    std::string buffer = argv[1]; 
} 

, но если вы хотите сделать это трудный путь, то это правильно

int main() 
{ 
    int length = strlen(argv[1]); 
    char* buffer = (char*)malloc(length + 1); 
} 

Не забудьте указать +1 для нулевого терминатора, используемого в строках стиля С.

Answer 5

Во-первых, если вы используете C++, я думаю, что лучше использовать new вместо malloc.

Во-вторых, размер malloc равен false: buffer = malloc(sizeof(char) * length);, потому что вы выделяете буфер char, а не char * buffer.

В-третьих, вы должны выделить 1 байт больше для конца своей строки и сохранить '\ 0'.

Наконец, sizeof получает только размер типа, а не строку, вы должны использовать strlen для получения размера строки.

Answer 6

В C++ вы можете сделать это, чтобы получить свои аргументы в хорошей структуре данных.

const std::vector<std::string>(argv, argv + argc)