Копирование массивов в C

Question

В моей программе на C я пытаюсь скопировать массив символов в другой массив, удалив первый элемент (элемент 0).

Я написал:

char array1[9]; 
char array2[8]; 
int i, j; 

for(i = 1, j = 0 ; i < 10, j < 9; i++, j++){ 
     array2[j] = array1[i]; 
} 
printf(array2); 

При печати array2, это дает мне переполнение стека.

Любые идеи?

Answer 1

Ваша строка не завершена нулевым символом, поэтому, когда она печатается, она продолжает печатать символы, предшествующие 8, которые вы выделили, и ищет их, но до этого времени заканчивается пространство стека. Вы также пишете на один символ больше, чем вы выделили, и ваши условия должны быть «объединены» с && - a , игнорирует результат первого выражения. Вы также должны избегать использования строковой переменной в качестве форматирования строк до printf.

Вот ваш код fixed:

char array1[10] = "123456789"; 
char array2[9]; 
int i, j; 
for(i = 1, j = 0 ; i < 10 && j < 9; i++, j++){ 
     array2[j] = array1[i]; 
} 
printf("%s\n", array2); 

Вы также можете упростить цикл, используя единый индекс переменной i и индексации array2 с i+. Вы также можете полностью удалить цикл, используя strncpy, но имейте в виду, что если n меньше длины строки + 1, он не добавит нулевой ограничитель.

Answer 2

Когда вы говорите printf(array2), он считает, что он печатает строку с нулевым символом. Поскольку есть (возможно) нет \0 в array2, printf продолжается до конца array2, блуждая по памяти, он не должен.

Answer 3

Для дальнейшего расширения ответа marcog: вы объявляете массив 1 с 9 элементами, 0-8, а затем записываете из 0-9 (10 элементов). То же самое с array2.

Answer 4

2 вопроса: Во-первых, при печати строки с printf и при работе с другими стандартными строковыми строками C, ваши массивы char должны быть завершены с нулевыми значениями, чтобы функции знали, где заканчивается строка. Вы также пишете один за концом своих массивов.

Во-вторых, при использовании printf почти всегда Плохая идея использовать строку, которую вы хотите распечатать, как строку формата. Использовать

printf("%s", array2); 

вместо этого. Если вы используете printf, как и в исходном примере, и на него может влиять массив2, ваша программа, скорее всего, уязвима к уязвимости строки формата.

Answer 5

Использование тетсру():

memcpy(array2, &array1[1], 8); 

Thats проще.

Answer 6

Просто используйте strcpy() (если они обе строки!) strcpy() хочет указатель на источник и указатель на пункт назначения.Если вы хотите, чтобы пропустить первый элемент исходного массива просто передать source + 1:

char source[] = "ffoo"; 
char dest[] = "barbar"; 

strcpy(dest, source + 1); 

// now dest is "foo" (since the ending \0 is copied too) 

printf("\n%s\n", dest); 

Answer 7

Это не нужно использовать дополнительные array2 как

printf("%.8s",array1+1);