RegexEscaper::getFirstBounduaryPosition() - Code Metrics - matecat/whole-text-finder - Measure and Improve Code Quality continuously with Scrutinizer

RegexEscaper::getFirstBounduaryPosition() A
last analyzed 2025-02-14 11:09 UTC

↳ Parent: RegexEscaper

Complexity

Conditions	3
Paths	3

Size

Total Lines	9
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	4
c	0
b	0
f	0
nc	3
nop	1
dl	0
loc	9
rs	10

<?php

namespace Matecat\Finder\Helper;

class RegexEscaper
{
    /**
     * @param string $needle
     *
     * @return string
     */
    public static function escapeWholeTextPattern($needle)
    {
        $escapedNeedle = self::escapeRegularPattern($needle);
        $splittedNeedle = Strings::split($escapedNeedle);

        $final = '';

        foreach ($splittedNeedle as $index => $letter) {
            if ($index === self::getFirstBounduaryPosition($splittedNeedle)) {
                $final .= "\\b";
            }

            $final .= $letter;

            if ($index === self::getLastBounduaryPosition($splittedNeedle)) {
                $final .= "\\b";
            }
        }

        return $final;
    }

    /**
     * @param array $splittedNeedle
     *
     * @return int
     */
    private static function getFirstBounduaryPosition($splittedNeedle)
    {
        for ($i=0; $i < count($splittedNeedle); $i++) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
            if (self::isBoundary($splittedNeedle[$i])) {
                return $i;
            }
        }

        return -1;
    }

    /**
     * @param array $splittedNeedle
     *
     * @return int
     */
    private static function getLastBounduaryPosition($splittedNeedle)
    {
        for ($i=(count($splittedNeedle)-1); $i >= 0; $i--) {
            if (self::isBoundary($splittedNeedle[$i])) {
                return $i;
            }
        }

        return -1;
    }

    /**
     * @param string $letter
     *
     * @return bool
     */
    private static function isBoundary($letter)
    {
        return (preg_match("/[\w]/iu", $letter) > 0) ? true : false;
    }

    /**
     * @param string $needle
     *
     * @return string
     */
    public static function escapeRegularPattern($needle)
    {
        $needle = preg_quote($needle);

        return str_replace("/", "\/", $needle);
    }
}


1			<?php
2
3			namespace Matecat\Finder\Helper;
4
5			class RegexEscaper
6			{
7			/**
8			* @param string $needle
9			*
10			* @return string
11			*/
12			public static function escapeWholeTextPattern($needle)
13			{
14			$escapedNeedle = self::escapeRegularPattern($needle);
15			$splittedNeedle = Strings::split($escapedNeedle);
16
17			$final = '';
18
19			foreach ($splittedNeedle as $index => $letter) {
20			if ($index === self::getFirstBounduaryPosition($splittedNeedle)) {
21			$final .= "\\b";
22			}
23
24			$final .= $letter;
25
26			if ($index === self::getLastBounduaryPosition($splittedNeedle)) {
27			$final .= "\\b";
28			}
29			}
30
31			return $final;
32			}
33
34			/**
35			* @param array $splittedNeedle
36			*
37			* @return int
38			*/
39			private static function getFirstBounduaryPosition($splittedNeedle)
40			{
41			for ($i=0; $i < count($splittedNeedle); $i++) {
			0 ignored issues – show Performance Best Practice introduced 2023-01-04 18:04 UTC by Report Bug Copy Issue Report It seems like you are calling the size function `count()` as part of the test condition. You might want to compute the size beforehand, and not on each iteration. If the size of the collection does not change during the iteration, it is generally a good practice to compute it beforehand, and not on each iteration: for ($i=0; $i<count($array); $i++) { // calls count() on each iteration } // Better for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once } Loading history...
42			if (self::isBoundary($splittedNeedle[$i])) {
43			return $i;
44			}
45			}
46
47			return -1;
48			}
49
50			/**
51			* @param array $splittedNeedle
52			*
53			* @return int
54			*/
55			private static function getLastBounduaryPosition($splittedNeedle)
56			{
57			for ($i=(count($splittedNeedle)-1); $i >= 0; $i--) {
58			if (self::isBoundary($splittedNeedle[$i])) {
59			return $i;
60			}
61			}
62
63			return -1;
64			}
65
66			/**
67			* @param string $letter
68			*
69			* @return bool
70			*/
71			private static function isBoundary($letter)
72			{
73			return (preg_match("/[\w]/iu", $letter) > 0) ? true : false;
74			}
75
76			/**
77			* @param string $needle
78			*
79			* @return string
80			*/
81			public static function escapeRegularPattern($needle)
82			{
83			$needle = preg_quote($needle);
84
85			return str_replace("/", "\/", $needle);
86			}
87			}
88

matecat / whole-text-finder

RegexEscaper::getFirstBounduaryPosition() A last analyzed 2025-02-14 11:09 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

RegexEscaper::getFirstBounduaryPosition() A
last analyzed 2025-02-14 11:09 UTC