1

所以，我正在尝试编写一个函数，该函数接受一个字符串并将所有小写值更改为大写。这是代码：

void lowerToUpper(char *s)                                                      
{                                                                               
    char *p;                                                                                                                                      

    for (p = s; *p; p++)                                                        
    {                                                                                                                                               
         if (islower(*p))                                                                             
             *p = toupper(*p);                                                                                                                                                                               
    }                                                                           
}                                                                               

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

    pa = "This is a test.";                                                                                                

    printf("The following string will be edited:\n");                          
    printf("%s\n%s\n%s\n", pa);                                         

    lowerToUpper(pa);                                                                                                                      

    printf("The string has been edited, and is now as follows:\n");          
    printf("%s\n%s\n%s", pa);                                           

    return EXIT_SUCCESS;                                                        
}

问题出在“*p = toupper(*p);”行，这是我遇到分段错误的地方。我的猜测是，问题出在尝试将 toupper(*p) 返回的值分配给 *p 时。在做了一些测试之后，似乎 toupper(*p) 可以工作，但是一旦我尝试将值分配给 *p，我就出现了错误？关于为什么会发生这种情况的任何想法？

4

2 回答 2

7 
 pa = "This is a test.";

在上面的行中，您将指针 pa 设置为指向只读字符串。当您的函数写入该内存时，您正在调用未定义的行为。（如果你用 -Wall 标志编译你的程序，你会得到一个警告，告诉你这个问题）

获得可写字符数组的正确方法是：

char pa[] = "This is a test.";

此外，您对 printf() 的调用指定了四个 %s 标记，但您只提供了一个字符串指针参数。您需要删除三个 %s 标记，或者在 pa 之后添加额外的字符串指针参数。

于 2014-05-15T19:54:58.083 回答
0

改变：

   pa = "This is a test.";

至：

   pa = strdup("This is a test.");

改变：

   printf("%s\n%s\n%s\n", pa);

至：

   printf("%s\n", pa);

而且，也许应该在“return”语句之前调用“free(pa)”。

于 2014-05-15T20:05:56.847 回答