C - 比较整数时出现分段错误

Question

这是我的代码的一部分。当我运行我的代码时，它会请求用户输入，然后将其与记录在我的结构中的另一个整数匹配。当用户输入匹配时，它工作正常。但是当用户输入错误的输入时，就会出现分段错误。我应该在哪里修改我的代码？

long int userInput,endCheck; // Input from user
int flag=0; // check for match
int status;
int c; // controlling ctrl+D
int position= 999; // position of where the user input and data matched

LABEL:
    printf("\n\t---------------------------------------\n");


    printf("\n\nPlease enter the student ID which you want to find(3-times CTRL+D for finish):\n");



   scanf("%d",&userInput);

   if( (c=getchar()) == EOF){
     exit(0);
     }




    for(i=0;i<lines,flag==0;i++){

       if(index[i].id == userInput){
          position=i;
          flag=1;
       }else{
          position=999;
        }
    }

    if(flag==0){
    printf("id not found");
    }


 studentInfo info; // for storing the information which we will take between determined offsets
    if(position!= 999){
           if ( (pos = lseek(mainFile,index[position].offset , SEEK_SET)) == -1)/*going to determined offset and setting it as starting offset*/
        { perror("classlist"); return 4; }


            while ( (ret= read(mainFile,&info, sizeof(info))) > 0  ){
           printf("\n\nStudent ID: %d, Student Name: %s\n\n",info.id,info.name);
                   break;// to not take another students' informations.
            }

   }
flag=0; 
goto LABEL;


printf("Program is terminated");

Answer 1

使用不需要的逗号进行循环的正确方法是这样的。当您找到正确的index[i].id 后，您可以使用break 提前退出循环。

for(i=0;i<lines;i++){
    if(index[i].id == userInput){
        position=i;
        flag=1;
        break;
    }
}

您不需要 else 分支，因为 position 从代码一开始就设置为 999。但实际上你不应该以这种方式使用位置。如果您有超过 999 条记录怎么办？您已经在使用 flag 来确定您是否已将 position 设置为有效值。您应该将if(position!= 999) 的任何实例替换为if(flag)。

或者由于position 是有符号的int，您可以使用负值并放弃flag。

Answer 2

原因可能是您到达一个在循环结束时不存在的索引，在带有迭代器“i”的“if”语句的时刻。

或者在最后一个 if 中，您访问数组的“位置”索引。检查这些限制。

另外，试试GDB，对于解决这类问题很有用。