Feature #7121

Extending the use of `require'

Added by Morgan Jones over 1 year ago. Updated over 1 year ago.

[ruby-core:47869]
Status:Assigned
Priority:Normal
Assignee:Yukihiro Matsumoto
Category:-
Target version:next minor

Description

=begin
I was playing with Ruby tonight and thought up an interesting idea to make (({require})) a bit better, so you can load multiple files sequentially using one method call.

Currently, (({require})) supports one argument, and throws a (({TypeError})) if you pass an array:

irb(main):001:0> require %w(json yaml)
TypeError: can't convert Array into String

However, there's a way to patch Kernel that makes it respond to multiple objects passed in an (({Array})).

module Kernel
@@require = method :require

def require *args
args.flatten!
args.collect! do |a|
raise ArgumentError.new "arguments to `require' must be strings or symbols" unless a.isa?(String) || a.isa?(Symbol)
@@require.call a.to_s
end

 args.length == 1 ? args.first : args

end
end

The new behavior doesn't actually require the modification of any code that calls (({require})) (pretty much anything, really), and new code can take advantage of the new functionality instantly.

irb> require %w(json yaml)
=> [true, false]
irb> require :pp
=> false
irb> require 'rails'
=> true
irb> require %w(json yaml), :pp, 'rails'
=> [true, false, false, true]
=end

Thanks for considering this.

History

#1 Updated by Tom Wardrop over 1 year ago

I personally don't mind your suggestion. It makes sense to me. I can't think of any potential negative side effects. On that note, I think the return value should always be a boolean, instead of a boolean OR an array of booleans depending on the input. #require should return true if any of the listed files were loaded, or false if none of them were. If you need to determine the loaded state of each file (the lesser common use case I'd imagine), then you should resort to looping over multiple calls to #require.

#2 Updated by Yusuke Endoh over 1 year ago

  • Status changed from Open to Assigned
  • Assignee set to Yukihiro Matsumoto
  • Target version set to next minor

#3 Updated by Thomas Sawyer over 1 year ago

It's ugly, as it makes code harder to read. Please no.

#4 Updated by Ilya Vorontsov over 1 year ago

Just an alternative idea. What about using Regexp as an alternative to String?
require /.*_helper/

Also available in: Atom PDF