Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:124119
Content-Type: text/plain;
	charset=us-ascii
Precedence: bulk
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3774.600.62\))
Subject: Re: [PHP-DEV] [RFC] [Discussion] Add bcdivmod to BCMath
In-Reply-To: <7f1ba41b-ace8-4af2-8efc-1412c47e34cb@app.fastmail.com>
Date: Mon, 1 Jul 2024 08:43:08 +0900
Cc: php internals <internals@lists.php.net>
Content-Transfer-Encoding: quoted-printable
Message-ID: <8BB31FFD-F44D-4259-A0AB-398B08770F74@sakiot.com>
References: <F709ADD3-502E-418B-82D3-34A681EBB7A0@sakiot.com>
 <668168A7.4040005@adviesenzo.nl>
 <e4a3fba1-e441-4f35-ac17-462f5f9c3fb9@app.fastmail.com>
 <3B23E039-D90F-4F84-9ECE-C6A9A3E145AF@sakiot.com>
 <7f1ba41b-ace8-4af2-8efc-1412c47e34cb@app.fastmail.com>
To: Larry Garfield <larry@garfieldtech.com>
From: saki@sakiot.com (Saki Takamachi)

Hi Larry,

> I agree an associative array is the second-worst option.  An inout =
by-ref argument is the absolute worst.
>=20
> Normally my default position is that when in doubt, make it a =
structured object with properly defined properties, and screw whatever =
micro-performance hit it is, you won't notice.  99% of the time I =
believe that is the correct approach.
>=20
> I can see the argument that this is the other 1%, since it's just two =
values, which will basically always be wanted separately but both wanted =
(meaning divmod(...)->divsor is kinda pointless), and their order should =
be fairly self-evident.
>=20
> However, in that case I would urge that both the RFC and the resulting =
documentation *always* use examples that return into a `[$foo, $bar]` =
destructured list.  Don't even suggest that people should use 0 and 1 =
indexes.  It's a tuple for deconstruction, that's it, if you're using it =
some other way you're probably wrong.  Politely ignore that it's even =
possible, lest we lead people down the dark path.
>=20
> (Which means removing the current index-using example and just keeping =
the pizza example.)
>=20
> --Larry Garfield =20

Thanks, agree. I rewrote the non-pizza example.

Regards,

Saki=