csvreader 1.2.4 → 1.2.5

data/lib/csvreader/parser_table.rb

@@ -1,123 +1,122 @@
-# encoding: utf-8
-
-class CsvReader
-
-class ParserTable
-
-###################################
-## add simple logger with debug flag/switch
-#
-#  use Parser.debug = true   # to turn on
-#
-#  todo/fix: use logutils instead of std logger - why? why not?
-
-def self.build_logger()
-  l = Logger.new( STDOUT )
-  l.level = :info    ## set to :info on start; note: is 0 (debug) by default
-  l
-end
-def self.logger() @@logger ||= build_logger; end
-def logger()  self.class.logger; end
-
-
-
-
-attr_reader   :config   ## todo/fix: change config to proper dialect class/struct - why? why not?
-
-##
-##  todo/check:
-##    null values - include NA - why? why not?
-##        make null values case sensitive or add an option for case sensitive
-##        or better allow a proc as option for checking too!!!
-def initialize( space: nil )
-  @config = {}   ## todo/fix: change config to proper dialect class/struct - why? why not?
-
-  ## e.g. treat/convert char to space e.g. _-+• etc
-  ##   Man_Utd   => Man Utd
-  ##  or use it for leading and trailing spaces without quotes
-  ##  todo/check: only use for unquoted values? why? why not?
-  @config[:space]   = space
-end
-
-
-#########################################
-## config convenience helpers
-def space=( value )       @config[:space]=value; end
-
-
-
-
-
-def parse( str_or_readable, **kwargs, &block )
-
-  ## note: input: required each_line (string or io/file for example)
-  ## note: kwargs NOT used for now (but required for "protocol/interface" by other parsers)
-
-  input = str_or_readable   ## assume it's a string or io/file handle
-
-  if block_given?
-    parse_lines( input, &block )
-  else
-    records = []
-
-    parse_lines( input ) do |record|
-      records << record
-    end
-
-    records
-  end
-end ## method parse
-
-
-
-private
-
-def parse_lines( input, &block )
-
-  space = config[:space]
-
-  ## note: each line only works with \n (windows) or \r\n (unix)
-  ##   will NOT work with \r (old mac, any others?) only!!!!
-  input.each_line do |line|
-
-    logger.debug  "line:"             if logger.debug?
-    logger.debug line.pretty_inspect  if logger.debug?
-
-
-    ##  note: chomp('') if is an empty string,
-    ##    it will remove all trailing newlines from the string.
-    ##    use line.sub(/[\n\r]*$/, '') or similar instead - why? why not?
-    line = line.chomp( '' )
-    line = line.strip         ## strip leading and trailing whitespaces (space/tab) too
-    logger.debug line.pretty_inspect    if logger.debug?
-
-    if line.empty?             ## skip blank lines
-      logger.debug "skip blank line"    if logger.debug?
-      next
-    end
-
-    if line.start_with?( "#" )  ## skip comment lines
-      logger.debug "skip comment line"   if logger.debug?
-      next
-    end
-
-    # note: string.split defaults to split by space (e.g. /\s+/) :-)
-    #          for  just make it "explicit" with /[ \t]+/
-
-    values = line.split( /[ \t]+/ )
-    logger.debug values.pretty_inspect   if logger.debug?
-
-    if space
-      ## e.g. translate _-+ etc. if configured to space
-      ##  Man_Utd => Man Utd etc.
-       values = values.map {|value| value.tr(space,' ') }
-    end
-
-    ## note: requires block - enforce? how? why? why not?
-    block.call( values )
-  end
-end # method parse_lines
-
-
-end # class ParserTable
-end # class CsvReader
+
+class CsvReader
+
+class ParserTable
+
+###################################
+## add simple logger with debug flag/switch
+#
+#  use Parser.debug = true   # to turn on
+#
+#  todo/fix: use logutils instead of std logger - why? why not?
+
+def self.build_logger()
+  l = Logger.new( STDOUT )
+  l.level = :info    ## set to :info on start; note: is 0 (debug) by default
+  l
+end
+def self.logger() @@logger ||= build_logger; end
+def logger()  self.class.logger; end
+
+
+
+
+attr_reader   :config   ## todo/fix: change config to proper dialect class/struct - why? why not?
+
+##
+##  todo/check:
+##    null values - include NA - why? why not?
+##        make null values case sensitive or add an option for case sensitive
+##        or better allow a proc as option for checking too!!!
+def initialize( space: nil )
+  @config = {}   ## todo/fix: change config to proper dialect class/struct - why? why not?
+
+  ## e.g. treat/convert char to space e.g. _-+• etc
+  ##   Man_Utd   => Man Utd
+  ##  or use it for leading and trailing spaces without quotes
+  ##  todo/check: only use for unquoted values? why? why not?
+  @config[:space]   = space
+end
+
+
+#########################################
+## config convenience helpers
+def space=( value )       @config[:space]=value; end
+
+
+
+
+
+def parse( str_or_readable, **kwargs, &block )
+
+  ## note: input: required each_line (string or io/file for example)
+  ## note: kwargs NOT used for now (but required for "protocol/interface" by other parsers)
+
+  input = str_or_readable   ## assume it's a string or io/file handle
+
+  if block_given?
+    parse_lines( input, &block )
+  else
+    records = []
+
+    parse_lines( input ) do |record|
+      records << record
+    end
+
+    records
+  end
+end ## method parse
+
+
+
+private
+
+def parse_lines( input, &block )
+
+  space = config[:space]
+
+  ## note: each line only works with \n (windows) or \r\n (unix)
+  ##   will NOT work with \r (old mac, any others?) only!!!!
+  input.each_line do |line|
+
+    logger.debug  "line:"             if logger.debug?
+    logger.debug line.pretty_inspect  if logger.debug?
+
+
+    ##  note: chomp('') if is an empty string,
+    ##    it will remove all trailing newlines from the string.
+    ##    use line.sub(/[\n\r]*$/, '') or similar instead - why? why not?
+    line = line.chomp( '' )
+    line = line.strip         ## strip leading and trailing whitespaces (space/tab) too
+    logger.debug line.pretty_inspect    if logger.debug?
+
+    if line.empty?             ## skip blank lines
+      logger.debug "skip blank line"    if logger.debug?
+      next
+    end
+
+    if line.start_with?( "#" )  ## skip comment lines
+      logger.debug "skip comment line"   if logger.debug?
+      next
+    end
+
+    # note: string.split defaults to split by space (e.g. /\s+/) :-)
+    #          for  just make it "explicit" with /[ \t]+/
+
+    values = line.split( /[ \t]+/ )
+    logger.debug values.pretty_inspect   if logger.debug?
+
+    if space
+      ## e.g. translate _-+ etc. if configured to space
+      ##  Man_Utd => Man Utd etc.
+       values = values.map {|value| value.tr(space,' ') }
+    end
+
+    ## note: requires block - enforce? how? why? why not?
+    block.call( values )
+  end
+end # method parse_lines
+
+
+end # class ParserTable
+end # class CsvReader

