佔用多個分隔符的代碼不起作用

Question

我有一個程序，我寫了一個字符串，並根據出現的分隔符分隔每個單詞並將其添加到數組中。

我已將其調整爲考慮''，'。'要麼 '。'。現在的目標是調整多個分隔符一起出現（如「狗，走路」），並且仍然只添加該單詞。雖然我的程序運行正常，並且不打印額外的分隔符，但每次遇到其他分隔符時，它都會在輸出中包含一個空格而不是忽略它們。

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[]從未分配terminarting空字符尚未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); 

正如你看到的，有很多問題需要擔心：

• 沒有內存管理（分配/免費），這是由垃圾收集handeled
• 沒有指針，所以沒有什麼可以去與他們錯了
• 強大的內置的字符串功能，如Split()
• foreach使循環更加簡單