| Conditions | 1 | 
| Paths | 1 | 
| Total Lines | 100 | 
| Code Lines | 54 | 
| Lines | 0 | 
| Ratio | 0 % | 
| Changes | 0 | ||
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | <?php | ||
| 58 | public function testGetCommandsText(): void | ||
| 59 |     { | ||
| 60 | $content = "/R14 30 Tf 0.999016 0 0 1 137.4 | ||
| 61 | 342.561 Tm | ||
| 62 | [(A)-168.854( BC D)-220.905(\\(E\\))20.905<20>] | ||
| 63 | TJ /R14 17.16 Tf <20> Tj | ||
| 64 | 0.999014 0 0 1 336.84 319.161 Tm T* ( \x00m)Tj | ||
| 65 | /R14 20.04 Tf | ||
| 66 | ET Q | ||
| 67 | q -124.774 124.127 5.64213 5.67154 930.307 4436.95 cm | ||
| 68 | BI"; | ||
| 69 | |||
| 70 | $offset = 0; | ||
| 71 | $parts = $this->getPdfObjectInstance(new Document())->getCommandsText($content, $offset); | ||
| 72 | $reference = [ | ||
| 73 | [ | ||
| 74 | self::TYPE => '/', | ||
| 75 | self::OPERATOR => 'Tf', | ||
| 76 | self::COMMAND => 'R14 30', | ||
| 77 | ], | ||
| 78 | [ | ||
| 79 | self::TYPE => '', | ||
| 80 | self::OPERATOR => 'Tm', | ||
| 81 | self::COMMAND => "0.999016 0 0 1 137.4\n342.561", | ||
| 82 | ], | ||
| 83 | [ | ||
| 84 | self::TYPE => '[', | ||
| 85 | self::OPERATOR => 'TJ', | ||
| 86 | self::COMMAND => [ | ||
| 87 | [ | ||
| 88 |                         self::TYPE => '(', | ||
| 89 | self::OPERATOR => '', | ||
| 90 | self::COMMAND => 'A', | ||
| 91 | ], | ||
| 92 | [ | ||
| 93 | self::TYPE => 'n', | ||
| 94 | self::OPERATOR => '', | ||
| 95 | self::COMMAND => '-168.854', | ||
| 96 | ], | ||
| 97 | [ | ||
| 98 |                         self::TYPE => '(', | ||
| 99 | self::OPERATOR => '', | ||
| 100 | self::COMMAND => ' BC D', | ||
| 101 | ], | ||
| 102 | [ | ||
| 103 | self::TYPE => 'n', | ||
| 104 | self::OPERATOR => '', | ||
| 105 | self::COMMAND => '-220.905', | ||
| 106 | ], | ||
| 107 | [ | ||
| 108 |                         self::TYPE => '(', | ||
| 109 | self::OPERATOR => '', | ||
| 110 | self::COMMAND => '\\(E\\)', | ||
| 111 | ], | ||
| 112 | [ | ||
| 113 | self::TYPE => 'n', | ||
| 114 | self::OPERATOR => '', | ||
| 115 | self::COMMAND => '20.905', | ||
| 116 | ], | ||
| 117 | [ | ||
| 118 | self::TYPE => '<', | ||
| 119 | self::OPERATOR => '', | ||
| 120 | self::COMMAND => '20', | ||
| 121 | ], | ||
| 122 | ], | ||
| 123 | ], | ||
| 124 | [ | ||
| 125 | self::TYPE => '/', | ||
| 126 | self::OPERATOR => 'Tf', | ||
| 127 | self::COMMAND => 'R14 17.16', | ||
| 128 | ], | ||
| 129 | [ | ||
| 130 | self::TYPE => '<', | ||
| 131 | self::OPERATOR => 'Tj', | ||
| 132 | self::COMMAND => '20', | ||
| 133 | ], | ||
| 134 | [ | ||
| 135 | self::TYPE => '', | ||
| 136 | self::OPERATOR => 'Tm', | ||
| 137 | self::COMMAND => '0.999014 0 0 1 336.84 319.161', | ||
| 138 | ], | ||
| 139 | [ | ||
| 140 | self::TYPE => '', | ||
| 141 | self::OPERATOR => 'T*', | ||
| 142 | self::COMMAND => '', | ||
| 143 | ], | ||
| 144 | [ | ||
| 145 |                 self::TYPE => '(', | ||
| 146 | self::OPERATOR => 'Tj', | ||
| 147 | self::COMMAND => " \x00m", | ||
| 148 | ], | ||
| 149 | [ | ||
| 150 | self::TYPE => '/', | ||
| 151 | self::OPERATOR => 'Tf', | ||
| 152 | self::COMMAND => 'R14 20.04', | ||
| 153 | ], | ||
| 154 | ]; | ||
| 155 | |||
| 156 | $this->assertEquals($parts, $reference); | ||
| 157 | $this->assertEquals(172, $offset); | ||
| 158 | } | ||
| 294 |