PHPCompatibility /
PHPCompatibility
| Conditions | 39 |
| Paths | 39 |
| Total Lines | 143 |
| Lines | 7 |
| Ratio | 4.9 % |
| 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 |
||
| 212 | public function processNonString(File $phpcsFile, $stackPtr, $tokens) |
||
| 213 | { |
||
| 214 | $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true); |
||
| 215 | if ($nextNonEmpty === false) { |
||
| 216 | return; |
||
| 217 | } |
||
| 218 | |||
| 219 | /* |
||
| 220 | * Deal with anonymous classes - `class` before a reserved keyword is sometimes |
||
| 221 | * misidentified as `T_ANON_CLASS`. |
||
| 222 | * In PHPCS < 2.3.4 these were tokenized as T_CLASS no matter what. |
||
| 223 | */ |
||
| 224 | if ($tokens[$stackPtr]['type'] === 'T_ANON_CLASS' || $tokens[$stackPtr]['type'] === 'T_CLASS') { |
||
| 225 | $prevNonEmpty = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 1), null, true); |
||
| 226 | if ($prevNonEmpty !== false && $tokens[$prevNonEmpty]['type'] === 'T_NEW') { |
||
| 227 | return; |
||
| 228 | } |
||
| 229 | } |
||
| 230 | |||
| 231 | /* |
||
| 232 | * PHP 5.6 allows for use const and use function, but only if followed by the function/constant name. |
||
| 233 | * - `use function HelloWorld` => move to the next token (HelloWorld) to verify. |
||
| 234 | * - `use const HelloWorld` => move to the next token (HelloWorld) to verify. |
||
| 235 | */ |
||
| 236 | elseif ($tokens[$stackPtr]['type'] === 'T_USE' |
||
| 237 | && isset($this->validUseNames[strtolower($tokens[$nextNonEmpty]['content'])]) === true |
||
| 238 | ) { |
||
| 239 | $maybeUseNext = $phpcsFile->findNext(Tokens::$emptyTokens, ($nextNonEmpty + 1), null, true, null, true); |
||
| 240 | if ($maybeUseNext !== false && $this->isEndOfUseStatement($tokens[$maybeUseNext]) === false) { |
||
| 241 | $nextNonEmpty = $maybeUseNext; |
||
| 242 | } |
||
| 243 | } |
||
| 244 | |||
| 245 | /* |
||
| 246 | * Deal with visibility modifiers. |
||
| 247 | * - `use HelloWorld { sayHello as protected; }` => valid, bow out. |
||
| 248 | * - `use HelloWorld { sayHello as private myPrivateHello; }` => move to the next token to verify. |
||
| 249 | */ |
||
| 250 | elseif ($tokens[$stackPtr]['type'] === 'T_AS' |
||
| 251 | && isset($this->allowedModifiers[$tokens[$nextNonEmpty]['code']]) === true |
||
| 252 | && $phpcsFile->hasCondition($stackPtr, \T_USE) === true |
||
| 253 | ) { |
||
| 254 | $maybeUseNext = $phpcsFile->findNext(Tokens::$emptyTokens, ($nextNonEmpty + 1), null, true, null, true); |
||
| 255 | if ($maybeUseNext === false || $this->isEndOfUseStatement($tokens[$maybeUseNext]) === true) { |
||
| 256 | return; |
||
| 257 | } |
||
| 258 | |||
| 259 | $nextNonEmpty = $maybeUseNext; |
||
| 260 | } |
||
| 261 | |||
| 262 | /* |
||
| 263 | * Deal with foreach ( ... as list() ). |
||
| 264 | */ |
||
| 265 | elseif ($tokens[$stackPtr]['type'] === 'T_AS' |
||
| 266 | && isset($tokens[$stackPtr]['nested_parenthesis']) === true |
||
| 267 | && $tokens[$nextNonEmpty]['code'] === \T_LIST |
||
| 268 | ) { |
||
| 269 | $parentheses = array_reverse($tokens[$stackPtr]['nested_parenthesis'], true); |
||
| 270 | View Code Duplication | foreach ($parentheses as $open => $close) { |
|
|
|
|||
| 271 | if (isset($tokens[$open]['parenthesis_owner']) |
||
| 272 | && $tokens[$tokens[$open]['parenthesis_owner']]['code'] === \T_FOREACH |
||
| 273 | ) { |
||
| 274 | return; |
||
| 275 | } |
||
| 276 | } |
||
| 277 | } |
||
| 278 | |||
| 279 | /* |
||
| 280 | * Deal with functions declared to return by reference. |
||
| 281 | */ |
||
| 282 | elseif ($tokens[$stackPtr]['type'] === 'T_FUNCTION' |
||
| 283 | && $tokens[$nextNonEmpty]['type'] === 'T_BITWISE_AND' |
||
| 284 | ) { |
||
| 285 | $maybeUseNext = $phpcsFile->findNext(Tokens::$emptyTokens, ($nextNonEmpty + 1), null, true, null, true); |
||
| 286 | if ($maybeUseNext === false) { |
||
| 287 | // Live coding. |
||
| 288 | return; |
||
| 289 | } |
||
| 290 | |||
| 291 | $nextNonEmpty = $maybeUseNext; |
||
| 292 | } |
||
| 293 | |||
| 294 | /* |
||
| 295 | * Deal with nested namespaces. |
||
| 296 | */ |
||
| 297 | elseif ($tokens[$stackPtr]['type'] === 'T_NAMESPACE') { |
||
| 298 | if ($tokens[$stackPtr + 1]['code'] === \T_NS_SEPARATOR) { |
||
| 299 | // Not a namespace declaration, but use of, i.e. `namespace\someFunction();`. |
||
| 300 | return; |
||
| 301 | } |
||
| 302 | |||
| 303 | $endToken = $phpcsFile->findNext(array(\T_SEMICOLON, \T_OPEN_CURLY_BRACKET), ($stackPtr + 1), null, false, null, true); |
||
| 304 | $namespaceName = trim($phpcsFile->getTokensAsString(($stackPtr + 1), ($endToken - $stackPtr - 1))); |
||
| 305 | if (empty($namespaceName) === true) { |
||
| 306 | return; |
||
| 307 | } |
||
| 308 | |||
| 309 | $namespaceParts = explode('\\', $namespaceName); |
||
| 310 | foreach ($namespaceParts as $namespacePart) { |
||
| 311 | $partLc = strtolower($namespacePart); |
||
| 312 | if (isset($this->invalidNames[$partLc]) === false) { |
||
| 313 | continue; |
||
| 314 | } |
||
| 315 | |||
| 316 | // Find the token position of the part which matched. |
||
| 317 | for ($i = ($stackPtr + 1); $i < $endToken; $i++) { |
||
| 318 | if ($tokens[$i]['content'] === $namespacePart) { |
||
| 319 | $nextNonEmpty = $i; |
||
| 320 | break; |
||
| 321 | } |
||
| 322 | } |
||
| 323 | } |
||
| 324 | unset($i, $namespacePart, $partLc); |
||
| 325 | } |
||
| 326 | |||
| 327 | $nextContentLc = strtolower($tokens[$nextNonEmpty]['content']); |
||
| 328 | if (isset($this->invalidNames[$nextContentLc]) === false) { |
||
| 329 | return; |
||
| 330 | } |
||
| 331 | |||
| 332 | /* |
||
| 333 | * Deal with PHP 7 relaxing the rules. |
||
| 334 | * "As of PHP 7.0.0 these keywords are allowed as property, constant, and method names |
||
| 335 | * of classes, interfaces and traits, except that class may not be used as constant name." |
||
| 336 | */ |
||
| 337 | if ((($tokens[$stackPtr]['type'] === 'T_FUNCTION' |
||
| 338 | && $this->inClassScope($phpcsFile, $stackPtr, false) === true) |
||
| 339 | || ($tokens[$stackPtr]['type'] === 'T_CONST' |
||
| 340 | && $this->isClassConstant($phpcsFile, $stackPtr) === true |
||
| 341 | && $nextContentLc !== 'class')) |
||
| 342 | && $this->supportsBelow('5.6') === false |
||
| 343 | ) { |
||
| 344 | return; |
||
| 345 | } |
||
| 346 | |||
| 347 | if ($this->supportsAbove($this->invalidNames[$nextContentLc])) { |
||
| 348 | $data = array( |
||
| 349 | $tokens[$nextNonEmpty]['content'], |
||
| 350 | $this->invalidNames[$nextContentLc], |
||
| 351 | ); |
||
| 352 | $this->addError($phpcsFile, $stackPtr, $tokens[$nextNonEmpty]['content'], $data); |
||
| 353 | } |
||
| 354 | } |
||
| 355 | |||
| 443 |
Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.
You can also find more detailed suggestions in the “Code” section of your repository.