C++ 菜鸟：计算两点之间的距离

Question

我正在尝试编写一些简单的 C++ 代码到 d 维度中点之间的 output 距离。 我定义了一个点结构，它有一个 function 用于计算到任何其他点的距离。

在两点上测试它，它不起作用。 我认为这可能是 sqrt function 不起作用，但即使是 function “测试”输出双精度也不起作用。

我正在使用 VS Code，而 output 是......什么都没有。

我确定我错过了一些简单的东西......

#include <iostream>
#include <cmath>

using namespace std;

struct point
{
        
        static int d;
        double *coords;

        point(){ //point class
            coords=new double[d];
            for (int i=0; i<d; i++){
                coords[i]=0;
            }
        }
        double dist(point &q){
                double squared=0;
                for (int i=0; i<d;i++){
                        squared+=(coords[i]-q.coords[i])*(coords[i]-q.coords[i]);
                }
                return sqrt(squared);
        }

        double test(){
                return 1.4;
        }
};

int point::d;

int main() {
        
    point p;
    int d=2;
    p.d=d;
    p.coords[0]=1;
    p.coords[1]=1;
    point q;
    q.d=d;
    q.coords[0]=0;
    q.coords[1]=2;
    
    std::cout << "\ndistance:" << p.dist(q)<<"\n\n";
    std::cout << "even this doesn't work!:" << p.test()<<"\n\n";

    return 0;
}

Answer 1

几个问题，有些是至关重要的，有些是常规的。

1. 您最好将方法作为const和const传递，以便您可以从 const object 中使用这些方法。
2. 为了改变d ，你应该写point::d = 2而不是int d = 2
3. d是维度，不能为负数，因此为std::size_t或unsigned int
4. 如果你在构造函数上分配，你应该在析构函数上释放。 如果您有析构函数，则需要复制构造函数和复制赋值运算符。 如果您不知道这些是什么，请不要在构造函数中分配:-)
5. 请注意，点构造函数使用的是point::d ，因此您应该在创建 p 的任何实例之前设置point::d 。

我附上了您的代码并应用了我的修复程序

struct point
{
    static std::size_t d;
    double *coords;

    point() { //point class
        coords = new double[d];
        for (int i = 0; i < d; i++) {
            coords[i] = 0;
        }
    }

    double dist(const point &q) const {
        double squared = 0;
        for (int i = 0; i < d; i++) {
            squared += (coords[i] - q.coords[i])*(coords[i] - q.coords[i]);
        }
        return sqrt(squared);
    }

    double test() const {
        return 1.4;
    }

    point(const point& p) {
        coords = new double[d];
        for (std::size_t i = 0; i < d; ++i) {
            coords[i] = p.coords[i];
        }
    }

    point& operator=(const point& p) {
        if (this == &p) {
            return *this;
        }

        // coords are already allocated
        for (std::size_t i = 0; i < d; ++i) {
            coords[i] = p.coords[i];
        }
    }

    ~point() {
        delete[] coords;
    }
};

std::size_t point::d;


int main() {

    point::d = 2;
    point p;
    p.coords[0] = 1;
    p.coords[1] = 1;

    point q;
    q.coords[0] = 0;
    q.coords[1] = 2;

    std::cout << "\ndistance:" << p.dist(q) << "\n\n";
    std::cout << "even this doesn't work!:" << p.test() << "\n\n";

    return 0;
}

Answer 2

您的代码什么也不做，因为在您将任何值分配给d之前将调用point的构造函数。 因此，意外地， d的值似乎为 0（静态变量默认为零初始化）。

这是修复此类代码的一种可能性：

#include <iostream>
#include <cmath>

using namespace std;

struct point
{
    constexpr static int d = 2;
    double coords[d];

    point(){ //point class
        for (int i=0; i<d; i++){
            coords[i]=0;
        }
    }
    double dist(point &q){
        double squared=0;
        for (int i=0; i<d;i++){
            squared+=(coords[i]-q.coords[i])*(coords[i]-q.coords[i]);
        }
        return sqrt(squared);
    }
};


int main(){

    point p;
    p.coords[0]=1;
    p.coords[1]=1;

    point q;
    q.coords[0]=0;
    q.coords[1]=2;

    std::cout << "\ndistance:" << p.dist(q)<<"\n\n";

    return 0;
}

实时代码

Answer 3

问题是您在构造函数运行后设置成员变量 d ，因此您的双精度数组始终是 0 数组。您应该在构造函数参数中提供 d 。