ProfileAvatar::randomFillIn() - Code Metrics - dariuszwrzesien/DwrAvatarBundle - Measure and Improve Code Quality continuously with Scrutinizer

ProfileAvatar::randomFillIn() B
last analyzed 2017-03-10 21:36 UTC

↳ Parent: ProfileAvatar

Complexity

Conditions	3
Paths	3

Size

Total Lines	29
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	29
rs	8.8571
c	0
b	0
f	0
cc	3
eloc	12
nc	3
nop	0

<?php

namespace Dwr\AvatarBundle\Model;

class ProfileAvatar extends Avatar {
    
    
    const PROFILE_IMAGE_RESOLUTION = 5;
    
    /**
     * @var int 
     */
    protected $size;
    
    /**
     * @var string 
     */
    protected $picture;
    
    /**
     * @var string
     */
    protected $canvas;

    /**
     * @param int $size
     */
    public function __construct($size) 
    {
        $this->size = $size;
        $this->canvas = imagecreatetruecolor($size, $size);

    }
    
    /**
     * @return Avatar
     */
    public function getAvatar() 
    {
        return $this;
    }
    
    protected function draw()
    {
        $this->createBackground();
        $this->createProfileImage();
        
        ob_start(); 
            imagejpeg($this->canvas);
            $image = ob_get_contents(); 
        ob_end_clean();
        
        $this->picture = $image;
        
        return $this->picture;
    }
    
    private function createBackground()
    {
        $backgroundColor = $this->randomizeColor($this->canvas, Avatar::$WHITE);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        imagefill($this->canvas, 0, 0, $backgroundColor);
    }
    
    private function createProfileImage()
    {
        $imageMapFields = $this->randomFillIn();
        $imageCellSize = $this->getSize()/self::PROFILE_IMAGE_RESOLUTION;
        $imageColor = $this->randomizeColor($this->canvas, Avatar::$GREY);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        
        $coordinates = array(
            'x1' => 0, 'y1' => 0, 
            'x2' => $imageCellSize, 'y2' => $imageCellSize,
        );
        
        $counter = 0;
        foreach ($imageMapFields as $field) {
            if ($counter % 5 !== 0) {
                $coordinates['x1'] += $imageCellSize;
                $coordinates['x2'] += $imageCellSize;
                $this->filledRectangle($coordinates, $imageColor, $field);
            } else {
                if ($counter > 0) {
                    $coordinates = array(
                        'x1' => 0, 'y1' => $coordinates['y1'] += $imageCellSize, 
                        'x2' => $imageCellSize, 'y2' => $coordinates['y2'] += $imageCellSize,
                    );
                }
                $this->filledRectangle($coordinates, $imageColor, $field);
            }
            $counter++;
        }
    }
    
    private function filledRectangle(array $coordinates, $color, $field)
    {
        if ($field) {
            imagefilledrectangle(
                $this->canvas, 
                $coordinates['x1'], 
                $coordinates['y1'], 
                $coordinates['x2'], 
                $coordinates['y2'], 
                $color
            );
        }
    }
    
    /**
     * @return array
     */
    private function randomFillIn()
    {
        /**
         * Output: 
         * $random = array(
         *     array((bool)rand(0,1), (bool)rand(0,1)),
         *     array((bool)rand(0,1), (bool)rand(0,1)),
         *     array((bool)rand(0,1), (bool)rand(0,1))
         *     array((bool)rand(0,1), (bool)rand(0,1))
         *     array((bool)rand(0,1), (bool)rand(0,1))
         * );
         */
        $random = array();
        for ($i = 0; $i < self::PROFILE_IMAGE_RESOLUTION; $i++) {
            for ($j = 0; $j < 2; $j++) {
               $random[$i][$j] = (bool)rand(0,1);
            }
        }
        
        $filledInFields = array(
            $random[0][0], $random[0][1], !$random[0][1], $random[0][1], $random[0][0],
            $random[1][0], $random[1][1], !$random[1][1], $random[1][1], $random[1][0],
            $random[2][0], $random[2][1], !$random[2][1], $random[2][1], $random[2][0],
            $random[3][0], $random[3][1], !$random[3][1], $random[3][1], $random[3][0],
            $random[4][0], $random[4][1], !$random[4][1], $random[4][1], $random[4][0],
        );
            
        return $filledInFields;
    }
    
}


