Unverified Commit 1bf565ea authored by ksss's avatar ksss

Argument more strictly

parent c28a963b
...@@ -215,13 +215,15 @@ module Enumerable ...@@ -215,13 +215,15 @@ module Enumerable
# Returns the first element, or the first +n+ elements, of the enumerable. # Returns the first element, or the first +n+ elements, of the enumerable.
# If the enumerable is empty, the first form returns <code>nil</code>, and the # If the enumerable is empty, the first form returns <code>nil</code>, and the
# second form returns an empty array. # second form returns an empty array.
def first(n=NONE) def first(*args)
if n == NONE case args.length
when 0
self.each do |*val| self.each do |*val|
return val.__svalue return val.__svalue
end end
return nil return nil
else when 1
n = args[0]
raise TypeError, "no implicit conversion of #{n.class} into Integer" unless n.respond_to?(:to_int) raise TypeError, "no implicit conversion of #{n.class} into Integer" unless n.respond_to?(:to_int)
i = n.to_int i = n.to_int
raise ArgumentError, "attempt to take negative size" if i < 0 raise ArgumentError, "attempt to take negative size" if i < 0
...@@ -233,6 +235,8 @@ module Enumerable ...@@ -233,6 +235,8 @@ module Enumerable
break if i == 0 break if i == 0
end end
ary ary
else
raise ArgumentError, "wrong number of arguments (given #{args.length}, expected 0..1)"
end end
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment