类变量的更新值-C＆＃x2B;＆＃x2B;

Question

我才刚刚开始学习如何编码，并且遇到了一个我似乎无法解决的问题。 更具体地说，问题发生在“借用”功能中。

在以下程序中，即使我使用了getters和setter，我也无法以某种方式更新公共类变量“库存”的价值。 它似乎在Cout上正确更新，但不能“永久保存”。我的猜测是，它正在修改变量的副本，而不是变量本身。

我已将代码附加到帖子上！请让我知道是否应该上传整个文件！

提前致谢！

void Book::borrows() {
    int searchid;
    bool isfound=false;
    cout<<"Please enter the unique ID of the book:\t\t";
    cin>>searchid;
    for(auto i:myBooks){
        if (i.id==searchid){
            cout<<"This book matches your search:\t"; print(i);
            if (i.stock==0) {
                cout<<"Book is out of stock!"<<endl;
            } else {
                setStock((i.stock-1));
                cout<<"Successfully borrowed!! Now there are only "<<getStock()<<" copies left in stock!"<<endl;
            }
            isfound=true;
        }
    }
    if (isfound== false){
        cout<<"++++\t\tBook not found++++\t\t"<<endl;
    }
    system("pause");
}

int Book::getStock() const {
    return stock;
}

void Book::setStock(int newstock) {
    Book::stock = newstock;
}

编辑1：

这是我的班级结构和我的向量：

class Book{
public:
    int id;
    string title;
    int stock;

    void add();
    void displayall();
    void displayspecific();
    void print(Book);
    void borrows();
    void returns();

    int getStock() const;

    void setStock(int stock);
};

vector<Book> myBooks;

Answer 1

您的实际问题是，您正在使用书籍对象的副本进行操作，而不是书籍成员的固定器和获取者。

for(auto i:myBooks){

您需要

for(auto &i:myBooks){

，但正如其他指出的那样，您需要2堂课，图书馆和书籍。

Answer 2

我同意托马斯·马修斯（Thomas Matthews）的评论。您当前的book结构没有意义，因为大多数方法用于使用book> book s的集合而不是特定的book> book

您要将书籍集作为library，那么下面的代码对于在book和library

#include <string>
#include <vector>

using namespace std;

class Book {
public:
    int id;
    string title;
    int stock;

    // These methods operate on this particular Book instance
    int getStock() const;
    void setStock(int);
};

int Book::getStock() const {
    return stock;
}

void Book::setStock(int newstock) {
    stock = newstock;
}

class Library {
public:
    vector<Book> myBooks;

    // These methods operate on the vector of books
    void add();
    void displayall();
    void displayspecific();
    //void print(Book); // This method probably belongs inside Book
    void borrows();
    void returns();
};

int main() {
    return 0;
}

重写借款根据库方法意味着在该library's mybooks vector上操作。当您找到特定的book您想要在矢量内部的时，您可以调用其getters和setter，可以很好地

解决您的问题，在当前代码中，这是在您的book s并给您每个副本：

for(auto i:myBooks)

您希望它得到参考而不是副本：

for(auto& i:myBooks)