Merge pull request #6926 from shyim/fix/performance-parser

feat: improve performance of sql parser

Author: morozov

src/SQL/Parser.php

@@ -7,14 +7,9 @@
 use Doctrine\DBAL\SQL\Parser\Visitor;
 
 use function array_merge;
-use function assert;
-use function current;
 use function implode;
-use function key;
-use function next;
 use function preg_last_error;
 use function preg_match;
-use function reset;
 use function sprintf;
 use function strlen;
 
@@ -45,6 +40,7 @@ final class Parser
     private const OTHER                = '[^' . self::SPECIAL_CHARS . ']+';
 
     private string $sqlPattern;
+    private string $tokenPattern;
 
     public function __construct(bool $mySQLStringEscaping)
     {
@@ -69,7 +65,12 @@ public function __construct(bool $mySQLStringEscaping)
             self::OTHER,
         ]);
 
-        $this->sqlPattern = sprintf('(%s)', implode('|', $patterns));
+        $this->sqlPattern   = sprintf('(%s)', implode('|', $patterns));
+        $this->tokenPattern = '~\\G'
+            . '(?P<named>' . self::NAMED_PARAMETER . ')'
+            . '|(?P<positional>' . self::POSITIONAL_PARAMETER . ')'
+            . '|(?P<other>' . $this->sqlPattern . '|' . self::SPECIAL . ')'
+            . '~s';
     }
 
     /**
@@ -79,40 +80,26 @@ public function __construct(bool $mySQLStringEscaping)
      */
     public function parse(string $sql, Visitor $visitor): void
     {
-        /** @var array<string,callable> $patterns */
-        $patterns = [
-            self::NAMED_PARAMETER => static function (string $sql) use ($visitor): void {
-                $visitor->acceptNamedParameter($sql);
-            },
-            self::POSITIONAL_PARAMETER => static function (string $sql) use ($visitor): void {
-                $visitor->acceptPositionalParameter($sql);
-            },
-            $this->sqlPattern => static function (string $sql) use ($visitor): void {
-                $visitor->acceptOther($sql);
-            },
-            self::SPECIAL => static function (string $sql) use ($visitor): void {
-                $visitor->acceptOther($sql);
-            },
-        ];
-
         $offset = 0;
-
-        while (($handler = current($patterns)) !== false) {
-            if (preg_match('~\G' . key($patterns) . '~s', $sql, $matches, 0, $offset) === 1) {
-                $handler($matches[0]);
-                reset($patterns);
-
-                $offset += strlen($matches[0]);
+        $length = strlen($sql);
+        while ($offset < $length) {
+            if (preg_match($this->tokenPattern, $sql, $matches, 0, $offset) === 1) {
+                $match = $matches[0];
+                if ($matches['named'] !== '') {
+                    $visitor->acceptNamedParameter($match);
+                } elseif ($matches['positional'] !== '') {
+                    $visitor->acceptPositionalParameter($match);
+                } else {
+                    $visitor->acceptOther($match);
+                }
+
+                $offset += strlen($match);
             } elseif (preg_last_error() !== PREG_NO_ERROR) {
                 // @codeCoverageIgnoreStart
                 throw RegularExpressionError::new();
                 // @codeCoverageIgnoreEnd
-            } else {
-                next($patterns);
             }
         }
-
-        assert($offset === strlen($sql));
     }
 
     private function getMySQLStringLiteralPattern(string $delimiter): string