C++ cout 给出意外的解引用指针

Question

所以，我有一个函数返回一个指向动态分配的双精度的指针，并且我需要打印它指向的值（我知道很愚蠢）。

double * ans1p = dot(v1, v2);
    double ans1 = *ans1p;
    cout << "Answer to problem 1:" << endl;
    cout << ans1 << endl;
    cout << *ans1p << endl;

第一个 cout 语句返回预期值。

第二个返回一些随机和意外的东西。有什么区别？似乎在这两种情况下打印的都是 ans1p 指向的值。

我宁愿完全消除 ans1 及其 cout 语句。

好的，这是点函数：

double * dot(Vector &v1, Vector &v2) {
if (getLength(v1) != getLength(v2) ) {
    std::cout << "Error: cannot perform dot product. Vectors must be " <<
        "equal length." << std::endl;
    return 0;
}
double result = 0;
for (unsigned int i = 1; i <= getLength(v1); ++i) {
    result += (v1.elements[i] * v2.elements[i]);
}
double * resultPtr = new double;
resultPtr = &result;
return resultPtr;
}

郑重声明，我不会尝试这样做，但它是作业的一部分。如果大小不匹配，我应该返回 0 指针。

好的，看看它，我可以看到当函数返回时结果超出了范围，而我返回的指针只是指向那里（对吗？）。但后来，我想不出如何做我想做的事。

我想我可以在函数外部声明一个双指针，将指针作为参数传递，然后将其设置在函数内部。但我有点想弄清楚如何按照我计划的方式去做。不过，想一想，当我了解了旧方式的情况后，这样做可能会更好，对吗？

Answer 1

你的问题就在那里：

double * resultPtr = new double;
resultPtr = &result;
return resultPtr;

你在堆上分配一个新的double...然后将局部变量的地址分配给resultPtr。您会遇到内存泄漏（使用 new double 分配的内容丢失），然后返回所述局部变量的地址。这是未定义的行为。这意味着任何事情都可能发生。

要解决此问题，您必须更改分配以实际使用您刚刚分配的内存：

double * resultPtr = new double;
*resultPtr = result; // assign the value to the value pointed to resultPtr
return resultPtr;

或者，更好的是：

return new double(result);

---

考虑OP必须使用指针，然后这是无关紧要的。留下作为奖励（因为无论如何我都写了这个，删除它会是一种浪费）。

或者你也可以放弃整个内存分配并返回结果。 double 非常轻，我真的不明白为什么你绝对必须返回一个指针而不是值本身：

double dot(Vector &v1, Vector &v2) {
    // check here and throw an exception if needed instead of returning 0

    double result = 0.0;
    for (unsigned int i = 1; i <= getLength(v1); ++i) {
        result += (v1.elements[i] * v2.elements[i]);
    }
    return result; // return the value
}