使用 strcpy() 在 C 中正确地为指针赋值

Question

我只需要从 char 数组中取出奇数值，然后使用指针将它们复制到大小正确的动态内存中。

但是，当运行我的程序时，它适用于某些输入字符串而不适用于其他输入字符串。有什么我做错了吗？我似乎无法弄清楚发生了什么。

/* A.) Include the necessary headers in our program */
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define MAX_STRING_LENGTH 32

int main() {
    /* B.) Declare char array with inital size of 32 */
    char input_string[MAX_STRING_LENGTH];

    /* C.) Recieve user input.
           Can save the first 31 characters in the array with 32nd reserved for '\0' */
    printf("Enter a string of characters: ");

    /* D.) Using the technique we discussed to limit the string to 31 charaters */
    scanf("%31s", input_string);
    printf("\n");

    /* Will be used to determine the exact amount of dynamic memory that will be allocated later */
    int odd_value_count = 0;
    printf("Odd Characters: ");
    for(int i = 0; i < strlen(input_string); i++) {
        if(i % 2 != 0) {
            printf("%c ", input_string[i]);
            odd_value_count++;
        }
    }

    printf("\n");
    printf("Odd value count: %d\n", odd_value_count);

    /* E.) Delecaring the pointer that will hold some part of the input_string
           Pointer will be a char type */
    char *string_pointer;

    /* G.) Allocating the space before the copy using our odd value count */
    /* H.) The exact amount of space needed is the sizeof(char) * the odd value count + 1 */
    string_pointer = (char *)malloc(sizeof(char) * (odd_value_count + 1));

    if (string_pointer == NULL) {
        printf("Error! Did not allocte memory on heap.");
        exit(0);
    }


    /* F.) Copying all charcters that are on the odd index of the input_string[] array
           to the memory space pointed by the pointer we delcared */
    printf("COPIED: ");
    for (int i = 0; i < strlen(input_string); ++i) {

        if(i % 2 != 0) {
            strcpy(string_pointer++, &input_string[i]);
            printf("%c ", input_string[i]);
        }
    }

    /* Printing out the string uses the pointer, however we must subtract odd_value_count to
       position the pointer back at the original start address */
    printf("\n%s\n", string_pointer - odd_value_count);

    return 0;

}

这个输入字符串：01030507 工作正常，复制和打印：1357

输入字符串：testing 复制etn，但打印etng。

我不明白为什么对于某些字符串，它会在最后打印出多余的字符，而我什至从不复制值。

Answer 1

你需要Null Terminate你的字符串，就像*string_pointer = '\0';，就在你复制完字符串指针中的奇数字符之后 - 在那个循环之后，null 终止你的字符串。

在How to add null terminator to char pointer, when using strcpy了解更多信息？

Answer 2

在您的例程结束时，您需要空终止字符串，否则您没有字符串，您只有一个 char 数组，您可以使用 string_pointer，它已经指向末尾的一个要保存的字符串：

//...
for (int i = 0; i < strlen(input_string); ++i) {

    if(i % 2 != 0) {
        strcpy(string_pointer++, &input_string[i]);
        //as you are copying characters, you can do this:
        //*string_pointer++ = input_string[i]; 
        //instead of strcpy
        printf("%c ", input_string[i]);
    }
}
*string_pointer = '\0'; // <-- here
//...