0

我在 C++ 类中有以下内容：在通过调用 isValid() 检查迭代器是否仍然有效后，我想使用迭代器通过调用 getNextQuestion() 函数来一次获取问题列表的一个元素。它给了我以下可怕的错误：

passing ‘const iterator {aka const std::_List_iterator<domain::Question>}’ as ‘this’ argument of ‘std::_List_iterator<_Tp>::_Self& 
std::_List_iterator<_Tp>::operator++() [with _Tp = domain::Question,std::_List_iterator<_Tp>::_Self = 
std::_List_iterator<domain::Question>]’ discards qualifiers [-fpermissive]

#ifndef TESTREPOSITORY_H_
#define TESTREPOSITORY_H_

#include <iostream>
#include <iterator>
#include <list>
#include <algorithm>
#include <fstream>
#include "../domain/question.h"

using namespace domain;

namespace repository{
template<class T>
class TestRepository{
std::string file;
std::list<T> questions;
typename std::list<T>::iterator it;
public:
TestRepository(std::string& file=""):file(file){
    this->questions = this->getQ();
    this->it = this->questions.begin();
};

std::list<T> getQ() const{
    std::list<T> listq;
    using namespace std;
    string line;
    std::ifstream fin(file.c_str());
    while(fin.good()){
        Question q;
        fin >> q;
        listq.push_back(q);
    }
    fin.close();
    return listq;
}

const bool isValid() const{
    return this->it != this->questions.end();
}

const T getNextQuestion() const{
    T q = (*this->it);
    ++this->it;
    return q;
}

};
}

#endif /* TESTREPOSITORY_H_ */

这是我调用这些函数的代码，也许这就是问题所在：

#include "TestController.h"
#include "../domain/test.h"
#include <iostream>
#include <list>
#include <iterator>

 namespace controller{

TestController::TestController(repository::TestRepository<domain::Question>* repo,int   testId){
this->repo = repo;
this->testId = 0;
}

const test TestController::getCurrentTest() const{
test test(this->testId,0,0);
return test;
}

const bool TestController::isValid() const{
return this->repo->isValid();
 }

const Question TestController::getNextQuestion() const{
return this->repo->getNextQuestion();
}



}
4

2 回答 2

1

这里：

const T getNextQuestion() const{
    T q = (*this->it);
    ++this->it;
    return q;
}

您正在更改字段it并且您不应该这样做，因为方法是const. 如果您只想const在“存储库”未被修改但其内部迭代器无关紧要的情况下使用，则可以使用mutable关键字：

mutable typename std::list<T>::iterator it;
于 2014-06-09T12:34:19.827 回答
0

您正在尝试修改const成员函数中的成员，这是被禁止的（这就是 const 成员函数的意义）：

const T getNextQuestion() const{
    T q = (*this->it);
    ++this->it;   // << Here
    return q;
}

这个方法应该是非常量的，或者考虑让你的成员迭代器是mutable：

mutable typename std::list<T>::iterator it;
于 2014-06-09T12:32:58.043 回答