Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:101711
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain jenningstechnology.com designates 209.85.218.47 as permitted sender)
MIME-Version: 1.0
References: <517F0F7B-CD01-4F35-9021-8336B7BA06E1@cschneid.com> <620452cc-09eb-ec95-bf8d-e1cddee988f1@fleshgrinder.com>
In-Reply-To: <620452cc-09eb-ec95-bf8d-e1cddee988f1@fleshgrinder.com>
Date: Fri, 26 Jan 2018 22:44:10 +0000
Message-ID: <CADHGybnN4BhjVECE=QQLD06i-BaV=KA7XkvV_SgLjutZnwBPqA@mail.gmail.com>
To: internals@lists.php.net
Content-Type: multipart/alternative; boundary="001a113dd9622a64040563b5a314"
Subject: Re: [PHP-DEV] Shorthand proposal for associative arrays
From: richard@jenningstechnology.com (Richard Jennings)

--001a113dd9622a64040563b5a314
Content-Type: text/plain; charset="UTF-8"

In my opinion this suggestion reduces readability and increases cognative
burden and as such I hope it is not included. Regards

On Fri, 26 Jan 2018 at 21:52, Fleshgrinder <php@fleshgrinder.com> wrote:

> On 1/26/2018 7:16 PM, Christian Schneider wrote:
> > Hi there,
> > I have a proposal for a shorthand notation of associative arrays
> borrowed from another language:
> >       :$foo
> > would be equivalent to
> >       'foo' => $foo
> > and would work with array, list or []
> >
> > Motivation behind it, maybe someone else finds more good uses:
> >
> > 1) Emulating named parameters with associative arrays like
> >       html::img([ 'src' => $src, 'alt' => $alt ]);
> >    could be written as
> >       html::img([ :$src, :$alt ]);
> >    which encourages consistent naming of variables and parameters
> >
> > 2) Simplifying list destructuring with non-integer keys, example taking
> from
> http://php.net/manual/en/migration71.new-features.php#migration71.new-features.support-for-keys-in-list
> >       foreach ($data as ["id" => $id, "name" => $name]) {
> >    becomes
> >       foreach ($data as [ :$id, :$name ]) {
> >    which reduces redundancy.
> >
> > I implemented a minimal patch (2 lines are added to the parser) to
> implement this which you can find at
> >       https://cschneid.com/php/php7_2/assoc_array_shorthand.patch
> >
> > What do you think, is this worth an RFC? I hope I didn't miss an
> existing one :-)
> >
> > Regards,
> > - Chris
> >
> >
>
> Hi Chris!
>
> I really like this proposal. `compact` is cumbersome to use, or lets
> say, almost impossible without an intelligent IDE. However, what is more
> important is the fact that is would allow for a very readable and usable
> approach for destructuring of associative arrays (as illustrated by your
> example) which is very, very neat. The change is also extremely minimal
> which speaks for it.
>
> I would support you in writing up an RFC for this (if desired).
>
> --
> Richard "Fleshgrinder" Fussenegger
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
> --
Richard Jennings
Jennings Technology Limited
Company Number: 09827512
VAT Number: 224 8864 84

--001a113dd9622a64040563b5a314--