某些输入上的分段错误

Question

我编写的函数中已经有了一些调试元素。当我输入“y”或“Y”作为输入时，运行时会出现分段错误。当我输入任何其他值代码运行。在扫描并给出响应之后，但在输出“扫描工作”行之前，seg故障会启动。不知道为什么它只会在这些值上起作用。如果任何人需要函数调用，我也有。

query_user(char *response [10]) 
{ 
    printf("response after query call before clear=%s\n",response); 
    strcpy(response,""); 
    printf("response after clearing before scan=%s\n",response); 
    printf("Enter another person into the line? y or n\n"); 
    scanf("%s", response); 
    printf("response after scan=%s\n",response); 
    printf("scan worked"); 
} 

main() 
{ 
    char response [10]; 
    strcpy(response,"y"); 
    printf("response=%s\n",response); 
    printf("When finished with program type \"done\" to exit\n"); 
    while (strcmp(response,"done") != 0) 
    { 
     printf("response after while loop and before query call=%s\n",response); 
     query_user(&response); 
    } 
} 

输出上的错误：在非错误

 
response after query call before clear=y 
response after clearing before scan= 
Enter another person into the line? y or n 
y 
response after scan=y 
Segmentation Fault (core dumped) 

输出：

 
response after query call before clear=y 
response after clearing before scan= 
Enter another person into the line? y or n 
n 
response after scan=n 
scan worked 
Cycle number 0 
(program continues to run outside this function) 

Answer 1

为query_user参数的你的宣言是错误的。你已经声明了一个指向char的指针数组。你需要一个简单的字符缓冲区。就像这样：

query_user(char response[]) 

或

query_user(char* response) 

使用你喜欢哪个。

当你调用该函数，你可以这样说：

query_user(response); 

另外我想指出的是您为main声明不正确。你应该使用

int main(void) 

Answer 2

你在做什么并不完全安全。

如果用户录制的磁带大于标签大小，会发生什么情况？ （可能是Segfault）。

你真的应该使用类似的scanf（ “％787-9”，响应）（你将不得不后，清空缓冲区！）：

（举例标签大小= 10）：

query_user(char *response) 
{ 
    printf("response after query call before clear=%s\n",response); 
    strcpy(response,""); 
    printf("response after clearing before scan=%s\n",response); 
    printf("Enter another person into the line? y or n\n"); 
    scanf("%9s", response); 
    clean_buffer(); 
    printf("response after scan=%s\n",response); 
    printf("scan worked"); 
} 

void clean_buffer() 
{ 
    int c; 
    while ((c = getchar()) != '\n' && c != EOF); 
} 

希望它有帮助！

Answer 3

首先，您发布的代码不会重现该问题。我在Linux上测试用gcc 4.6.2和Windows上使用Visual Studio 2010在这两种情况下，你确切的代码我的输出是：

response=y
When finished with program type "done" to exit
response after while loop and before query call=y
response after query call before clear=y
response after clearing before scan=
Enter another person into the line? y or n
y
response after scan=y
scan workedresponse after while loop and before query call=y
response after query call before clear=y
response after clearing before scan=
Enter another person into the line? y or n

因此，为了更好地诊断，我们需要的代码，一个完整的工作组，显示该问题。有很多与发布代码的警告（不知道这些都是在你的基地也一样），有时忽略了警告，可以让你在一个不好的地方：

• main应该返回int：int main()
• 认沽在结束return 0;
• query_user在主返回需要void返回类型：void query_user()
• strcpy需要char *你给它一个char **

对于最后一点，你可以通过数组传递给你的函数，如修复：

query_user(response); // remove the & operator 

和你的query_user()需求标题来更改，如：

query_user(char response [10]) // remove the * operator 

而且请注意，scanf会在stdin上留下一个'\n'字符，以确保您不会消耗掉可以在%s之前放置单个空格：

scanf(" %s", response); 

我不认为任何这些都会产生很大的影响，但请确保您进行了更改。请发布更新到您的代码，实际显示错误。