Add benchmarks, improve performance of Message::parse()

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## 2.2.1 under development
 
+- Enh #132: Add benchmarks, improve performance of `Message::parse()` (@samdark)
 - Bug #130: Updated `Message::parse()` to correctly support multiple placeholders (@technicated)
 - Chg #130, #133: Changed `Message::parse()` to conform to PSR-3 (@technicated, @vjik)
 

composer.json

@@ -36,6 +36,7 @@
     },
     "require-dev": {
         "maglnet/composer-require-checker": "^4.4",
+        "phpbench/phpbench": "^1.0",
         "phpunit/phpunit": "^9.6.23",
         "rector/rector": "^2.0.17",
         "roave/infection-static-analysis-plugin": "^1.25",
@@ -82,6 +83,7 @@
     },
     "scripts": {
         "test": "phpunit --testdox --no-interaction",
-        "test-watch": "phpunit-watcher watch"
+        "test-watch": "phpunit-watcher watch",
+        "bench": "phpbench run --report=default"
     }
 }

phpbench.json

@@ -0,0 +1,7 @@
+{
+    "runner.bootstrap": "vendor/autoload.php",
+    "runner.path": "tests/Benchmark",
+    "runner.revs": 1000,
+    "runner.iterations": 10,
+    "runner.retry_threshold": 2
+}

src/Message.php

@@ -206,6 +206,10 @@ private function parse(string|Stringable $message, array $context): string
     {
         $message = (string) $message;
 
+        if (!str_contains($message, '{')) {
+            return $message;
+        }
+
         /** @var string */
         return preg_replace_callback(
             '/\{([\w\.\\\\_]+)\}/',

tests/Benchmark/LoggerBench.php

@@ -0,0 +1,43 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Yiisoft\Log\Tests\Benchmark;
+
+use Psr\Log\LogLevel;
+use Yiisoft\Log\Logger;
+use Yiisoft\Log\Target;
+
+final class LoggerBench
+{
+    private Logger $logger;
+
+    public function __construct()
+    {
+        $this->logger = new Logger([new class () extends Target {
+            protected function export(): void
+            {
+                // noop
+            }
+        }]);
+    }
+
+    public function benchLogSimple(): void
+    {
+        $this->logger->log(LogLevel::INFO, 'simple message');
+    }
+
+    public function benchLogWithPlaceholder(): void
+    {
+        $this->logger->log(LogLevel::INFO, 'has {foo} placeholder', ['foo' => 'some']);
+    }
+
+    public function benchLogWithMultiplePlaceholders(): void
+    {
+        $this->logger->log(
+            LogLevel::INFO,
+            'Placeholder 1: {p1} - Placeholder 2: {p2}',
+            ['p1' => 'hello', 'p2' => 'world'],
+        );
+    }
+}

tests/Benchmark/MessageBench.php

@@ -0,0 +1,55 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Yiisoft\Log\Tests\Benchmark;
+
+use Psr\Log\LogLevel;
+use Yiisoft\Log\Message;
+
+final class MessageBench
+{
+    public function benchNoPlaceholder(): void
+    {
+        new Message(LogLevel::INFO, 'no placeholder', ['foo' => 'some']);
+    }
+
+    public function benchSinglePlaceholderScalar(): void
+    {
+        new Message(LogLevel::INFO, 'has {foo} placeholder', ['foo' => 'some']);
+    }
+
+    public function benchPlaceholderMissing(): void
+    {
+        new Message(LogLevel::INFO, 'has {foo} placeholder', []);
+    }
+
+    public function benchPlaceholderNull(): void
+    {
+        new Message(LogLevel::INFO, 'has "{foo}" placeholder', ['foo' => null]);
+    }
+
+    public function benchPlaceholderArray(): void
+    {
+        new Message(LogLevel::INFO, 'has "{foo}" placeholder', ['foo' => ['bar' => 7]]);
+    }
+
+    public function benchNestedPlaceholder(): void
+    {
+        new Message(LogLevel::INFO, 'has "{foo.bar}" placeholder', ['foo' => ['bar' => 7]]);
+    }
+
+    public function benchDeeplyNestedPlaceholder(): void
+    {
+        new Message(LogLevel::INFO, 'has "{foo.bar.baz}" placeholder', ['foo' => ['bar' => ['baz' => 7]]]);
+    }
+
+    public function benchMultiplePlaceholders(): void
+    {
+        new Message(
+            LogLevel::INFO,
+            'Placeholder 1: {p1} - Placeholder 2: {p2}',
+            ['p1' => 'hello', 'p2' => 'world'],
+        );
+    }
+}