Forum PHP.pl

Przeglądają bibliotekę od sms api trafiłem na taką klasę
https://github.com/smsapi/smsapi-php-client...SendSmssBag.php
Uproszczony fragment kodu:

[PHP] pobierz, plaintext 
/**
 * @api
 * @property string $from
 * @property string $message
.....
 * @property string $timeRestriction
 */
#[\AllowDynamicProperties]
class SendSmssBag
{
    /** @var string */
    public $encoding = 'utf-8';
 
    public static function withMessage(array $receivers, string $message): self
    {
        $bag = new self();
        $bag->to = $receivers;
        $bag->message = $message;
 
        return $bag;
    }
 
}
 
[PHP] pobierz, plaintext 

I tak się zastanawiam jaki jest tu sens użycia AllowDynamicProperties i "definiowanie" właściwości w @property?
Przecież skoro lista tych właściwości jest znana to było można je zdefiniować tak jak to zrobili z $encoding.

Czy w tym jest jakiś geniusz którego ja nie dostrzegam? Jakiś wzorzec którego nie znam / nie rozumiem?

Macie pomysł "po co tak robić"?

Dynamiczne wlasciwosci sa deprecated od php8.2
Wiec to pewnie jakis stary kod, ktory przeniesli na php8 wiec musieli dodac tag AllowDynamicProperties by sie nie wykrzaczal
https://www.php.net/manual/en/class.allowdy...cproperties.php

oczywiscie mogli to poprawic porzadnie, no ale po co (IMG:style_emoticons/default/wink.gif)

edit:
O prosze, nawet jeden commit sprzed dwoch lat mowi dokladnie to co ja wlasnie powiedzialem
"Fixes dynamic property deprecations"
(IMG:style_emoticons/default/biggrin.gif)

Tu masz wszystkie zmiany z tego nieszczesnego commita
https://github.com/smsapi/smsapi-php-client...a4d152a811d4e11
poszli po najmniejszej linii oprou i poprostu pododawali wszedzie #[\AllowDynamicProperties]

A już myślałem że coś genialnego, a nie przykrywanie g... papierkiem...

Tylko to jeszcze bardziej zastanawiające po co, poprawienie tego zajęło by jakieś 10 sekund.

Cóż czyli jednak lepiej nie szukać przejawów geniuszu na siłę ;p