C++ 将输入值与向量中的值进行比较

C++ 将输入值与向量中的值进行比较,c++,C++,我试图输入一种数据验证形式,当用户输入一本书的ISBN号时,如果它已经被存储了,那么它将输出一条错误消息。然而,我做这件事有困难。我不确定是否正确重载==运算符,也不确定如何比较store_ISBN()函数中的向量值 代码如下: #include "std_lib_facilities.h" // Classes --------------------------------------------------------------------- class Book{ public:

我试图输入一种数据验证形式,当用户输入一本书的ISBN号时,如果它已经被存储了,那么它将输出一条错误消息。然而,我做这件事有困难。我不确定是否正确重载==运算符,也不确定如何比较store_ISBN()函数中的向量值

代码如下:

#include "std_lib_facilities.h"

// Classes ---------------------------------------------------------------------

class Book{
public:
       vector<Book> books; // stores book information
       Book() {}; // constructor
       friend ostream& operator<<(ostream& out, const Book& b);
       bool operator==(const Book& d);
       string what_title();
       string what_author();
       int what_copyright();
       void store_ISBN();
       void is_checkout();
private:
        char check;
        int ISBNfirst, ISBNsecond, ISBNthird;
        char ISBNlast;
        string title;
        string author;
        int copyright;
};

// Class Functions -------------------------------------------------------------

string Book::what_title()
{
       cout << "Title: ";
       getline(cin,title);
       cout << endl;
       return title;
}

string Book::what_author()
{
       cout << "Author: ";
       getline(cin,author);
       cout << endl;
       return author;
}

int Book::what_copyright()
{
    cout << "Copyright Year: ";
    cin >> copyright;
    cout << endl;
    return copyright;
}

void Book::store_ISBN()
{
     bool test = false;
     cout << "Enter ISBN number separated by spaces: ";
     while(!test){
     cin >> ISBNfirst >> ISBNsecond >> ISBNthird >> ISBNlast;
     for(int i = 0; i < books.size(); ++i)
             if(ISBNfirst == books[i]) cout << "test"; // no idea how to implement this line            
     if((ISBNfirst<0 || ISBNfirst>9) || (ISBNsecond<0 || ISBNsecond>9) || (ISBNthird<0 || ISBNthird>9))
                   error("Invalid entry.");
     else if(!isdigit(ISBNlast) && !isalpha(ISBNlast))
          error("Invalid entry.");
     else test = true;}     
     cout << endl;
}

void Book::is_checkout()
{
     bool test = false;
     cout << "Checked out?(Y or N): ";
     while(!test){
     cin >> check;
     if(check == 'Y') test = true;
     else if(check == 'N') test = true;                                
     else error("Invalid value.");}
     cout << endl;
}

// Operator Overloading --------------------------------------------------------

bool Book::operator==(const Book& d){ // is this right???
     if((ISBNfirst == d.ISBNfirst) && (ISBNsecond == d.ISBNsecond) 
             && (ISBNthird == d.ISBNthird) && (ISBNlast == d.ISBNlast)) return true;
     else return false;
}

ostream& operator<<(ostream& out, const Book& b){
         out << "Title: " << b.title << endl;
         out << "Author: " << b.author << endl;
         out << "ISBN: " << b.ISBNfirst << "-" << b.ISBNsecond << "-" << b.ISBNthird << "-" << b.ISBNlast << endl;
         out << endl;
         return out;
}

// Main ------------------------------------------------------------------------     

