src/GameState.cpp

@@ -31,39 +31,6 @@ void GameState::clearUnits(){
     units.clear();
 }
 
-GameState &GameState::operator=(const GameState &other) {
-    if (this == &other)
-        return *this;
-
-    clearUnits();
-    for (const Unit *const unit : other.units)
-        units.push_back(unit->clone());
-    hero = other.hero;
-    return *this;
-}
-
-GameState::GameState(const GameState &other) : hero(other.hero) {
-    clearUnits();
-    for (const Unit *const unit : other.units)
-        units.push_back(unit->clone());
-}
-
-GameState &GameState::operator=(GameState &&other) noexcept {
-    if (this == &other)
-        return *this;
-    clearUnits();
-
-    hero = std::move(other.hero);
-    units = std::move(other.units);
-    other.units.clear();
-
-    return *this;
-}
-
-GameState::GameState(GameState &&other) noexcept : units(std::move(other.units)), hero(std::move(other.hero)) {
-    other.units.clear();
-}
-
 void GameState::draw(QGraphicsScene *scene) const {
     scene->clear();
     hero.draw(scene);
@@ -100,6 +67,23 @@ void GameState::updateCrashes() {
     }
 }
 
+GameState &GameState::operator=(const GameState &other) {
+    if (this == &other)
+        return *this;
+
+    clearUnits();
+    for (const Unit *const unit : other.units)
+        units.push_back(unit->clone());
+    hero = other.hero;
+    return *this;
+}
+
+GameState::GameState(const GameState &other) : hero(other.hero) {
+    clearUnits();
+    for (const Unit *const unit : other.units)
+        units.push_back(unit->clone());
+}
+
 int GameState::countToBeJunked() const {
     int numberDestroyed = 0;
     for (const Unit *const unit: units)

src/GameState.h

@@ -27,14 +27,8 @@ public:
 
     ~GameState();
 
-    // Copy
     GameState &operator=(const GameState &other);
     GameState(const GameState &other);
-
-    // Move
-    GameState &operator=(GameState &&other) noexcept;
-    GameState(GameState &&other) noexcept;
-
     /*
      * Clear and redraw entire playing field
      */

src/Hero.cpp

@@ -17,7 +17,3 @@ void Hero::draw(QGraphicsScene *scene) const {
 Unit *Hero::clone() const {
     return new Hero(*this);
 }
-
-bool Hero::isAlive() const { return true; }
-bool Hero::isToBeJunked() const { return false; }
-void Hero::doCrash() {}

src/Hero.h

@@ -18,9 +18,7 @@ public:
     * Draws this hero onto the given QGraphicsScene.
     */
     void draw(QGraphicsScene *scene) const override;
-    bool isAlive() const override;
+
-    bool isToBeJunked() const override;
-    void doCrash() override;
 };
 
 #endif // HERO_H

src/Unit.cpp

@@ -59,4 +59,7 @@ void Unit::checkBounds() {
 }
 
 // Default implementation
-
+bool Unit::isAlive() const { return true; }
+bool Unit::isToBeJunked() const { return false; }
+void Unit::doCrash() {}
+void Unit::draw(QGraphicsScene *) const {}

src/Unit.h

@@ -57,10 +57,10 @@ public:
     */
     double distanceTo(const Unit& u) const;
 
-    virtual bool isAlive() const = 0;
+    virtual bool isAlive() const;
-    virtual bool isToBeJunked() const = 0;
+    virtual bool isToBeJunked() const;
-    virtual void doCrash() = 0;
+    virtual void doCrash();
-    virtual void draw(QGraphicsScene *scene) const = 0;
+    virtual void draw(QGraphicsScene *scene) const;
 private:
     int x;  // x position of this unit
     int y;  // y position of this unit