0

好的,这次我有一个并不总是出现的非常奇怪的错误。这是实际包含问题的函数。它所做的只是对向量的元素求和。它在大多数情况下都有效,但在少数情况下,它往往会变得非常有问题。

int sumvec(vect v) {
    int i = 0;
    int sum = 0;
    int l = v.length();
    std::cout << "Sum of vector " << v.tostring(3) << std::endl;
    for (; i < l; i++) {
        sum += v[i];
        std::cout << v[i] << " " << sum << " ";
    };
    std::cout << std::endl;
    return sum;
}

这里vect是使用定义的typedef alglib::real_1d_array vect;。好的,那我会得到什么?呵呵..

Sum of vector [1.000,1.000,0.000,1.000,1.000,1.000] 1 0 1 0 0 0 1 0 1 0 1 1

什么?!!!!!

4

2 回答 2

1

正如所评论的,如果您使用浮点整数,请使用双精度数来存储总和。使用整数将导致变量被隐式转换为一个 int ,它只是削减尾数:

0.9999998 -> 0

根据 cout::precision,0.99999 将打印为 1.0000(四舍五入)或没有 std::fixed 就像 1 在您的示例中可能发生的情况。

double a = 0.999;
std::cout.precision(2);
std::cout << a << std::endl; /* This prints 1 */
std::cout << std::fixed;
std::cout << a << endl; /* This prints 1.00 */
于 2018-10-16T15:48:29.177 回答
1

由于您的 sum 变量是一个整数,因此在对向量中非整数的元素求和时,您可能无法获得预期的结果。

如果您的元素具有的值0.999999999而不是1.00000打印它们可以四舍五入,1.00000但是当您将它们添加到整数时,该值将被截断为 0。

1从提供的输出判断,除了最后一个大于或等于 1 之外,您的所有值都小于。

有两种可能的解决方案:

  1. 将 的类型更改sumfloatdouble
  2. 将您的计算更改为:sum += static_cast<int>(round( v[i] ));

请注意,您的编译器可能会警告将双精度数截断为整数。注意编译器警告,它们通常表明存在错误。

于 2018-10-16T15:58:09.223 回答