| Conditions | 79 |
| Paths | > 20000 |
| Total Lines | 399 |
| Code Lines | 184 |
| Lines | 0 |
| Ratio | 0 % |
| Tests | 245 |
| CRAP Score | 79 |
| 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 |
||
| 17 | 263 | public function parse() |
|
| 18 | { |
||
| 19 | 263 | $this->matchSetextLines(); |
|
| 20 | |||
| 21 | 263 | $codeFence = null; |
|
| 22 | 263 | $codeIndent = 4; |
|
| 23 | 263 | $codeTag = null; |
|
| 24 | 263 | $lineIsEmpty = true; |
|
| 25 | 263 | $lists = []; |
|
| 26 | 263 | $listsCnt = 0; |
|
| 27 | 263 | $newContext = false; |
|
| 28 | 263 | $quotes = []; |
|
| 29 | 263 | $quotesCnt = 0; |
|
| 30 | 263 | $textBoundary = 0; |
|
| 31 | |||
| 32 | 263 | $regexp = '/^(?:(?=[-*+\\d \\t>`~#_])((?: {0,3}> ?)+)?([ \\t]+)?(\\* *\\* *\\*[* ]*$|- *- *-[- ]*$|_ *_ *_[_ ]*$|=+$)?((?:[-*+]|\\d+\\.)[ \\t]+(?=\\S))?[ \\t]*(#{1,6}[ \\t]+|```+[^`\\n]*$|~~~+[^~\\n]*$)?)?/m'; |
|
| 33 | 263 | preg_match_all($regexp, $this->text, $matches, PREG_OFFSET_CAPTURE | PREG_SET_ORDER); |
|
| 34 | |||
| 35 | 263 | foreach ($matches as $m) |
|
| 36 | { |
||
| 37 | 263 | $matchPos = $m[0][1]; |
|
| 38 | 263 | $matchLen = strlen($m[0][0]); |
|
| 39 | 263 | $ignoreLen = 0; |
|
| 40 | 263 | $quoteDepth = 0; |
|
| 41 | |||
| 42 | // If the last line was empty then this is not a continuation, and vice-versa |
||
| 43 | 263 | $continuation = !$lineIsEmpty; |
|
| 44 | |||
| 45 | // Capture the position of the end of the line and determine whether the line is empty |
||
| 46 | 263 | $lfPos = $this->text->indexOf("\n", $matchPos); |
|
| 47 | 263 | $lineIsEmpty = ($lfPos === $matchPos + $matchLen && empty($m[3][0]) && empty($m[4][0]) && empty($m[5][0])); |
|
| 48 | |||
| 49 | // If the line is empty and it's the first empty line then we break current paragraph. |
||
| 50 | 263 | $breakParagraph = ($lineIsEmpty && $continuation); |
|
| 51 | |||
| 52 | // Count quote marks |
||
| 53 | 263 | if (!empty($m[1][0])) |
|
| 54 | 263 | { |
|
| 55 | 31 | $quoteDepth = substr_count($m[1][0], '>'); |
|
| 56 | 31 | $ignoreLen = strlen($m[1][0]); |
|
| 57 | 31 | if (isset($codeTag) && $codeTag->hasAttribute('quoteDepth')) |
|
| 58 | 31 | { |
|
| 59 | 4 | $quoteDepth = min($quoteDepth, $codeTag->getAttribute('quoteDepth')); |
|
| 60 | 4 | $ignoreLen = $this->computeQuoteIgnoreLen($m[1][0], $quoteDepth); |
|
| 61 | 4 | } |
|
| 62 | |||
| 63 | // Overwrite quote markup |
||
| 64 | 31 | $this->text->overwrite($matchPos, $ignoreLen); |
|
| 65 | 31 | } |
|
| 66 | |||
| 67 | // Close supernumerary quotes |
||
| 68 | 263 | if ($quoteDepth < $quotesCnt && !$continuation) |
|
| 69 | 263 | { |
|
| 70 | 30 | $newContext = true; |
|
| 71 | |||
| 72 | do |
||
| 73 | { |
||
| 74 | 30 | $this->parser->addEndTag('QUOTE', $textBoundary, 0) |
|
|
1 ignored issue
–
show
|
|||
| 75 | 30 | ->pairWith(array_pop($quotes)); |
|
| 76 | } |
||
| 77 | 30 | while ($quoteDepth < --$quotesCnt); |
|
| 78 | 30 | } |
|
| 79 | |||
| 80 | // Open new quotes |
||
| 81 | 263 | if ($quoteDepth > $quotesCnt && !$lineIsEmpty) |
|
| 82 | 263 | { |
|
| 83 | 30 | $newContext = true; |
|
| 84 | |||
| 85 | do |
||
| 86 | { |
||
| 87 | 30 | $tag = $this->parser->addStartTag('QUOTE', $matchPos, 0, $quotesCnt - 999); |
|
| 88 | 30 | $quotes[] = $tag; |
|
| 89 | } |
||
| 90 | 30 | while ($quoteDepth > ++$quotesCnt); |
|
| 91 | 30 | } |
|
| 92 | |||
| 93 | // Compute the width of the indentation |
||
| 94 | 263 | $indentWidth = 0; |
|
| 95 | 263 | $indentPos = 0; |
|
| 96 | 263 | if (!empty($m[2][0]) && !$codeFence) |
|
| 97 | 263 | { |
|
| 98 | 39 | $indentStr = $m[2][0]; |
|
| 99 | 39 | $indentLen = strlen($indentStr); |
|
| 100 | do |
||
| 101 | { |
||
| 102 | 39 | if ($indentStr[$indentPos] === ' ') |
|
| 103 | 39 | { |
|
| 104 | 37 | ++$indentWidth; |
|
| 105 | 37 | } |
|
| 106 | else |
||
| 107 | { |
||
| 108 | 4 | $indentWidth = ($indentWidth + 4) & ~3; |
|
| 109 | } |
||
| 110 | } |
||
| 111 | 39 | while (++$indentPos < $indentLen && $indentWidth < $codeIndent); |
|
| 112 | 39 | } |
|
| 113 | |||
| 114 | // Test whether we're out of a code block |
||
| 115 | 263 | if (isset($codeTag) && !$codeFence && $indentWidth < $codeIndent && !$lineIsEmpty) |
|
| 116 | 263 | { |
|
| 117 | 18 | $newContext = true; |
|
| 118 | 18 | } |
|
| 119 | |||
| 120 | if ($newContext) |
||
| 121 | 263 | { |
|
| 122 | 45 | $newContext = false; |
|
| 123 | |||
| 124 | // Close the code block if applicable |
||
| 125 | 45 | if (isset($codeTag)) |
|
| 126 | 45 | { |
|
| 127 | 18 | if ($textBoundary > $codeTag->getPos()) |
|
| 128 | 18 | { |
|
| 129 | // Overwrite the whole block |
||
| 130 | 16 | $this->text->overwrite($codeTag->getPos(), $textBoundary - $codeTag->getPos()); |
|
| 131 | |||
| 132 | 16 | $endTag = $this->parser->addEndTag('CODE', $textBoundary, 0, -1); |
|
| 133 | 16 | $endTag->pairWith($codeTag); |
|
| 134 | 16 | } |
|
| 135 | else |
||
| 136 | { |
||
| 137 | // The code block is empty |
||
| 138 | 2 | $codeTag->invalidate(); |
|
| 139 | } |
||
| 140 | |||
| 141 | 18 | $codeTag = null; |
|
| 142 | 18 | $codeFence = null; |
|
| 143 | 18 | } |
|
| 144 | |||
| 145 | // Close all the lists |
||
| 146 | 45 | foreach ($lists as $list) |
|
| 147 | { |
||
| 148 | 2 | $this->closeList($list, $textBoundary); |
|
|
1 ignored issue
–
show
|
|||
| 149 | 45 | } |
|
| 150 | 45 | $lists = []; |
|
| 151 | 45 | $listsCnt = 0; |
|
| 152 | |||
| 153 | // Mark the block boundary |
||
| 154 | if ($matchPos) |
||
| 155 | 45 | { |
|
| 156 | 45 | $this->text->markBoundary($matchPos - 1); |
|
| 157 | 45 | } |
|
| 158 | 45 | } |
|
| 159 | |||
| 160 | 263 | if ($indentWidth >= $codeIndent) |
|
| 161 | 263 | { |
|
| 162 | 19 | if (isset($codeTag) || !$continuation) |
|
| 163 | 19 | { |
|
| 164 | // Adjust the amount of text being ignored |
||
| 165 | 18 | $ignoreLen += $indentPos; |
|
| 166 | |||
| 167 | 18 | if (!isset($codeTag)) |
|
| 168 | 18 | { |
|
| 169 | // Create code block |
||
| 170 | 18 | $codeTag = $this->parser->addStartTag('CODE', $matchPos + $ignoreLen, 0, -999); |
|
| 171 | 18 | } |
|
| 172 | |||
| 173 | // Clear the captures to prevent any further processing |
||
| 174 | 18 | $m = []; |
|
| 175 | 18 | } |
|
| 176 | 19 | } |
|
| 177 | else |
||
| 178 | { |
||
| 179 | 263 | $hasListItem = !empty($m[4][0]); |
|
| 180 | |||
| 181 | 263 | if (!$indentWidth && !$continuation && !$hasListItem) |
|
| 182 | 263 | { |
|
| 183 | // Start of a new context |
||
| 184 | 263 | $listIndex = -1; |
|
| 185 | 263 | } |
|
| 186 | 262 | elseif ($continuation && !$hasListItem) |
|
| 187 | { |
||
| 188 | // Continuation of current list item or paragraph |
||
| 189 | 262 | $listIndex = $listsCnt - 1; |
|
| 190 | 262 | } |
|
| 191 | 30 | elseif (!$listsCnt) |
|
| 192 | { |
||
| 193 | // We're not inside of a list already, we can start one if there's a list item |
||
| 194 | // and it's either not in continuation of a paragraph or immediately after a |
||
| 195 | // block |
||
| 196 | 30 | if ($hasListItem && (!$continuation || $this->text->charAt($matchPos - 1) === "\x17")) |
|
| 197 | 30 | { |
|
| 198 | // Start of a new list |
||
| 199 | 27 | $listIndex = 0; |
|
| 200 | 27 | } |
|
| 201 | else |
||
| 202 | { |
||
| 203 | // We're in a normal paragraph |
||
| 204 | 4 | $listIndex = -1; |
|
| 205 | } |
||
| 206 | 30 | } |
|
| 207 | else |
||
| 208 | { |
||
| 209 | // We're inside of a list but we need to compute the depth |
||
| 210 | 20 | $listIndex = 0; |
|
| 211 | 20 | while ($listIndex < $listsCnt && $indentWidth > $lists[$listIndex]['maxIndent']) |
|
| 212 | { |
||
| 213 | 6 | ++$listIndex; |
|
| 214 | 6 | } |
|
| 215 | } |
||
| 216 | |||
| 217 | // Close deeper lists |
||
| 218 | 263 | while ($listIndex < $listsCnt - 1) |
|
| 219 | { |
||
| 220 | 26 | $this->closeList(array_pop($lists), $textBoundary); |
|
|
1 ignored issue
–
show
|
|||
| 221 | 26 | --$listsCnt; |
|
| 222 | 26 | } |
|
| 223 | |||
| 224 | // If there's no list item at current index, we'll need to either create one or |
||
| 225 | // drop down to previous index, in which case we have to adjust maxIndent |
||
| 226 | 263 | if ($listIndex === $listsCnt && !$hasListItem) |
|
| 227 | 263 | { |
|
| 228 | 1 | --$listIndex; |
|
| 229 | 1 | } |
|
| 230 | |||
| 231 | 263 | if ($hasListItem && $listIndex >= 0) |
|
| 232 | 263 | { |
|
| 233 | 27 | $breakParagraph = true; |
|
| 234 | |||
| 235 | // Compute the position and amount of text consumed by the item tag |
||
| 236 | 27 | $tagPos = $matchPos + $ignoreLen + $indentPos; |
|
| 237 | 27 | $tagLen = strlen($m[4][0]); |
|
| 238 | |||
| 239 | // Create a LI tag that consumes its markup |
||
| 240 | 27 | $itemTag = $this->parser->addStartTag('LI', $tagPos, $tagLen); |
|
| 241 | |||
| 242 | // Overwrite the markup |
||
| 243 | 27 | $this->text->overwrite($tagPos, $tagLen); |
|
| 244 | |||
| 245 | // If the list index is within current lists count it means this is not a new |
||
| 246 | // list and we have to close the last item. Otherwise, it's a new list that we |
||
| 247 | // have to create |
||
| 248 | 27 | if ($listIndex < $listsCnt) |
|
| 249 | 27 | { |
|
| 250 | 20 | $this->parser->addEndTag('LI', $textBoundary, 0) |
|
|
1 ignored issue
–
show
|
|||
| 251 | 20 | ->pairWith($lists[$listIndex]['itemTag']); |
|
| 252 | |||
| 253 | // Record the item in the list |
||
| 254 | 20 | $lists[$listIndex]['itemTag'] = $itemTag; |
|
| 255 | 20 | $lists[$listIndex]['itemTags'][] = $itemTag; |
|
| 256 | 20 | } |
|
| 257 | else |
||
| 258 | { |
||
| 259 | 27 | ++$listsCnt; |
|
| 260 | |||
| 261 | if ($listIndex) |
||
| 262 | 27 | { |
|
| 263 | 5 | $minIndent = $lists[$listIndex - 1]['maxIndent'] + 1; |
|
| 264 | 5 | $maxIndent = max($minIndent, $listIndex * 4); |
|
| 265 | 5 | } |
|
| 266 | else |
||
| 267 | { |
||
| 268 | 27 | $minIndent = 0; |
|
| 269 | 27 | $maxIndent = $indentWidth; |
|
| 270 | } |
||
| 271 | |||
| 272 | // Create a 0-width LIST tag right before the item tag LI |
||
| 273 | 27 | $listTag = $this->parser->addStartTag('LIST', $tagPos, 0); |
|
| 274 | |||
| 275 | // Test whether the list item ends with a dot, as in "1." |
||
| 276 | 27 | if (strpos($m[4][0], '.') !== false) |
|
| 277 | 27 | { |
|
| 278 | 10 | $listTag->setAttribute('type', 'decimal'); |
|
| 279 | |||
| 280 | 10 | $start = (int) $m[4][0]; |
|
| 281 | 10 | if ($start !== 1) |
|
| 282 | 10 | { |
|
| 283 | 2 | $listTag->setAttribute('start', $start); |
|
| 284 | 2 | } |
|
| 285 | 10 | } |
|
| 286 | |||
| 287 | // Record the new list depth |
||
| 288 | 27 | $lists[] = [ |
|
| 289 | 27 | 'listTag' => $listTag, |
|
| 290 | 27 | 'itemTag' => $itemTag, |
|
| 291 | 27 | 'itemTags' => [$itemTag], |
|
| 292 | 27 | 'minIndent' => $minIndent, |
|
| 293 | 27 | 'maxIndent' => $maxIndent, |
|
| 294 | 'tight' => true |
||
| 295 | 27 | ]; |
|
| 296 | } |
||
| 297 | 27 | } |
|
| 298 | |||
| 299 | // If we're in a list, on a non-empty line preceded with a blank line... |
||
| 300 | 263 | if ($listsCnt && !$continuation && !$lineIsEmpty) |
|
| 301 | 263 | { |
|
| 302 | // ...and this is not the first item of the list... |
||
| 303 | 23 | if (count($lists[0]['itemTags']) > 1 || !$hasListItem) |
|
| 304 | 23 | { |
|
| 305 | // ...every list that is currently open becomes loose |
||
| 306 | 5 | foreach ($lists as &$list) |
|
| 307 | { |
||
| 308 | 5 | $list['tight'] = false; |
|
| 309 | 5 | } |
|
| 310 | 5 | unset($list); |
|
| 311 | 5 | } |
|
| 312 | 23 | } |
|
| 313 | |||
| 314 | 263 | $codeIndent = ($listsCnt + 1) * 4; |
|
| 315 | } |
||
| 316 | |||
| 317 | 263 | if (isset($m[5])) |
|
| 318 | 263 | { |
|
| 319 | // Headers |
||
| 320 | 36 | if ($m[5][0][0] === '#') |
|
| 321 | 36 | { |
|
| 322 | 17 | $startLen = strlen($m[5][0]); |
|
| 323 | 17 | $startPos = $matchPos + $matchLen - $startLen; |
|
| 324 | 17 | $endLen = $this->getAtxHeaderEndTagLen($matchPos + $matchLen, $lfPos); |
|
|
1 ignored issue
–
show
|
|||
| 325 | 17 | $endPos = $lfPos - $endLen; |
|
| 326 | |||
| 327 | 17 | $this->parser->addTagPair('H' . strspn($m[5][0], '#', 0, 6), $startPos, $startLen, $endPos, $endLen); |
|
| 328 | |||
| 329 | // Mark the start and the end of the header as boundaries |
||
| 330 | 17 | $this->text->markBoundary($startPos); |
|
| 331 | 17 | $this->text->markBoundary($lfPos); |
|
| 332 | |||
| 333 | if ($continuation) |
||
| 334 | 17 | { |
|
| 335 | 2 | $breakParagraph = true; |
|
| 336 | 2 | } |
|
| 337 | 17 | } |
|
| 338 | // Code fence |
||
| 339 | 19 | elseif ($m[5][0][0] === '`' || $m[5][0][0] === '~') |
|
| 340 | { |
||
| 341 | 19 | $tagPos = $matchPos + $ignoreLen; |
|
| 342 | 19 | $tagLen = $lfPos - $tagPos; |
|
| 343 | |||
| 344 | 19 | if (isset($codeTag) && $m[5][0] === $codeFence) |
|
| 345 | 19 | { |
|
| 346 | 19 | $endTag = $this->parser->addEndTag('CODE', $tagPos, $tagLen, -1); |
|
| 347 | 19 | $endTag->pairWith($codeTag); |
|
| 348 | |||
| 349 | 19 | $this->parser->addIgnoreTag($textBoundary, $tagPos - $textBoundary); |
|
|
1 ignored issue
–
show
|
|||
| 350 | |||
| 351 | // Overwrite the whole block |
||
| 352 | 19 | $this->text->overwrite($codeTag->getPos(), $tagPos + $tagLen - $codeTag->getPos()); |
|
| 353 | 19 | $codeTag = null; |
|
| 354 | 19 | $codeFence = null; |
|
| 355 | 19 | } |
|
| 356 | 19 | elseif (!isset($codeTag)) |
|
| 357 | { |
||
| 358 | // Create code block |
||
| 359 | 19 | $codeTag = $this->parser->addStartTag('CODE', $tagPos, $tagLen); |
|
| 360 | 19 | $codeFence = substr($m[5][0], 0, strspn($m[5][0], '`~')); |
|
| 361 | 19 | $codeTag->setAttribute('quoteDepth', $quoteDepth); |
|
| 362 | |||
| 363 | // Ignore the next character, which should be a newline |
||
| 364 | 19 | $this->parser->addIgnoreTag($tagPos + $tagLen, 1); |
|
| 365 | |||
| 366 | // Add the language if present, e.g. ```php |
||
| 367 | 19 | $lang = trim(trim($m[5][0], '`~')); |
|
| 368 | 19 | if ($lang !== '') |
|
| 369 | 19 | { |
|
| 370 | 4 | $codeTag->setAttribute('lang', $lang); |
|
| 371 | 4 | } |
|
| 372 | 19 | } |
|
| 373 | 19 | } |
|
| 374 | 36 | } |
|
| 375 | 263 | elseif (!empty($m[3][0]) && !$listsCnt && $this->text->charAt($matchPos + $matchLen) !== "\x17") |
|
| 376 | { |
||
| 377 | // Horizontal rule |
||
| 378 | 9 | $this->parser->addSelfClosingTag('HR', $matchPos + $ignoreLen, $matchLen - $ignoreLen); |
|
| 379 | 9 | $breakParagraph = true; |
|
| 380 | |||
| 381 | // Mark the end of the line as a boundary |
||
| 382 | 9 | $this->text->markBoundary($lfPos); |
|
|
1 ignored issue
–
show
|
|||
| 383 | 9 | } |
|
| 384 | 263 | elseif (isset($this->setextLines[$lfPos]) && $this->setextLines[$lfPos]['quoteDepth'] === $quoteDepth && !$lineIsEmpty && !$listsCnt && !isset($codeTag)) |
|
| 385 | { |
||
| 386 | // Setext-style header |
||
| 387 | 11 | $this->parser->addTagPair( |
|
| 388 | 11 | $this->setextLines[$lfPos]['tagName'], |
|
| 389 | 11 | $matchPos + $ignoreLen, |
|
| 390 | 11 | 0, |
|
| 391 | 11 | $this->setextLines[$lfPos]['endPos'], |
|
| 392 | 11 | $this->setextLines[$lfPos]['endLen'] |
|
| 393 | 11 | ); |
|
| 394 | |||
| 395 | // Mark the end of the Setext line |
||
| 396 | 11 | $this->text->markBoundary($this->setextLines[$lfPos]['endPos'] + $this->setextLines[$lfPos]['endLen']); |
|
| 397 | 11 | } |
|
| 398 | |||
| 399 | if ($breakParagraph) |
||
| 400 | 263 | { |
|
| 401 | 262 | $this->parser->addParagraphBreak($textBoundary); |
|
|
1 ignored issue
–
show
|
|||
| 402 | 262 | $this->text->markBoundary($textBoundary); |
|
|
1 ignored issue
–
show
|
|||
| 403 | 262 | } |
|
| 404 | |||
| 405 | 263 | if (!$lineIsEmpty) |
|
| 406 | 263 | { |
|
| 407 | 263 | $textBoundary = $lfPos; |
|
| 408 | 263 | } |
|
| 409 | |||
| 410 | if ($ignoreLen) |
||
| 411 | 263 | { |
|
| 412 | 45 | $this->parser->addIgnoreTag($matchPos, $ignoreLen, 1000); |
|
| 413 | 45 | } |
|
| 414 | 263 | } |
|
| 415 | 263 | } |
|
| 416 | |||
| 511 | } |
If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check:
If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue.