Указатель на динамический массив указателей на объекты - PullRequest
1 голос
/ 18 февраля 2012

Заголовок класса карты

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: бинарный '=': не найден оператор, который принимает правый операндтипа 'card *' (или нет приемлемого преобразования) \ projects \ cards \ cards \ card.h (27): может быть 'card & card :: operator = (const card &)' при попытке сопоставить список аргументов'(карточка, карточка *)'

Если я использую (в настоящее время закомментированную) «более глубокую копию», известную как:

decklist[used].set_rank(new_card->rtn_rnk());

, то это приведет к ошибке в deckPrint ()

Первоначально я написал это с помощью векторов для простой реализации класса карт, а затем мы начали изучать динамические массивы - и у меня появилась блестящая идея попытаться добавить указатели в микс.Опять же, это не домашняя работа - это для личного роста.

Если есть пропущенное мной учебное пособие, в котором четко изложено, что я пытаюсь сделать, не стесняйтесь указывать мне на это тоже - у меня естьзалил 30 лучших результатов Google и тонны постов здесь.Половину времени люди рекомендуют использовать векторы, что опять-таки не моя цель.

Спасибо за ваше время, я знаю, что это долго.

1 Ответ

2 голосов
/ 18 февраля 2012

Я думаю, у вас есть небольшая ошибка в функции построения класса вашей колоды.

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

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

здесь карта ** decklist - это новый указатель на указатель, я думаю, вы хотите инициализировать приватную переменную decklist.

так что вы можете изменить его на

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

Здесь есть закрытая переменная.

и измените объявление

private:
   card **cardlist;
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...