Bill::__construct() - Code Metrics - Inspection of "Finished `DbMergingAggregator`" - hiqdev/php-billing - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 6ed56d...9a6247 )

by Andrii

created 2018-09-12 15:33 UTC

Bill::__construct() A

↳ Parent: Bill

Complexity

Conditions	1
Paths	1

Size

Total Lines	22
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	1

Importance

Changes

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

How to fix Many Parameters

<?php
/**
 * PHP Billing Library
 *
 * @link      https://github.com/hiqdev/php-billing
 * @package   php-billing
 * @license   BSD-3-Clause
 * @copyright Copyright (c) 2017-2018, HiQDev (http://hiqdev.com/)
 */

namespace hiqdev\php\billing\bill;

use DateTimeImmutable;
use hiqdev\php\billing\charge\ChargeInterface;
use hiqdev\php\billing\customer\CustomerInterface;
use hiqdev\php\billing\plan\PlanInterface;
use hiqdev\php\billing\target\TargetInterface;
use hiqdev\php\billing\type\TypeInterface;
use hiqdev\php\units\QuantityInterface;
use Money\Money;

/**
 * Bill.
 *
 * @author Andrii Vasyliev <[email protected]>
 */
class Bill implements BillInterface
{
    /** @var int|string */
    protected $id;

    /** @var TypeInterface */
    protected $type;

    /** @var DateTimeImmutable */
    protected $time;

    /** @var Money */
    protected $sum;

    /** @var QuantityInterface */
    protected $quantity;

    /** @var CustomerInterface */
    protected $customer;

    /** @var TargetInterface */
    protected $target;

    /** @var PlanInterface */
    protected $plan;

    /** @var ChargeInterface[] */
    protected $charges = [];

    /** @var BillState */
    protected $state;

    /** @var string */
    protected $comment;

    public function __construct(
                            $id,
        TypeInterface       $type,
        DateTimeImmutable   $time,
        Money               $sum,
        QuantityInterface   $quantity,
        CustomerInterface   $customer,
        TargetInterface     $target = null,
        PlanInterface       $plan = null,
        array               $charges = [],
        BillState           $state = null
    ) {
        $this->id           = $id;
        $this->type         = $type;
        $this->time         = $time;
        $this->sum          = $sum;
        $this->quantity     = $quantity;
        $this->customer     = $customer;
        $this->target       = $target;
        $this->plan         = $plan;
        $this->charges      = $charges;
        $this->state        = $state;
    }

    /**
     * Provides unique string.
     * Can be used to compare or aggregate bills.
     */
    public function getUniqueString(): string
    {
        $parts = [
            'currency'  => $this->sum->getCurrency()->getCode(),
            'buyer'     => $this->customer->getUniqueId(),
            'target'    => $this->target ? $this->target->getUniqueId() : null,
            'type'      => $this->type->getUniqueId(),
            'time'      => $this->time->format('c'),
            'plan'      => $this->plan ? $this->plan->getUniqueId() : null,
        ];

        return implode('-', $parts);
    }

    public function calculatePrice()
    {
        $quantity = $this->quantity->getQuantity();

        return $quantity ? $this->sum->divide($quantity) : $this->sum;
    }

    /**
     * @return int|string
     */
    public function getId()
    {
        return $this->id;
    }

    public function setId($id)
    {
        if ($this->id === $id) {
            return;
        }
        if ($this->id !== null) {
            throw new \Exception('cannot reassign bill id');
        }
        $this->id = $id;
    }

    /**
     * @return TypeInterface
     */
    public function getType(): TypeInterface
    {
        return $this->type;
    }

    /**
     * @return DateTimeImmutable
     */
    public function getTime(): DateTimeImmutable
    {
        return $this->time;
    }

    /**
     * @return TargetInterface
     */
    public function getTarget()
    {
        return $this->target;
    }

    /**
     * @return CustomerInterface
     */
    public function getCustomer()
    {
        return $this->customer;
    }

    /**
     * @return QuantityInterface
     */
    public function getQuantity()
    {
        return $this->quantity;
    }

    /**
     * @return BillInterface
     */
    public function setQuantity(QuantityInterface $quantity): BillInterface
    {
        $this->quantity = $quantity;

        return $this;
    }

    /**
     * @return Money
     */
    public function getSum()
    {
        return $this->sum;
    }

    /**
     * @return PlanInterface
     */
    public function getPlan()
    {
        return $this->plan;
    }

    /**
     * @return bool
     */
    public function hasCharges(): bool
    {
        return $this->charges !== [];
    }

    /**
     * @return ChargeInterface[]
     */
    public function getCharges(): array
    {
        return $this->charges;
    }

    /**
     * @param ChargeInterface[] $prices
     * @return self
     * @throws \Exception
     */
    public function setCharges(array $charges): self
    {
        if ($this->hasCharges()) {
            throw new \Exception('cannot reassign charges for bill');
        }
        $this->charges = $charges;

        return $this;
    }

    public function getState(): ?BillState
    {
        return $this->state;
    }

    public function setFinished(): void
    {
        $this->state = BillState::finished();
    }

    public function isFinished(): ?bool
    {
        return $this->state === null ? null : $this->state->isFinished();
    }

    public function getComment()
    {
        return $this->comment;
    }

    public function setComment(string $comment)
    {
        $this->comment = $comment;
    }

    public function jsonSerialize()
    {
        return get_object_vars($this);
    }
}