1			<?php
2
3			namespace Dwr\AvatarBundle\Model;
4
5			class ProfileAvatar extends Avatar {
6
7
8			const PROFILE_IMAGE_RESOLUTION = 5;
9
10			/**
11			* @var int
12			*/
13			protected $size;
14
15			/**
16			* @var string
17			*/
18			protected $picture;
19
20			/**
21			* @var string
22			*/
23			protected $canvas;
24
25			/**
26			* @param int $size
27			*/
28			public function __construct($size)
29			{
30			$this->size = $size;
31			$this->canvas = imagecreatetruecolor($size, $size);
			0 ignored issues – show Documentation Bug introduced 2017-02-02 12:26 UTC by Report Bug Copy Issue Report It seems like `imagecreatetruecolor($size, $size)` of type `resource` is incompatible with the declared type `string` of property `$canvas`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
32			}
33
34			/**
35			* @return Avatar
36			*/
37			public function getAvatar()
38			{
39			return $this;
40			}
41
42			protected function draw()
43			{
44			$this->createBackground();
45			$this->createProfileImage();
46
47			ob_start();
48			imagejpeg($this->canvas);
49			$image = ob_get_contents();
50			ob_end_clean();
51
52			$this->picture = $image;
53
54			return $this->picture;
55			}
56
57			private function createBackground()
58			{
59			$backgroundColor = $this->randomizeColor($this->canvas, Avatar::$WHITE);
			0 ignored issues – show Documentation introduced 2017-02-02 12:26 UTC by Report Bug Copy Issue Report `$this->canvas` is of type `string`, but the function expects a `resource`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
60			imagefill($this->canvas, 0, 0, $backgroundColor);
61			}
62
63			private function createProfileImage()
64			{
65			$imageMapFields = $this->randomFillIn();
66			$imageCellSize = $this->getSize()/self::PROFILE_IMAGE_RESOLUTION;
67			$imageColor = $this->randomizeColor($this->canvas, Avatar::$GREY);
			0 ignored issues – show Documentation introduced 2017-02-02 12:26 UTC by Report Bug Copy Issue Report `$this->canvas` is of type `string`, but the function expects a `resource`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
68
69			$coordinates = array(
70			'x1' => 0, 'y1' => 0,
71			'x2' => $imageCellSize, 'y2' => $imageCellSize,
72			);
73
74			$counter = 0;
75			foreach ($imageMapFields as $field) {
76			if ($counter % 5 !== 0) {
77			$coordinates['x1'] += $imageCellSize;
78			$coordinates['x2'] += $imageCellSize;
79			$this->filledRectangle($coordinates, $imageColor, $field);
80			} else {
81			if ($counter > 0) {
82			$coordinates = array(
83			'x1' => 0, 'y1' => $coordinates['y1'] += $imageCellSize,
84			'x2' => $imageCellSize, 'y2' => $coordinates['y2'] += $imageCellSize,
85			);
86			}
87			$this->filledRectangle($coordinates, $imageColor, $field);
88			}
89			$counter++;
90			}
91			}
92
93			private function filledRectangle(array $coordinates, $color, $field)
94			{
95			if ($field) {
96			imagefilledrectangle(
97			$this->canvas,
98			$coordinates['x1'],
99			$coordinates['y1'],
100			$coordinates['x2'],
101			$coordinates['y2'],
102			$color
103			);
104			}
105			}
106
107			/**
108			* @return array
109			*/
110			private function randomFillIn()
111			{
112			/**
113			* Output:
114			* $random = array(
115			* array((bool)rand(0,1), (bool)rand(0,1)),
116			* array((bool)rand(0,1), (bool)rand(0,1)),
117			* array((bool)rand(0,1), (bool)rand(0,1))
118			* array((bool)rand(0,1), (bool)rand(0,1))
119			* array((bool)rand(0,1), (bool)rand(0,1))
120			* );
121			*/
122			$random = array();
123			for ($i = 0; $i < self::PROFILE_IMAGE_RESOLUTION; $i++) {
124			for ($j = 0; $j < 2; $j++) {
125			$random[$i][$j] = (bool)rand(0,1);
126			}
127			}
128
129			$filledInFields = array(
130			$random[0][0], $random[0][1], !$random[0][1], $random[0][1], $random[0][0],
131			$random[1][0], $random[1][1], !$random[1][1], $random[1][1], $random[1][0],
132			$random[2][0], $random[2][1], !$random[2][1], $random[2][1], $random[2][0],
133			$random[3][0], $random[3][1], !$random[3][1], $random[3][1], $random[3][0],
134			$random[4][0], $random[4][1], !$random[4][1], $random[4][1], $random[4][0],
135			);
136
137			return $filledInFields;
138			}
139
140			}
141

dariuszwrzesien / DwrAvatarBundle

ProfileAvatar::randomFillIn() B last analyzed 2017-03-10 21:36 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

ProfileAvatar::randomFillIn() B
last analyzed 2017-03-10 21:36 UTC