CharacterAttackSkill::getDamage() - Code Metrics - Inspection of "simplified TextCombatLogRender::renderItem()" - heroesofabenez/combat - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( dd3471...ff1e5a )

by Jakub

created 2018-12-21 15:22 UTC

CharacterAttackSkill::getDamage() A

↳ Parent: CharacterAttackSkill

Complexity

Conditions	3
Paths	4

Size

Total Lines	9
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	3

Importance

Changes

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

<?php
declare(strict_types=1);

namespace HeroesofAbenez\Combat;

/**
 * Character skill attack
 *
 * @author Jakub Konečný
 * @property-read SkillAttack $skill
 * @property-read int $damage
 * @property-read int $hitRate
 */
final class CharacterAttackSkill extends BaseCharacterSkill {
  /** @var SkillAttack */
  protected $skill;
  
  public function __construct(SkillAttack $skill, int $level) {
    parent::__construct($skill, $level);
  }

  public function getSkillType(): string {
    return "attack";
  }

  /**
   * @return SkillAttack
   */
  public function getSkill(): SkillAttack {
    return $this->skill;
  }
  
  public function getDamage(): int {
    $damage = 0;
    if(substr($this->skill->baseDamage, -1) === "%") {
      $damage += (int) $this->skill->baseDamage;
    }
    if(substr($this->skill->damageGrowth, -1) === "%") {
      $damage += (int) $this->skill->damageGrowth * ($this->level - 1);
    }
    return $damage;
  }
  
  public function getHitRate(): int {
    if(is_string($this->skill->hitRate) AND substr($this->skill->hitRate, -1) === "%") {
      return (int) $this->skill->hitRate;
    }
    return 100;
  }
}
?>

1		<?php
2		declare(strict_types=1);
3
4		namespace HeroesofAbenez\Combat;
5
6		/**
7		* Character skill attack
8		*
9		* @author Jakub Konečný
10		* @property-read SkillAttack $skill
11		* @property-read int $damage
12		* @property-read int $hitRate
13		*/
14	1	final class CharacterAttackSkill extends BaseCharacterSkill {
15		/** @var SkillAttack */
16		protected $skill;
17
18		public function __construct(SkillAttack $skill, int $level) {
19	1	parent::__construct($skill, $level);
20	1	}
21
22		public function getSkillType(): string {
23	1	return "attack";
24		}
25
26		/**
27		* @return SkillAttack
28		*/
29		public function getSkill(): SkillAttack {
30	1	return $this->skill;
31		}
32
33		public function getDamage(): int {
34	1	$damage = 0;
35	1	if(substr($this->skill->baseDamage, -1) === "%") {
36	1	$damage += (int) $this->skill->baseDamage;
37		}
38	1	if(substr($this->skill->damageGrowth, -1) === "%") {
39	1	$damage += (int) $this->skill->damageGrowth * ($this->level - 1);
40		}
41	1	return $damage;
42		}
43
44		public function getHitRate(): int {
45	1	if(is_string($this->skill->hitRate) AND substr($this->skill->hitRate, -1) === "%") {
46	1	return (int) $this->skill->hitRate;
47		}
48	1	return 100;
49		}
50		}
51		?>

heroesofabenez / combat

Push — master ( dd3471...ff1e5a )

CharacterAttackSkill::getDamage() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like