【发布时间】:2012-03-09 10:36:11
【问题描述】:
卡片类头
class card {
public:
string rtn_suit() { return suit; }
int rtn_rank() { return rank; }
void set_suit(string new_suit){ suit = new_suit; }
void set_rank(int new_rank) { rank = new_rank; }
card();
card(string suit, int rank);
card(const card& copyCARD);
~card();
private:
string suit;
int rank;
};
#endif
card.cpp
#include "card.h"
card::card()
{
set_suit("default");
set_rank(0);
}
card::card(string new_suit, int new_rank)
{
// allows for using private class member variables
set_suit(new_suit); // can be anything (string)
set_rank(new_rank); // anticipating simple number ranking (int)
}
card::~card() {}
甲板类标题
class deck {
public:
static const int default_cap = 52;
void addCard(card *new_card);
deck & operator=(const deck &sourceDECK);
void deckPrint();
// ----------------------------------------------------
deck(int init_cap = default_cap);
deck(const deck& sourceDECK);
~deck(){ delete [] decklist ; }
private:
card *decklist;
int used;
int capacity;
};
#endif
甲板类
deck::deck(int init_cap)
{
card **decklist = new card*[init_cap];
for(int i=0;i<init_cap;i++)
{
decklist[i]=new card;
}
capacity = init_cap;
used=0;
}
deck::deck(const deck& sourceDECK)
{
card **newDECK;
if (capacity != sourceDECK.capacity)
{
newDECK = new card*[sourceDECK.capacity];
for(int i=0;i<sourceDECK.capacity;i++) { newDECK[i]=new card(); }
decklist = /*reinterpret_cast<card*>(*/newDECK/*)*/;
capacity = sourceDECK.capacity;
}
used = sourceDECK.used;
copy(sourceDECK.decklist, sourceDECK.decklist+ used, decklist);
}
deck& deck::operator= (const deck& sourceDECK)
{
if (this == &sourceDECK)
return *this;
card ** newDECK;
if (capacity != sourceDECK.capacity)
{
newDECK = new card*[sourceDECK.capacity];
for(int i=0;i<sourceDECK.capacity;i++) { newDECK[i]=new card(); }
for (int i=0; i<capacity; i++) { delete &decklist[i]; }
delete [ ] decklist;
decklist = /*reinterpret_cast<card*>(*/newDECK/*)*/;
capacity = sourceDECK.capacity;
}
// Copy the data from the source array:
used = sourceDECK.used;
copy(sourceDECK.decklist, sourceDECK.decklist + used, decklist);
return *this;
}
void deck::addCard(card* new_card)
{
//------- Not using vectors----
//deckList.push_back(new_card);
//cout << "Card added."<<endl;
decklist[used] = new_card;
//decklist[used].set_rank(new_card->rtn_rank());
//decklist[used].set_suit(new_card->rtn_suit());
++used;
cout << "Card added."<<endl;
}
void deck::deckPrint()
{
if ( capacity > 0 )
{
for(int i = 0; i < capacity; i++)
{
cout << "----------"<<endl;
cout << decklist[i].rtn_rank() << " ";
cout << decklist[i].rtn_suit() << endl;
cout << "----------"<<endl;
}
}
else\
{
cout << "There are no cards in the deck."<<endl;
}
}
最后是 main()
int main ()
{
string new_suit, del_suit;
int new_rank, del_rank;
deck newDeck;
card *temp_card;
cout<<"Enter the card's suit: ";
cin>>new_suit;
cout<<endl<<"Enter the card's rank: ";
cin>>new_rank;
cin.clear();
cin.ignore(1000, '\n');
temp_card = new card(new_suit, new_rank);
newDeck.addCard(temp_card);
newDeck.deckPrint();
return 0;
}
不知何故,我没有正确初始化或分配。或者也许我搞砸了指针......
按原样,由于以下原因无法编译:
错误 C2679:二进制“=”:未找到采用“卡片 *”类型的右侧操作数的运算符(或没有可接受的转换) \projects\cards\cards\card.h(27): 可能是 'card &card::operator =(const card &)' 同时尝试匹配参数列表 '(card, card *)'
如果我使用(当前已注释掉)“更深的副本”又名:
decklist[used].set_rank(new_card->rtn_rnk());
它会在deckPrint()中抛出错误
我最初用向量编写了这个简单的卡片类实现,然后我们开始学习动态数组 - 我有一个好主意,尝试在混合中抛出指针。同样,这不是家庭作业——这是为了个人成长。
如果有我错过的教程清楚地概述了我正在尝试做的事情,请随时向我指出 - 我已经倾注了前 30 名谷歌搜索结果和大量帖子。有一半的人推荐使用向量,这又不是我的目标。
感谢您的宝贵时间,我知道这很长。
【问题讨论】:
-
请做一个最小尺寸的例子来问你的问题。人们不太可能花时间阅读所有这些代码来帮助您。
-
检查您的
deck复制构造函数/赋值运算符。它们分配指针数组,但不为这些指针分配实际对象。将其与deck构造函数(两者都执行)进行比较。赋值运算符也删除指针数组而不删除底层对象... -
@Carl - 我减掉了所有的脂肪。
-
@ AzzA - 我会解决这些问题,谢谢 - 我还没有开始清理工作。我的问题在 copy-cstr/assignment-op 中吗?我以为他们在别处……
-
@mate0x 很遗憾,您删除了复制构造函数和赋值运算符 - 它们是错误的根源。如果我没记错的话,在分配
card *指针数组后,您试图将对象card复制到其中。这就是为什么你得到binary '=' : no operator found which takes a right-hand operand of type 'card *'... 遵循deck构造函数的逻辑:用new分配指针数组,然后为循环中的每个指针创建另一个new的对象。以相反的顺序删除 - 在循环中删除对象,然后是指针数组。
标签: c++ arrays pointers dynamic