class_from_son 0.1.8 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 64442ffab338e5be82ed2a8dd9da60904777f17c6aa397a1c41f3a03617823cb
-  data.tar.gz: 9f6145f03094e312ed5efd05c23cd93e67a5112ddd920b177a4dd91383e2ba6f
+  metadata.gz: 7fe5463050c5bf5076026c3035dd5bf5e470659327ea19f6d8306042d96660bf
+  data.tar.gz: ea0270609324bd424d40ab09fa3b4da93d02689356e89d80b43c5a67cf5c6200
 SHA512:
-  metadata.gz: a8e7f6a45207f44a91043bda76f45c367d116c8498466d51e6946b14057de66e39ced63717304abcb0bf86623d19510a4b32023e5f3822c950ff419e4b0c97a4
-  data.tar.gz: 49894af656eb32ac4ce2d68637bc4998d737a3b2d7d2ab2f2afaf335dfa1153c09b1e62d136bf43318da898a330f34be30a28b9b195d3deb61183034a4b72b62
+  metadata.gz: 43cdeb8b35bd888eb0a21516e815e961d84d6714a522f49e16c628752954e67e4159afcd8552ba0741d204d26d824fd4b1b38b68dab8329ecf6ebdf34a1c642f
+  data.tar.gz: 549407b5e6149da0da5a826658591fdb76cc672b01bb157f81189fe66fe9fb37ab8de1424b80834f0758997045307133e44dbe2dfef3525b3a000574e18d411a

data/README.md

@@ -31,6 +31,12 @@ or
 ruby -e "require 'class_from_son'; ClassFromSON.generate :ruby, my_json_string, :json"
 ```
 
+Advanced use :
+
+```
+ruby -e "require 'class_from_son'; ClassFromSON.generate :ruby, my_json_string, :json, true, false, true, '../write/files/somewhere/else'"
+```
+
 Method parameter explanations :
 
 ```
@@ -42,7 +48,10 @@ Method parameter explanations :
 # source_lang is symbol or nil (if nil, source language will be determined from the file extension)
 # make_file flag defaults to true; set to false if you do not want files to be created by this method
 # force_file flag is false; set to true if you wish to overwrite matching destination files (use with caution!)
-def ClassFromSON.generate_from_file(dest_lang, file, source_lang, make_file = true, force_file = false)
+# lenient_mode flag is true; if the SON contains different objects with the same name (e.g. "data") then these will be treated
+#    as different objects with a _1, _2, etc. suffix. If this flag is false and these different objects are present, then errors will occur
+# custom_file_path is nil; set to an absoulte or relative path to have the new files be written to that location
+def ClassFromSON.generate_from_file(dest_lang, file, source_lang = nil, make_file = true, force_file = false, lenient_mode = true, custom_file_path = nil)
 ```
 
 ```
