在C中将奇数索引元素从一个数组复制到另一个数组

Question

所以我有一个程序，其中有用户输入字符串（char 数组），我需要使用分配内存的指向 char 数组的指针将所有奇数索引元素复制到另一个数组。

（charPointer 需要分配给使用 malloc 所需的确切空间量）

char charArray[16];

printf("Enter string: ");
scanf("%16s", charArray);

char *charPointer = malloc((sizeof(charArray) / 2 ));
char *d = charPointer;

for (char *p = charArray; *p != '\0'; p++) {

    // Calculates odd-index element using bit operator
    if ((p - charArray) % 2 != 0){
        // Shows which elements should be copied
        printf("%c", *p);

        // Copy *p value to charPointer
        charPointer[p-charArray] = *p;
        charPointer++;
    }
}

while(*d != '\0')
   printf("%c\n",*d++);

但是我得到了奇怪的结果，比如 charPointer 它只复制第一个奇数索引而不是其余的。我以为我理解指针，但这真的让我很困惑。感谢您的帮助！

Answer 1

对于初学者来说这个电话

scanf("%16s", charArray);
       ^^^^ 

不正确。你需要写

scanf("%15s", charArray);
       ^^^^

这个内存分配

char *charPointer = malloc((sizeof(charArray) / 2 ));

分配冗余内存。你应该写

char *charPointer = malloc( strlen(charArray) / 2 + 1 );

这些陈述

    charPointer[p-charArray] = *p;
    charPointer++;

没有意义，因为它们不会在动态指向的字符数组中顺序写入字符。你至少应该这样写

    *charPointer++ = *p;

由于终止零字符 '\0' 没有附加到动态分配的数组中，所以这个 while 循环

while(*d != '\0')
   printf("%c\n",*d++);

调用未定义的行为。

你也应该释放分配的内存。

这是一个演示程序。

#include <stdio.h>
#include <string.h>
#include <stdlib.h>

int main(void) 
{
    char charArray[16]; 
    
    printf( "Enter a string: " );
    scanf( "%15s", charArray );
    
    char *charPointer = malloc( strlen( charArray ) / 2 + 1 );
    
    char *q = charPointer;
    
    for ( const char *p = charArray; *p != '\0'; ++p  )
    {
        if ( ( p - charArray ) % 2 != 0 )
        {
            *q++ = *p;
        }
    }
    
    *q = '\0';
    
    puts( charPointer );
    
    free( charPointer );
    
    return 0;
}

程序输出可能看起来像

Enter a string: 012345678901234
1357913