Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:119592
MIME-Version: 1.0
References: <CABAhh3Ed1uwS8bY75Rz_4FkhU4Bq7Fx0TexnLgQQcvhygcTgvQ@mail.gmail.com>
 <CAAwdEzDH_NoRcNwWh1MULqNAHu6g0S0oi8j2XtgZOKWYA+fPwQ@mail.gmail.com>
 <CABAhh3HpcuHPBKwd+8_-_GFy2gVy4-s9_+6ytU2ijZQKF4RD=w@mail.gmail.com> <CAAwdEzBYFPtf8WBvkYm8gsOeANFC-pMN6NHQqLRUsTF7SgzLKw@mail.gmail.com>
In-Reply-To: <CAAwdEzBYFPtf8WBvkYm8gsOeANFC-pMN6NHQqLRUsTF7SgzLKw@mail.gmail.com>
Date: Tue, 21 Feb 2023 21:04:03 +0200
Message-ID: <CABAhh3EHjiS4saWL0c+R5cDQWR6nKkuhi9PvD6KKbQdU2omj3w@mail.gmail.com>
To: =?UTF-8?Q?Alexandru_P=C4=83tr=C4=83nescu?= <drealecs@gmail.com>
Cc: php internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000013385005f53a722e"
Subject: Re: [PHP-DEV] Class Re-implementation Mechanism
From: someniatko@gmail.com (someniatko)

--00000000000013385005f53a722e
Content-Type: text/plain; charset="UTF-8"

>
>
>> So it's mostly a syntactic sugar that allows defining an interface for
> the current class that is already implemented by the current class.
>

Yeah, while answering to you, I've actually came to the same conclusion.

>

> The interface name is the same as the class name and based on the context,
> you either use one or the other.
>

I think we should distinguish this thing from an interface though.

There are some places where this could get in conflict like (new
> ReflectionClass(A::class))->isInterface(). But also internally things might
> be difficult to sort out to know what to use one or the other.
>

Yeah, I also thought about reflection, exactly about this method. I believe
it's better not to lie, and make isInterface() return false. However,
another method should be added, e.g. isClassInterface() - which will return
true only in case of `class interface`, or, maybe a more general one like
isImplementable(), which will also return true for interfaces.


> To have a perfect syntax-sugar, why not also allow the interface name to
> be specified?
> Something like:
> final class interface(ServiceInterface) Service {
>     public function method1() {}
>     protected function method2() {}
> }
>
> that would generate:
> interface ServiceInterface {
>     public function method1();
> }
> final class Service implements ServiceInterface {
>     public function method1() {}
>     protected function method2() {}
> }
>

Well that goes against my initial concern of not introducing additional
entities to the system. However it indeed reduces the amount of code to
write and is more "transparent" type of syntaxic sugar, like the promoted
properties in constructors - this feature also translates to PHP code. If
my suggestion ever becomes an RFC, this way of implementing it may become a
separate vote I suppose.


Regards,
Illia / someniatko

--00000000000013385005f53a722e--