Project

General

Profile

Actions

Feature #16296

open

Alternative behavior for `...` in method body if `...` is not in method definition

Added by Dan0042 (Daniel DeLorme) about 5 years ago. Updated about 5 years ago.

Status:
Open
Assignee:
-
Target version:
-
[ruby-core:95705]

Description

In #16253 we settled on a syntax where the remainder arguments captured via ... in the method definition can be forwarded via ... in the method body. I think that was the correct decision.

But I can't forget about the use case presented by zverok (and in #15049) where the method definition is used to specify mandatory and default parameters and then forward all of them to another method. I've also experienced that same use case in my code. Using the current syntax we would need to do this:

def get(path:, accept: :json, headers: {}, ...)
  _request(method: :get, path: path, accept: accept, headers: headers, ...)
end

def post(path:, body:, accept: :json, headers: {}, ...)
  _request(method: :post, path: path, body: body, accept: accept, headers: headers, ...)
end

Which feels pointlessly repetitive to me. So I was thinking that maybe if ... is not present in the method definition, then in the method body ... could take on the meaning of "all arguments of the method". Then the code would look like this:

def get(path:, accept: :json, headers: {}, **opts)
  _request(method: :get, ...)
end

def post(path:, body:, accept: :json, headers: {}, **opts)
  _request(method: :post, ...)
end

In those examples (no positional parameters) it would also allow Hash[...] or {}.replace(...) to get the hash of all keyword arguments.

Pro: it allows a new useful and powerful behavior
Con: some may consider it 'unclean' to change the behavior of ... based on the method definition


Related issues 1 (0 open1 closed)

Related to Ruby master - Feature #16253: Shorthand "forward everything" syntaxClosedActions
Actions

Also available in: Atom PDF

Like0
Like0Like0Like0