BaseCharacterSkill::isUsable() - Code Metrics - Inspection of "replaced method BaseCharacterSkill::canUse() with..." - heroesofabenez/combat - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 9ad4f6...7fb37f )

by Jakub

created 2018-12-22 19:16 UTC

BaseCharacterSkill::isUsable() A

↳ Parent: BaseCharacterSkill

Complexity

Conditions	1
Paths	1

Size

Total Lines	2
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	1
CRAP Score	1

Importance

Changes

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

<?php
declare(strict_types=1);

namespace HeroesofAbenez\Combat;

use Nexendrie\Utils\Numbers;

/**
 * Base character skill
 *
 * @author Jakub Konečný
 * @property-read BaseSkill $skill
 * @property int $level
 * @property-read int $cooldown
 * @property-read string $skillType
 * @property-read bool $usable
 */
abstract class BaseCharacterSkill {
  use \Nette\SmartObject;
  
  /** @var BaseSkill */
  protected $skill;
  /** @var int */
  protected $level;
  /** @var int */
  protected $cooldown = 0;
  
  public function __construct(BaseSkill $skill, int $level) {
    $this->skill = $skill;
    $this->setLevel($level);
    $this->resetCooldown();
  }

  abstract public function getSkillType(): string;
  
  public function getLevel(): int {
    return $this->level;
  }
  
  public function getCooldown(): int {
    return $this->cooldown;
  }
  
  public function setLevel(int $level): void {
    $this->level = Numbers::range($level, 0, $this->skill->levels);
  }
  
  public function isUsable(): bool {
    return ($this->cooldown < 1);
  }
  
  public function resetCooldown(): void {
    $this->cooldown = $this->skill->cooldown;
  }
  
  public function decreaseCooldown(): void {
    if($this->cooldown > 0) {
      $this->cooldown--;
    }
  }
}
?>

1		<?php
2		declare(strict_types=1);
3
4		namespace HeroesofAbenez\Combat;
5
6		use Nexendrie\Utils\Numbers;
7
8		/**
9		* Base character skill
10		*
11		* @author Jakub Konečný
12		* @property-read BaseSkill $skill
13		* @property int $level
14		* @property-read int $cooldown
15		* @property-read string $skillType
16		* @property-read bool $usable
17		*/
18	1	abstract class BaseCharacterSkill {
19	1	use \Nette\SmartObject;
20
21		/** @var BaseSkill */
22		protected $skill;
23		/** @var int */
24		protected $level;
25		/** @var int */
26		protected $cooldown = 0;
27
28		public function __construct(BaseSkill $skill, int $level) {
29	1	$this->skill = $skill;
30	1	$this->setLevel($level);
31	1	$this->resetCooldown();
32	1	}
33
34		abstract public function getSkillType(): string;
35
36		public function getLevel(): int {
37	1	return $this->level;
38		}
39
40		public function getCooldown(): int {
41		return $this->cooldown;
42		}
43
44		public function setLevel(int $level): void {
45	1	$this->level = Numbers::range($level, 0, $this->skill->levels);
46	1	}
47
48		public function isUsable(): bool {
49	1	return ($this->cooldown < 1);
50		}
51
52		public function resetCooldown(): void {
53	1	$this->cooldown = $this->skill->cooldown;
54	1	}
55
56		public function decreaseCooldown(): void {
57	1	if($this->cooldown > 0) {
58	1	$this->cooldown--;
59		}
60	1	}
61		}
62		?>

heroesofabenez / combat

Push — master ( 9ad4f6...7fb37f )

BaseCharacterSkill::isUsable() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like