奇怪的向量<long>加法'0'作为'-1'

Question

我正在编写一个基本的国际象棋程序来计算你可以用给定的国际象棋数字做多少组。数据文件：

4
22 3 5 6 2 0
1 1 1 1 1 1
8 4 4 4 1 2
5 3 3 3 0 2

代码：

#include <iostream>
#include <fstream>
#include <vector>

int main
(int argc, char *argv[])
{
    std::fstream data_file;
    size_t i, k;
    std::vector<long> chess;
    long t, n;

    data_file.open("U1.txt", std::ios::in);

    data_file >> n;

    for (i = 0; i < n; i++)
        chess.push_back(0);

    for (i = 0; i < n; i++) {
        for (k = 0; k < 6; k++) {
            data_file >> t;
            std::cout << t << " ";
            chess[k] += t;
        }
        std::cout << std::endl;
    }

    data_file.close();

    for (int i = 0; i < 6; i++)
        std::cout << chess[i] << " ";
    std::cout << std::endl;

    data_file.open("U1rez.txt", std::ios::out);
    data_file << n;
    std::cout << n << std::endl;
    data_file.close();

    return EXIT_SUCCESS;
}

输出：

22 3 5 6 2 0 
1 1 1 1 1 1 
8 4 4 4 1 2 
5 3 3 3 0 2 
36 11 13 14 3 4 
4

为什么我在第 5 行的 36、11、13 和 14 之后的最终结果中得到 3 和 4？当我打印测试值时，我似乎得到了正确的数字，但是在向量容器中添加它们时出现了严重错误。

Answer 1

顺便说一句，如果你放弃 C 强加的一些规则，你会更轻松地使用 C++。

• C++ API 使用范围有界的资源管理；也就是说，这里不需要显式关闭文件句柄，因为当在声明范围的末尾调用其析构函数时，该类会为您执行此操作。
• 首选std::cout << "\n" 而不是std::cout << std::endl，除非您确实打算插入换行符并刷新缓冲区。许多 C++ 程序员仍然使用printf，我个人认为它比 C++ 丑陋的 IO 标准解决方案要优雅得多——请随意使用printf。
• 当您在使用变量的位置（例如for (size_type i = 0; ... ; ...)）而不是在方法的开头声明变量时，您是否发现代码更易于管理和阅读？这也允许编译器对寄存器的使用做出更好的选择，因为它有更多关于临时变量范围的信息。

以下是 C++11 的一些功能，它们也可以为您节省一些时间：

• 自动类型推断：当编译器可以推断变量的类型时，您不必显式指定它；您可以改用auto（例如auto x = parse_data();）。
• Range-for：如果您在提供全局定义 begin() 和 end() 的容器上进行操作（例如任何标准容器），则不要这样：

typedef typename container_type::const_iterator iter;
for (iter i = begin(container), iter l = end(container); i != l; ++i) { ... }

你可以这样做：

for (const auto& x : container) { ... }

我只是想给你一些快速的指导，可以节省你一些时间。

Answer 2

for (i = 0; i < n; i++)
    chess.push_back(0);

。 . .

    for (i = 0; i < n; i++) {
    for (k = 0; k < 6; k++) {
        data_file >> t;
        std::cout << t << " ";
        chess[k] += t;
    }
    std::cout << std::endl;
}

在这里，您初始化了向量中的 n(=4) 个位置，但在这里您正在访问导致加法问题的向量国际象棋的索引 4 和 5。