代码有未定义的行为，因为char savStr[strL];定义了一个长度为1的char数组，刚好够空字符串。复制任何更大的内容会导致未定义的行为。请改用char savStr[1000];。阅读完其余代码后，似乎根本不需要这个额外的数组。
还要注意，char *valueArry;定义了一个指向char的指针，您可以将其初始化为指向8个char的数组，而应该定义一个指向char *的数组的指针并正确地初始化它。
您的教授希望您使用动态数组，因此如果要存储的元素数量超过分配的长度，您可能需要重新分配它们。
以下是修改后的版本：

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

int main(int argc, char **argc) {
    int userNumber;     //to take in user key and key in file
    char userStr[1000]; //initial string to take value from user and string
    FILE *fp = fopen("values.txt", "r");
    if (fp == NULL) {
        fprintf(stderr, "cannot open file values.txt: %s\n", strerror(errno));
        return 1;
    }
    int size = 8;
    int curr = 0;
    int *keyArry = calloc(size, sizeof(*keyArray));
    char **valueArry = calloc(size, sizeof(*valueArry));
    if (keyArry == NULL || valueArry == NULL) {
        fprintf(stderr, "cannot allocate arrays\n");
        return 1;
    }

    while (fscanf(fp, "%d,%999s", &userNumber, userStr) == 2) {
        if (curr >= size) {
            // must reallocate the arrays
            size += size;
            keyArry = realloc(keyArry, size * sizeof(*keyArry);
            valueArry = realloc(valueArry, size * sizeof(*valueArry);
            if (keyArry == NULL || valueArry == NULL) {
                fprintf(stderr, "cannot reallocate arrays\n");
                return 1;
            }
        }
        printf("%d - %s\n", cuserNumber, userStr);
        keyArry[curr] = userNumber;
        valueArry[curr] = strdup(userStr);
        curr++;
    }
    fclose(fp);
    for (int i = 0; i < curr; i++) {
        printf("%d - %s \n", keyArry[i], &valueArry[i]);
    }
    return 0;
}