2011-03-23 165 views
1

我遇到的问题是使用标准输入从文件中读取多行整数。这些文件看起来像:C从文件读取多行文件

123 
423 
235 
523 
..etc 

我此刻的代码是:

/* 
* Read in the initial puzzle configuration. 
* Each line is 4 characters long: 
* Row as a character '0' .. '9' 
* Column as character '0' .. '9' 
* Digit as character '0' .. '9' 
* Terminating newline. 
* Exits with an error message if there are syntactic 
* or semantic errors with any configuration line. 
*/ 

void configure(FILE *puzzle_file) { 
     int row; 
     int column; 
     int value; 

     while((fscanf(puzzle_file, "%i%i%i\n", row, column, value)) != EOF){ 
       fscanf(puzzle_file, "%i%i%i\n", row, column, value); 
       puzzle[row][column] = value; 
       fixed[row][column] = 1; 
     } 

} 

我试图用的fscanf因为该文件的格式是否正确(如根据功能配置上面的评论)但我无法得到它的工作。

如果有不同的,更简单的方法来解决这个可以看到的解决方案。

语言:C

编辑:

在编译错误:

[email protected]:~/350/sudoku$ make 
gcc -c puzzle.c 
puzzle.c: In function ‘configure’: 
puzzle.c:95: warning: format ‘%i’ expects type ‘int *’, but argument 3 has type ‘int’ 
puzzle.c:95: warning: format ‘%i’ expects type ‘int *’, but argument 4 has type ‘int’ 
puzzle.c:95: warning: format ‘%i’ expects type ‘int *’, but argument 5 has type ‘int’ 
puzzle.c:96: warning: format ‘%i’ expects type ‘int *’, but argument 3 has type ‘int’ 
puzzle.c:96: warning: format ‘%i’ expects type ‘int *’, but argument 4 has type ‘int’ 
puzzle.c:96: warning: format ‘%i’ expects type ‘int *’, but argument 5 has type ‘int’ 
gcc -o sudoku main.o puzzle.o arguments.o 

在我的测试误差的运行:

[email protected]:~/350/sudoku$ make test_l2 
./sudoku -e p+s/good_puzzle.txt < p+s/script_good_quit.txt 
/bin/sh: line 1: 9143 Segmentation fault  ./sudoku -e p+s/good_puzzle.txt < p+s/script_good_quit.txt 
make: *** [good_configured] Error 139 
+0

这将无法编译为拼图没有定义 - 你的代码如何不工作 - 什么是错误信息或其他可见行为? – Mark 2011-03-23 16:44:24

+0

我将编译和测试错误添加到问题中。 – DestroyerDust 2011-03-23 16:48:23

回答

5

你有两个问题跟你是什么做:

首先,您将跳过文件中的一堆行,因为您在while循环中调用fscanf,然后在检查循环条件之后立即调用它。您只需要在while循环条件下调用一次。

while((fscanf(puzzle_file, "%i%i%i\n", row, column, value)) != EOF){ 
      // fscanf(puzzle_file, "%i%i%i\n", row, column, value); REMOVE THIS! 
      puzzle[row][column] = value; 
      fixed[row][column] = 1; 
    } 

其次,你会想要读取每个整数作为单独的字符,即。 %c%c%c而不是%i%i%i,并将这些字符代码转换为整数值。即。减((int)ch - 48)其中ch是由fscanf读入的字符之一。

UPDATE:

您还传递了错误的值FSCANF,你想通过你的变量不是它们的值的存储位置。

char row,value,column; 
fscanf(puzzle_file, "%c%c%c\n", &row, &column, &value); 

更新2:

还检查了ladenedge评论我的答案使用宽度说明符整数值有关,而不是阅读字符,然后转换它们。

+1

我相信'%1i%1i%1i'也可以工作,不需要转换。 – ladenedge 2011-03-23 16:52:18

+0

@ladenedge:哦,我不知道这很好。 – GWW 2011-03-23 16:52:56

+1

@ladenedge:那么%c%c%c然后将它们转换为我需要的字符值,%1i%li%li和它们与int的相同将会做同样的事情? – DestroyerDust 2011-03-23 16:58:35

1

警告清楚地表明什么可能出错在运行时 -

puzzle.c:95: warning: format ‘%i’ expects type ‘int *’, but argument 3 has type ‘int’ 

因此改变

(fscanf(puzzle_file, "%i%i%i\n", row, column, value) 

(fscanf(puzzle_file, "%i%i%i\n", &row, &column, &value) 
// Added & symbol 
0

作为一个很好的做法,始终使用伪代码!在上面砌/规格来看,我会做一些事情,如:

  1. 打开文件
  2. 抢第一行(这可能会是一个字符串),而你不是在EOF
  3. 该3位数字分离成单独的变量
  4. 拼图[NUM1] [NUM2 = NUM​​3
  5. 固定[NUM1] [NUM2] = 1
  6. 转到#2

因为我没有完全检查规格,可能会在一两个地方关闭。