【问题标题】:Pointer to dynamic array of pointers to objects指向对象指针的动态数组的指针
【发布时间】: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


【解决方案1】:

我认为你的牌组构造函数中有一个小错误。

deck::deck(int init_cap)
{
    card **decklist = new card*[init_cap];

    for(int i=0;i<init_cap;i++)
    {
        decklist[i]=new card;
    }
}

这里的card **decklist是一个新的指针,我猜你要初始化私有变量decklist。

所以你可以把它改成

deck::deck(int init_cap)
{
    decklist = new card*[init_cap];

decklist 这里是私有变量。

并更改声明

private:
   card **cardlist;

【讨论】:

  • 漂亮,非常感谢!编译并运行!仍然有错误,但你让我克服了障碍。感谢您的宝贵时间!
猜你喜欢
  • 1970-01-01
  • 2017-02-07
  • 2021-03-29
  • 1970-01-01
  • 2016-12-23
  • 1970-01-01
  • 2013-04-06
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多