CombatActionSelector::getAllowedActions() - Code Metrics - Inspection of "use combat action selector to choose action in Com..." - heroesofabenez/combat - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a5d74c...946f06 )

by Jakub

created 2018-06-04 14:40 UTC

CombatActionSelector::getAllowedActions() A

↳ Parent: CombatActionSelector

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	4
ccs	3
cts	3
cp	1
rs	10
c	0
b	0
f	0
cc	1
eloc	3
nc	1
nop	0
crap	1

<?php
declare(strict_types=1);

namespace HeroesofAbenez\Combat;

use Nexendrie\Utils\Constants;

/**
 * CombatActionSelector
 *
 * @author Jakub Konečný
 */
final class CombatActionSelector implements ICombatActionSelector {
  public function getAllowedActions(): array {
    $allowedActions = Constants::getConstantsValues(CombatAction::class, "ACTION_");
    return array_values(array_filter($allowedActions, function(string $value) {
      return ($value !== CombatAction::ACTION_POISON);
    }));
  }
  
  public function chooseAction(CombatBase $combat, Character $character): ?string {
    if($character->hitpoints < 1) {
      return NULL;
    } elseif(in_array($character, $combat->findHealers()->toArray(), true) AND !is_null($combat->selectHealingTarget($character))) {

      return CombatAction::ACTION_HEALING;
    }
    $attackTarget = $combat->selectAttackTarget($character);
    if(is_null($attackTarget)) {
      return NULL;
    }
    if(count($character->usableSkills) > 0) {
      $skill = $character->usableSkills[0];
      if($skill instanceof CharacterAttackSkill) {
        return CombatAction::ACTION_SKILL_ATTACK;
      } elseif($skill instanceof  CharacterSpecialSkill) {
        return CombatAction::ACTION_SKILL_SPECIAL;
      }
    }
    return CombatAction::ACTION_ATTACK;
  }
}
?>

1		<?php
2		declare(strict_types=1);
3
4		namespace HeroesofAbenez\Combat;
5
6		use Nexendrie\Utils\Constants;
7
8		/**
9		* CombatActionSelector
10		*
11		* @author Jakub Konečný
12		*/
13	1	final class CombatActionSelector implements ICombatActionSelector {
14		public function getAllowedActions(): array {
15	1	$allowedActions = Constants::getConstantsValues(CombatAction::class, "ACTION_");
16	1	return array_values(array_filter($allowedActions, function(string $value) {
17	1	return ($value !== CombatAction::ACTION_POISON);
18	1	}));
19		}
20
21		public function chooseAction(CombatBase $combat, Character $character): ?string {
22	1	if($character->hitpoints < 1) {
23		return NULL;
24	1	} elseif(in_array($character, $combat->findHealers()->toArray(), true) AND !is_null($combat->selectHealingTarget($character))) {
		0 ignored issues – show introduced 2018-04-07 08:50 UTC by Report Bug Copy Issue Report Line exceeds 120 characters; contains 132 characters Loading history...
25	1	return CombatAction::ACTION_HEALING;
26		}
27	1	$attackTarget = $combat->selectAttackTarget($character);
28	1	if(is_null($attackTarget)) {
29		return NULL;
30		}
31	1	if(count($character->usableSkills) > 0) {
32	1	$skill = $character->usableSkills[0];
33	1	if($skill instanceof CharacterAttackSkill) {
34	1	return CombatAction::ACTION_SKILL_ATTACK;
35	1	} elseif($skill instanceof CharacterSpecialSkill) {
36	1	return CombatAction::ACTION_SKILL_SPECIAL;
37		}
38		}
39	1	return CombatAction::ACTION_ATTACK;
40		}
41		}
42		?>

heroesofabenez / combat

Push — master ( a5d74c...946f06 )

CombatActionSelector::getAllowedActions() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like