magician 0.2.0 → 0.2.1

data/lib/magician/array.rb

@@ -57,13 +57,12 @@ class Array
 	#
 	# @return [Numeric] the median of the numbers in the array
 	def median
-		nums = numerics
-		return nil if nums.empty?
-		sorted = nums.sort
+		sorted = numerics.sort
+		return nil if sorted.empty?
 		if sorted.length.odd?
-			nums[nums.length/2]
+			sorted[sorted.length/2]
 		else
-			(nums[nums.length/2-1] + nums[nums.length/2]) / 2.0
+			(sorted[sorted.length/2-1] + sorted[sorted.length/2]) / 2.0
 		end
 	end
 
@@ -87,8 +86,7 @@ class Array
 	# @return [Hash] a hash table of the occurrences of each item from the original
 	# array
 	def occurences
-		occurences = {}
-		each { |item| occurences[item] = 0 }
+		occurences = Hash.new(0)
 		each { |item| occurences[item] += 1 }
 		occurences
 	end

data/lib/magician/integer.rb

@@ -11,21 +11,13 @@ class Integer
 		return nil if self == 0
 		return [1] if abs == 1
 		#first half
-		factors = []
-		for i in 1..abs
-			if abs % i == 0
-				if i < abs/i
-					factors << i
-				else break
-				end
+		factors = [1]
+		2.upto((abs/2).to_i) do |i|
+			if abs%i == 0
+				factors << i 
 			end
 		end
-		#second half
-		factors_old = factors.dup
-		until factors_old.length == 0
-			factors << abs/factors_old.pop
-		end
-		factors
+		factors << abs
 	end
 
 	# Gets the factorial of the integer, which is equivalent to the product of all
@@ -46,10 +38,18 @@ class Integer
 	# @return [Boolean] true if the integer is prime
 	def prime?
 		return false if self <= 1
-		for i in 2..Math.sqrt(self)
+		(2..Math.sqrt(self)).each do |i|
 			return false if self % i == 0
 		end
 		true
 	end
 
+	# Returns true if the number is pandigital. That is the number contains the
+	# each of the digits 1 through 9 exactly once.
+	#
+	# @return [Boolean] true if pandigital
+	def pandigital?
+		to_s.split(//).sort!.join.to_i == 123456789
+	end
+
 end

data/lib/magician/numeric.rb

@@ -22,8 +22,10 @@ class Numeric
 	#
 	# @return [Integer] substring of the number (using []), converted to an
 	# Integer
+	# 
+	# @deprecated because it's useless and wrong
 	def digits selection
 		to_s[selection].to_i
 	end
 
-end
+end

data/lib/magician/shortcuts.rb

@@ -5,3 +5,6 @@ PI = Math::PI
 
 # Alias Math::E to E
 E = Math::E
+
+# the Golden ratio
+GOLDEN_RATIO = (1 + Math.sqrt(5))/2

data/magician.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = "magician"
-  s.version = "0.2.0"
+  s.version = "0.2.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Nicolas McCurdy"]
-  s.date = "2012-12-03"
+  s.date = "2012-12-10"
   s.description = "A suite of handy methods for doing calculations in irb."
   s.email = "thenickperson@gmail.com"
   s.extra_rdoc_files = [

data/spec/array_spec.rb

@@ -47,7 +47,7 @@ describe Array do
 		[].median.should == nil
 		[4].median.should == 4.0
 		['string', 4].median.should == 4.0
-		[2, 1, 3, 5, 4].median.should == 3.0
+		[2, 1, 5, 4, 3].median.should == 3.0
 		[1, 2, 3, 4].median.should == 2.5
 	end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: magician
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-03 00:00:00.000000000 Z
+date: 2012-12-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -154,7 +154,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -956776375334094677
+      hash: -1827626556693680297
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: