-1

マルチマップから印刷しようとしています。私のマルチマップは次のとおりです。multimap<int,Questions*> map; 私がしたいのは、質問を呼び出すときです。その 3 つの質問をランダムな順序で出力します。しかし、これまでに取得したのは、printQuestion が呼び出されたときの実行時エラーだけです。

Run time error: 
Debug Assertion Failed!
Expression: map/set iterator may not be dereferencable

これが私のコードです：

#include <iostream>
#include "Questions.h"
using namespace std;

Questions :: Questions()
{
}

Questions::Questions(string question,string correctAnswer, string wrongAnswer1,string wrongAnswer2,string wrongAnswer3)
{
    this->question = question;
    this->pAnswers = new string[4];
    this->pAnswers[0]=wrongAnswer1;
    this->pAnswers[1]=wrongAnswer2;
    this->pAnswers[2]=wrongAnswer3;
    this->pAnswers[3] =correctAnswer;
    this->shuffle(this->pAnswers,4);
    this->correctAnswer = correctAnswer;
}

void Questions::shuffle(string *array, int n)
{
    random_shuffle(&this->pAnswers[0],&this->pAnswers[4]);
}

string Questions::getQuestion()
{
    return this->question;
}

string Questions::getCorrectAnswer()
{
    return this->correctAnswer;
}

string* Questions::getAnswers()
{
    return this->pAnswers;
}

bool Questions::checkAnswer(string answer)
{
    if(this->correctAnswer.compare(answer)==0)
    {
        return true;
    }
    return false;
}

void Questions::questionStore()
{
    Questions *q1 = new Questions("Whats the oldest known city in the world?", "Sparta" , "Tripoli" , "Rome", "Demascus");
    Questions *q2 = new Questions("What sport in the olympics are beards dissallowed?", "Judo", "Table Tennis" , "Volleyball", "Boxing");
    Questions *q3 = new Questions("What does an entomologist study?", "People" , "Rocks" , "Plants", "Insects");
    Questions *q4 = new Questions("Where would a cowboy wear his chaps?", "Hat" , "Feet" , "Arms", "Legs");
    Questions *q5 = new Questions("which of these zodiac signs is represented as an animal that does not grow horns?", "Aries" , "Tauris" , "Capricorn", "Aquarius");
    Questions *q6 = new Questions("Former Prime Minister Tony Blair was born in which country?", "Northern Ireland" , "Wales" , "England", "Scotland");
    Questions *q7 = new Questions("Duffle coats are named after a town in which country?", "Austria" , "Holland" , "Germany", "Belgium");
    Questions *q8 = new Questions("The young of which creature is known as a squab?", "Horse" , "Squid" , "Octopus", "Pigeon");
    Questions *q9 = new Questions("The main character in the 2000 movie ""Gladiator"" fights what animal in the arena?", "Panther" , "Leopard" , "Lion", "Tiger");

    addQuestion(1,q1);
    addQuestion(1,q2);
    addQuestion(1,q3);
    addQuestion(2,q4);
    addQuestion(2,q5);
    addQuestion(2,q6);
    addQuestion(3,q7);
    addQuestion(3,q8);
    addQuestion(3,q9);
}

void Questions::addQuestion(int level, Questions *question)
{
    map.insert(pair<int,Questions*>(level,question));
}


Questions* Questions::printQuestion(int level)
{
    multimap<int, Questions*>::iterator iter;
    pair<multimap<int, Questions*>::iterator,multimap<int, Questions*>::iterator> constIter;
    for (multimap< int, Questions*, less< int > >::const_iterator iter =map.begin();
    iter != map.end(); ++iter )
      cout << iter->first << '\t' << iter->second << '\n';



        /*constIter = map.equal_range(level);
    size_t sz = distance(constIter.first, ret.second);
    size_t idx = rand();
    if(ret.first != ret.second)
    advance(ret.first, idx);
    it =ret.first;
    Questions* question = (*it).second;
    return (*it).second;
       cout << question->getQuestion() << std::endl;*/
       return iter->second;
}

誰でもこれで私を助けることができますか？

4

1 に答える 1

1

「実行時エラー」が発生したと言うとき、私は関数がprintsとintegerとポインターのアドレスを想定していると思います。おそらく、質問へのポインターを出力する前に逆参照するつもりでした。

*it->second

もう 1 つの問題は、 を返すときに末尾イテレータを逆参照することです。ステートメントiter->secondの前のループは、 のときに終了します。過去の終了イテレータを逆参照することは違法であり、未定義の動作を引き起こします (これは実際にアサートされているエラーです)。returniter == map.end()

もちろん、Questionオブジェクトを最初に格納し、ヒープに割り当てないようにする必要があります。オブジェクトはリークする可能性があります。また、"Question* s *" オブジェクトに "question" を格納するのも適切ではないようです。おそらく、個別の質問用のクラスと複数の質問を収集するクラスを分けた方がよいでしょう。

于 2012-12-18T22:59:50.553 に答える