How can I avoid repetition of array shape declaration for method return value?

Question

If, for example, I have a class like this:

class Foo
{
   /**
    * @var array<string, array{name: string, age: int}>
    */
   private array $things;

   /**
    * @return array
    */
   public function getThings(): array
   {
      return $this->things;
   }
}

Then phpstan will give me something along the lines of Method Foo::getThings() return type has no value type specified in iterable type array.

Of course, I can resolve this by adding the array shape definition to the @return, but given that I've already defined this on the property, is there a way to avoid duplication here that I'm missing?

Can Vural · Accepted Answer · 2022-10-08 21:36:31Z

9

No. PHPStan does not read the method body like that to understand what it returns.

You can use local type aliases to reduce the duplication:

/** @phpstan-type Things array<string, array{name: string, age: int}> */
class Foo
{
   /**
    * @var Things
    */
   private array $things;

   /**
    * @return Things
    */
   public function getThings(): array
   {
      return $this->things;
   }
}

answered Oct 8, 2022 at 21:36

Can Vural

2,1321 gold badge28 silver badges51 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Mark G Over a year ago

awesome, thank you! As an aside question/opinion (for better understanding): do you think it would be a reasonable addition to phpstan? Or, could you foresee any problems with handling this sort of thing automatically?

Can Vural Over a year ago

@MarkG Funny enough, someone asked the same question recently. And the answer is no: github.com/phpstan/phpstan/discussions/… As it's mentioned, maybe Rector already can handle this automatically.

Collectives™ on Stack Overflow

How can I avoid repetition of array shape declaration for method return value?

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related