多个分隔符的代码会计不起作用

Question

我编写了一个程序来获取一串单词，并根据出现的分隔符分隔每个单词并将其添加到数组中。

我已经对其进行了调整，以考虑到 ' ' ， '.'或者 '。'。现在的目标是针对同时出现的多个分隔符进行调整（如“the dog,,,was walk”）并且仍然只添加单词。虽然我的程序可以工作，并且它不会打印出额外的分隔符，但每次遇到额外的分隔符时，它都会在输出中包含一个空格，而不是忽略它们。

int main(int argc, const char * argv[]) {

    char *givenString = "USA,Canada,Mexico,Bermuda,Grenada,Belize";

    int stringCharCount;

    //get length of string to allocate enough memory for array
    for (int i = 0; i < 1000; i++) {
        if (givenString[i] == '\0') {
            break;
        }
        else {
            stringCharCount++;
        }
    }



    // counting # of commas in the original string
    int commaCount = 1;
    for (int i = 0; i < stringCharCount; i++) {
        if (givenString[i] == ',' || givenString[i] == '.' || givenString[i] == ' ') {
            commaCount++;
        }
    }


    //declare blank Array that is the length of commas (which is the number of elements in the original string)
    //char *finalArray[commaCount];



    int z = 0;
    char *finalArray[commaCount] ;
    char *wordFiller = malloc(stringCharCount);


    int j = 0;
    char current = ' ';

    for (int i = 0; i <= stringCharCount; i++) {

        if (((givenString[i] == ',' || givenString[i] == '\0' || givenString[i] == ',' || givenString[i] == ' ') && (current != (' ' | '.' | ',')))) {
            finalArray[z] = wordFiller;
            wordFiller = malloc(stringCharCount);
            j=0;
            z++;
            current = givenString[i];
        }

        else {
            wordFiller[j++] = givenString[i];
        }
    }


    for (int i = 0; i < commaCount; i++) {
        printf("%s\n", finalArray[i]);
    }
    return 0;
}

这个计划花了我几个小时才聚在一起（在更有经验的开发人员的帮助下），我不禁感到沮丧。我正在尽我最大的能力使用调试器，但肯定需要更多的经验。

/////////

我又回到了便笺簿和纸上，重写了我的代码。现在我试图将分隔符存储在一个数组中，并将该数组的元素与当前字符串值进行比较。如果它们相等，那么我们遇到了一个新单词，我们将它添加到最终的字符串数组中。我正在努力弄清楚我将用于此的“for”循环的位置和内容。

    char * original = "USA,Canada,Mexico,Bermuda,Grenada,Belize";

    //creating two intialized variables to count the number of characters and elements to add to the array (so we can allocate enough mmemory)
    int stringCharCount = 0;
    //by setting elementCount to 1, we can account for the last word that comes after the last comma
    int elementCount = 1;

    //calculate value of stringCharCount and elementCount to allocate enough memory for temporary word storage and for final array
    for (int i = 0; i < 1000; i++) {
        if (original[i] == '\0') {
            break;
        }
        else {
            stringCharCount++;
            if (original[i] == ',') {
                elementCount++;
            }
        }
    }

    //account for the final element
    elementCount = elementCount;

    char *tempWord = malloc(stringCharCount);
    char *finalArray[elementCount];
    int a = 0;
    int b = 0;
    //int c = 0;
    //char *delimiters[4] = {".", ",", " ", "\0"};

        for (int i = 0; i <= stringCharCount; i++) {
            if (original[i] == ',' || original[i] == '\0') {
                finalArray[a] = tempWord;
                tempWord = malloc(stringCharCount);
                tempWord[b] = '\0';
                b = 0;
                a++;
            }
            else {
                tempWord[b++] = original[i];
            }
            }

    for (int i = 0; i < elementCount; i++) {
        printf("%s\n", finalArray[i]);
    }
    return 0;
}

Answer 1

很多问题。建议将代码分成小块，先调试。

--

取消初始化数据。

// int stringCharCount;
int stringCharCount = 0;
... 
stringCharCount++;

或者

int stringCharCount  = strlen(givenString);

还有其他问题：finalArray[] 从未分配过终止空字符，但 printf("%s\n", finalArray[i]); 已使用。

char *的用法不清楚

char *wordFiller = malloc(stringCharCount);

wordFiller = malloc(stringCharCount);

Answer 2

代码中的错误多于行数。

我建议你从更简单的开始。

通过练习完成一本基本的编程书籍。

编辑

或者，如果这是关于学习编程，尝试另一种更简单的编程语言：

在C# 中，您的任务看起来相当简单：

string givenString = "USA,Canada Mexico,Bermuda.Grenada,Belize";
string [] words = string.Split(new char[] {' ', ',', '.'});
foreach(word in words)
    Console.WriteLine(word);

如您所见，有很多问题需要担心：

• 没有内存管理（分配/释放），这是由垃圾收集器处理的
• 没有指针，所以它们不会出错
• 强大的内置字符串功能，如Split()
• foreach 使循环更简单