Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:114626
MIME-Version: 1.0
References: <CAOwTYAtc_xD6n=efmamkAy4ro4KqPKxq18-zXsNriY0280FY2g@mail.gmail.com>
In-Reply-To: <CAOwTYAtc_xD6n=efmamkAy4ro4KqPKxq18-zXsNriY0280FY2g@mail.gmail.com>
Date: Wed, 26 May 2021 22:27:45 +0100
Message-ID: <CA+p1FaYfFUE13xOszP48ySeE-MV414GgX+=N=9XjxbTxRjKrNg@mail.gmail.com>
To: Joe Watkins <krakjoe@php.net>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000ef4bb105c34250ca"
Subject: Re: [PHP-DEV] Consensus Gathering: is_initialized
From: davidgebler@gmail.com (David Gebler)

--000000000000ef4bb105c34250ca
Content-Type: text/plain; charset="UTF-8"

On Wed, May 26, 2021 at 11:14 AM Joe Watkins <krakjoe@php.net> wrote:

> Hi internals,
>
> In response to: https://bugs.php.net/bug.php?id=78480
>
> I implemented: https://github.com/php/php-src/pull/7029
>
> Not absolutely convinced that it's a good idea, I asked Nikita to review,
> and he's unconvinced also and suggested a discussion should be started.
>
>
I would just like to second (or third, or fourth) the notion that the very
possibility of accessing uninitialized state is a bug in the user's code,
not a problem for which the language should provide a workaround. If a
property is required (i.e. not nullable) it should absolutely and always be
initialized in the constructor.

I do appreciate PHP being how it is, there might be "valid" use cases for
is_initialized() but I also think it will encourage bad practice.

Regards
Dave

--000000000000ef4bb105c34250ca--