nested_hash 0.1.4 → 0.2.0

data/Changelog

@@ -1,3 +1,7 @@
+2012-05-25  Guillermo Alvarez <guillermo@cientifico.net>
+
+	* nested_hash.rb (get/set): Implement get and set methods.
+
 2012-05-24  Guillermo Alvarez <guillermo@cientifico.net>
 
 	* nested_hash.rb (process): [bug] When there is only one level, the key was

data/lib/nested_hash.rb

@@ -3,31 +3,60 @@ require "nested_hash/version"
 class NestedHash < Hash
 
   def initialize(hash = {})
-    hash.each do |key,v|
-      key = sanitize_long_key(key)
-
-      if is_valid_key?(key)
-        process(key,v)
-      elsif copy_invalid_keys?
-        copy(key,v)
-      end
+    hash.each do |key,value|
+      set(key,value)
     end
     post_process
   end
 
-  protected
+  def set(key,value)
+    key = sanitize_long_key(key)
+    if is_valid_key?(key)
+      process(key,value)
+    elsif copy_invalid_keys?
+      copy(key,value)
+    end
+  end
 
-  def post_process
-    compact_arrays
+  def get(key)
+    key = sanitize_long_key(key)
   end
 
-  def compact_arrays(element = self)
+  def compact_arrays!(element = self)
     element.each do |item|
       item.compact! if item.is_a?(Array)
-      compact_arrays(item) if item.is_a?(Array) || item.is_a?(Hash)
+      compact_arrays!(item) if item.is_a?(Array) || item.is_a?(Hash)
     end
   end
 
+
+  def set(key,value, write = true)
+    keys = key.split(".").map{|k| sanitize_key(k)}
+
+    previous = keys.shift
+    top = (keys.inject(self) do |memo,key|
+      memo[previous] ||= ( is_for_array?(key) ? [] : {} )
+      memo = memo[previous]
+      previous = key
+      memo
+    end)
+
+    write ? top[previous] = value : top[previous]
+  rescue => e
+    handle_exception(e, key, value)
+  end
+
+  def get(key)
+    set(key,nil, false)
+  end
+
+  protected
+
+  def post_process
+    compact_arrays!
+  end
+
+
   def copy_invalid_keys?
     true
   end
@@ -37,7 +66,7 @@ class NestedHash < Hash
   end
 
   def sanitize_key(key)
-    key
+    ( key =~ /^\d+$/ ) ? key.to_i : key
   end
 
   def is_valid_key?(key)
@@ -61,27 +90,6 @@ class NestedHash < Hash
     v
   end
 
-  def process_nested(key, value)
-    keys = key.split(".")
-    previous = sanitize_key(keys.shift)
-    top = keys.inject(self) do |memo,key|
-      if is_for_array?(key)
-        memo[previous] ||= []
-        key = key.to_i
-      else
-        key = sanitize_key(key)
-        memo[previous] ||= {}
-      end
-      a = memo[previous]
-      previous = key
-      a
-    end
-
-    top[previous] = value
-  rescue => e
-    handle_exception(e, key, value)
-  end
-
   def handle_exception(e, key, value)
     log_exception(e, key, value) if log_exceptions?
     raise e unless continue_on_exceptions?
@@ -100,7 +108,7 @@ class NestedHash < Hash
   end
 
   def is_for_array?(key)
-    key =~ /\A\d+\z/
+    key.is_a?(Numeric)
   end
 
 

data/lib/nested_hash/version.rb

@@ -1,3 +1,3 @@
 class NestedHash < Hash
-  VERSION = "0.1.4"
+  VERSION = "0.2.0"
 end

data/test/test_simple_to_complex_hash.rb

@@ -36,4 +36,20 @@ class NestedHashTest < Test::Unit::TestCase
     assert_equal( {"name" => "Guillermo", "lives" => 3} , hash["game"][0], 'should merge elements')
     assert_equal( {"lives" => 5}, hash["game"][1])
   end
+
+
+  def test_set_and_get
+   nh = NestedHash.new()
+   nh.set("person.2.name", "guillermo")
+
+   assert_equal({"person"=>[nil, nil, {"name"=>"guillermo"}]}, nh)
+  end
+
+  def test_compact_array
+   nh = NestedHash.new()
+   nh.set("person.2.name", "guillermo")
+
+   nh.compact_arrays!
+   assert_equal({"person"=>[{"name"=>"guillermo"}]}, nh)
+  end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: nested_hash
 version: !ruby/object:Gem::Version 
-  hash: 19
+  hash: 23
   prerelease: 
   segments: 
   - 0
-  - 1
-  - 4
-  version: 0.1.4
+  - 2
+  - 0
+  version: 0.2.0
 platform: ruby
 authors: 
 - "Guillermo A\xCC\x81lvarez"
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-05-24 00:00:00 +00:00
+date: 2012-05-25 00:00:00 +00:00
 default_executable: 
 dependencies: []