使用参数不明确的函数

Question

我想实现一个 walk(x,y) 函数，让精灵向上、向下、向左或向右移动正好一个单位。游戏板存储在一个数组中，其中每个图块都位于索引 y*width+x 处。所以只要知道当前位置、宽度以及 x 和 y，我就可以立即计算出新位置（并检查障碍物或其他东西）。这对我来说似乎是一个非常好的解决方案，但这会是糟糕的代码吗？或者在这种情况下可以吗？

（更易读的解决方案是使用 "direction" 作为参数，但这会使函数本身的定义膨胀，这也可能降低性能）

提前致谢！

Answer 1

如果你的walk(x, y)函数允许输入大于1的距离，这种情况也应该实现。否则你将不得不检查错误的输入。我肯定会说编写仅部分实现的函数是一种不好的做法。

如果您只想移动单个图块，有一个非常优雅的解决方案：

enum class Direction : unsigned {
    UP, RIGHT, DOWN, LEFT
};

constexpr int X[4] {
    0, 1, 0, -1
};
constexpr int Y[4] {
    1, 0, -1, 0
};

void move(Direction d) {
    int x = X[static_cast<size_t>(d)];
    int y = Y[static_cast<size_t>(d)];

    // move by x and y
}

在这种情况下，性能成本几乎为零。你真是多虑了。您还可以使用单个 table 而不是两个单独的

void move(Direction d) {
    int x = X[static_cast<size_t>(d)];
    int y = X[(static_cast<size_t>(d) + 1) & 0b11];

    // move by x and y
}

使用参数不明确的函数

Using functions with unclear parameters

c++

coding-style