C指針和內存 - 寫了一個代碼...拋出異常

Question

• 我正在寫一個代碼，用word-Y代替word-X。

  • 正文中的每個單詞長度相同。
  • 讓說我的字符串，X，Y分別是：

  「AAA BBB CCC DDD QQQ」，X = 2，Y = 5

所以它會打印：「AAA QQQ CCC ddd bbb「

但由於某些原因，我不理解它會拋出異常。 我在代碼中寫了錯誤。 我知道我的代碼有點混亂，所以如果你也有建議，我很樂意聽到。

謝謝!!!

void changeWords(char *s,int X, int Y) 
{ 
    int len,words,i,count; 
    len = count = words = i = 0; 
    bool flag = false; 
    while (s[len] != ' ') 
     len++; 
    char *p1 = (char*)malloc(sizeof(char)*(len+1)); 
if (p1== NULL) 
{ 
    printf("Error: memory did not allocated"); 
    exit(1); 
} 
    char *p2 = (char*)malloc(sizeof(char)*(len+1)); 
if (p2== NULL) 
{ 
    printf("Error: memory did not allocated"); 
    exit(1); 
} 
    while (flag == false) 
    { 
     if (count == (X-1)) 
     { 
      for(int x = 0; x< len;x++,i++) 
       p1[x] = s[i]; 
     } 
     else if (count == (Y-1)) 
     { 
      for (int x = 0; x< len; x++,i++) 
       p2[x] = s[i]; 
      flag = true; 
     } 
     if (s[i] == ' ') 
      count++; 
     i++; 
    } 
    p1[len] = p2[len] = '\0'; 
    i = count = 0; 
    flag = false; 
    while (flag == false) 
    { 
     if (count == (X-1)) 
     { 
      for (int x = 0; x< len; x++, i++) 
       s[i] = p1[x]; // here it throw an error "Unhandled exception thrown:.." 
     } 
     else if (count == (Y-1)) 
     { 
      for (int x = 0; x< len; x++, i++) 
       s[i] = p2[x]; 
      flag = true; 
     } 
     if (s[i] == ' ') 
      count++; 
     i++; 
    } 
    puts(s); 
    free(p1); free(p2); 
} 
void main() 
{ 
char*str = (char*)malloc(sizeof(char)); 
if (str == NULL) 
{ 
    printf("Error: memory did not allocated"); 
    exit(1); 
} 
char ch; 
int i = 0; 
printf("Enter a string: "); 
while ((ch = getchar()) != '\n') 
{ 
    str[i] = ch; 
    i++; 
    str = realloc(str, sizeof(char) * (i + 1)); 
    if (str == NULL) 
    { 
     printf("Error: memory did not allocated"); 
     exit(1); 
    } 
} 
str[i] = '\0'; 
func(str,3,5); 
printf("new string: %s\n", str); 

free(str); 
system("pause"); 

} 
} 

Answer 1

當你這樣做：

char * str = "aaa bbb ccc ddd qqq"; 

字符串字面"aaa bbb ccc ddd qqq"被放置在只讀數據段，並str由指向它。在這一點上，改變str指向的是未定義的行爲，並希望能夠引發段錯誤。

如果你絕對要使用指針，試試這個來代替：

char * str = strdup("aaa bbb ccc ddd qqq"); 

這將使字符串的副本堆上，這將是可寫的文字。

一旦你完成它，但你需要free(str);釋放內存。

Answer 2

更改此：

char * str = "aaa bbb ccc ddd qqq"; 

這樣：

char str[] = "aaa bbb ccc ddd qqq"; 

，因爲第一個是字符串文字。

字符串常量不能被修改，因此您的代碼調用未定義行爲，在你的函數，其中s實際上是字符串文字：

s[i] = p1[x]; 

---

PS：無關您的問題：What should main() return in C and C++?int。

---

編輯：

如果你真的想使用指針，那麼你可以創建數組，並點的指針的第一個元素。或者，你可以動態地爲你的字符串分配內存，這將是一個矯枉過正的問題。

Answer 3

• 的變量temp不使用
• 長度計算是錯誤的。使用來自文庫string.h中的strlen：int len = strlen(s)

• 我也改變了主要功能：

  INT主（）{ 炭STR [20] = 「AAA BBB CCC DDD QQQ」;

  changeWords(str, 3, 5); 
  printf("new string: %s\n", str); 
  return 0; 
  

  }

• 當我執行，它沒有任何異常。但是你有邏輯問題。你必須找到用''來打破字符串的單詞。你也可以使用strtok函數。