1

我有一个指向对象的指针向量。每个对象都存储一个值并具有一个 toString() 函数，该函数将该值作为字符串返回。我有一个迭代器来遍历向量，我需要通过调用 toString() 来提取每个对象的值。问题是，我不知道如何获得价值。

该函数最终应该将数字写入文件，但我正在使用 cout 进行测试。

    void writeNumbers(vector<Integer*>& input)
    {
        ofstream write;
        write.open("Integers.txt");
        vector<Integer*>::iterator iter = input.begin();
        for (iter; iter < input.end(); iter++)
        {
            **std::cout << (*iter)->toString() << std::endl;**
        }
        write.close();

I get an Access Violation error which points me to the toString() function:

    std::string Integer::toString()
    {
        std::stringstream ss;
        ss << *(this)->value;
        return ss.str();
    }

toString() works fine whenever I don't try to access it through the iterator.

编辑： toString 中的值实际上是一个指向数字的指针。

Edit2：新的 writeNumbers：

void writeNumbers(vector<Integer*>& input)
{
    ofstream write;
    write.open("Integers.txt");
    vector<Integer*>::iterator iter = input.begin();
    for (iter; iter != input.end(); iter++)
    {
        std::cout << (*iter)->toString() << std::endl;
    }
    write.close();
}

最终编辑：好吧，问题原来是一个糟糕的构造函数，它未能正确初始化指针，所以我离问题的实际位置很远。:)

Integer::Integer(string input)
{
if(isNaN(input))
value = new int(atoi(input.c_str()));
}

所以它应该是！isNaN，另外我修复了在错误输入时初始化它的问题：

//New constructor, works 100%
Integer::Integer(string input)
{
if(!isNaN(input))
    value = new int(atoi(input.c_str()));
else
    value = new int(0);
}
4

2 回答 2

2

您的 toSting() 有问题。改变

ss <<*(this)->value;

至

ss << value;
于 2012-11-13T12:18:30.860 回答
0

编辑：这不是错误，而是使用迭代器时的一般建议。不要用于<检查结束，使用!=.

iter < input.end()

它应该是这样的：

iter != input.end()

这是因为对于某些容器， < 运算符不会按照您的预期执行。结果，在某些时候，您可能会取消引用 input.end() 本身，它什么都没有。

于 2012-11-13T12:10:40.447 回答