Row::setCells() - Code Metrics - Inspection of "Row objects and Cell styling" - box/spout - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Pull Request — develop_3.0 (#434)

by Adrien

created 2017-10-21 16:26 UTC

Row::setCells() A

↳ Parent: Row

Complexity

Conditions	2
Paths	2

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	2

Importance

Changes

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

<?php

namespace Box\Spout\Writer\Common\Entity;

use Box\Spout\Writer\Common\Entity\Style\Style;
use Box\Spout\Writer\Common\Manager\RowManager;

class Row
{
    /**
     * The cells in this row
     * @var Cell[]
     */
    protected $cells = [];

    /**
     * The row style
     * @var Style|null
     */
    protected $style;

    /**
     * Thw row manager
     * @var RowManager
     */
    protected $rowManager;

    /**
     * Row constructor.
     * @param Cell[] $cells
     * @param Style|null $style
     * @param RowManager $rowManager
     */
    public function __construct(array $cells, $style, RowManager $rowManager)
    {
        $this
            ->setCells($cells)
            ->setStyle($style);
function notNullable(stdClass $x) { }

// Unsafe
function withoutCheck(stdClass $x = null) {
    notNullable($x);
}

// Safe - Alternative 1: Adding Additional Type-Check
function withCheck(stdClass $x = null) {
    if ($x instanceof stdClass) {
        notNullable($x);
    }
}

// Safe - Alternative 2: Changing Parameter
function withNonNullableParam(stdClass $x) {
    notNullable($x);
}

        $this->rowManager = $rowManager;
    }

    /**
     * @return Cell[] $cells
     */
    public function getCells()
    {
        return $this->cells;
    }

    /**
     * @param Cell[] $cells
     * @return $this
     */
    public function setCells(array $cells)
    {
        $this->cells = [];
        foreach ($cells as $cell) {
            $this->addCell($cell);
        }

        return $this;
    }

    /**
     * @return Style
     */
    public function getStyle()
    {
        return $this->style;
    }

    /**
     * @param Style $style
     * @return Row
     */
    public function setStyle($style)
    {
        $this->style = $style ?: new Style();

        return $this;
    }

    /**
     * @param Style $style
     * @return Row
     */
    public function applyStyle($style)
    {
        $this->rowManager->applyStyle($this, $style);

        return $this;
    }

    /**
     * @param Cell $cell
     * @return Row
     */
    public function addCell(Cell $cell)
    {
        $this->cells[] = $cell;

        return $this;
    }

    /**
     * Returns whether a row has cells
     *
     * @return bool
     */
    public function hasCells()
    {
        return $this->rowManager->hasCells($this);
    }

    /**
     * Detect whether this row is considered empty.
     * An empty row has either no cells at all - or only empty cells
     *
     * @return bool
     */
    public function isEmpty()
    {
        return $this->rowManager->isEmpty($this);
    }
}


1		<?php
2
3		namespace Box\Spout\Writer\Common\Entity;
4
5		use Box\Spout\Writer\Common\Entity\Style\Style;
6		use Box\Spout\Writer\Common\Manager\RowManager;
7
8		class Row
9		{
10		/**
11		* The cells in this row
12		* @var Cell[]
13		*/
14		protected $cells = [];
15
16		/**
17		* The row style
18		* @var Style\|null
19		*/
20		protected $style;
21
22		/**
23		* Thw row manager
24		* @var RowManager
25		*/
26		protected $rowManager;
27
28		/**
29		* Row constructor.
30		* @param Cell[] $cells
31		* @param Style\|null $style
32		* @param RowManager $rowManager
33		*/
34	94	public function __construct(array $cells, $style, RowManager $rowManager)
35		{
36		$this
37	94	->setCells($cells)
38	94	->setStyle($style);
		0 ignored issues – show Bug introduced 2017-05-30 10:40 UTC by Report Bug Copy Issue Report It seems like `$style` defined by parameter `$style` on line `34` can be `null`; however, `Box\Spout\Writer\Common\Entity\Row::setStyle()` does not accept `null`, maybe add an additional type check? It seems like you allow that null is being passed for a parameter, however the function which is called does not seem to accept null. We recommend to add an additional type check (or disallow null for the parameter): function notNullable(stdClass $x) { } // Unsafe function withoutCheck(stdClass $x = null) { notNullable($x); } // Safe - Alternative 1: Adding Additional Type-Check function withCheck(stdClass $x = null) { if ($x instanceof stdClass) { notNullable($x); } } // Safe - Alternative 2: Changing Parameter function withNonNullableParam(stdClass $x) { notNullable($x); } Loading history...
39
40	94	$this->rowManager = $rowManager;
41	94	}
42
43		/**
44		* @return Cell[] $cells
45		*/
46	82	public function getCells()
47		{
48	82	return $this->cells;
49		}
50
51		/**
52		* @param Cell[] $cells
53		* @return $this
54		*/
55	94	public function setCells(array $cells)
56		{
57	94	$this->cells = [];
58	94	foreach ($cells as $cell) {
59	89	$this->addCell($cell);
60		}
61
62	94	return $this;
63		}
64
65		/**
66		* @return Style
67		*/
68	62	public function getStyle()
69		{
70	62	return $this->style;
71		}
72
73		/**
74		* @param Style $style
75		* @return Row
76		*/
77	94	public function setStyle($style)
78		{
79	94	$this->style = $style ?: new Style();
80
81	94	return $this;
82		}
83
84		/**
85		* @param Style $style
86		* @return Row
87		*/
88		public function applyStyle($style)
89		{
90		$this->rowManager->applyStyle($this, $style);
91
92		return $this;
93		}
94
95		/**
96		* @param Cell $cell
97		* @return Row
98		*/
99	90	public function addCell(Cell $cell)
100		{
101	90	$this->cells[] = $cell;
102
103	90	return $this;
104		}
105
106		/**
107		* Returns whether a row has cells
108		*
109		* @return bool
110		*/
111	71	public function hasCells()
112		{
113	71	return $this->rowManager->hasCells($this);
114		}
115
116		/**
117		* Detect whether this row is considered empty.
118		* An empty row has either no cells at all - or only empty cells
119		*
120		* @return bool
121		*/
122	32	public function isEmpty()
123		{
124	32	return $this->rowManager->isEmpty($this);
125		}
126		}
127

box / spout

Pull Request — develop_3.0 (#434)

Row::setCells() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like