Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:122403
Precedence: bulk
MIME-Version: 1.0
Date: Sat, 17 Feb 2024 09:56:00 +0800
Message-ID: <CAKkgovSFUMwhdVERHHb-5Pw7Bm_hBcYbZqPH58JX1mvhW6d=rg@mail.gmail.com>
Subject: [PHP-DEV] RFC Proposal : Allows calling non-static public methods through the __callStatic magic method instead of throwing an error.
To: internals@lists.php.net
Content-Type: multipart/alternative; boundary="00000000000038175006118a2a42"
From: daddyofsky@gmail.com (=?UTF-8?B?7ZWY64qY7JWE67aA7KeA?=)

--00000000000038175006118a2a42
Content-Type: text/plain; charset="UTF-8"

Hi.
I'd like to propose an RFC, but I don't have the authority.
Below is my suggestion.
If you think this makes sense, please write an RFC for me.
Sincerely.

----------------------------------------------------------

===== Introduction =====
Allows calling non-static public methods through the __callStatic magic
method instead of throwing an error.

===== Proposal =====

From a conceptual perspective:
It goes without saying that calling a non-static method statically will
result in an error.
However, a confusing situation occurred when the __callStatic magic method
exists.
Non-public methods can be called, but public methods cannot.
This is the opposite of the method visibility policy.

From a practical point of view:
If you can call Non-static public methods through the __callStatic magic
method, you can write code like Laravel ORM more simply and tidy.

<code>
User::foo()->bar();
</code>

==== Before ====

<code>
class Foo
{
    protected static ?Foo $instance = null;

    public static function __callStatic($method, $args)
    {
        $instance = self::$instance ?? self::$instance = new static();
        return $instance->__call($method, $args);
    }

    public function __call($method, $args)
    {
        if (method_exists($this, $method)) {
            return $instance->$method(...$args);
        }

        return $this;
    }

    protected function bar()
    {
        echo __METHOD__ . '<br />';

        return $this;
    }

    protected function baz()
    {
        echo __METHOD__ . '<br />';

        return $this;
    }
}

Foo::bar()->baz();
(new Foo())->bar()->baz();
</code>

There is no error, but the concept of method visibility is broken.
All Non-public methods can be called at instance scope.

==== After ====

<code>
class Foo
{
    protected static ?Foo $instance = null;

    public static function __callStatic($method, $args)
    {
        $instance = self::$instance ?? self::$instance = new static();

        if (method_exists($instance, $method)) {
            return $instance->$method(...$args);
        }

        return $instance;
    }

    public function bar()
    {
        echo __METHOD__ . '<br />';

        return $this;
    }

    public function baz()
    {
        echo __METHOD__ . '<br />';

        return $this;
    }
}

Foo::bar()->baz();
(new Foo())->bar()->baz();
</code>

This is more tidy.
Only public methods are callable at instance scope.

--00000000000038175006118a2a42
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div>Hi.<br>I&#39;d like to propose an RFC, but I don&#39;=
t have the authority.<br>Below is my suggestion.<br>If you think this makes=
 sense, please write an RFC for me.<br></div>Sincerely.<br><div><div><br>--=
--------------------------------------------------------<br><br>=3D=3D=3D=
=3D=3D Introduction =3D=3D=3D=3D=3D<br>Allows calling non-static public met=
hods through the __callStatic magic method instead of throwing an error.<br=
><br>=3D=3D=3D=3D=3D Proposal =3D=3D=3D=3D=3D<br><br>From a conceptual pers=
pective:<br>It goes without saying that calling a non-static method statica=
lly will result in an error.<br>However, a confusing situation occurred whe=
n the __callStatic magic method exists.<br>Non-public methods can be called=
, but public methods cannot.<br>This is the opposite of the method visibili=
ty policy.<br><br>From a practical point of view:<br>If you can call Non-st=
atic public methods through the __callStatic magic method, you can write co=
de like Laravel ORM more simply and tidy.<br><br>&lt;code&gt;<br>	User::foo=
()-&gt;bar();<br>&lt;/code&gt;<br><br>=3D=3D=3D=3D Before =3D=3D=3D=3D<br><=
br>&lt;code&gt;<br>class Foo<br>{<br>=C2=A0 =C2=A0 protected static ?Foo $i=
nstance =3D null;<br><br>=C2=A0 =C2=A0 public static function __callStatic(=
$method, $args)<br>=C2=A0 =C2=A0 {<br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 $instance=
 =3D self::$instance ?? self::$instance =3D new static();<br>=C2=A0 =C2=A0 =
=C2=A0 =C2=A0 return $instance-&gt;__call($method, $args);<br>=C2=A0 =C2=A0=
 }<br><br>=C2=A0 =C2=A0 public function __call($method, $args)<br>=C2=A0 =
=C2=A0 {<br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 if (method_exists($this, $method)) =
{<br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 return $instance-&gt;$method=
(...$args);<br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 }<br><br>=C2=A0 =C2=A0 =C2=A0 =
=C2=A0 return $this;<br>=C2=A0 =C2=A0 }<br><br>=C2=A0 =C2=A0 protected func=
tion bar()<br>=C2=A0 =C2=A0 {<br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 echo __METHOD_=
_ . &#39;&lt;br /&gt;&#39;;<br><br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 return $this=
;<br>=C2=A0 =C2=A0 }<br><br>=C2=A0 =C2=A0 protected function baz()<br>=C2=
=A0 =C2=A0 {<br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 echo __METHOD__ . &#39;&lt;br /=
&gt;&#39;;<br><br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 return $this;<br>=C2=A0 =C2=
=A0 }<br>}<br><br>Foo::bar()-&gt;baz();<br>(new Foo())-&gt;bar()-&gt;baz();=
<br>&lt;/code&gt;<br><br>There is no error, but the concept of method visib=
ility is broken.<br>All Non-public methods can be called at instance scope.=
<br><br>=3D=3D=3D=3D After =3D=3D=3D=3D<br><br>&lt;code&gt;<br>class Foo<br=
>{<br>=C2=A0 =C2=A0 protected static ?Foo $instance =3D null;<br><br>=C2=A0=
 =C2=A0 public static function __callStatic($method, $args)<br>=C2=A0 =C2=
=A0 {<br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 $instance =3D self::$instance ?? self:=
:$instance =3D new static();<br><br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 if (method_=
exists($instance, $method)) {<br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
return $instance-&gt;$method(...$args);<br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 }<br=
><br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 return $instance;<br>=C2=A0 =C2=A0 }<br><b=
r>=C2=A0 =C2=A0 public function bar()<br>=C2=A0 =C2=A0 {<br>=C2=A0 =C2=A0 =
=C2=A0 =C2=A0 echo __METHOD__ . &#39;&lt;br /&gt;&#39;;<br><br>=C2=A0 =C2=
=A0 =C2=A0 =C2=A0 return $this;<br>=C2=A0 =C2=A0 }<br><br>=C2=A0 =C2=A0 pub=
lic function baz()<br>=C2=A0 =C2=A0 {<br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 echo _=
_METHOD__ . &#39;&lt;br /&gt;&#39;;<br><br>=C2=A0 =C2=A0 =C2=A0 =C2=A0 retu=
rn $this;<br>=C2=A0 =C2=A0 }<br>}<br><br>Foo::bar()-&gt;baz();<br>(new Foo(=
))-&gt;bar()-&gt;baz();<br>&lt;/code&gt;<br><br>This is more tidy.<br>Only =
public methods are callable at instance scope.<br></div></div></div>

--00000000000038175006118a2a42--