C 字符串到数组 - 大写字符错误

Question

关闭。此题需要 details or clarity 。目前不接受答案。

---

想要改进这个问题吗？通过 editing this post 添加详细信息并澄清问题.

已关闭 7 年前。

Improve this question

给定这个字符串“red, blue, green”，创建一个包含这些颜色作为其元素的数组。我在下面编写的代码可以工作，但是当我将颜色的第一个字母更改为大写时，我得到输出 - Red, Blu\301-!Wree\316。如何使此代码更加动态地处理以大写开头的单词？谢谢。

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


int findLength(char string[]){
    int l =0;

    for(l = 0; string[l]!='\0'; l++){

    }
    return l;
};


char *stringToArray(char string[]){
    int i = 0;
    int j = 0;
    char c = ',';
    int n = 0;
    int l = findLength(string);

    char *str = (char *)malloc(l * sizeof(char));

    while(string[i] != l){
        if(string[i] == c || string[i] != '\0'){
            for(n = j; n < i; n++){
                str[j++] += string[n];
            }

        }
        i++;

    }
    printf("%s\n", str);
    str = '\0';
    return str;
}


int main(int argc, const char * argv[]) {
    char *string = "red, blue, green";
    //char *string = "Red, Blue, Green"; 
    char *str = stringToArray(string);
    free(str);

    return 0;

}

Answer 1

奇怪的行为与你的字符串是否有大写字母没有任何关系。 stringToArray 中循环的终止条件是错误的:

    int l = findLength(string);

    while (string[i] != l) ...

条件应该是

    while (i < l) ...

或者，正如您已在 findLength 中使用的那样:

    while (string[i] != '\0') ...

因为条件是错误的 - 在您的情况下 l 是 16，并且没有一个字母的 ASCII 值为 16 - 您超出了字符串的有效范围，这会导致未定义的行为。

目前，您只需将旧字符串复制到新字符串，尽管以一种非常奇怪的方式。您的内部循环使用三个变量，其中两个变量递增。这非常令人困惑。它可能也不会按照您的想法进行，因为条件是:

    if (string[i] == c || string[i] != '\0') ..

对于字符串的所有字母都成立，前提是 opuer 循环应仅考虑有效字符，但不包括字符串末尾。

最后，如果你想复制字符串，你应该为终止符分配süpace:

    char *str = malloc(l + 1);

当您想要附加最后一个空字符时:

    str = '\0';

您实际上将 while allocate 字符串设置为 null，这会导致内存泄漏。 (main 中的 free 不会产生错误，因为 free 可以合法地将“NULL”作为参数。)而是使用:

    str[l] = '\0';

通过这些修复，您现在有了一个可以复制原始字符串的程序。 (POSIX) 库函数 strdup 可以更有效地完成此操作。如果要返回字符串数组，则必须反射(reflect)您的函数以返回指向字符指针的指针。

下面是该行为的可能实现。 (它使用该方法为堆上的所有内容分配内存。如果您总是期望三个短字符串，这可能不是最好的解决方案。)

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

    char **stringToArray(const char *str)
    {
        char **res;
        const char *p = str;
        int n = 0;

        while (p) {
            p = strchr(p, ',');
            if (p) p++;
            n++;
        }

        res = malloc((n + 1) * sizeof(*res));

        p = str;
        n = 0;
        while (p) {
            const char *begin;
            size_t len;

            while (*p == ' ') p++;
            begin = p;

            p = strchr(p, ',');
            if (p) {
                len = p - begin;
                p++;
            } else {
                len = strlen(begin);
            }

            res[n] = malloc(len + 1);
            memcpy(res[n], begin, len);
            res[n][len] = '\0';

            n++;
        }

        res[n] = NULL;
        return res;
    }


    int main(int argc, const char * argv[])
    {
        char *str = "Vermilion, Ultramarine, Chartreuse"; 
        char **res = stringToArray(str);
        int i = 0;

        for (i = 0; res[i]; i++) {
            puts(res[i]);
            free(res[i]);
        }
        free(res);

        return 0;
    }