Question

我需要一个项目的Vector2结构，我对数学不是很强，这个类似乎工作但是我非常感谢有关如何优化代码或者我是否犯了任何错误的任何指针。还有比Math.Sqrt更便宜的距离计算方法吗？

    private struct Vector2
    {
        public float X;
        public float Y;

        public Vector2(float x, float y)
        {
            this.X = x;
            this.Y = y;
        }

        public static Vector2 operator +(Vector2 v1, Vector2 v2)
        {
            return new Vector2(v1.X + v2.X, v1.Y + v2.Y);
        }

        public static Vector2 operator -(Vector2 v1, Vector2 v2)
        {
            return new Vector2(v1.X - v2.X, v1.Y - v2.Y);
        }

        public static Vector2 operator *(Vector2 v1, float m)
        {
            return new Vector2(v1.X * m, v1.Y * m);
        }

        public static float operator *(Vector2 v1, Vector2 v2)
        {
            return v1.X * v2.X + v1.Y * v2.Y;
        }

        public static Vector2 operator /(Vector2 v1, float m)
        {
            return new Vector2(v1.X / m, v1.Y / m);
        }

        public static float Distance(Vector2 v1, Vector2 v2)
        {
            return (float)Math.Sqrt(Math.Pow(v1.X - v2.X, 2) + Math.Pow(v1.Y - v2.Y, 2));
        }

        public float Length()
        {
            return (float)Math.Sqrt(X * X + Y * Y);
        }
    }

Answer 1

只有3条评论：

1）在C＃中，我们通常会公开属性而不是字段。也就是说，

    public float X {get; private set;}
    public float Y {get; private set;}

2）double类型比float更精确，所以我建议你使用double而不是float。

3）您将一个运算符重载为：

    public static float operator *(Vector2 v1, Vector2 v2)
    {
        return v1.X * v2.X + v1.Y * v2.Y;
    }

我认为它是两个2d实数向量的点（内积）。在编程中，当我们重载操作符时，重载应该执行操作符所做的操作。也就是说，如果我们要重载运算符*（Vector2 v1，Vector2 v2），则重载应该将这两个向量相乘。但是，对于两个向量，没有明确的“乘法”定义。在线性代数中使用术语“乘法”是非常模糊的。您可能指的是张量产品，外部产品，内部产品，交叉产品，标量产品等。因此，为了减少您的代码混乱，我建议您实施内部产品运营商

public static float InnerProduct(Vector2 v1, Vector v2)

代替。

对于平方根函数的更好实现，Math.Sqrt可能是最快的一个（在.NET框架中）。如果您正在考虑提高类的速度，那么在本机C / C ++中编写它，然后为它编写托管包装器可能会给您带来最大的改进。

C＃Vector2代码

1 个答案: