bitary 0.1.6 → 0.1.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bed2641b2bb380989c96dcc95710be56b64711c28723f3ca5f3f82fd7119bc7e
-  data.tar.gz: 8b7aaf05d552014d77d42707453c5c4c6d84aefb2bd5d382376cdb152ce52068
+  metadata.gz: 537a58c702f90bfeade22439d4134d921df709d4d474279f29adae869ff08b0d
+  data.tar.gz: 90910632d7a7f40f89774a8d0ba2b1a6bbf04f5b281454ab18313359151f58b0
 SHA512:
-  metadata.gz: fb6ad75f7e41ef52bdd71a78ac341381671264eb86e9b90cea66d36cbdbf751ff28e0a072a1479a9474bb68e111c983bcf093936ce08b1fac60b584ec2b7e5d0
-  data.tar.gz: bb66c485748c97732da3def70727dbc822704d21e1f2aad31cf810179db6e4154e3aa0c079a3a96a203c6e7e332865e8302428fada04b30f0671f314cbd8d772
+  metadata.gz: 3c65d9bcd2b757bf0a3e2d8b25b51b6a50b239e589efba142eb3c1c7539d42ac88c311361aa934f6ecf5be06c18da53c4e3c138dda801f0c89330da7c030a745
+  data.tar.gz: 10169aa8520c4d2dc4ddc92e419b77b745688c14bc08214770f9da25ed35b66d66655db37ef32daa7967119514ad3cacb618d06be5bd3c69df45e2ed435e9fb1

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## [0.1.6] - 2024-03-31
+
+- enhance global perf by remove useless and heavy error handling
+
 ## [0.1.5] - 2024-03-31
 
 - enhance performance of #each_byte (4 seconds faster on average)

data/lib/bitary/bitwarr.rb

@@ -5,101 +5,68 @@ class Bitary
     attr_reader :bpi, :bitsize
 
     def initialize(initial_data, bpi: Bitary::LONG)
-      check_initial_data(initial_data)
-      check_bpi(bpi)
+      @bitsize = init_bitsize(initial_data, bpi)
+      @array = init_array(initial_data, @bitsize, bpi)
+      @bpi = init_bpi(initial_data, bpi)
 
-      @bpi = bpi
-      @bitsize = init_bitsize(initial_data)
-      @array = init_array(initial_data)
+      self.bpi = bpi
     end
 
-    def method_missing(method, *, &)
-      @array.respond_to?(method) ? @array.send(method, *, &) : super
-    end
-
-    def respond_to_missing?(method, include_all = false)
-      @array.respond_to?(method, include_all) || super
-    end
-
-    def [](bit_index)
-      @array[item_index(bit_index)]
-    end
+    def [](bit_index) = @array[item_index(bit_index)]
+    def bit_at(index) = operate_bit_at(:get, index)
+    def bit_at!(index) = operate_bit_at!(:set, index)
+    def unbit_at!(index) = operate_bit_at!(:unset, index)
+    def to_s = @array.map { |item| to_binstr(item) }.join(' ')
 
     def []=(bit_index, value)
-      raise ArgumentError unless value.is_a?(Integer)
-
       @array[item_index(bit_index)] = value
     end
 
-    def item_index(bit_index)
-      check_bit_index(bit_index)
-
-      bit_index / @bpi
-    end
-
-    def bit_at(index)
-      operate_bit_at(:get, index)
-    end
-
-    def bit_at!(index)
-      operate_bit_at!(:set, index)
-    end
-
-    def unbit_at!(index)
-      operate_bit_at!(:unset, index)
-    end
-
-    def each_byte
+    def each_byte(&proc)
       @array.each do |item|
-        (@bpi / Bitary::BYTE).times do |i|
-          byte = (item >> (@bpi - (Bitary::BYTE * (i + 1))))
-          yield byte
-        end
+        explode_item(item, Bitary::BYTE, @bpi, &proc)
       end
     end
 
-    def to_s
-      @array.map { |item| format("%0#{@bpi}d", item.to_s(2)) }.join(' ')
-    end
-
     def bpi=(value)
-      check_bpi(value)
+      return if value == @bpi
 
       update_items_size!(value)
-
       @bpi = value
     end
 
+    def method_missing(method, *, &)
+      @array.respond_to?(method) ? @array.send(method, *, &) : super
+    end
+
+    def respond_to_missing?(method, include_all = false)
+      @array.respond_to?(method, include_all) || super
+    end
+
     private
 
-    def init_bitsize(initial_data)
-      initial_data.is_a?(Array) ? @bpi * initial_data.length : initial_data
+    def init_bpi(initial_data, bpi)
+      initial_data.is_a?(Array) ? Bitary::BYTE : bpi
     end
 
-    def init_array(initial_data)
+    def init_bitsize(initial_data, bpi)
+      initial_data.is_a?(Array) ? bpi * initial_data.length : initial_data
+    end
+
+    def init_array(initial_data, bitsize, bpi)
       if initial_data.is_a?(Array)
         initial_data.clone
       else
-        [0] * (@bitsize / @bpi.to_f).ceil
+        [0] * (bitsize / bpi.to_f).ceil
       end
     end
 
-    def check_initial_data(initial_data)
-      raise ArgumentError unless [Array, Integer].include?(initial_data.class)
-    end
-
-    def check_bit_index(bit_index)
-      raise ArgumentError unless bit_index.is_a?(Integer)
-      raise IndexError if bit_index.negative? || bit_index >= @bitsize
+    def item_index(bit_index)
+      bit_index / @bpi
     end
 
-    def check_bpi(bpi)
-      raise ArgumentError unless [
-        Bitary::BYTE,
-        Bitary::SHORT,
-        Bitary::INT,
-        Bitary::LONG
-      ].include?(bpi)
+    def to_binstr(item)
+      format("%0#{@bpi}d", item.to_s(2))
     end
 
     def operate_bit_at(operation, index)
@@ -116,29 +83,34 @@ class Bitary
     end
 
     def update_items_size!(value)
-      if value > @bpi
-        increase_items_size!(value)
-      else
-        decrease_items_size!(value)
-      end
+      value > @bpi ? increase_items_size!(value) : decrease_items_size!(value)
+    end
+
+    def append_bits(item, bpi, addend)
+      Factory.make('Handler::Append', item).execute(
+        offset: bpi,
+        value: addend
+      )
     end
 
     def increase_items_size(array, new_size, bpi)
       processed_bits = 0
-      array.each_with_object([0]) do |value, acc|
-        offset = bpi
-        if processed_bits >= new_size
-          offset = 0
+      res = array.each_with_object([0]) do |item, acc|
+        if processed_bits == new_size
           acc << 0
           processed_bits = 0
         end
 
-        acc[-1] = Factory.make('Handler::Append', acc[-1]).execute(
-          offset:,
-          value:
-        )
+        acc[-1] = append_bits(acc[-1], bpi, item)
+        processed_bits += bpi
+      end
+
+      while processed_bits < new_size
+        res[-1] = append_bits(res[-1], bpi, 0)
         processed_bits += bpi
       end
+
+      res
     end
 
     def increase_items_size!(value)
@@ -147,7 +119,9 @@ class Bitary
 
     def decrease_items_size(array, new_size, bpi)
       array.each_with_object([]) do |item, acc|
-        acc.concat(explode_item(item, new_size, bpi))
+        explode_item(item, new_size, bpi) do |new_item|
+          acc << new_item
+        end
       end
     end
 
@@ -156,16 +130,10 @@ class Bitary
     end
 
     def explode_item(item, new_size, bpi)
-      res = []
-      offset = bpi
       mask = (2**new_size) - 1
-
-      while offset.positive?
-        offset -= new_size
-        res << ((item >> offset) & mask)
+      (bpi / new_size).times do |i|
+        yield ((item >> (bpi - (new_size * (i + 1)))) & mask)
       end
-
-      res
     end
   end
 end

data/lib/bitary/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 class Bitary
-  VERSION = '0.1.6'
+  VERSION = '0.1.7'
 end

data/lib/bitary.rb

@@ -12,17 +12,24 @@ class Bitary
   include Size
 
   def initialize(initial_data, bpi: LONG)
-    @internal_array = Factory.make('Bitwarr', initial_data, bpi:)
+    check_initial_data(initial_data)
+    check_bpi(bpi)
+
+    @bitwarr = Factory.make('Bitwarr', initial_data, bpi:)
   end
 
   def [](index)
-    @internal_array.bit_at(index)
+    check_bit_index(index)
+
+    @bitwarr.bit_at(index)
   end
 
   def []=(index, value)
+    check_bit_index(index)
+
     case Factory.make('Mapper::ObjToBit').map(value)
-    when 0 then @internal_array.unbit_at!(index)
-    else @internal_array.bit_at!(index)
+    when 0 then @bitwarr.unbit_at!(index)
+    else @bitwarr.bit_at!(index)
     end
   end
 
@@ -35,27 +42,44 @@ class Bitary
   end
 
   def each_byte(&)
-    @internal_array.each_byte(&)
+    @bitwarr.each_byte(&)
   end
 
   def to_a
-    @internal_array.to_a
+    @bitwarr.to_a
   end
 
   def to_s
-    @internal_array.to_s
+    @bitwarr.to_s
   end
 
   def bpi=(value)
-    @internal_array.bpi = value
+    check_bpi(value)
+
+    @bitwarr.bpi = value
   end
 
   def size
-    @internal_array.bitsize
+    @bitwarr.bitsize
   end
 
   def bpi
-    @internal_array.bpi
+    @bitwarr.bpi
+  end
+
+  private
+
+  def check_initial_data(initial_data)
+    raise ArgumentError unless [Array, Integer].include?(initial_data.class)
+  end
+
+  def check_bpi(bpi)
+    raise ArgumentError unless [BYTE, SHORT, INT, LONG].include?(bpi)
+  end
+
+  def check_bit_index(bit_index)
+    raise ArgumentError unless bit_index.is_a?(Integer)
+    raise IndexError if bit_index.negative? || bit_index >= @bitwarr.bitsize
   end
 
   alias at []

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bitary
 version: !ruby/object:Gem::Version
-  version: 0.1.6
+  version: 0.1.7
 platform: ruby
 authors:
 - Maximilien Ballesteros
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-03-31 00:00:00.000000000 Z
+date: 2024-04-01 00:00:00.000000000 Z
 dependencies: []
 description: Ruby-based implementation of the bit array data structure
 email: