X-Mailer: iPhone Mail (21E236) From: saki@sakiot.com (Saki Takamachi) --Apple-Mail-A22DD69B-27D9-4E5F-8F81-626FE5FF68E8 Content-Type: text/html; charset=utf-8 Content-Transfer-Encoding: quoted-printable

Hi a= ll,

On Tue, Apr 23= , 2024 at 11:26=E2=80=AFAM Stephen Reay <php-lists@koalephant.com> wrote:

Sent from my iPhon= e

On 23 Apr 2024, at 18:= 21, Lynn <kjarli@gm= ail.com> wrote:

<= div dir=3D"ltr">=EF=BB=BF

On Tue, Apr 23= , 2024 at 10:21=E2=80=AFAM Bilge <bilge@scriptfusion.com> wrote:
On 21/04/2024 14:00, Saki Takamachi w= rote:
> Hi internals,
>
> Recently I've been working on an RFC regarding object support for BCMat= h. While working on that, I learned of the following RFC:
> https://wiki.php.net/rfc/namespaces_in_b= undled_extensions
>
> If we follow this RFC, is it reasonable to place subclasses of PDO unde= r the namespace "PDO=E2=80=9D?
>
> e.g.
> ```
> PdoMysql =3D> PDO\Mysql
> PdoPgsql =3D> PDO\Pgsql
> PdoSqlite =3D> PDO\Sqlite
> PdoOdbc =3D> PDO\Odbc
> PdoDblib =3D> PDO\Dblib
> PdoFirebird =3D> PDO\Firebird
> ```
>
> We'll probably get a BC Break if try to fix this after 8.4 is released,= so before it's released is last chance to fix this safely.
>
> If Tim's RFC under discussion is passed, the namespace will be "Pdo" in= stead of "PDO=E2=80=9D.
> https://wiki.php.net/rfc/class-naming-acronyms<= br> >
> I would appreciate hearing your opinions.
>
> Regards,
>
> Saki

Hi Saki,

Consider that adding a namespace does not/should not change the class
name. That is, `MyClass` once namespaced becomes `MyNamespace\MyClass`.
= Ergo, `PdoMysql` becomes `Pdo\PdoMysql`. The class name should still
make sense and be a "strong name" (without conflict) once imported.

To state it more concretely, I believe it is normal and correct to
include 1-3 namespace components within the class name itself, in order
= to create such a "strong name". As a more concrete example of this,
consider `HttpClient`, `FtpClient` and `SoapClient`. Far too often, we
see user libraries (incorrectly) namespace these as `Http\Client`,
`Ftp\Client` and `Soap\Client` (or similar) where the leaf name just
becomes `Client`. "Client", by itself is a meaningless moniker, but that is all we see once the name is imported, notwithstanding importing
multiple of these clients in one file causes conflicts that now need to
= be resolved with local aliases. In general, I believe aliasing to be an
= anti-pattern that points to a failure to create strong names and thus
should be avoided by including some of the namespace portion in the
class name to make the class name more meaningful. Once imported, we do
= not see the namespace portion within the body of the file any more;
`HttpClient` and `FtpClient` make much more sense by themselves, whether or not they would otherwise conflict.

Kind regards,
Bilge

The code base I work in has 25 cla= sses that are called "Line". They have namespaces like `App\Model\Invoice\Li= ne`, this is cumbersome to work with so I would also prefer something like `= Pdo\PdoMysql`, even if it's not likely to conflict with a name such as Mysql= .

I don't think it's appropriate to claim that a n= amespace like MyLib\HTTP\Client is categorically "wrong".

The argument that "Client" is meaningless becomes pretty moot when yo= u realise that you can import a *namespace* and use it relatively, if you so= wish:

```

import MyLib\HTTP;

$a =3D new HTTP\Client(...);

```

I'm not claiming that any particular import pattern is more "cor= rect" (and I think it's a bad idea to suggest that other usage patterns are "= incorrect") but adding repetitive prefixes kind of defeats the purpose of na= mespaces.

You may as well just go back to MyLib_HTT= P_Client, and ignore that namespaces exist.

In your own codebase you should do what works best for you= . When it comes to vendor classes I don't want to have to scroll through a l= ist of 20 identical classes to find the one in the right namespace. It requi= res extra development and review effort to figure out if the right class cal= led "Client" or "Factory" is used. For reference, I have 12 "Response", 12 "= Client", 20 "Factory", and 20 "TestCase" classes in this project, of wh= ich most are vendors.

Using partially imp= orted namespaces and aliases causes inconsistencies that makes it harder to f= ind things. Nobody here is advocating for MyLibHttpClient class names either= . Even in an IDE like Intellij with a project the size I'm working with it's= hard to quickly get the right classes, especially in legacy code where some= classes live in the global namespace.

First, there are two things I want to make clear= :

1. RFC: Class Naming [1] clearly defines that cla= ss names should be in camel case, excluding acronyms, so currently `PDO` can= not be `Pdo'. However, if the currently voting RFC: Casing of acronyms= in class and method names [2] passes, acronyms will also need to be camel c= ase, so we'll use `Pdo`.

2. For example, `HttpClien= t` becomes `Http\Client` if follow the guidelines laid out in RFC: Namespace= s in bundled PHP extensions [3]. These rules are clearly stated in the RFC.<= /div>

[1] https://wiki.php.net/rfc/class-naming

[2] https://wiki.php.net/rfc/class-nam= ing-acronyms

[3] https://wiki.php.net/rfc/namespaces_in_bundled= _extensions

Regards,

S= aki

= --Apple-Mail-A22DD69B-27D9-4E5F-8F81-626FE5FF68E8--