c++ - 无法编译 C++ 构造函数

Question

谁能向我解释这段代码有什么问题以及如何修复它？这是一个更大项目的一部分，如果您需要更多信息以便给出答案，请告诉我。

我得到的错误是：

  g++ -c search.cpp
    search.cpp: In constructor ‘Search::Search()’:
    search.cpp:26:26: error: ‘callnumber’ was not declared in this scope
    make: *** [search.o] Error 1



Search::Search()
        {
            ifstream input;
            input.open("data.txt", ios::in);

            if(!input.good())
                //return 1;
            string callnumber;
            string title;
            string subject;
            string author;
            string description;
            string publisher;
            string city;
            string year;
            string series;
            string notes;

            while(! getline(input, callnumber, '|').eof())
            {   
                getline(input, title, '|');
                getline(input, subject, '|');
                getline(input, author, '|');
                getline(input, description, '|');
                getline(input, publisher, '|');
                getline(input, city, '|');
                getline(input, year, '|');
                getline(input, series, '|');
                getline(input, notes, '|');


            Book *book = new Book(callnumber, title, subject, author, description, publisher, city, year, series, notes);
            Add(book);
            }   

                input.close();

        }

Answer 1

在这条线上：

if(!input.good())
    //return 1;
string callnumber;

您用分号注释掉了该行并且您没有使用大括号，并且 C++ 在标记之间的空格和换行符¹对空格不敏感，因此通过删除注释（并添加缩进）我们可以看到相当于

if (!input.good())
    string callnumber;

我们可以看到 的声明callnumber是本地的if. 添加大括号或分号以使声明位于callnumber外部if（或完全删除它）：

if (!input.good()) { // or ;
    //return 1;
}
string callnumber;

---

¹宏除外

Answer 2

因为你评论了这一行

if(!input.good())
    //return 1;

字符串 callnumber 仅在该范围内声明，在其外部无法使用。

也删除 if 语句或取消注释返回行

Answer 3

        if(!input.good())
            //return 1;
        string callnumber;

有效地

        if(!input.good()) {
            //return 1;
            string callnumber;
        }

因为注释掉的//return 1;不是声明。所以callnumber被创建然后立即消失在范围之外。