int main()
{
    Book store;
    string question;
    while(true){
        store.what_title();
        store.what_author();
        store.what_copyright();
        store.store_ISBN();
        store.is_checkout();
        store.books.push_back(store);
        cout << "Are you finished?(Y or N): ";
        cin >> question;
        if(question == "Y") break;
        else if(question == "N"){
              cout << endl;
              cin.ignore();}
        else error("Invalid value.");
        }
    cout << endl;
    cout << "Books stored -\n" << endl;
    for(int i = 0; i < store.books.size(); ++i)
            cout << store.books[i];
    keep_window_open();
}
#包括“std_lib_facilities.h”
//班级---------------------------------------------------------------------
课堂用书{
公众:
矢量图书;//存储图书信息
Book(){};//构造函数
friend ostream&operator>ISBNlast;
for(int i=0;iif(ISBNfirst==books[i])coutbooks引用Book类的向量。您正在将Book与整数进行比较,这是未定义的行为。您需要先取消对Book对象的引用,然后才能访问其数据成员

首先,不要使用下标[]表示法访问向量。这样做效率低,而且会使生活变得困难。请使用迭代器(例如,不确定要如何实现):

但是,这不是您的问题。您可以使用
->
操作符取消对对象的引用以访问其成员。但是,您将成员设置为私有,因此您需要一个get函数,如

ISBNf() { return ISBNfirst; }
或者将您的成员公开,但这不是一个好主意(人们可以愚弄您的数据)。不过,为简单起见,假设他们是公开的,这就是您想要的:

for (std::vector::iterator it = books.begin(); it != books.end(); ++it)
{
    if (*this == *it) cout << "test"; 
}
相反,请使用isalphnum()函数:

else if (!isalphnum(ISBNlast));

已发布;我将编辑我的帖子以指出您代码中的所有缺陷。

books引用Book类的向量。您正在将Book与整数进行比较,这是未定义的行为。您需要先取消对Book对象的引用,然后才能访问其数据成员

首先,不要使用下标[]表示法访问向量。这样做效率低,而且会使生活变得困难。请使用迭代器(例如,不确定要如何实现):

但是,这不是您的问题。您可以使用
->
操作符取消对对象的引用以访问其成员。但是,您将成员设置为私有,因此您需要一个get函数,如

ISBNf() { return ISBNfirst; }
或者将您的成员公开,但这不是一个好主意(人们可以愚弄您的数据)。不过,为简单起见,假设他们是公开的,这就是您想要的:

for (std::vector::iterator it = books.begin(); it != books.end(); ++it)
{
    if (*this == *it) cout << "test"; 
}
相反,请使用isalphnum()函数:

else if (!isalphnum(ISBNlast));

发布;我将编辑我的帖子,指出您代码中的所有缺陷。

我不太确定用户希望为ISBN键入什么

从一个流读入一个整数将把数字读入一个空格,然后将结果转换成整数(如果一切顺利的话)。读入一个字符将存储字符值。因此,目前你正在验证一个ISBN看起来像三个单位数(0-9),然后是下一个字符。我认为ISBN不是这样的

您的
运算符==
看起来正常,但请注意,对于bool返回值

if (X) return true;
else return false;
可以替换为

return X;
因为条件句已经是bool类型

设置ISBN值(以及您计划在
operator==
中使用的任何其他字段,如果尚未完成),在商店中查找匹配书籍的方法是:

for(int i = 0; i < books.size(); ++i)
    if(*this == books[i]) cout << "test";
for(int i=0;i如果(*this==books[i])不能我不确定用户希望为ISBN键入什么

从一个流读入一个整数将把数字读入一个空格,然后将结果转换成整数(如果一切顺利的话)。读入一个字符将存储字符值。因此,目前你正在验证一个ISBN看起来像三个单位数(0-9),然后是下一个字符。我认为ISBN不是这样的

您的
运算符==
看起来正常,但请注意,对于bool返回值

if (X) return true;
else return false;
可以替换为

return X;
因为条件句已经是bool类型

设置ISBN值(以及您计划在
operator==
中使用的任何其他字段,如果尚未完成),在商店中查找匹配书籍的方法是:

for(int i = 0; i < books.size(); ++i)
    if(*this == books[i]) cout << "test";
for(int i=0;iif(*this==books[i])不能“使用下标[]表示法访问向量。这是低效的”.神话。在这种情况下,通过索引进行访问不太可能慢得多,而且比迭代器代码要简洁得多。你能给我提供一些关于这方面的信息吗?我想在停止使用迭代器之前得到一些证明。此外,我认为解引用运算符更清晰。另外,你注意到了吗store_ISBN是Book?的一个成员函数,因此不需要访问器、公共成员或->来访问成员变量。我并不赞成这种设计,但我认为您声称语法问题并不存在。请注意,
(*it)->ISBNFirst
不起作用,因为它是书的向量,而不是书的向量。
(*it)
是一本书,
(*it)。ISBNFirst是访问其字段的正确方式。哦,谢谢。我搞混了(我一直在使用Node,这类东西太多了)使用迭代器,因为你认为它们更清楚。我不想说服你反对它,它对通用代码至关重要。如果你想看看它对实际代码有什么不同,请检查这两个不同的循环的分解——通常是操作符[]。每次访问都会导致额外的负载,读取向量缓冲区的地址,并进行乘法运算。在性能关键的循环中,当然,迭代器可能会快一点。但在我看来,我们不应该训练初学者对小向量中的微小性能差异大惊小怪。不要优化无法测量的内容。“不要使用下标[]符号访问向量。这是低效的”。神话。在这种情况下,通过索引访问不太可能慢得多,而且比迭代器代码要简洁得多。你能把我链接到