2013-05-08 98 views
-1

我的程序一行一行地读取一个文本文件,以获得关于一群捐献者的信息。它使用一系列结构来存储信息(名称,捐赠金额)。文本文件的第一行是贡献者的数量。从那时起,代表捐赠的名称和价值。例如:我的程序给了我一个分段错误(C++)

3 
Kyle Butler 
10340 
James Wright 
5006 
John Smith 
10000 

然后,如果他们的捐款为10,000或更多,他们的名字和捐款值输出到标题下的屏幕“大主顾”,否则,它出现在标题下的“食客”

问题是,当我运行程序时,它给了我一个分段错误,没有证据显示程序输出任何东西到屏幕上。有人可以告诉我这里发生了什么吗?我对这门语言很陌生。

#include <iostream> 
#include <string> 
#include <fstream> 
#include <cstring> 
#include <cctype> 
#include <cstdlib> 

using namespace std; 

struct contribute { 
    char Name[100]; 
    int contribution; 
}; 

int main() 
{ 

    char tempstore[100]; 
    int linecount=1; 
    int pointerindex=0; 
    ifstream inputfile("myfile.txt"); 

    if (!inputfile.is_open()){ 
     cout << "Error opening file"; 
    } 

    inputfile.getline(tempstore, 20); 
    int contributors = atoi(tempstore); 
    contribute carray[contributors]; 

    while (!inputfile.eof()){ 
     if(linecount ==1){ 
      linecount++; 
      continue; 
     } 

     inputfile.getline(tempstore, 20); 


     if ((linecount % 2) == 0){ 
      strcpy((carray[pointerindex]).Name, tempstore); 

     } 
     else { 
      (carray[pointerindex]).contribution = atoi(tempstore); 
     } 


     ++linecount; 
     ++pointerindex; 
    } 




    cout << "\n#####--#-Grand Patrons-#--#####\n"; 

    for (int i=0; i<contributors; i++){ 
     if (carray[i].contribution >= 10000){ 
      cout << "Name: " << carray[i].Name << endl; 
      cout << "Contribution: " << carray[i].contribution << endl << endl; 
     } 
    } 

    cout << "\n#####--#-Patrons-#--#####\n"; 

    for (int d=0; d<contributors; d++){ 
     if (carray[d].contribution < 10000){ 
      cout << "Name: " << carray[d].Name << endl; 
      cout << "Contribution: " << carray[d].contribution << endl << endl; 
     } 
    } 

    inputfile.close(); 

    return 0; 
} 
+2

什么是断层?你有没有试过在调试器中运行它?你在使用什么IDE,OS? – OldProgrammer 2013-05-08 02:25:19

+1

贡献carray [贡献者];你的编译器让你这么做?它不应该是贡献* carray =新贡献[贡献者]; ? – 2013-05-08 02:30:25

+0

@JonathanHenson 不,这不是一个动态数组,只是一个结构数组,我原来使用了一个动态数组,但我认为这可能是segfault背后的原因,所以我改为静态数组。无论哪种方式。 – 2013-05-08 02:36:20

回答

0

你递增每个pointerindex通过循环,而不是你的贡献量读取之后。因此,当你在5006阅读时,你有pointerindex == 3,这导致在carray[pointerindex]上的段错误。

++pointerindex移至else声明中。

+0

非常感谢!我知道这会是一件简单的事情,它突然变得非常有意义! – 2013-05-08 02:36:59