Probleme mit Pointern

  • C++

Es gibt 5 Antworten in diesem Thema. Der letzte Beitrag () ist von Gonger96.

    Probleme mit Pointern

    Servus, hab grad das Problem, dass mir en Fehler bei dem Code geschmissen wird den ich nicht nachvollziehen kann.

    C-Quellcode

    1. #ifndef PLAYER_INPUT_COMPONENT_H_
    2. #define PLAYER_INPUT_COMPONENT_H_
    4. #include <SFML/Graphics.hpp>
    5. #include <memory>
    6. #include "InputComponent.h"
    7. #include "../Commands/PlayerJumpCommand.h"
    9. namespace t1
    10. {
    11. //forward declaration
    12. class Player;
    14. using Key = sf::Keyboard::Key;
    16. //Component class for handling Player input
    17. class PlayerInputComponent : public InputComponent<Player>
    18. {
    19. public:
    20. PlayerInputComponent() {};
    21. std::unique_ptr<Command<Player>> handleInput() override;
    23. private:
    24. };
    27. std::unique_ptr<Command<Player>> PlayerInputComponent::handleInput()
    28. {
    30. if(sf::Keyboard::isKeyPressed(Key::W))
    31. return std::unique_ptr<PlayerJumpCommand> (new PlayerJumpCommand{});
    32. else if(sf::Keyboard::isKeyPressed(Key::A))
    33. return nullptr;
    34. else if(sf::Keyboard::isKeyPressed(Key::D))
    35. return nullptr;
    36. else
    37. return nullptr;
    38. }
    39. }
    40. #endif


    Der Vollständigkeit halber hier noch die wichtigen anderen Klassen:

    C-Quellcode

    1. #ifndef COMMAND_H_
    2. #define COMMAND_H_
    4. namespace t1
    5. {
    7. template<class Actor>
    8. class Command
    9. {
    10. public:
    11. virtual ~Command() {};
    12. virtual void execute(Actor &actor) = 0;
    13. };
    14. }
    15. #endif


    C-Quellcode

    1. #ifndef PLAYER_JUMP_COMMAND_H_
    2. #define PLAYER_JUMP_COMMAND_H_
    4. #include "Command.h"
    5. #include "../Player.h"
    7. namespace t1
    8. {
    9. class PlayerJumpCommand : Command<Player>
    10. {
    11. public:
    12. void execute(Player &player);
    13. };
    15. void PlayerJumpCommand::execute(Player &player)
    16. {
    17. player.jump();
    18. }
    19. }
    21. #endif


    C-Quellcode

    1. #ifndef INPUT_COMPONENT_H_
    2. #define INPUT_COMPONENT_H_
    4. #include "../Commands/Command.h"
    5. #include <memory>
    7. namespace t1
    8. {
    9. template<class T>
    10. class InputComponent
    11. {
    12. public:
    13. virtual ~InputComponent();
    14. virtual std::unique_ptr<Command<T>> handleInput();
    15. };
    17. }
    18. #endif


    Bei dem Code bekomm ich folgenden Fehler und kann mir aber nicht erklären wieso. Blickt einer von euch C++ Gurus durch was da schief läuft ?

    Quellcode

    1. include/Components/PlayerInputComponent.h: In member function ‘virtual std::unique_ptr<t1::Command<t1::Player>, std::default_delete<t1::Command<t1::Player> > > t1::PlayerInputComponent::handleInput()’:
    2. include/Components/PlayerInputComponent.h:30:70: error: could not convert ‘std::unique_ptr<t1::PlayerJumpCommand>((operator new(8ul), (<statement>, ((t1::PlayerJumpCommand*)<anonymous>))))’ from ‘std::unique_ptr<t1::PlayerJumpCommand>’ to ‘std::unique_ptr<t1::Command<t1::Player>, std::default_delete<t1::Command<t1::Player> > >’
    3. return std::unique_ptr<PlayerJumpCommand> (new PlayerJumpCommand{});
    4. ^
    5. make: *** [debug] Fehler 1

    Hatte ich auch schon versucht, kommt dann Folgender Fehler bei raus:
    Passt hab bei PlayerJumpCommand die Vererbung nicht public gemacht. Jetzt passts, danke.

    Jo das funktioniert jetzt. Noch ne Frage wie ist das eigentlich Performance Technisch. Schlägt das arg auf die Performance jedes mal Speicher neu zu allozieren ? Der unique_ptr hat ja soweit ich weiß nicht viel overhead, könnte aber bei vielen Iterationen doch en problem werden oder ?

    new ist immer sehr teuer. Schlägt dementsprechend auch stark auf die Performance. Ich würde garnicht von Command<> erben, sondern eine Methode im Template zur Verfügung stellen und Command<> einfach übern Stack zurückgeben, oder als RValue etc.