extracted battle logic into a class

[l2e.git] / src / battle / TargetSelection.cpp

diff --git a/src/battle/TargetSelection.cpp b/src/battle/TargetSelection.cpp
index 5fbf78525d21d1318b636b5b63beb6f756c57c5c..573200a799c1caf96e3819f47850e5eb67fd4a29 100644 (file)
--- a/src/battle/TargetSelection.cpp
+++ b/src/battle/TargetSelection.cpp
@@ -1,25 +1,31 @@
-/*
- * TargetSelection.cpp
- *
- *  Created on: Aug 9, 2012
- *      Author: holy
- */
-
 #include "TargetSelection.h"
 
-#include "BattleState.h"
+#include "Battle.h"
 #include "../common/TargetingMode.h"
 
+#include <cassert>
+
+
 namespace battle {
 
-TargetSelection::TargetSelection(BattleState *battle, bool multiple, bool atEnemy)
+TargetSelection::TargetSelection()
+: battle(0)
+, selected()
+, selection(-1)
+, cursor(0)
+, multiple(false)
+, enemy(true) {
+
+}
+
+TargetSelection::TargetSelection(Battle *battle, bool multiple, bool atEnemy)
 : battle(battle)
-, selected(battle ? (battle->MaxMonsters() > battle->NumHeroes() ? battle->MaxMonsters() : battle->NumHeroes()) : 0, State())
+, selected(battle->MaxMonsters() > battle->NumHeroes() ? battle->MaxMonsters() : battle->NumHeroes(), State())
 , selection(-1)
 , cursor(0)
 , multiple(multiple)
 , enemy(atEnemy) {
-       if (battle && enemy) {
+       if (enemy) {
                FindNextEnemy();
        }
 }
@@ -27,7 +33,7 @@ TargetSelection::TargetSelection(BattleState *battle, bool multiple, bool atEnem
 
 void TargetSelection::ReadMode(const common::TargetingMode &tm) {
        if (tm.TargetsEnemy()) {
-               SelectEnemies();
+               SelectMonsters();
        } else {
                SelectHeroes();
        }
@@ -39,8 +45,8 @@ void TargetSelection::ReadMode(const common::TargetingMode &tm) {
 }
 
 
-void TargetSelection::SelectEnemies() {
-       if (TargetsEnemies()) return;
+void TargetSelection::SelectMonsters() {
+       if (TargetsMonsters()) return;
        enemy = true;
        UnselectAll();
        cursor = 0;
@@ -58,23 +64,24 @@ void TargetSelection::SelectHeroes() {
 void TargetSelection::Reset() {
        UnselectAll();
        cursor = 0;
-       if (TargetsEnemies()) {
+       if (TargetsMonsters()) {
                FindNextEnemy();
        }
 }
 
 
 void TargetSelection::MoveUp() {
-       if (TargetsEnemies()) return;
+       if (TargetsMonsters()) return;
        if (cursor < 2) {
-               SelectEnemies();
+               SelectMonsters();
        } else {
                cursor -= 2;
        }
 }
 
 void TargetSelection::MoveRight() {
-       if (TargetsEnemies()) {
+       assert(battle);
+       if (TargetsMonsters()) {
                cursor = (cursor + 1) % battle->MaxMonsters();
                while (!battle->MonsterPositionOccupied(cursor)) {
                        cursor = (cursor + 1) % battle->MaxMonsters();
@@ -85,7 +92,8 @@ void TargetSelection::MoveRight() {
 }
 
 void TargetSelection::MoveDown() {
-       if (TargetsEnemies()) {
+       assert(battle);
+       if (TargetsMonsters()) {
                SelectHeroes();
                return;
        }
@@ -96,7 +104,8 @@ void TargetSelection::MoveDown() {
 }
 
 void TargetSelection::MoveLeft() {
-       if (TargetsEnemies()) {
+       assert(battle);
+       if (TargetsMonsters()) {
                cursor = (cursor + battle->MaxMonsters() - 1) % battle->MaxMonsters();
                FindNextEnemy();
        } else {
@@ -105,6 +114,7 @@ void TargetSelection::MoveLeft() {
 }
 
 void TargetSelection::FindNextEnemy() {
+       assert(battle);
        int start(cursor);
        while (!battle->MonsterPositionOccupied(cursor)) {
                cursor = (cursor + battle->MaxMonsters() - 1) % battle->MaxMonsters();