为什么我的C# class代码是死循环？ 我能做些什么来修复它？

Question

namespace Random_walk_2d
{
    internal class Program
    {
        static void Main(string[] args)
        {
            Point C = new Point(2, 5);

            C.Move('n');
            Point.Print(C);
        }
    }

    public class Point
    {
        public int x;
        public int y;

        readonly public Dictionary<char, Point> directions = new Dictionary<char, Point>()
        {
            { 'e', new Point(1, 0) },
            { 'n', new Point(0, 1) },
            { 'w', new Point(-1, 0) },
            { 's', new Point(0, -1) },
        };

        public Point(int x, int y)
        {
            this.x = x;
            this.y = y;
        }

        public static Point operator +(Point A, Point B)
        {
            return new Point(A.x + B.x, A.y + B.y);
        }

        public static void Print(Point A)
        {
            Console.WriteLine($"({A.x}, {A.y})");
        }

        public void Move(char dir)
        {
            Point displacement = directions[dir];
            this.x += displacement.x;
            this.y += displacement.y;
        }

    }
}

我可以看到问题出在字典或 Move() 方法中的某个地方，但我不明白的是幕后到底发生了什么？ 是什么导致无限循环发生？

我该如何解决这个问题？

我的朋友建议我使用 arrays 的 2 个整数而不是字典中的点，但是......这有点违背整个 class 的目的。如果我只能使用 2 个整数 arrays 为什么要使用点？ 我想要一个不这样做的解决方案。

如果您有任何其他类型的建议，请告诉我。

编辑：我应该马上提到这一点，但是当我运行它时，我在控制台中收到无限量的at Random_walk_2d.Point..ctor(Int32, Int32)消息。

EDIT2：我只是在尝试上课。 对于这个项目，我想在二维网格上创建一个随机游走。 2d 网格可以很好地由 class 和 2d 点对象建模。 我需要Move()方法向右（东 = 'e'）、向上（北 = 'n'）等移动 1 个单位。

Answer 1

问题是堆栈溢出，这在这里完成

public class Point {
    public int x;
    public int y;

    readonly public Dictionary<char, Point> directions = new Dictionary<char, Point>()
    {
        { 'e', new Point(1, 0) },
        { 'n', new Point(0, 1) },
        { 'w', new Point(-1, 0) },
        { 's', new Point(0, -1) },
    };

当您创建一个点时，它会创建一个“方向”成员，这又会创建 4 个新点，每个新点都会创建自己的“方向”，每个点都会创建 4 个点.....

目前尚不清楚您要做什么，但这已解决

public class Point {
    public int x;
    public int y;

    static readonly public Dictionary<char, Point> directions = new Dictionary<char, Point>()
    {
        { 'e', new Point(1, 0) },
        { 'n', new Point(0, 1) },
        { 'w', new Point(-1, 0) },
        { 's', new Point(0, -1) },
    };

通过声明 'directions' static整个点只有一个 class，也许这就是你想要的

编辑 - 是的，这就是你想要的'方向' map 实际上是一个常数，所以 static 工作正常

Answer 2

您正在递归地、无限期地创建Point类型的新对象，因为在创建Point时，您正在创建一个包含 4 个点的directions ，每个点都有一个包含 4 个以上点的字典，等等。

我不确定你到底想达到什么目的，因为这个问题并不清楚。 所以我帮不上忙了。

Answer 3

我建议使用 static 变量来保存您的特定方向，而不是字典。 我还建议，不要使用类，而是使用struct ，以确保数据不会因我的意外而改变。 struct具有值语义，这意味着(x,y) go 对始终在一起。

不是字典，而是 function 和返回正确步骤的 switch 语句。 然后我做了一个技巧并覆盖Move()中的this关键字以更改一个 go 中的(x,y)的值。如果您已经定义了operator + ，不妨使用它。

static class Program
{
    static void Main(string[] args)
    {
        Point C = new Point(2, 5);
        C.Move('n');
        Console.WriteLine(C);
    }
}

public struct Point
{
    public readonly int x;
    public readonly int y;

    public Point(int x, int y)
    {
        this.x = x;
        this.y = y;
    }

    public static readonly Point North = new Point(0, 1);
    public static readonly Point South = new Point(0, -1);
    public static readonly Point East = new Point(1, 0);
    public static readonly Point West = new Point(-1, 0);

    public Point GetDirection(char dir)
    {
        switch (dir)
        {
            case 'n': return North;
            case 'e': return East;
            case 's': return South;
            case 'w': return West;
            default:
                throw new ArgumentException("Invalid Direction.", nameof(dir));
        }
    }

    public static Point operator +(Point a, Point b) => new Point(a.x + b.x, a.y + b.y);

    public void Move(char dir)
    {
        Point delta = GetDirection(dir);
        this += delta;
    }
    public override string ToString()
    {
        return $"( {x}, {y} )";
    }
}