On Mon, Sep 2, 2013 at 9:17 AM, Stas Malyshev <[email protected]>wrote:
> Hi!
>
> I've finally took some time to revive the skipping parameter RFC and
> patch. For those who doesn't remember what it is please see:
> https://wiki.php.net/rfc/skipparams
> TLDR version:
>
> The idea is to allow skipping parameters in function with optional
> arguments so that this:
> function create_query($where, $order_by, $join_type='INNER', $execute
> = false, $report_errors = true)
>
> can be called like this:
> create_query("deleted=0", "name", default, default,
> /*report_errors*/ true);
>
> Instead of trying to remember what the defaults are.
> The patch is here:
>
> https://github.com/php/php-src/pull/426
>
> Any comments or feedback on the RFCs and the code are welcome,
> especially pointing out the cases where it may not work (which means we
> need more phpt's there :)
>
I'm -1 on this proposal.
I think this doesn't really help readability. Right now you should
implement functions with many options with an $options array.
If we want to change something here, we should skip this step and go right
for named arguments. I think I'll give implementing them a try.
Nikita