class RuboCop::Cop::Style::NumberedParametersLimit
hash.each { use_only_hash_value_as_numbered_parameter(_2) }
array.each { use_array_element_as_numbered_parameter(_1) }
# good
use_multiple_numbered_parameters { _1.call(_2, _3, _4) }
# bad
@example Max: 1 (default)
parameter but this maximum can be configured by setting ‘Max`.
The cop defaults to registering an offense if there is more than 1 numbered
cryptic and hard to read.
single block. Having too many numbered parameters can make code too
Detects use of an excessive amount of numbered parameters in a
def max_count
def max_count max = cop_config.fetch('Max', DEFAULT_MAX_VALUE) # Ruby does not allow more than 9 numbered parameters [max, 9].min end
def numbered_parameter_nodes(node)
def numbered_parameter_nodes(node) node.each_descendant(:lvar).select do |lvar_node| lvar_node.source.match?(NUMBERED_PARAMETER_PATTERN) end end
def on_numblock(node)
def on_numblock(node) param_count = numbered_parameter_nodes(node).uniq.count return if param_count <= max_count parameter = max_count > 1 ? 'parameters' : 'parameter' message = format(MSG, max: max_count, parameter: parameter, count: param_count) add_offense(node, message: message) { self.max = param_count } end