data/lib/csvreader/parser_yaml.rb

@@ -1,24 +1,23 @@
-# encoding: utf-8
-
-
-class CsvReader
-
-class ParserYaml
-
-def parse( data, **kwargs, &block )
-  ## note: kwargs NOT used for now (but required for "protocol/interface" by other parsers)
-
-  ## note: input: required each_line (string or io/file for example)
-  ## assume data is a string or io/file handle
-  csv = CsvYaml.new( data )
-
-  if block_given?
-    csv.each( &block )
-  else
-    csv.to_a
-  end
-end ## method parse
-
-
-end # class ParserYaml
-end # class CsvReader
+
+
+class CsvReader
+
+class ParserYaml
+
+def parse( data, **kwargs, &block )
+  ## note: kwargs NOT used for now (but required for "protocol/interface" by other parsers)
+
+  ## note: input: required each_line (string or io/file for example)
+  ## assume data is a string or io/file handle
+  csv = CsvYaml.new( data )
+
+  if block_given?
+    csv.each( &block )
+  else
+    csv.to_a
+  end
+end ## method parse
+
+
+end # class ParserYaml
+end # class CsvReader

data/lib/csvreader/reader.rb

@@ -1,4 +1,3 @@
-# encoding: utf-8
 
 class CsvReader
 
@@ -155,11 +154,11 @@ class CsvReader
 
          ## check array / pipeline of converters is empty (size=0 e.g. is [])
          if @converters.empty?
-           @parser.parse( @io, kwargs, &block )
+           @parser.parse( @io, **kwargs, &block )
          else
            ## add "post"-processing with converters pipeline
            ##   that is, convert all strings to integer, float, date, ... if wanted
-           @parser.parse( @io, kwargs ) do |raw_record|
+           @parser.parse( @io, **kwargs ) do |raw_record|
              record = []
              raw_record.each_with_index do | value, i |
                record << @converters.convert( value, i )

data/lib/csvreader/reader_hash.rb

@@ -1,4 +1,3 @@
-# encoding: utf-8
 
 class CsvHashReader
 
@@ -169,7 +168,7 @@ def_delegators :@io,
      kwargs[:width] = @kwargs[:width]    if @parser.is_a?( ParserFixed )
 
 
-     @parser.parse( @io, kwargs ) do |raw_values|     # sep: sep
+     @parser.parse( @io, **kwargs ) do |raw_values|     # sep: sep
         if @names.nil?    ## check for (first) headers row
           if @header_converters.empty?
             @names = raw_values   ## store header row / a.k.a. field/column names

data/lib/csvreader/version.rb

@@ -1,32 +1,30 @@
-# encoding: utf-8
-
-
-class CsvReader   ## note: uses a class for now - change to module - why? why not?
-
-  module Version
-    MAJOR = 1    ## todo: namespace inside version or something - why? why not??
-    MINOR = 2
-    PATCH = 4
-
-    ## self.to_s  - why? why not?
-  end
-
-  VERSION = [Version::MAJOR,
-             Version::MINOR,
-             Version::PATCH].join('.')
-
-  def self.version   ## keep (as an alternative to VERSION) - why? why not?
-    VERSION
-  end
-
-
-
-  def self.banner
-    "csvreader/#{VERSION} on Ruby #{RUBY_VERSION} (#{RUBY_RELEASE_DATE}) [#{RUBY_PLATFORM}]"
-  end
-
-  def self.root
-    File.expand_path( File.dirname(File.dirname(File.dirname(__FILE__))) )
-  end
-
-end # class CsvReader
+
+class CsvReader   ## note: uses a class for now - change to module - why? why not?
+
+  module Version
+    MAJOR = 1    ## todo: namespace inside version or something - why? why not??
+    MINOR = 2
+    PATCH = 5
+
+    ## self.to_s  - why? why not?
+  end
+
+  VERSION = [Version::MAJOR,
+             Version::MINOR,
+             Version::PATCH].join('.')
+
+  def self.version   ## keep (as an alternative to VERSION) - why? why not?
+    VERSION
+  end
+
+
+
+  def self.banner
+    "csvreader/#{VERSION} on Ruby #{RUBY_VERSION} (#{RUBY_RELEASE_DATE}) [#{RUBY_PLATFORM}] in (#{root})"
+  end
+
+  def self.root
+    File.expand_path( File.dirname(File.dirname(File.dirname(__FILE__))) )
+  end
+
+end # class CsvReader

data/lib/csvreader.rb

@@ -1,4 +1,3 @@
-# encoding: utf-8
 
 
 ## our own code (without "top-level" shortcuts e.g. "modular version")

data/test/test_parser_formats.rb

@@ -1,66 +1,66 @@
-# encoding: utf-8
-
-###
-#  to run use
-#     ruby -I ./lib -I ./test test/test_parser_formats.rb
-
-
-require 'helper'
-
-class TestParserFormats < MiniTest::Test
-
-
-def parser
-  CsvReader::Parser
-end
-
-
-def test_parse_whitespace
-   records = [["a", "b", "c"],
-              ["1", "2", "3"]]
-
-   ## don't care about newlines (\r\n) ??? - fix? why? why not?
-   assert_equal records, parser.default.parse( "a,b,c\n1,2,3" )
-   assert_equal records, parser.default.parse( "a,b,c\n1,2,3\n" )
-   assert_equal records, parser.default.parse( " a, b ,c \n\n1,2,3\n" )
-   assert_equal records, parser.default.parse( " a, b ,c \n \n1,2,3\n" )
-
-   assert_equal [["a", "b", "c"],
-                 [""],
-                 ["1", "2", "3"]], parser.default.parse( %Q{a,b,c\n""\n1,2,3\n} )
-   assert_equal [["", ""],
-                 [""],
-                 ["", "", ""]], parser.default.parse( %Q{,\n""\n"","",""\n} )
-
-
-   ## strict rfc4180 - no trim leading or trailing spaces or blank lines
-   assert_equal records,   parser.strict.parse( "a,b,c\n1,2,3" )
-   assert_equal [["a", "b", "c"],
-                 [""],
-                 ["1", "2", "3"]], parser.strict.parse( "a,b,c\n\n1,2,3" )
-   assert_equal [[" a", " b ", "c "],
-                 [""],
-                 ["1", "2", "3"]], parser.strict.parse( " a, b ,c \n\n1,2,3" )
-    assert_equal [[" a", " b ", "c "],
-                  [" "],
-                  ["",""],
-                  ["1", "2", "3"]], parser.strict.parse( " a, b ,c \n \n,\n1,2,3" )
-end
-
-
-def test_parse_empties
-    assert_equal [], parser.default.parse( "\n \n \n" )
-
-    ## strict rfc4180 - no trim leading or trailing spaces or blank lines
-    assert_equal [[""],
-                  [" "],
-                  [" "]], parser.strict.parse( "\n \n \n" )
-    assert_equal [[""],
-                  [" "],
-                  [" "]], parser.strict.parse( "\n \n " )
-
-    assert_equal [[""]], parser.strict.parse( "\n" )
-    assert_equal [],     parser.strict.parse( "" )
-end
-
-end # class TestParserFormats
+# encoding: utf-8
+
+###
+#  to run use
+#     ruby -I ./lib -I ./test test/test_parser_formats.rb
+
+
+require 'helper'
+
+class TestParserFormats < MiniTest::Test
+
+
+def parser
+  CsvReader::Parser
+end
+
+
+def test_parse_whitespace
+   records = [["a", "b", "c"],
+              ["1", "2", "3"]]
+
+   ## don't care about newlines (\r\n) ??? - fix? why? why not?
+   assert_equal records, parser.default.parse( "a,b,c\n1,2,3" )
+   assert_equal records, parser.default.parse( "a,b,c\n1,2,3\n" )
+   assert_equal records, parser.default.parse( " a, b ,c \n\n1,2,3\n" )
+   assert_equal records, parser.default.parse( " a, b ,c \n \n1,2,3\n" )
+
+   assert_equal [["a", "b", "c"],
+                 [""],
+                 ["1", "2", "3"]], parser.default.parse( %Q{a,b,c\n""\n1,2,3\n} )
+   assert_equal [["", ""],
+                 [""],
+                 ["", "", ""]], parser.default.parse( %Q{,\n""\n"","",""\n} )
+
+
+   ## strict rfc4180 - no trim leading or trailing spaces or blank lines
+   assert_equal records,   parser.strict.parse( "a,b,c\n1,2,3" )
+   assert_equal [["a", "b", "c"],
+                 [""],
+                 ["1", "2", "3"]], parser.strict.parse( "a,b,c\n\n1,2,3" )
+   assert_equal [[" a", " b ", "c "],
+                 [""],
+                 ["1", "2", "3"]], parser.strict.parse( " a, b ,c \n\n1,2,3" )
+    assert_equal [[" a", " b ", "c "],
+                  [" "],
+                  ["",""],
+                  ["1", "2", "3"]], parser.strict.parse( " a, b ,c \n \n,\n1,2,3" )
+end
+
+
+def test_parse_empties
+    assert_equal [], parser.default.parse( "\n \n \n" )
+
+    ## strict rfc4180 - no trim leading or trailing spaces or blank lines
+    assert_equal [[""],
+                  [" "],
+                  [" "]], parser.strict.parse( "\n \n \n" )
+    assert_equal [[""],
+                  [" "],
+                  [" "]], parser.strict.parse( "\n \n " )
+
+    assert_equal [[""]], parser.strict.parse( "\n" )
+    assert_equal [],     parser.strict.parse( "" )
+end
+
+end # class TestParserFormats