1

我正在为 MIT OCW 课程做一些事情,它要求编写一个“图书馆”课程。现在我有这个:

#include <iostream>

using namespace std;

class Book{
    string title;
    bool rented;
public:
    Book(string bookTitle){
        title = bookTitle;
    }
    void borrowed(){
        rented = true;
    }
    void returned(){
        rented = false;
    }
    bool isBorrowed(){
        return rented;
    }
    string getTitle(){
        return title;
    }
};

class Library{
    string Lname;
    Book bookList[100000000];
    int numOfBooks = 0;
public:
    Library(string name){
        Lname = name;
    }
    void addBook(string bookName){
        bookList[numOfBooks] = Book(bookName);
        numOfBooks += 1;
    }
    void returnInfo(){
    cout << "Library hours:" << "\n" << "Libraries are open daily from 9am to 5pm." << "\n" << "Library addresses:" << "\n" << "10 Main St." << "\n" << "228 Liberty St.";
    }

};
int main()
{
    Library l = Library("Hi");
    return 0;
}

当我编译它给我错误 no matching function for call to 'Book::Book()' for line 35。

4

3 回答 3

2

编译器抱怨你没有为 Book 定义默认构造函数,你可以试试这个:

 explicit Book(const string& bookTitle = std::string("")) 
 : title(bookTitle) 
 {
 }

在堆栈上定义 100000000 我的原因溢出,

Book bookList[100000000];

尝试使用向量代替

std::vector<Book> bookList;

在 addBook 中,您可以简单地使用std::vector::push_back添加新书。要获得总书数,请使用std::vector::size()

void addBook(const string& bookName){
    bookList.push_back(Book(bookName));
}

建议:

  1. 要将参数传递给 std::string 之类的函数,请通过 const 传递
  2. 参考是更好的方法。如果可能,使用成员初始化列表
 Library(const string& name) : Lname(name) {  }
 void addBook(const string& bookName){ }
于 2013-06-07T04:46:45.007 回答
0
Book bookList[100000000]; 

将尝试使用默认构造函数创建 100000000 个对象..

于 2013-06-07T04:51:31.600 回答
0

bookList是作为对象数组而不是字符串数组创建的,然后您将字符串 (bookName)分配给bookList,所以我认为问题出在此处。

于 2013-06-07T04:58:36.800 回答