0

这是我的 operator>> 重载代码。它应该将数字增加到一个分号并将它们放入一个 bigint。

std::istream& operator>>(std::istream& is, bigint& bi) {

    int i = 0;
    char ch;
    char temp[SIZE];

    // grabs the first character in the file
    is >> ch;
    temp[i] = ch;
    ++i;

    // while loop grabs the rest of the characters
    // up to the semicolon
    while(ch != ';') {
        is >> ch;
        temp[i] = ch;
        ++i;
    }

    // temp is stored in the bigint ref
    bi = bigint(temp);

    return is;
}

我遇到的问题是，当我运行它时，它会给我额外的输出。例如：当我输入“34;”时 作为输入，生成的 bigint 将为“3411”。谁能告诉我我做错了什么？

4

2 回答 2

0

您保证temp最后有一个分号。分号可能会搞乱bigint对该字符串的任何解析。在将分号插入之前更改循环以测试分号temp：

std::istream& operator>>(std::istream& is, bigint& bi)
{
    char temp[SIZE] = {}; // zero out the array so the end is null terminated
    char c;

    for(int i = 0; i < SIZE-1 && is >> c && c != ';'; ++i)
    {
        temp[i] = c;
    }

    bi = bigint(temp);
    return is;
}
于 2013-10-02T00:41:58.923 回答
0

你不是空终止你的 string temp。添加这个：

temp[i - 1] = '\0';
bi = bigint(temp);

请注意，这-1将删除您可能也不需要的分号。如果您出于某种原因想保留分号，请将其更改为temp[i].

您还应该在 while 循环中添加一个检查，以确保您不会溢出缓冲区大小。

于 2013-10-02T00:45:04.487 回答