2015-07-10 83 views
1

似乎一切都在我的程序试图返回0这是当我得到的错误,除了运行良好:信号SIGABRT 0

Thread_1: signal SIGABRT 

我不知道我在做什么错,但我认为,它可能是我如何使用我的指针(通过引用传递一个双精度数组)。我相信这与当我的记忆被释放时有关,我有点新,所以很难弄清楚这一点。谢谢!

编辑:readGrades()被从文本文件中读取input.txt中4个整数,并将它们添加到在

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

/* 
* readGrades() 
* INPUT: double array of grades (double grades[]) 
* OUTPUT: number of grades read (int numOfGradesRead) 
*/ 
int readGrades(double (*grades)[]) { 

int numOfGradesRead = 0, 
    count = 0, 
    numRead; 

char buf[1000]; 
FILE *file = fopen("input.txt", "r"); 

if (file == NULL) { 
    perror("Can't open file"); 
} else { 
    while (fgets(buf, sizeof(buf), file)) { 

     // Convert buf to integer 
     numRead = atoi(buf); 

     // Add number read to grades[] 
     if (numRead != -999) { 
      (*grades)[count] = numRead; 
      numOfGradesRead++; 
      count++; 
     } 
    } 
} 

fclose(file); 

return numOfGradesRead; 
} 

void frequency(double grades[], int numOfGrades) { 

} 

int main() { 

double grades[100]; 
int i; 

// Initialize grades values to 0 
for (i = 0; i < sizeof(grades)/sizeof(int); i++) { 
    grades[i] = 0; 
} 

int numOfGradesRead = readGrades(&grades); 

for (i = 0; i < 4; i++) { 
    printf("%f", grades[i]); 
} 

return 0; 
} 

回答

4

一个主要错误传递的阵列的行:

for (i = 0; i < sizeof(grades)/sizeof(int); i++) { 

由于该错误,您正在使用无界索引设置grades元素的值,这会导致未定义的行为。

应该

for (i = 0; i < sizeof(grades)/sizeof(double); i++) { 
            // ^^^^^^^ Needs to be double not int 

可以使用常规

for (i = 0; i < sizeof(grades)/sizeof(grades[0]); i++) { 

使你的代码更健壮。

而且,使用for循环中的硬编码编号4来打印等级,您可能需要使用numOfGradesRead。另外,在等级之间打印空格或换行符以使输出更易于阅读。

for (i = 0; i < numOfGradesRead; i++) { 
    printf("%f\n", grades[i]); 
} 
+0

嗯看起来像这样也解决了这个问题!非常感谢,我没有发现这个错误。谢谢你的其他指针,真的很有帮助。 – Brejuro