1

我正在编写一个程序，允许用户输入要读取的人员数据库的文件名；然后，程序会在每个状态链接中创建状态对象的链接列表和人员对象的链接列表，以组织文件中的信息。

我知道链表部分有效，因为我能够直接在文件名中编码并打印出州列表和每个州的人员列表；但是，当我尝试允许用户输入文件名作为命令时，我收到了总线错误。当我在 gdb 中运行代码时，它告诉我的只是：

Program received signal SIGBUS, Bus error.
0x280df0bd in std::operator>><char, std::char_traits<char> > ()
   from /usr/lib/libstdc++.so.5

我什至没有一个行号可以离开！任何帮助将不胜感激。这是我的代码的命令和读取部分：

List<State*>* read(char* filename) {
    string fname, lname, birthday, state;
    int ssn;
    List<State*>* state_list = new List<State*>();

    ifstream file(filename);
    if (file.fail()) {
        cerr << "Error reading file.\n";
        exit(1);
    }

    while (!file.eof()) {
        file >> birthday >> ssn >> fname >> lname >> state;
        Link<State*>* searchres = searchList(state, state_list);
        Person* p = new Person(fname, lname, ssn, birthday, state);
        if (searchres == NULL) // create new state
        {
            State* addedstate = state_list->addLink(new State(state))->data;
            addedstate->res_list.addLink(p);
        }

        else // add to pre-existing state
        {
            searchres->data->res_list.addLink(p);
        }
    }
    return state_list;
}

void main() {
    string cmd;
    cout << "Type your command in all lowercase letters.\n";
    cin >> cmd;
    if (cmd == "read") {
        char* filnm;
        cin >> filnm;
        List<State*>* state_ls = read(filnm);
        Link<Person*>* counter = state_ls->first->data->res_list.first;
        while (counter != NULL) {
            cout << counter->data->ssn << "\n";
            counter = counter->next;
        }
    }
}
4

1 回答 1

7

马上，你有一个问题：

char* filnm;
cin >> filnm;

指针未初始化，但您使用该指针读取信息。

使用std::string，或适当大小的 char 数组。

要std::string在文件的打开中使用 a：

std::string filnm;
cin >> filnm;
read(filnm.c_str());

您的read函数还应该将参数更改为const char*而不是char *. 您没有更改正在传递的字符数组的内容，所以它应该是const.

编辑：您实际上不需要使用c_str()，因为std::string有一个带有const char*. 不过，将参数更改为const char *filename函数中的read()。

于 2015-04-16T20:31:08.620 回答