Codor\Sniffs\Files\FunctionNameContainsAndOrSniff

Complexity

Total Complexity

8

Size/Duplication

Total Lines	76
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	register()	0	4	1
A	process()	0	12	2
A	containsKeywords()	0	10	3
A	contains()	0	4	1
A	getErrorMessage()	0	4	1

<?php

namespace Codor\Sniffs\Files;

use PHP_CodeSniffer_Sniff;
use PHP_CodeSniffer_File;

class FunctionNameContainsAndOrSniff implements PHP_CodeSniffer_Sniff
{

    /**
     * The forbidden strings this sniff looks for.
     * @var array
     */
    protected $keywords = ['And', '_and', 'Or', '_or'];

    /**
     * Returns the token types that this sniff is interested in.
     * @return array
     */
    public function register()
    {
        return [T_FUNCTION];
    }

    /**
     * Processes the tokens that this sniff is interested in.
     *
     * @param PHP_CodeSniffer_File $phpcsFile The file where the token was found.
     * @param integer              $stackPtr  The position in the stack where
     *                                    the token was found.
     * @return void
     */
    public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
    {
        $tokens = $phpcsFile->getTokens();
        $functionNameToken = $tokens[$stackPtr + 2];
        $functionName = $functionNameToken['content'];
        
        if (! $this->containsKeywords($functionName)) {
            return;
        }

        $phpcsFile->addError($this->getErrorMessage(), $stackPtr);
    }

    /**
     * Determines if the provided $string contains any of the
     * strings in the $this->keywords property.
     * @param  string $string The string to check.
     * @return boolean
     */
    protected function containsKeywords($string)
    {
        foreach ($this->keywords as $keyword) {
            if ($this->contains($keyword, $string)) {
                return true;
            }
        }

        return false;
    }

    /**
     * Checks if the $haystack contains the $needle.
     * @param  string $needle   Needle string.
     * @param  string $haystack Haystack string.
     * @return boolean
     */
    protected function contains($needle, $haystack)
    {
        return strpos($haystack, $needle) !== false;
    }

    /**
     * Gets the error message for this sniff.
     * @return string
     */
    protected function getErrorMessage()
    {
        return "Your function contains 'and' or 'or' which indicates it might be doing more than one thing.";
    }
}