PHPStan报assign.readOnlyProperty错误：只读属性已赋值问题排查

开发者在使用PHPStan分析App\Payload类时，遇到了只读属性已赋值的错误提示，推测错误和validationValue方法调用有关，但无法明确具体原因，以下是问题解析：

代码示例 #

<?php

namespace App;

use Illuminate\Support\Str;
use stdClass;

class Payload
{
    private(set) readonly array $validation;
    
    private array $ids = [];


    public function __construct(private readonly stdClass $payload)
    {
        $this->parseValidation();
    }

    private function parseValidation(): void
    {
        $validations = [];

        foreach ($this->payload->validation as $validation) {

            $validations[] = [
                'id' => Str::orderedUuid()->toString(),
                'field_id' => $this->ids[$validation->field_id],
                'rule' => $validation->rule,
                'value' => $this->validationValue($validation->value),
                'message' => $validation->message,
            ];
        }

        $this->validation = $validations;
    }

    private function validationValue(mixed $value): mixed
    {
        if (is_null($value)) {
            return null;
        }

        return $this->ids[$value] ?? $value;
    }
}

PHPStan 分析结果 #

$ phpstan analyse app/Payload.php
 1/1 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

 ------ -----------------------------------------------------------------
  Line   Payload.php
 ------ -----------------------------------------------------------------
  35     Readonly property App\Payload::$validation is already assigned.
         🪪  assign.readOnlyProperty
 ------ -----------------------------------------------------------------

问题原因与解决方法 #

错误关联点澄清 #

这个错误和validationValue方法完全无关，是PHPStan对PHP 8.1+只读属性语法的检测逻辑偏差导致的。

核心问题 #

你使用了private(set) readonly array $validation;的组合语法，意图让属性仅能在类内部赋值，但PHPStan的静态分析逻辑出现了误判：

• 当在构造函数内调用的方法中给只读属性赋值时，PHPStan可能无法正确追踪赋值路径，误以为属性被多次赋值；
• 另一种可能是private(set)和readonly的组合在部分PHPStan版本中存在解析问题——虽然PHP官方支持该组合，但静态分析工具的兼容逻辑没跟上。

可行修复方案 #

有两种简单的修复方式：

1. 把赋值逻辑内联到构造函数中
   跳过中间的parseValidation方法，直接在构造函数里完成$validation的赋值，让PHPStan能清晰看到属性仅被赋值一次：

   public function __construct(private readonly stdClass $payload)
   {
       $validations = [];
       foreach ($this->payload->validation as $validation) {
           $validations[] = [
               'id' => Str::orderedUuid()->toString(),
               'field_id' => $this->ids[$validation->field_id],
               'rule' => $validation->rule,
               'value' => $this->validationValue($validation->value),
               'message' => $validation->message,
           ];
       }
       $this->validation = $validations;
   }
   

   之后可以直接删除parseValidation方法。

2. 去掉冗余的private(set)声明
   因为readonly属性本身就只能在构造函数内赋值，类内部其他方法也无法修改它，所以private(set)在这里是多余的，修改属性声明为：

   private readonly array $validation;
   

   这种方式下，PHPStan能正确识别赋值逻辑，不会再触发误报。

额外潜在问题提醒 #

代码中$this->ids[$validation->field_id]直接访问数组键，若$validation->field_id不存在于$ids数组中，会抛出Undefined array key错误，建议添加空值处理：

'field_id' => $this->ids[$validation->field_id] ?? null,

内容的提问来源于stack exchange，提问作者Sebastian Sulinski