2011-06-10 40 views
-1

即时试图从一个文件的文件的设置是这样读出的数据:将数据读入2个阵列

3050 76 
2030 60 
1600 70 
2222 50 
2430 60 
2800 50 
0 0 

第一个数字代表一个学生ID,第二个数字代表学生的 级时程序达到0 0时,应该停止从文件读取

这里是一个程序的例子,它没有从文件中读取,因为出现一个称为SEGMENTATION FAULT的错误。我正在使用Ubuntu的,这是发生此错误

// this is the grading sorting program for part one 


#include <stdio.h> 
#include <stdlib.h> 

#define SIZE 50 

int main(int argc, char **argv) 
{ 
    FILE* fp; // file pointer, points to the file 
    char file_name[32]; // store file name 

    int ID[SIZE]; 
    int grade[SIZE]; 

    int a,b; // index variables 

    int student_id, grades; 

    // opening file by asking the user for the file name 
    printf("Enter the name of the file containing the grades\n"); 
    scanf("%s",file_name); 

    fp=fopen("file_name", "r"); 

    /*fp = fopen("grades.txt", "r");*/ 

    // read in data into the arrays 
    for (a = 0; a <= SIZE; a++) 
    { 
     fscanf(fp,"%d", &student_id); 
     ID[a] = student_id; 

     for(b = 0; b <= SIZE; b++) 
     { 
      fscanf(fp,"%d", &grades); 
      grade[b] = grades; 
     } 

     if(ID[a] == 0 && grade[b] == 0) 
      break; 
    } 

    fclose(fp); 
    return 0; 
} 
+2

这看起来像功课 – rerun 2011-06-10 21:22:14

+0

是文件名超过32个字符?另外,如果这是家庭作业,您应该这样标记它。 – 2011-06-10 21:22:49

回答

0

一个提示你正在阅读许多值。你有一个嵌套for循环,将尝试读取大小*大小实体。

0

你的问题可能是这一部分:

fp=fopen("file_name", "r"); 

您要使用的变量FILE_NAME代替:

fp=fopen(file_name, "r"); 

使你的程序不容易出错,您应经常检查该文件已经成功打开:

fp=fopen("file_name", "r"); 
if (!fp) 
    return -1; 
0

它没有解决问题,但改为(a = 0; a < = SIZE; a ++) to for(a = 0; a < SIZE; a ++)

0

您正在打开名为“file_name”的文件,该文件可能不是从命令行扫描的文件。

为了解决这个问题删除就行了引号

fp=fopen("file_name", "r"); 
0

你不需要内部循环,而你从像“〜/ FILE_NAME”读书,这可能不存在,因此fp将是NULL(你不检查),所以任何尝试使用它都会导致崩溃。

+0

这怎么修复 – user793384 2011-06-10 22:24:18

1

这听起来像是一个很好的机会,熟悉gdb。既然你在Ubuntu上,它应该默认安装。

-gdb3编译程序并将其加载到GDB:它使用r和gdb

gdb -q ./a.out 

运行会告诉你在哪里发生段错误,从该网站使用print语句检查你的变量。

$ gdb -q a.out 
Reading symbols from /home/user/tmp/SO/bytes/a.out...done. 
(gdb) r 
Starting program: /home/user/tmp/SO/bytes/a.out 

Program received signal SIGSEGV, Segmentation fault. 
0x080483a4 in main() at sf.c:7 
7  *p = 2; 
(gdb) print p 
$1 = (int *) 0x0 
+0

你是什么意思编译和运行 - gdb3 – user793384 2011-06-10 22:16:36

+0

GDB是一个命令行调试器。调试器是帮助您修复程序并允许您一次运行一行的工具。 – rerun 2011-06-11 01:23:58

1

有许多东西都是在这里下车,如:

  • 文件名可能是不正确的 - fopen("file_name",...总是会打开一个名为“filename

  • 您正在创建长度为50(SIZE)的ID和等级数组 - 以C表示,这意味着数组索引的范围为0到49,但您的a和b for循环从0到50。

  • 您没有对fscanf的输出进行任何完整性检查,也没有检查是否可以真正打开文件。

我建议读一下C数组,字符串处理和文件I/O操作 - 有大量的教程可以通过Google找到。

1

首先 这是不正确

fp=fopen("file_name", "r"); 

也许你的意思

fp=fopen(file_name, "r"); 

你这样做从这个循环不会退出您已经阅读了大小乘

整数前
for(b = 0; b < SIZE; b++) 
     { 
      fscanf(fp,"%d", &grades); 
      grade[b] = grades; 
     } 

你不必做一个循环:我觉得

fscanf(fp,"%d", &grades); 

取而代之的,是你真正需要的阅读成绩。

记住,总是从的fscanf(测试返回代码),而fopen()

例子:

if ((fp=fopen(file_name,"r"))==NULL) exit(-9); 
do { 
    if (fscanf(fp,"%d",&(ID[a]))!=1) break; 
    if (fscanf(fp,"%d",&(grade[a]))!=1) break; 
} 
while((grade[a]!=0) && (ID[a++]!=0));