Update PrintfHelper.php

Make `public function getScanfPlaceholdersCount(string $format): ?int` returning the sscanf() vetted number of placeholders that give/return/assign conversions.

refs:

- #5591
- phpstan/phpstan#14567

Author: hakre

src/Rules/Functions/PrintfHelper.php

@@ -5,12 +5,16 @@
 use Nette\Utils\Strings;
 use PHPStan\DependencyInjection\AutowiredService;
 use PHPStan\Php\PhpVersion;
+use ValueError;
 use function array_filter;
 use function array_keys;
 use function count;
 use function in_array;
 use function max;
+use function restore_error_handler;
+use function set_error_handler;
 use function sprintf;
+use function sscanf;
 use function strlen;
 use const PREG_SET_ORDER;
 
@@ -26,24 +30,44 @@ public function __construct(private PhpVersion $phpVersion)
 
 	public function getPrintfPlaceholdersCount(string $format): ?int
 	{
-		return $this->getPlaceholdersCount(self::PRINTF_SPECIFIER_PATTERN, $format, false);
+		return $this->getPlaceholdersCount(self::PRINTF_SPECIFIER_PATTERN, $format);
 	}
 
 	/** @phpstan-return array<int, non-empty-list<PrintfPlaceholder>> parameter index => placeholders */
 	public function getPrintfPlaceholders(string $format): ?array
 	{
-		return $this->parsePlaceholders(self::PRINTF_SPECIFIER_PATTERN, $format, false);
+		return $this->parsePlaceholders(self::PRINTF_SPECIFIER_PATTERN, $format);
 	}
 
 	public function getScanfPlaceholdersCount(string $format): ?int
 	{
-		return $this->getPlaceholdersCount('(?<specifier>[cdDeEfinosuxX%s]|\[[^\]]+\])', $format, true);
+		$throws = $this->phpVersion->throwsValueErrorForInternalFunctions();
+		try {
+			if ($throws === false) {
+				set_error_handler(
+					static function ($s, $m) {
+						throw new ValueError($m, 0);
+					},
+				);
+			}
+			$result = sscanf('', '%*n' . $format);
+			if ($result === null) {
+				return null;
+			}
+			return count($result);
+		} catch (ValueError) {
+			return null;
+		} finally {
+			if ($throws === false) {
+				restore_error_handler();
+			}
+		}
 	}
 
 	/**
 	 * @phpstan-return array<int, non-empty-list<PrintfPlaceholder>>|null parameter index => placeholders
 	 */
-	private function parsePlaceholders(string $specifiersPattern, string $format, bool $isScanf): ?array
+	private function parsePlaceholders(string $specifiersPattern, string $format): ?array
 	{
 		$addSpecifier = '';
 		if ($this->phpVersion->supportsHhPrintfSpecifier()) {
@@ -72,10 +96,6 @@ private function parsePlaceholders(string $specifiersPattern, string $format, bo
 			$showValueSuffix = false;
 
 			if (isset($placeholder['width']) && $placeholder['width'] !== '') {
-				if ($isScanf) {
-					// In scanf, * means assignment suppression - skip this placeholder entirely
-					continue;
-				}
 				$parsedPlaceholders[] = new PrintfPlaceholder(
 					sprintf('"%s" (width)', $placeholder[0]),
 					$parameterIdx++,
@@ -136,9 +156,9 @@ private function getAcceptingTypeBySpecifier(string $specifier): string
 		return 'mixed';
 	}
 
-	private function getPlaceholdersCount(string $specifiersPattern, string $format, bool $isScanf): ?int
+	private function getPlaceholdersCount(string $specifiersPattern, string $format): ?int
 	{
-		$placeholdersMap = $this->parsePlaceholders($specifiersPattern, $format, $isScanf);
+		$placeholdersMap = $this->parsePlaceholders($specifiersPattern, $format);
 		if ($placeholdersMap === null) {
 			return null;
 		}