2017-07-24 1293 views
0

我正在进行网站中的示例测试:https://www.testdome.com/for-developers/solve-question/9808 我为基类和派生类分别添加了两个析构函数以释放由构造函数分配的内存。这个问题的前两个要求是成功解决,但结果失败,因为:使用定时多项选择测试作为多项选择测试:超过内存限制在C++测试示例中超出内存限制

我修改后的代码,如果您可以帮助修复失败...

#include <iostream> 
#include <string> 

class MultipleChoiceTest 
{ 
public: 
    MultipleChoiceTest(int questionsCount) 
    { 
     this->questionsCount = questionsCount; 
     answers = new int[questionsCount]; 
     for (int i = 0; i < questionsCount; i++) 
     { 
      answers[i] = -1; 
     } 
    } 

    void setAnswer(int questionIndex, int answer) 
    { 
     answers[questionIndex] = answer; 
    } 

    int getAnswer(int questionIndex) const 
    { 
     return answers[questionIndex]; 
    } 
    ~MultipleChoiceTest() 
    { 
     delete answers; // release memory 
    } 
protected: 
    int questionsCount; 

private: 
    int* answers; 
}; 

class TimedMultipleChoiceTest : public MultipleChoiceTest 
{ 
public: 
    TimedMultipleChoiceTest(int questionsCount) 
     : MultipleChoiceTest(questionsCount) 
    { 
     times = new int[questionsCount]; 
     for (int i = 0; i < questionsCount; i++) 
     { 
      times[i] = 0; 
     } 
    } 

    void setTime(int questionIndex, int time) 
    { 
     times[questionIndex] = time; 
    } 

    int getTime(int questionIndex) const 
    { 
     return times[questionIndex]; 
    } 
    ~TimedMultipleChoiceTest() 
    { 
     delete times; // release memory 
    } 
private: 
    int* times; 
}; 

#ifndef RunTests 
void executeTest() 
{ 
    MultipleChoiceTest test(5); 
    for (int i = 0; i < 5; i++) 
    { 
     test.setAnswer(i, i); 
    } 

    for (int i = 0; i < 5; i++) 
    { 
     std::cout << "Question " << i + 1 << ", correct answer: " << test.getAnswer(i) << "\n"; 
    } 
} 

int main() 
{ 
    for (int i = 0; i < 3; i++) 
    { 
     std::cout << "Test: " << i + 1 << "\n"; 
     executeTest(); 
    } 
} 
#endif 
+2

不知道如果它解决了问题,但对于一个你应该清理由'删除[答案]回答'而不是'删除答案'(和'次'相同) – CompuChip

+0

作为一个附注,你应该在你的代码中进行边界检查。调用'getAnswer(-1);'此刻是未定义的行为。 – IlBeldus

回答

2

你应该使用delete []而不是delete解除分配动态数组。

而且,你似乎没有使用派生类,但尽管如此,在MultipleChoiceTest析构函数应该是虚拟

+0

我还补充说,最好是使用'std :: vector answers',以避免手动内存分配/释放。 –

+0

@IIBeldus如何获得MultipleChoiceTest作为虚拟? –

+0

这两个类中都有* no *虚函数,所以它们很可能不会多形地使用。在这种情况下,虚拟析构函数并不重要。 –