C ++错误(C2280)尝试访问已删除的功能

时间:2019-03-21 19:04:08

标签: c++ sfml deleted-functions

所以,我试图用opengl和sfml制作2D游戏, 所以我在输入名称空间中创建了一个按钮类, 我在其中创建了render()函数,但是当我调用它时(无论我使用指针还是不使用指针),即使我传递了所有必需的参数,它仍然给出 给我一个错误,说我正在尝试访问已删除的功能 这是Button标头:

#pragma once

#include <SFML/Graphics.hpp>
#include "InputManager.h"

namespace input {

class Button {
private:
    bool m_Selected, m_Clicked;
    sf::Vector2f m_Position;
    sf::Sprite m_Sprite;
    sf::Texture m_Texture;
public:
    Button(sf::Vector2f position, sf::Sprite sprite);
    Button(sf::Vector2f position, sf::Texture texture);
    Button(sf::Vector2f position);
    Button();
    ~Button();
    void update(engine::InputManager inputManager);
    void render(sf::RenderWindow window);
    inline bool isClicked() { return m_Clicked; }
    inline bool isSelected() { return m_Selected; }
    inline sf::Vector2f getPosition() { return m_Position; }
    void setPosition(sf::Vector2f position) { m_Position = position; }
};

}

这是Button.cpp:

#include "Button.h"

namespace input {

Button::Button(sf::Vector2f position, sf::Sprite texture) : m_Position(position), m_Sprite(texture) {
    m_Sprite.setPosition(m_Position);
}

Button::Button(sf::Vector2f position, sf::Texture texture) : m_Position(position), m_Texture(texture) {

    m_Sprite.setTexture(m_Texture);
    m_Sprite.setPosition(m_Position);
}

Button::Button(sf::Vector2f position) : m_Position(position) {
    m_Sprite.setPosition(m_Position);
}

void Button::update(engine::InputManager inputManager) {}

void Button::render(sf::RenderWindow window) {
    window.draw(m_Sprite);
}

Button::~Button() {
    delete &m_Position;
    delete &m_Texture;
    delete &m_Clicked;
    delete &m_Selected;
}
}

这是main.cpp代码:

#include <iostream>
#include <vector>
#include "InputManager.h"
#include "Button.h"

#define WIDTH 800
#define HEIGHT 600
#define TITLE "C++ Platformer"

int main() {

sf::RenderWindow window(sf::VideoMode(WIDTH, HEIGHT), TITLE, sf::Style::Close | sf::Style::Titlebar);

sf::Texture ButtonTexture;
ButtonTexture.loadFromFile("Texture.png");
sf::Sprite sprite;
sprite.setTexture(ButtonTexture);

input::Button* button = new input::Button(sf::Vector2f(100.f, 100.f), sprite);

int fps = 0;

int ticks = 0;

sf::Clock clock;
sf::Clock FpsClock;
sf::Time time;
sf::Time Fpstime;

long delta = 0;
long target = 1000000 / 60;

engine::InputManager IManager;

sf::Event ev;
while (window.isOpen()) {
    while (window.pollEvent(ev)) {
        if (ev.type == sf::Event::Closed) window.close();
        if (ev.key.code == sf::Keyboard::Escape) window.close();
        if (ev.type == sf::Event::KeyPressed) IManager.onKeyDown(ev.key.code);
    }
    time = clock.getElapsedTime();
    if (target > time.asMicroseconds()) {
        clock.restart();
        // UPDATE
        button->update(IManager);

        delta = time.asMicroseconds() % 60;
        target = 1000000 / 60 + delta;
        ticks++;
    } Fpstime = FpsClock.getElapsedTime();
    if (Fpstime.asSeconds() >= 1) { 
        std::cout << "FPS: " << fps << " Ticks: " << ticks << std::endl; 
        FpsClock.restart();
        fps = 0;
    }
    fps++;
    // RENDER
    button->render(window);
}

return EXIT_SUCCESS;
}

我已经搜索了Microsoft文档和其他stackoverflow问题,但找不到像我这样的情况,希望有人可以提供帮助,谢谢

2 个答案:

答案 0 :(得分:3)

首先。什么...

Button::~Button() {
    delete &m_Position;
    delete &m_Texture;
    delete &m_Clicked;
    delete &m_Selected;
}

您不应该那样做。

但是回到您的错误。它具有描述性。您正在尝试使用最有可能被明确删除的功能。像这样:

struct Test {
    Test(const Test & other) = delete; 
};

很明显,由于删除了复制构造函数,我现在无法复制Test

sf::RenderWindow通过继承sf::NonCopyable类来做什么。

并且由于您的render()方法采用复制而不是引用的方式。显然,它试图做被禁止的事情。

您应该对此进行更改:

void Button::render(sf::RenderWindow window)

对此:

void Button::render(sf::RenderWindow & window)

答案 1 :(得分:3)

问题是sf::RenderWindow是不可复制的。这是一件好事,因为这意味着您不会意外对拥有多个窗口感到困惑,并且在创建新窗口以及该窗口的拥有位置时很明显。但是,在类似

的函数签名中
void Button::render(sf::RenderWindow window);

您接受的是sf::RenderWindow 按值。这意味着只要您调用Button::render并传递一个窗口,该窗口就会被复制,然后由函数接收。看到问题了吗?

您需要通过引用接受渲染窗口,以确保您不尝试创建副本:

void Button::render(sf::RenderWindow& window);

此外:正如NathanOliver指出的那样,即使您不是delete析构函数中的所有实例成员,它们也不是指针,并且您并没有使用{ {1}}。 Buttonnewdelete / new则完全避免。

如果您不确定通过引用传递的含义,或者newdelete的含义,我建议您拿起good C++ book的副本。 / p>