继承SFML中的Transformable和Drawable

我试图继承 SFML 中的 Transformable 和 Drawable 以使我的对象......嗯，可变形和可绘制。我正在制作一个简单的突破游戏，但也许我的做法是错误的。这是我的代码：

#include <SFML/Graphics.hpp>
#include <SFML/System.hpp>

class Player : public sf::Transformable, public sf::Drawable {
    public:
        Player(int x, int y);
        ~Player() {};

        sf::RectangleShape p_rect;

        void doMovement(const sf::RenderWindow& window);
        sf::FloatRect getGlobalBounds() const;
    private:
        virtual void draw(sf::RenderTarget& target, sf::RenderStates states) const {
            states.transform *= getTransform();
            target.draw(p_rect, states);
        }

};

class Ball : public sf::Transformable, public sf::Drawable {
    public:
        Ball(int r, int x, int y);
        ~Ball() {};

        sf::CircleShape b_circle;

        void doXMovement();
        void doYMovement();
        bool doXCollisions(const Player& player);
        bool doYCollisions(const Player& player);
        sf::FloatRect getGlobalBounds() const;
    private:
        virtual void draw(sf::RenderTarget& target, sf::RenderStates states) const {
            states.transform *= getTransform();
            target.draw(b_circle, states);
        }

        bool right;
        bool up;
};

Player::Player(int x, int y) {
    p_rect = sf::RectangleShape(sf::Vector2f(x, y));
}

void Player::doMovement(const sf::RenderWindow& window) {
    setPosition(sf::Mouse::getPosition(window).x, 500);
    if (getPosition().x < 0)
        setPosition(0, 500);
    else if (getPosition().x > 720)
        setPosition(720, 500);
}

sf::FloatRect Player::getGlobalBounds() const {
    return getTransform().transformRect(p_rect.getGlobalBounds());
}

Ball::Ball(int r, int x, int y) {
    b_circle = sf::CircleShape(r);
    b_circle.setPosition(x, y);
    right = true;
    up = false;
}

void Ball::doXMovement() {
    if (right)
        move(1, 0);
    else
        move(-1, 0);
}

void Ball::doYMovement() {
    if (up)
        move(0, -1);
    else
        move(0, 1);
}

bool Ball::doXCollisions(const Player& player) {
    bool coll;
    if (getGlobalBounds().intersects(player.getGlobalBounds())) {
        right = !right;
        coll = true;
    } else
        coll = false;

    if (getPosition().x >= 800 - b_circle.getRadius())
        right = false;
    else if (getPosition().x <= 0)
        right = true;
    return coll;
}

bool Ball::doYCollisions(const Player& player) {
    bool coll;
    if (getGlobalBounds().intersects(player.getGlobalBounds())) {
        up = !up;
        coll = true;
    } else
        coll = false;
    if (getPosition().x <= 0)
        up = false;
    return coll;
}

sf::FloatRect Ball::getGlobalBounds() const {
    return getTransform().transformRect(b_circle.getGlobalBounds());
}

int main() {
    sf::RenderWindow window(sf::VideoMode(800, 600), "Breakout");
    window.setMouseCursorVisible(false);
    Player player(80, 10);
    Ball ball(3, 100, 100);
    sf::Clock clock;
    while (window.isOpen()) {
        sf::Event event;
        while (window.pollEvent(event)) {
            if (event.type == sf::Event::Closed)
                window.close();
        }
        player.doMovement(window);
        if (clock.getElapsedTime().asMilliseconds() >= 3) {
            clock.restart();
            if (!ball.doYCollisions(player))
                ball.doXCollisions(player);
            ball.doYMovement();
            ball.doXMovement();
        }
        window.clear(sf::Color::Black);
        window.draw(player);
        window.draw(ball);
        window.display();
    }
    return 0;
}

现在移动和绘图工作（几乎）按预期进行，但是碰撞有点不稳定。首先是我的碰撞问题：

1. 我需要按照我的方式实现 getGlobalBounds 函数吗？或者是否有更好的方法来处理 Transformable 和 Drawable 中包含的内容？
2. 我应该直接对形状执行转换，还是应该像现在一样将转换传递给绘图函数？

绘图中也发生了一些奇怪的事情，这可能是一个快速修复。现在， getPosition 方法为我的球对象返回不正确的值。它返回的区域似乎向下并向右移动了一点。有什么可能的原因吗？

感谢您能够提供的任何帮助！

编辑：也欢迎任何一般的 C++ 技巧，我仍然是初学者。

如果我是你，我会定义一个新类，称为TransformableAndDrawable像这样：

class TransformableAndDrawable : public sf::Transformable, public sf::Drawable {
    // Your code here
}

在此类中，您应该定义可转换类和可绘制类通常需要的所有成员。此外，在此类中，您应该定义可在可转换类和可绘制类中普遍实现的所有方法。然后，你的类应该继承自TransformableAndDrawable， 像这样：

class Player : TransformableAndDrawable {
    // Your code here
}

现在，第一个问题的答案是：我将在TransformableAndDrawable如果给定方法是通用方法，则对给定方法进行类化，因此所有继承自的类TransformableAndDrawable会有这个方法。

而不是给出不同的名称，例如p_rect and p_circle，用相同的名字命名这些成员，例如p_shape，所以你不会有命名问题。另外，我相信您可以将您的 p_shape 声明为祖先类或接口（我不知道您正在使用的库中定义了哪些类），并且仅在需要时指定形状的性质（无论是圆形或矩形或其他）。

至于第二个问题：我喜欢你实现事情的方式，但你犯了两个错误：

• 它不可扩展：我们想要一个通用的解决方案，一个可以用于您现在和将来使用的任何形状的类，不是吗？
• 它不够通用：当我想知道形状的全局边界时，我对形状的性质不感兴趣，我希望你的代码在我不知道的情况下处理形状的性质

简而言之，您应该执行以下操作：

1. 创建一个包装类，该类将继承自Transformable and Drawable

2. 在你的包装类中，对形状的性质不可知，尽可能通用，希望有一些类或接口是两者的祖先RectangleShape and CircleShape.

3. 从包装类继承所有可绘制和可变换类，因此您将在类之间拥有共享功能

4. 如果包装类中的某些内容不适用于从其继承的类，请覆盖该类中的方法。

EDIT:

我更详细地研究了您正在使用的库，发现有一个名为 Shape 的类，它是两者的祖先CircleShape and RectangleShape。因此，不要使用这些类Shape你的代码将更加通用和可重用。