C++ 将输入值与向量中的值进行比较
我试图输入一种数据验证形式,当用户输入一本书的ISBN号时,如果它已经被存储了,那么它将输出一条错误消息。然而,我做这件事有困难。我不确定是否正确重载==运算符,也不确定如何比较store_ISBN()函数中的向量值 代码如下:C++ 将输入值与向量中的值进行比较,c++,C++,我试图输入一种数据验证形式,当用户输入一本书的ISBN号时,如果它已经被存储了,那么它将输出一条错误消息。然而,我做这件事有困难。我不确定是否正确重载==运算符,也不确定如何比较store_ISBN()函数中的向量值 代码如下: #include "std_lib_facilities.h" // Classes --------------------------------------------------------------------- class Book{ public:
#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;i if(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;i if(*this==books[i])不能“使用下标[]表示法访问向量。这是低效的”.神话。在这种情况下,通过索引进行访问不太可能慢得多,而且比迭代器代码要简洁得多。你能给我提供一些关于这方面的信息吗?我想在停止使用迭代器之前得到一些证明。此外,我认为解引用运算符更清晰。另外,你注意到了吗store_ISBN是Book?的一个成员函数,因此不需要访问器、公共成员或->来访问成员变量。我并不赞成这种设计,但我认为您声称语法问题并不存在。请注意,(*it)->ISBNFirst
不起作用,因为它是书的向量,而不是书的向量。(*it)
是一本书,(*it)。ISBNFirst是访问其字段的正确方式。哦,谢谢。我搞混了(我一直在使用Node,这类东西太多了)使用迭代器,因为你认为它们更清楚。我不想说服你反对它,它对通用代码至关重要。如果你想看看它对实际代码有什么不同,请检查这两个不同的循环的分解——通常是操作符[]。每次访问都会导致额外的负载,读取向量缓冲区的地址,并进行乘法运算。在性能关键的循环中,当然,迭代器可能会快一点。但在我看来,我们不应该训练初学者对小向量中的微小性能差异大惊小怪。不要优化无法测量的内容。“不要使用下标[]符号访问向量。这是低效的”。神话。在这种情况下,通过索引访问不太可能慢得多,而且比迭代器代码要简洁得多。你能把我链接到