资本化一个单词的所有副本：为什么此代码在它自己的函数中失败？

Question

我有两个C风格字符串：

char st[100] = "to be or not to be "; 
char sub_s[100] = "be"; 

我需要找到开头的“是”与strstr(st, sub_s)并将其更改为大写字母。新的字符串需要''是或不是'“;

我管理与做出来的功能，像这样：

void main() 
{ 
    char st[100] = "to be or not to be "; 
    char sub_s[100] = "be"; 
    char* p; 
    int i; 

    while (p = strstr(st, sub_s)) 
    { 
     for (i = 0; i < strlen(sub_s); i++) 
     { 
      p[i] -= 32; 
     } 
    } 

    printf("%s\n", st); 
} 

但是，当我把这个代码到它自己的功能不工作了：

void main() 
{ 
    char st[100] = "to be or not to be "; 
    char sub_s[100] = "be"; 
    replaceSubstring(st, sub_s); 
} 

void replaceSubstring(char* str, char* substr) 
{ 
    int* p; 
    int i; 

    while (p = strstr(str, substr)) 
    { 
     for (i = 0; i < strlen(substr); i++) 
     { 
      p[i] -= 32; 
     } 
    } 

    printf("%s\n", st); 
} 

这是怎么回事在这？

Answer 1

在您编写的函数中，您已将p的类型设置为int*而不是char *。这意味着，当你写

p[i] -= 32; 

编译器将假定在由p每个元素指出为int，因此采取尺寸sizeof(int)的在存储器中的步骤，而不是尺寸1的存储器中的步骤。换句话说，代码被解释为

Start at the location pointed at by p. 
Jump forward i * sizeof(int) bytes. 
Read an integer value from that location. 
Subtract 32 from it. 
Write it back 

而非

Start at the location pointed at by p. 
Find the character i steps down from there. 
Subtract 32 from that character. 

为了解决这个问题，更改的p类型为char*，不int*。

这是一种错误，如果您将编译器警告级别设置为最大值，可能很容易检测到这种错误。当你学习编码时，我强烈建议你这样做，然后询问你在不了解它们时得到的警告。

其他一些杂散注意事项：

1. 的main返回类型应该是int，不void。
2. 不是从每个字符中减去32个字符，但这不是这个字中最清晰的东西，可以考虑使用<ctype.h>标头中的tolower函数。
3. 如果您要搜索的子字符串仅包含非字母字符（例如，":-)"），则此代码可能导致无限循环。你明白为什么？想想你如何解决它。