@@ -54,5 +63,8 @@ def ClassFromSON.generate_from_file(dest_lang, file, source_lang, make_file = tr
 # source_lang is symbol
 # make_file flag defaults to true; set to false if you do not want files to be created by this method
 # force_file flag is false; set to true if you wish to overwrite matching destination files (use with caution!)
-def ClassFromSON.generate(dest_lang, source, source_lang, make_file = true, force_file = false)
+# lenient_mode flag is true; if the SON contains different objects with the same name (e.g. "data") then these will be treated
+#    as different objects with a _1, _2, etc. suffix. If this flag is false and these different objects are present, then errors will occur
+# custom_file_path is nil; set to an absoulte or relative path to have the new files be written to that location
+def ClassFromSON.generate(dest_lang, source, source_lang, make_file = true, force_file = false, lenient_mode = true, custom_file_path = nil)
 ```

data/lib/class_from_SON.rb

@@ -172,6 +172,7 @@ class ClassFromSON
 
 	# Returns code representing the start of the class
 	def generate_class_start(name)
+		# TODO make this more readable
 		case @language
 		when :java
 			start = <<-START
@@ -199,7 +200,16 @@ import lombok.Setter;
 public class #{convert_custom_class_type(name)} {
 START
 		when :ruby
-			start = "class #{convert_custom_class_type(name)}"
+			case @mode
+			when :json
+			start = <<-START
+require 'json'
+
+class #{convert_custom_class_type(name)}
+START
+			else 
+				error_and_exit "Cannot parse mode #{@mode}"
+			end
 		else 
 			error_and_exit "Could not convert to output language #{@language}"
 		end
@@ -285,7 +295,7 @@ START
 	def generate_ruby_code_from_attributes(attributes)
 		code = []
 		names = []
-		attributes.each {|att| names << att[:name]}
+		attributes.each {|att| names << att[:name].snakecase}
 
 		# Instance variables
 		names.each do |name|
@@ -294,12 +304,74 @@ START
 		code << "" # An empty string is enough to trigger a newline
 
 		# Constructor
-		code << "\tdef initialize(#{names.join(", ")})"
+		# This is deliberately commented out, in favour of self.from_hash
+		code << "\t# Using self.from_hash(hash) is usually better, but this code is here in case you prefer this style of constructor"
+		code << "\t# def initialize(#{names.join(", ")})"
 		names.each do |name|
-			code << "\t\t@#{name} = #{name}"
+			code << "\t\t# @#{name} = #{name}"
 		end
+		code << "\t# end"
+		code
+	end
+
+	# Returns code for a from_SON and to_SON method
+	def generate_from_and_to_methods(classname, attributes)
+		case @language
+		when :java, :java_lombok
+			return generate_java_from_and_to_methods(classname, attributes)
+		when :ruby
+			return generate_ruby_from_and_to_methods(classname, attributes)
+		else 
+			error_and_exit "Could not convert to output language #{@language}"
+		end
+	end
+
+	# Returns Java code for a from_SON and to_SON method
+	def generate_java_from_and_to_methods(classname, attributes)
+		code = []
+		# TODO
+		code
+	end
+
+	# Returns Ruby code for a from_SON and to_SON method
+	def generate_ruby_from_and_to_methods(classname, attributes)
+		code = []
+		names = []
+		attributes.each {|att| names << att[:name].snakecase}
+
+		# from_hash method
+		code << ""
+		code << "\tdef self.from_hash(h)"
+		code << "\t\to = self.new"
+		names.each do |name|
+			code << "\t\to.#{name} = h[:#{name}]"
+		end
+		code << "\t\to"
 		code << "\tend"
 
+		# from_SON method
+		code << ""
+		code << "\tdef self.from_#{@mode}(#{@mode})"
+		case @mode
+		when :json
+			code << "\t\tself.from_hash(JSON.parse(#{@mode}))"
+			# TODO other input languages, e.g. XML, YAML
+		end
+		code << "\tend"
+		
+		# to_SON method
+		code << ""
+		code << "\tdef to_#{@mode}"
+		code << "\t\th = {}"
+		names.each do |name|
+			code << "\t\th[:#{name}] = @#{name}"
+		end
+		case @mode
+		when :json
+			code << "\t\tJSON.generate(h)"
+			# TODO other input languages, e.g. XML, YAML
+		end
+		code << "\tend"
 		code
 	end
 
@@ -337,6 +409,7 @@ START
 		end
 
 		lines << generate_code_from_attributes(attributes)
+		lines << generate_from_and_to_methods(classname, attributes)
 		lines << generate_class_end
 		lines.flatten!
 		this_file[:contents] = lines.join("\n")
@@ -356,13 +429,16 @@ START
 	# source_lang is symbol or nil (if nil, source language will be determined from the file extension)
 	# make_file flag defaults to true; set to false if you do not want files to be created by this method
 	# force_file flag is false; set to true if you wish to overwrite matching destination files (use with caution!)
-	def ClassFromSON.generate_from_file(dest_lang, file, source_lang = nil, make_file = true, force_file = false)
+	# lenient_mode flag is true; if the SON contains different objects with the same name (e.g. "data") then these will be treated
+	#    as different objects with a _1, _2, etc. suffix. If this flag is false and these different objects are present, then errors will occur
+	# custom_file_path is nil; set to an absoulte or relative path to have the new files be written to that location
+	def ClassFromSON.generate_from_file(dest_lang, file, source_lang = nil, make_file = true, force_file = false, lenient_mode = true, custom_file_path = nil)
 
 		error_and_exit "Could not locate file #{file}" unless File.exists?(file)
 
 		source_lang ||= File.extname(file).gsub(".", "")
 		source = File.readlines(file).join
-		ClassFromSON.generate(dest_lang, source, source_lang, make_file, force_file)
+		ClassFromSON.generate(dest_lang, source, source_lang, make_file, force_file, lenient_mode, custom_file_path)
 
 		# o = ClassFromSON.new
 		# o.generate(dest_lang, source, source_lang, make_file)
@@ -376,9 +452,12 @@ START
 	# source_lang is symbol
 	# make_file flag defaults to true; set to false if you do not want files to be created by this method
 	# force_file flag is false; set to true if you wish to overwrite matching destination files (use with caution!)
-	def ClassFromSON.generate(dest_lang, source, source_lang, make_file = true, force_file = false)
+	# lenient_mode flag is true; if the SON contains different objects with the same name (e.g. "data") then these will be treated
+	#    as different objects with a _1, _2, etc. suffix. If this flag is false and these different objects are present, then errors will occur
+	# custom_file_path is nil; set to an absoulte or relative path to have the new files be written to that location
+	def ClassFromSON.generate(dest_lang, source, source_lang, make_file = true, force_file = false, lenient_mode = true, custom_file_path = nil)
 		o = ClassFromSON.new
-		o.generate(dest_lang, source, source_lang, make_file, force_file)
+		o.generate(dest_lang, source, source_lang, make_file, force_file, lenient_mode, custom_file_path)
 	end
 
 	# Will generate classes from a SON string. 
@@ -389,7 +468,10 @@ START
 	# source_lang is symbol
 	# make_file flag defaults to true; set to false if you do not want files to be created by this method
 	# force_file flag is false; set to true if you wish to overwrite matching destination files (use with caution!)
-	def generate(dest_lang, source, source_lang, make_file = true, force_file = false)
+	# lenient_mode flag is true; if the SON contains different objects with the same name (e.g. "data") then these will be treated
+	#    as different objects with a _1, _2, etc. suffix. If this flag is false and these different objects are present, then errors will occur
+	# custom_file_path is nil; set to an absoulte or relative path to have the new files be written to that location
+	def generate(dest_lang, source, source_lang, make_file = true, force_file = false, lenient_mode = true, custom_file_path = nil)
 
 		error_and_exit "Please supply first argument as a Symbol" unless dest_lang.class == Symbol
 
@@ -434,16 +516,46 @@ START
 		top_level_classname = generate_top_level_name
 		output_classes = generate_output_classes(hash, top_level_classname).flatten # returns an array
 
+		# Set the directory that the files will be written into
+		if custom_file_path
+			# This caters for both absolute & relative file paths
+			file_path = File.absolute_path(custom_file_path)
+		else
+			file_path = Dir.getwd
+		end
+
+		# Track the names of the classes/files we have written so far
+		written_file_names = []
+
 		if make_file
 			output_classes.each do |out|
 				name = out[:name_with_ext]
+				# Check the name against the files we have already written
+				if written_file_names.include?(name)
+					if lenient_mode
+						# Let us increment the name, e.g. "data.rb" -> "data_1.rb", "data_2.rb", etc.
+						increment = 1
+						new_name = name.gsub(@extension, "_#{increment}#{@extension}")
+						while written_file_names.include?(new_name)
+							increment += 1
+							new_name = name.gsub(@extension, "_#{increment}#{@extension}")
+						end
+						name = new_name
+					else
+						message = "Want to generate output file #{name}, but a file with that name has already been written by this process. Your SON structure contains 2+ different classes with the same name"
+						error_and_exit(message) 
+					end
+				end
+
+				filename = file_path + File::SEPARATOR + name
 				contents = out[:contents]
 				unless force_file
 					error_and_exit "Want to generate output file #{name}, but that file already exists" if File.exists?(name)
 				end
-				File.open(name, "w+") do |f|
+				File.open(filename, "w+") do |f|
 					f.puts contents
 				end
+				written_file_names << name
 				puts "Wrote out file #{name}"
 			end
 

data/test/test_class_from_son.rb

@@ -19,7 +19,7 @@ class TestEx < Test::Unit::TestCase
 		
 		address_filename = "address.rb"
 		top_level_filename = "generated_from_json.rb"
-		phonenumbers_filename = "phonenumbers.rb"
+		phonenumbers_filename = "phone_numbers.rb"
 
 		expected_address = File.readlines(address_filename).join.chomp # trim off any trailing whitespace (irrelevent for this test)
 		expected_example = File.readlines(top_level_filename).join.chomp # trim off any trailing whitespace (irrelevent for this test)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: class_from_son
 version: !ruby/object:Gem::Version
-  version: 0.1.8
+  version: 0.2.0
 platform: ruby
 authors:
 - Richard Morrisby
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-02-03 00:00:00.000000000 Z
+date: 2021-07-06 00:00:00.000000000 Z
 dependencies: []
 description: This gem will attempt to generate code of a class of an object representing
   the contents of a Serialised-Object-Notation (SON) string (or file). E.g. it will