如何创建包含对象向量的类?

时间:2015-08-18 15:34:03

标签: c++ segmentation-fault

我在FlashcardFlashcard.h中声明了课程Flashcard.cpp。我想创建一个类CardList,它应该存储std::vector<Flashcard>以及其他一些信息,并提供一些操作它的方法。

CardList.h

#include <vector>
#ifndef LC_FLASHCARD
    #include "Flashcard.h"
#endif

class CardList
{
    public:
        /* Constructor */
        CardList (int number_of_cards = 1);
        /* Returns the j'th card. */
        Flashcard getCard ( int j );
        /* Replaces the j'th card with new_card. */
        void setCard ( int j, Flashcard new_card );
        /* ... */

    private:
        std::vector<Flashcard> card_list;
        /* ... */
};

CardList.cpp

#include "CardList.h"

CardList::CardList ( int number_of_cards ) {
    std::vector<Flashcard> card_list(number_of_cards);
}

CardList::~CardList ( void ) { }

Flashcard CardList::getCard ( int j ) {
    return this->card_list[j];
}

void CardList::setCard ( int j, Flashcard new_card ) {
    this->card_list[j] = new_card;
}

Flashcard CardList::drawCard ( void ) {
    return this->getCard(0);
}

问题

每当我拨打CardList::getCardCardList::setCard时,我都会遇到段错误。例如:

#include "Flashcard.h"
#include "CardList.h"
/* ... */

int main( int argc, char** argv ) {
    /* Creates flashcard card */
    Flashcard card;
    /* (do things to card) */ 

    CardList card_list(7);
    std::cout << "Declaration and initialisation of card_list successful" << std::endl;

    card_list.setCard(0, card); // *** SEGFAULT ***
    std::cout << "Assignment successful" << std::endl;

    /* ... */
    return 0;
}

我认为问题在于我的构造函数CardList::CardList,但我该如何解决?

2 个答案:

答案 0 :(得分:1)

这段代码完全不符合您的想法:

CardList::CardList ( int number_of_cards ) {
    std::vector<Flashcard> card_list(number_of_cards);
}

你应该做的是:

CardList::CardList ( int number_of_cards ) :
    card_list(number_of_cards)
{
}

您的seg错误是因为您从未调整过card_list的大小。您的构造函数版本中的代码创建了一个不同的临时card_list,一旦构造函数退出就会被销毁。同时,card_list成员的默认构造大小为零。

答案 1 :(得分:0)

您的班级中有一个名为card_list的媒体资源,但您未在构造函数中正确进行初始化。在您的代码中,您有:

CardList::CardList ( int number_of_cards ) {
    std::vector<Flashcard> card_list(number_of_cards);
}

在这种情况下,您正在初始化名为local variable的{​​{1}}而不是类{&#39}。财产card_list

正确的方法如下:

card_list