令牌化C中的用戶輸入（存儲在** arg中）？

Question

我試圖編寫一個簡單的shell接口，它接受用戶輸入（通過字符），並通過指針指向一個指針*（確切地說argv是如何工作的）。這裏是我的代碼：

int i = 0; 
for (i = 0; i < wordCount; i++) 
    printf("Word %i: %s\n", i, argvInput[i]); 

argvInput的值[I]是什麼最後輸入分配是所有連接：通過argvInput

char input[100]; 
char **argvInput; 
char ch; 
int charLoop = 0; 
int wordCount = 0; 

argvInput = malloc(25 * sizeof(char *)); 

while((ch = getc(stdin))) { 
    if ((ch == ' ' || ch == '\n') && charLoop != 0) { 
     input[charLoop] = '\0'; 
     argvInput[wordCount] = malloc((charLoop + 1) * sizeof(char)); 
     argvInput[wordCount] = input; 
     charLoop = 0; 
     wordCount++; 

     if (ch == '\n') { 
      break; 
     } 

    } else if (ch != ' ' && ch != '\n') { 
      input[charLoop] = ch; 
      charLoop++; 
     } else { 
      break; 
     } 
    } 

如果我循環。所以，如果我輸入： 「幸福的日子即將到來」，環路的輸出是：

Word 0: soon 
Word 1: soon 
Word 2: soon 
Word 3: soon 
Word 4: soon 

我不知所措。顯然，每個循環覆蓋以前的值，但我盯着屏幕，無法弄清楚爲什麼...

Answer 1

此行是你的死穴：

argvInput[wordCount] = input; 

不要緊，你分配新的空間，如果你打算用另一個替換指針（即input）。

相反，使用strncpy來提取input的部分爲argvInput[wordCount]。

Answer 2

argvInput[wordCount] = input;僅使指針argvInput[wordCount]指向input的內存，而不是將輸入的內容複製到新分配的內存中。您應該使用memcpy或strcpy來更正您的程序。

指針分配後的內存狀態如下圖所示。由malloc((charLoop + 1) * sizeof(char));分配的內存（圖中灰色部分）無法再被您的程序訪問，這會導致一些內存泄漏問題。請注意這一點。

Answer 3

我建議打印您argvInput指針與％P，而不是％S，找出這個問題：printf("Word %i: %p\n", i, (void *) argvInput[i]);

你注意到它打印出的值是什麼？這與argv的行爲有何不同？嘗試打印argv的指針：for (size_t x = 0; x < argc; x++) { printf("Word %zu: %p\n", x, (void *) argv[x]); }

現在您已經觀察到問題了，解釋它可能會變得更容易。

此代碼分配內存，並且存儲一個指向在argvInput [的wordCount]認爲存儲器：argvInput[wordCount] = malloc((charLoop + 1) * sizeof(char));（順便說一下，焦炭的sizeof是總是 1 C，所以你通過1不必要地相乘）。

這段代碼用一個指向輸入的指針替換指向已分配內存的指針：argvInput[wordCount] = input; ...因此，所有項目都包含一個指向同一個數組的輸入：input，並且由於失去對它的引用而導致分配的內存泄漏。顯然，這是有問題的路線;它不會做你最初認爲它所做的事情。

有人建議您使用strdup調用替換您的malloc調用，並刪除有問題的行。我不喜歡這個建議，因爲strdup不在C標準中，所以不需要存在。

strncpy可以工作，但它不必要的複雜。由於目標數組的大小足以存儲字符串，因此strcpy可以保證正常工作。因此，我建議用strcpy(argvInput[wordCount], input);替換有問題的行。

另一個沒有詳細解釋的選項是strtok。看來這是目前最好的，尚未開發，因爲它需要對代碼進行太多的修改。

我有一塊骨頭可以用此代碼挑選：char ch; ch = getc(stdin);是錯誤的。 getc返回一個int，原因是：任何成功的字符讀取都將以unsigned char值的形式返回，這不可能是負數。如果getc遇到EOF或錯誤，它將返回一個負值。一旦您將返回值分配給ch，您如何區分錯誤和成功？

你有沒有想過如果第一個字符是''會發生什麼？目前，你的代碼會跳出循環。這看起來像一個bug，如果你的代碼是模仿常見的argv解析行爲。適應這個代碼來解決你的問題可能是一個好主意：

for (int c = getc(stdin); c >= 0; c = getc(stdin)) { 
    if (c == '\n') { 
     /* Terminate your argv array and break out of the loop */ 
    } 
    else if (c != ' ') { 
     /* Copy c into input */ 
    } 
    else if (charLoop != 0) { 
     /* Allocate argvInput[wordCount] and copy input into it, 
     * reset charLoop and increment wordCount */ 
    } 
}