将文件指针传递给函数，文件无法正确读取

Question

我认为我的代码存在问题，文件未正确传递。输入是一个包含三行的文件 1 2 3； 4 5 6； 7 8 9； 并且输出是分段错误（核心转储），输出应该打印第一行 1 2 3。

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

int getNum();
int getLine();
int getMatrix();
int det1();
int det2();
int det3();
int det4();
int det5();
int det6();


main(){
  FILE *infile;
    infile = fopen("matrix.txt","r");
  int line[6];
  int lineSize;
  int error;
  getLine(line,lineSize,infile);
  printf("%d %d\n", line[0],line[1]);
  fclose(infile);
}

/***********************************************
Name : getLine
Description : To get the line of numbers
Arguments :  infile - the file pointer with numbers inside
             line[]  - the line of numbers
             lineSize - size of line
Returns : 1   - If no errors were encountered
          2 - If END OF FILE was reached
          -1 if non number detected

*************************************************/
int getLine(int line[], int lineSize, FILE *infile){
  int value;
  int l;
  lineSize=0;

  while(value != '\n'){
    value=0;
    l=getNum(value,*infile);
    if (value==EOF){
      return(2);
    }
    line[lineSize]=value;
    lineSize++;
  }
  if (l == -1){
    return(-1);
  }
  return(1);

}


/***********************************************
Name : getNum
Description : To get the Next number from file
Arguments :  infile - the file with numbers inside
             value  - the value of number grabed
Returns : 1   - If no errors were encountered
          -1  - If letter or non number detected
*************************************************/
int getNum(int value, FILE *infile){

  int c;
  int error=1;

  while ((c=getc(infile)) != EOF){
    if (c=='\n'){
      value = '\n';
      return(1);
    }
    if(c==32){//checking for space
      if (error == -1){
        return(-1);
      }
      else{
        return(1);
      }
    }
    else {
      value = 10*value + c - '0';
    }
    if((c<=47)||(c>=58)){
      printf("incorrect number input %d\n",c);
      error = -1;
    }
  }
  value = EOF;
  return(1);

}

Answer 1

浏览你的代码......

int getNum();
int getLine();
int getMatrix();
int det1();
/* ... */

这些声明对编译器说：“嘿编译器，请注意我将使用这些名称（getNum，getLine，getMatrix，det1，...）调用函数，并且它们返回 int< /code>，但我不会告诉你它们接受什么参数，当我使用它们时请相信我”

如果你在向编译器引入函数时正确使用原型，那就更好了

int getNum(int value, FILE *infile);
int getLine(int line[], int lineSize, FILE *infile);
/* ... */

这些声明对编译器说：“嘿编译器” ，请注意，我将使用这些名称调用函数，它们返回 int 并接受这些参数，如果我犯了错误，请抱怨让我知道我的错误”

...继续里面。 main()

      /* ... */
      int lineSize;
      int error;
      getLine(line,lineSize,infile);
      /* ... */

声明了 lineSize 但没有为变量提供值。当程序调用 getLine 时，lineSize 的值几乎肯定是错误的值（甚至可能在调用该函数之前使您的计算机崩溃）。在使用（几乎）所有变量之前初始化它们。

      /* ... */
      int lineSize = 0;
      int error = 0;
      getLine(line,lineSize,infile);
      /* ... */

我没有浏览更多...

建议：提高编译器警告级别，并且在编译产生警告时不要运行程序。

Answer 2

在 getLine() 中，当您将 infile FILE* 赋予 getNum() 函数时，您取消引用它：

 l=getNum(value,*infile);

但是 getNum() 只期望一个正常的 FILE*，而不是取消引用的文件。因此，将 infile 不变地传递给该函数：

 l=getNum(value,infile);

---

此外，while(value != '\n') 循环可能会永远运行，写入超过 的末尾lines 数组，直到出现分段错误。 value 控制循环何时终止，它永远不会被修改（它也不会被初始化，使其以任意值开始）。 getNum() 函数可能应该修改 value，它获取作为参数传递的整数的副本，然后修改此副本。原始值永远不会改变。

如果您希望函数更改 value 变量，则必须使用指向 value 的指针，该指针用于修改该变量：

int getNum(int *value, ...) {
   *value = 5;
   ...
}

l=getNum(&value, infile);

此外，还有一点值得怀疑： value（一个整型变量）被赋值并与 '\n'（一个字符文字）进行比较。您确定要使用 '\n' 的整数值作为循环的终止条件吗？

Answer 3

虽然不是直接答案，但我建议在随机位置粘贴一些 printf 语句；这将使您相对快速地缩小崩溃的确切点。移动它们，直到有两个 printfs 将一行代码括起来，然后您就知道该代码是崩溃的罪魁祸首，这将让您更好地诊断。