Refactor ActorManager::validateBeing()

Squashed with:
* More docs for validateBeing

* addendum documentation for validateBeing

****
mana/plus!154

1 files changed, 26 insertions, 6 deletions

diff --git a/src/actormanager.cpp b/src/actormanager.cpp
index 2e1079487..e33798bab 100644
--- a/src/actormanager.cpp
+++ b/src/actormanager.cpp
@@ -1386,12 +1386,32 @@ bool ActorManager::validateBeing(const Being *const aroundBeing,
 {
     if (localPlayer == nullptr)
         return false;
-    return (being != nullptr) && ((being->getType() == type
-        || type == ActorType::Unknown) && (being->isAlive()
-        || (mTargetDeadPlayers && type == ActorType::Player))
-        && being != aroundBeing) && being != excluded
-        && (type != ActorType::Monster || !mTargetOnlyReachable
-        || localPlayer->isReachable(being, maxCost));
+
+    if (being == nullptr)
+        return false;
+
+    if (being == aroundBeing)
+        return false;
+
+    if (being == excluded)
+        return false;
+
+    // If specific being type is given, it must match.
+    if (!(being->getType() == type || type == ActorType::Unknown))
+        return false;
+
+    if (!being->isAlive())
+        if (!mTargetDeadPlayers || type != ActorType::Player)
+            return false;
+
+    // Why are we ignoring real reachability checks for non-monsters?
+    if (type != ActorType::Monster)
+        return true;
+
+    if (!mTargetOnlyReachable)
+        return true;
+
+    return localPlayer->isReachable(being, maxCost);
 }
 
 #ifdef TMWA_SUPPORT