1		<?php
2		/**
3		* PHP Billing Library
4		*
5		* @link https://github.com/hiqdev/php-billing
6		* @package php-billing
7		* @license BSD-3-Clause
8		* @copyright Copyright (c) 2017-2018, HiQDev (http://hiqdev.com/)
9		*/
10
11		namespace hiqdev\php\billing\bill;
12
13		use DateTimeImmutable;
14		use hiqdev\php\billing\charge\ChargeInterface;
15		use hiqdev\php\billing\customer\CustomerInterface;
16		use hiqdev\php\billing\plan\PlanInterface;
17		use hiqdev\php\billing\target\TargetInterface;
18		use hiqdev\php\billing\type\TypeInterface;
19		use hiqdev\php\units\QuantityInterface;
20		use Money\Money;
21
22		/**
23		* Bill.
24		*
25		* @author Andrii Vasyliev <[email protected]>
26		*/
27		class Bill implements BillInterface
28		{
29		/** @var int\|string */
30		protected $id;
31
32		/** @var TypeInterface */
33		protected $type;
34
35		/** @var DateTimeImmutable */
36		protected $time;
37
38		/** @var Money */
39		protected $sum;
40
41		/** @var QuantityInterface */
42		protected $quantity;
43
44		/** @var CustomerInterface */
45		protected $customer;
46
47		/** @var TargetInterface */
48		protected $target;
49
50		/** @var PlanInterface */
51		protected $plan;
52
53		/** @var ChargeInterface[] */
54		protected $charges = [];
55
56		/** @var BillState */
57		protected $state;
58
59		/** @var string */
60		protected $comment;
61
62	1	public function __construct(
63		$id,
64		TypeInterface $type,
65		DateTimeImmutable $time,
66		Money $sum,
67		QuantityInterface $quantity,
68		CustomerInterface $customer,
69		TargetInterface $target = null,
70		PlanInterface $plan = null,
71		array $charges = [],
72		BillState $state = null
73		) {
74	1	$this->id = $id;
75	1	$this->type = $type;
76	1	$this->time = $time;
77	1	$this->sum = $sum;
78	1	$this->quantity = $quantity;
79	1	$this->customer = $customer;
80	1	$this->target = $target;
81	1	$this->plan = $plan;
82	1	$this->charges = $charges;
83	1	$this->state = $state;
84	1	}
85
86		/**
87		* Provides unique string.
88		* Can be used to compare or aggregate bills.
89		*/
90	1	public function getUniqueString(): string
91		{
92		$parts = [
93	1	'currency' => $this->sum->getCurrency()->getCode(),
94	1	'buyer' => $this->customer->getUniqueId(),
95	1	'target' => $this->target ? $this->target->getUniqueId() : null,
96	1	'type' => $this->type->getUniqueId(),
97	1	'time' => $this->time->format('c'),
98	1	'plan' => $this->plan ? $this->plan->getUniqueId() : null,
99		];
100
101	1	return implode('-', $parts);
102		}
103
104		public function calculatePrice()
105		{
106		$quantity = $this->quantity->getQuantity();
107
108		return $quantity ? $this->sum->divide($quantity) : $this->sum;
109		}
110
111		/**
112		* @return int\|string
113		*/
114	1	public function getId()
115		{
116	1	return $this->id;
117		}
118
119		public function setId($id)
120		{
121		if ($this->id === $id) {
122		return;
123		}
124		if ($this->id !== null) {
125		throw new \Exception('cannot reassign bill id');
126		}
127		$this->id = $id;
128		}
129
130		/**
131		* @return TypeInterface
132		*/
133	1	public function getType(): TypeInterface
134		{
135	1	return $this->type;
136		}
137
138		/**
139		* @return DateTimeImmutable
140		*/
141	1	public function getTime(): DateTimeImmutable
142		{
143	1	return $this->time;
144		}
145
146		/**
147		* @return TargetInterface
148		*/
149	1	public function getTarget()
150		{
151	1	return $this->target;
152		}
153
154		/**
155		* @return CustomerInterface
156		*/
157	1	public function getCustomer()
158		{
159	1	return $this->customer;
160		}
161
162		/**
163		* @return QuantityInterface
164		*/
165	1	public function getQuantity()
166		{
167	1	return $this->quantity;
168		}
169
170		/**
171		* @return BillInterface
172		*/
173		public function setQuantity(QuantityInterface $quantity): BillInterface
174		{
175		$this->quantity = $quantity;
176
177		return $this;
178		}
179
180		/**
181		* @return Money
182		*/
183	1	public function getSum()
184		{
185	1	return $this->sum;
186		}
187
188		/**
189		* @return PlanInterface
190		*/
191	1	public function getPlan()
192		{
193	1	return $this->plan;
194		}
195
196		/**
197		* @return bool
198		*/
199		public function hasCharges(): bool
200		{
201		return $this->charges !== [];
202		}
203
204		/**
205		* @return ChargeInterface[]
206		*/
207	1	public function getCharges(): array
208		{
209	1	return $this->charges;
210		}
211
212		/**
213		* @param ChargeInterface[] $prices
214		* @return self
215		* @throws \Exception
216		*/
217		public function setCharges(array $charges): self
218		{
219		if ($this->hasCharges()) {
220		throw new \Exception('cannot reassign charges for bill');
221		}
222		$this->charges = $charges;
223
224		return $this;
225		}
226
227		public function getState(): ?BillState
228		{
229		return $this->state;
230		}
231
232		public function setFinished(): void
233		{
234		$this->state = BillState::finished();
235		}
236
237		public function isFinished(): ?bool
238		{
239		return $this->state === null ? null : $this->state->isFinished();
240		}
241
242		public function getComment()
243		{
244		return $this->comment;
245		}
246
247		public function setComment(string $comment)
248		{
249		$this->comment = $comment;
250		}
251
252		public function jsonSerialize()
253		{
254		return get_object_vars($this);
255		}
256		}
257

hiqdev / php-billing

Push — master ( 6ed56d...9a6247 )

Bill::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Many Parameters

Many Parameters

Duplication Side-by-Side

Filter issues like