romloader 0.0.0 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6f557300ce6914298480061925476c3bdb8e1b05
-  data.tar.gz: 19343b3980b59dc344cfb1c8e11182076d7f5009
+  metadata.gz: eb0581eebe31d910a5866868c6767f86e59f90c1
+  data.tar.gz: b2ba39f821db26cbad47104292155e813a772d03
 SHA512:
-  metadata.gz: 10c416b0f71e1176aca6bb24c4dc68087ce1640a6feca1da6345edf01b2b96cbd922555f90e5958b45265c9a2f93ac98f80c75fce1e098494d158b23bd0c3656
-  data.tar.gz: c67d64422a5cd3c7a14497d49c7063656c9e7f87e9162a6e26cdbb0815587c52ba03c10d19fbf4e0a101f4c3b3ae75b26530bd396cb603bcf033fdd6327174de
+  metadata.gz: c8fef42a55aff86ac56bf4626c8d442d1d22399068b3ea2e156575c9dcca6a6c1d69e3fd686ab0d61f3b28e3de1e94756242c0ef766290ae9088e991153d991b
+  data.tar.gz: 17d045707749160b27a5c425ed247ef69f75dde87253731bed178d264ff6329f4d21bf61c03e593b55563dd86705a7f6029d29173d9306e338ea872ae587f6ac

data/README.md

@@ -25,6 +25,11 @@ This will download any dependencies for development. Rspec is among the dependen
 
 Bug reports and pull requests are welcome on GitHub at https://github.com/jinstrider2000/romloader-cli-gem.
 
+## Version Changes
+
+v. 1.0: Changed the namespacing of classes, added zip/7-zip extraction and rom download directory management. Also, added the ability to open the newly downloaded game from the command line (requires a emulator)
+
+v. 0.0: Basic rom listing from Freeroms.com, and rom download feature
 
 ## License
 

data/lib/romloader.rb

@@ -1,7 +1,21 @@
+module RomLoader
+
+end
+
+require 'open-uri'
+require 'nokogiri'
+require 'cgi'
+require 'zip'
+require 'seven_zip_ruby'
+require 'fileutils'
+require_relative 'romloader/game_rom.rb'
+require_relative 'romloader/game_system.rb'
+require_relative 'romloader/archive_extractor.rb'
+require_relative 'romloader/freeroms_scraper.rb'
+require_relative 'romloader/scraping_error/errors.rb'
 require_relative 'romloader/romloader_cli.rb'
 
 def run
   Dir.mkdir(File.join(Dir.home,"videogame_roms")) unless Dir.exist?(File.join(Dir.home,"videogame_roms"))
-  RomloaderCli.new.start
-end
-
+  RomLoader::RomLoaderCli.new.start
+end

data/lib/romloader/archive_extractor.rb

@@ -0,0 +1,56 @@
+
+class RomLoader::ArchiveExtractor
+
+  #Extracts zip or 7-zip rom files, manages the extracted dirs, then deletes archive files
+  def self.extract(dir,game_obj)
+    file_or_dir_to_open = ""
+    /(?<=\().+(?=\))/.match(game_obj.system.name) ? system_dir = /(?<=\().+(?=\))/.match(game_obj.system.name)[0] : system_dir = nil
+    system_dir ||= game_obj.system.name
+    system_dir = system_dir.rstrip.gsub(/[[[:space:]]\/]/, "_").downcase
+    Dir.mkdir(File.join(Dir.home,"videogame_roms",system_dir)) unless Dir.exist?(File.join(Dir.home,"videogame_roms",system_dir))
+    
+    if game_obj.system.name != "MAME"
+      puts "Extracting #{game_obj.filename}"
+      if game_obj.file_ext == ".zip"
+        Zip::File.open(dir) do |zip_archive|
+          zip_archive.glob("*htm").each { |entry| zip_archive.remove(entry) }
+          Dir.mkdir(File.join(Dir.home,"videogame_roms",system_dir,game_obj.filename.split(game_obj.file_ext)[0])) if zip_archive.size > 1 && !Dir.exist?(File.join(Dir.home,"videogame_roms",system_dir,game_obj.filename.split(game_obj.file_ext)[0]))
+          zip_archive.each_entry do |rom|
+            if Dir.exist?(File.join(Dir.home,"videogame_roms",system_dir,game_obj.filename.split(game_obj.file_ext)[0]))
+              rom.extract(File.join(Dir.home,"videogame_roms",system_dir,game_obj.filename.split(game_obj.file_ext)[0],rom.name)) unless File.exist?(File.join(Dir.home,"videogame_roms",system_dir,game_obj.filename.split(game_obj.file_ext)[0],rom.name))
+            else
+              rom.extract(File.join(Dir.home,"videogame_roms",system_dir,rom.name)) unless File.exist?(File.join(Dir.home,"videogame_roms",system_dir,rom.name))
+            end
+            zip_archive.size == 1 ? file_or_dir_to_open = File.join(Dir.home,"videogame_roms",system_dir,"\"#{rom.name}\"") : file_or_dir_to_open = File.join(Dir.home,"videogame_roms",system_dir,game_obj.filename.split(game_obj.file_ext)[0])
+          end
+        end
+      else
+        File.open(dir, "rb") do |seven_zip_archive|
+          Dir.mkdir(File.join(Dir.home,"videogame_roms",system_dir,game_obj.filename.split(game_obj.file_ext)[0])) unless Dir.exist?(File.join(Dir.home,"videogame_roms",system_dir,game_obj.filename.split(game_obj.file_ext)[0]))
+          SevenZipRuby::Reader.open(seven_zip_archive) do |szr|
+            files_already_in_dir = Dir.entries(File.join(Dir.home,"videogame_roms",system_dir,game_obj.filename.split(game_obj.file_ext)[0]))
+            if files_already_in_dir.size > 0
+              files_to_be_extracted = szr.entries.select do |entry|
+                file_match = /(file, |dir, |anti, )[.[^\.]]+\..+(?=>)/.match(entry.inspect)
+                file_match ? file_name = file_match[0].split(file_match[1])[1] : file_name = nil
+                !files_already_in_dir.any? { |file|  file == file_name || file_name =~ /\.htm/}
+              end
+              szr.extract(files_to_be_extracted, File.join(Dir.home,"videogame_roms",system_dir,game_obj.filename.split(game_obj.file_ext)[0]))
+            else
+              szr.extract_all(File.join(Dir.home,"videogame_roms",system_dir,game_obj.filename.split(game_obj.file_ext)[0]))
+            end
+          end
+          Dir.chdir(File.join(Dir.home,"videogame_roms",system_dir,game_obj.filename.split(game_obj.file_ext)[0])) do
+            Dir.entries(".").size == 1 ? file_or_dir_to_open = File.join(Dir.pwd,"\"#{Dir.entries(".").first}\"") : file_or_dir_to_open = Dir.pwd
+          end
+        end
+      end
+      File.delete(dir)
+      file_or_dir_to_open
+    else
+      FileUtils.move "#{dir}", "#{File.join(Dir.home,"videogame_roms",system_dir)}"
+      file_or_dir_to_open = File.join(Dir.home,"videogame_roms",system_dir)
+    end
+  end
+  
+end

data/lib/romloader/freeroms_scraper.rb

@@ -1,20 +1,9 @@
-require 'open-uri'
-require 'nokogiri'
-require 'cgi'
 
 # The class which facilitates scraping freeroms.com. Uses the nokogiri gem to scrape the site
-class FreeromsScraper
-  # Class Variables: none
-  #
-  # Instance Variables: none
-
-  # Purpose: To retrieve the names and main rom urls of the game systems currently begin served by freeroms.com.  Returns this information in the form of an array of hashes
+class RomLoader::FreeromsScraper
+  
   def self.system_scrape(url)
-    # Arguments:
-    # => 1. url (String): Contains the url "http://freeroms.com"
-    #
-    # Return:
-    # => Array<Hash>
+    
     system_list = Nokogiri::HTML(open(url)).css("dt.leftside > a")
     [].tap do |game_system|
       system_list.each do |system_info|
@@ -32,13 +21,9 @@ class FreeromsScraper
     end
   end
   
-  # Purpose: To retrieve the names and main rom urls of the individual games currently begin served by freeroms.com.  Returns this information in the form of an array of hashes
+  # To retrieve the names and main rom urls of the individual games currently begin served by freeroms.com.  Returns this information in the form of an array of hashes
   def self.rom_scrape(url)
-    # Arguments:
-    # => 1. url (String): Contains the url for each game
-    #
-    # Return:
-    # => Array<Hash>
+    
     game_list = Nokogiri::HTML(open(url)).css("tr[class^=\"game\"] > td[align=\"left\"]")
     [].tap do |rom_list|
       game_list.each do |game_info|
@@ -58,13 +43,9 @@ class FreeromsScraper
     end
   end
 
-  # Purpose: To retrieve the detailed information of individual games currently begin served by freeroms.com.  Returns this information in the form of a hash
+  # To retrieve the detailed information of individual games currently begin served by freeroms.com.  Returns this information in the form of a hash
   def self.rom_details(url)
-    # Arguments:
-    # => 1. url (String): Contains the url for each game
-    #
-    # Return:
-    # => Hash
+    
     direct_download = Nokogiri::HTML(open(url))
     {}.tap do |game|
       unless direct_download.css("td#romss > script").empty?
@@ -77,6 +58,7 @@ class FreeromsScraper
           if game_url
             game[:download_url] = game_url[0]
             game[:file_ext] = game_url[1]
+            game[:filename] = /[.[^\/]]+(\.zip|\.7z)\Z/.match(game_url[0])[0]
             begin
               game[:size] = direct_download.css("td#rom + td[colspan=\"2\"]").first.children.first.text.strip
             rescue NoMethodError
@@ -90,11 +72,7 @@ class FreeromsScraper
 
   # Purpose: To retrieve the letter indices for the roms of the game systems currently begin served by freeroms.com.  Returns this information in the form of a hash
   def self.rom_index_scrape(url)
-    # Arguments:
-    # => 1. url (String): Contains the url for the letter indices
-    #
-    # Return:
-    # => Hash
+    
     rom_letter_list = Nokogiri::HTML(open(url)).css("tr.letters > td[align=\"center\"] > font > a")
     {}.tap do |letter_hash|
       rom_letter_list.each do |letter_list|

data/lib/romloader/game_rom.rb

@@ -1,52 +1,27 @@
 
 # The class whose instances represent an individual game rom (e.g. Chrono Trigger object)
-class GameRom
-  # Class Variables: none
-  #
-  # Instance Variables:
-  # => 1. name (String): Contains the system's name (e.g. "Sega Genesis")
-  # => 2. download_url (String): Contains a string with the url to download the game (e.g. "http://download.freeroms.com/genesis/a_roms/altered_beast.zip") 
-  # => 3. rom_detail_url (String): Contains a string with the url to details of the game
-  # => 4. size (String): Contains a string with the size of download, as published on freeroms.com
-  # => 5. file_ext (String): Contains a string with the download file index (e.g. ".zip")
-  # => 6. system (GameSystem): Points to the GameSystem object that contains this GameRom instance 
+class RomLoader::GameRom
+  
   attr_accessor :name, :system
-  attr_reader :rom_detail_url ,:download_url, :size, :file_ext
+  attr_reader :rom_detail_url ,:download_url, :size, :file_ext, :filename
 
-  # Purpose: Create individual game rom objects from information scraped from freeroms.com, and sets the required name and rom_detail_url instance variables
+  # Create individual game rom objects from information scraped from freeroms.com, and sets the required name and rom_detail_url instance variables
   def initialize(name:, rom_detail_url:)
-    # Arguments:
-    # => 1. name (String): Contains the game's name (e.g. "Super Mario Bros 3")
-    # => 2. rom_detail_url (String): Contains a string with the url to details of the game
-    #
-    # Return:
-    # => GameRom object
     @name = name
     @rom_detail_url = rom_detail_url
   end
 
-  # Purpose: Creates an array of GameRom objects from an array
+  # Creates an array of GameRom objects from an array
   def self.create_collection(game_array)
-    # Arguments:
-    # => 1. game_array (Array<Hash>): Contains the game's information
-    #
-    # Return:
-    # => Array<GameRom>
     game_array.collect {|game_details| self.new(game_details)}    
   end
 
-  # Purpose: Sets all additional, optional rom details
-  def set_rom_details(download_url: nil, size: nil, file_ext: nil)
-    # Arguments:
-    # => 1. download_url [optional] (String): Contains a string with the url to download the game
-    # => 2. size [optional] (String): Contains a string with the size of download, as published on freeroms.com
-    # => 3. file_ext (String): Contains a string with the download file index (e.g. ".zip")
-    #
-    # Return:
-    # => nil
+  # Sets all additional, optional rom details
+  def set_rom_details(download_url: nil, size: nil, file_ext: nil, filename: nil)
     @download_url = download_url
     @size = size
     @file_ext = file_ext
+    @filename = filename
     nil
   end
   

data/lib/romloader/game_system.rb

@@ -1,28 +1,14 @@
 
 # The class whose instances represent an individual game system (e.g. Sega Genesis object) 
-class GameSystem
-  # Class Variables:
-  # => 1. all (Array<GameSystem>): Contains an array of all GameSystem objects that have been instantiated.
-  #
-  # Instance Variables:
-  # => 1. name (String): Contains the system's name (e.g. "Sega Genesis")
-  # => 2. rom_indices (Hash): Contains key/value pairs of a rom index letter and a url (e.g. {"A" => "http://freeroms.com/genesis_games_that_start_with_a.html"}) 
-  # => 3. roms (Hash): Contains key/value pairs of a rom index letter and an array of GameRom objects (e.g. {"S" => [Sonic the Hedgehog, Streets of Rage,...], "T" => [...], etc})
-  # => 4. rom_index_url (String): Contains the url that leads to the rom index for the given system on http://freeroms.com
+class RomLoader::GameSystem
   
   attr_accessor :name, :rom_index_url
   attr_writer :rom_indices
 
   @@all = []
   
-  # Purpose: Create individual game system objects from information scraped from http://freeroms.com, sets all instance variables
+  # Create individual game system objects from information scraped from http://freeroms.com, sets all instance variables
   def initialize(name:, rom_index_url:)
-    # Arguments:
-    # => 1. name (String): Contains the system's name (e.g. "Sega Genesis")
-    # => 2. rom_index_url (String): Contains the url that leads to the rom index for the given system on http://freeroms.com
-    #
-    # Return:
-    # => GameSystem object
     @name = name
     @rom_index_url = rom_index_url
     @rom_indices = {}
@@ -30,61 +16,35 @@ class GameSystem
     @@all << self
   end
 
-  # Purpose: Creates multiple GameSystem objects from information scraped from http://freeroms.com
+  # Creates multiple GameSystem objects from information scraped from http://freeroms.com
   def self.create_from_collection(system_array)
-    # Arguments:
-    # => 1. system_array (Array<Hash>): Contains an array of key/value pairs of the GameSystem instance variables and values (e.g. [{:name =>"NES"...},{:name => "SNES"..,},etc])
-    #
-    # Return:
-    # => nil
     system_array.each { |system_details| self.new(system_details)}
     nil
   end
 
-  # Purpose: Retrieves an array of all GameSystem objects
+  # Retrieves an array of all GameSystem objects
   def self.all
-    # Arguments: None
-    #
-    # Return:
-    # => Array<GameSystem>
     @@all    
   end
 
-  # Purpose: Retrieves an array of all GameRom objects starting which the provided letter index (e.g. [Sonic the Hedgehog, Streets of Rage,...])
+  # Retrieves an array of all GameRom objects starting which the provided letter index (e.g. [Sonic the Hedgehog, Streets of Rage,...])
   def get_roms_by_letter(letter_index)
-    # Arguments:
-    # => 1. letter_index (String): Letter used as key to retrieve array
-    # Return:
-    # => Array<GameRom>
     @roms[letter_index]
   end
 
-  # Purpose: Retrieves an array of the indicies for the roms (i.e. ["A","B","C"...])
+  # Retrieves an array of the indicies for the roms (i.e. ["A","B","C"...])
   def get_rom_indices
-    # Arguments: None
-    #
-    # Return:
-    # => Array<String>
     @rom_indices.keys
   end
 
-  # Purpose: Retrieves the url for roms of a particular letter index (e.g. "A" => "http://freeroms.com/genesis_games_that_start_with_a.html")
+  # Retrieves the url for roms of a particular letter index (e.g. "A" => "http://freeroms.com/genesis_games_that_start_with_a.html")
   def get_rom_collection_url(letter_index)
-    # Arguments:
-    # => 1. letter_index (String): Letter used as key to retrieve url
-    # Return:
-    # => String
     @rom_indices[letter_index]
   end
 
-  # Purpose: Add the game collection scraped from http://freeroms.com to the GameSystem object to the roms (Hash)
+  # Add the game collection scraped from http://freeroms.com to the GameSystem object to the roms (Hash)
   def add_roms_to_collection_by_letter(letter_index, game_obj_array)
-    # Arguments:
-    # => 1. letter_index (String): Letter to be used as a key for the roms Hash
-    # => 2. game_obj_array (Array<GameRom>): Contains array of GameRom objects (e.g. [Sonic the Hedgehog, Streets of Rage,...]
-    #
-    # Return:
-    # => Array<GameRom>
+
     game_obj_array.each { |game| game.system = self }
     @roms[letter_index] = game_obj_array
   end

data/lib/romloader/romloader_cli.rb

@@ -1,40 +1,24 @@
-require_relative 'freeroms_scraper.rb'
-require_relative 'game_rom.rb'
-require_relative 'game_system.rb'
-require_relative 'scraping_error/no_element_found.rb'
 
 # The CLI class
-class RomloaderCli
-  # Class Variables: none
-  #
-  # Instance Variables: none
+class RomLoader::RomLoaderCli
 
-  # Purpose: Instantiates a GameSystem collection from information scraped from http://freeroms.com
   def initialize
-    # Arguments: None
-    #
-    # Return:
-    # => RomloaderCli object
-    GameSystem.create_from_collection(FreeromsScraper.system_scrape("http://freeroms.com"))
-    raise ScrapingError::NoElementFound.exception("System index is currently unavailable. Exiting the program.") if GameSystem.all.size == 0
+    RomLoader::GameSystem.create_from_collection(RomLoader::FreeromsScraper.system_scrape("http://freeroms.com"))
+    raise RomLoader::ScrapingError::NoElementFound.exception("System index is currently unavailable. Exiting the program.") if RomLoader::GameSystem.all.size == 0
   end
 
-  # Purpose: Starts the CLI, called in romloader.rb
+  # Starts the CLI, called in romloader.rb
   def start
-    # Arguments: None
-    #
-    # Return:
-    # => nil
     input_stack = []
     control_flow_level = 1
 
-    puts "Thanks for using RomLoader, powered by freeroms.com!\nConnecting to freeroms.com and retrieving the system index...\n\n"
-    sleep 1
+    puts "Thanks for using RomLoader, powered by freeroms.com!\nNOTE: To play these games, please download an emulator for the desired system.\nConnecting to freeroms.com and retrieving the systems index...\n\n"
+    sleep 3
     while control_flow_level > 0 
       case control_flow_level
       when 1
         list_systems
-        input = input_prompt("Select a system (1-#{GameSystem.all.size}) [exit]:",1..GameSystem.all.size)
+        input = input_prompt("Select a system (1-#{RomLoader::GameSystem.all.size}) [exit]:",1..RomLoader::GameSystem.all.size)
         if input == "exit"
           control_flow_level = 0
         else
@@ -46,7 +30,7 @@ class RomloaderCli
         list_system_index(system)
         if system.get_rom_indices.empty?
           begin
-            raise ScrapingError::NoElementFound.exception("Requested system is currently unavailable. Try another one.")
+            raise RomLoader::ScrapingError::NoElementFound.exception("Requested system is currently unavailable. Try another one.")
           rescue
             control_flow_level -= 1
             input_stack.shift
@@ -59,7 +43,7 @@ class RomloaderCli
         game_collection = select_game_collection_by_index(system,input_stack[0].upcase)
         if game_collection.empty?
           begin
-            raise ScrapingError::NoElementFound.exception("Requested game index is currently unavailable. Try another one.")
+            raise RomLoader::ScrapingError::NoElementFound.exception("Requested game index is currently unavailable. Try another one.")
           rescue
             control_flow_level -= 1
             input_stack.shift
@@ -73,16 +57,23 @@ class RomloaderCli
         game = select_game(game_collection,input_stack[0].to_i)
         if game.download_url == nil
           begin
-            raise ScrapingError::NoElementFound.exception("Requested game is currently unavailable. Try another one.")
+            raise RomLoader::ScrapingError::NoElementFound.exception("Requested game is currently unavailable. Try another one.")
           rescue
             control_flow_level -= 1
             input_stack.shift
           end
         else
           display_rom_details(game)
-          input = input_prompt("Download (Y/n) [exit]:", /[yn]/, control_flow_level)
+          input = input_prompt("Download? (Y/n) [exit]:", /[yn]/, control_flow_level)
           if input == 'y' || input == ""
-            download_rom(game)
+            file_or_dir_to_open = download_rom(game)
+            if /\".+\"/.match(file_or_dir_to_open)
+              game_file = /\".+\"/.match(file_or_dir_to_open)[0]
+              input = input_prompt("Play #{game_file}? (Y/n) [exit]:", /[yn]/,control_flow_level)
+            else
+              input = input_prompt("Open #{file_or_dir_to_open}? (Y/n) [exit]:", /[yn]/,control_flow_level)
+            end
+            system("open #{file_or_dir_to_open}") if input == 'y' || input == ""
           end
           input_stack.shift
           input == "exit" ? control_flow_level = 0 : control_flow_level -= 1
@@ -93,15 +84,8 @@ class RomloaderCli
     puts "Happy Gaming!"
   end
 
-  # Purpose: Sets control_flow_level in RomloaderCli#start, manipulates input_stack in RomloaderCli#start
+  # Sets control_flow_level in RomLoaderCli#start, manipulates input_stack in RomLoaderCli#start
   def flow_controller(input,control_flow_level,input_stack)
-    # Arguments:
-    # => 1. input (String): Current user input from the CLI
-    # => 2. control_flow_level (Fixnum): Indicator of current user progress through the CLI
-    # => 3. input_stack (Array<String>): Buffer of previous user input from the CLI
-    #
-    # Return:
-    # => Fixnum
     if input == "exit"
       0
     elsif input == "back"
@@ -113,35 +97,21 @@ class RomloaderCli
     end
   end
 
-  # Purpose: Lists the game systems scraped from http://freeroms.com and saved in GameSystem.all (e.g. 1. Amiga, 2. Atari, etc...)
-  def list_systems
-    # Arguments: None
-    #
-    # Return:
-    # => nil 
-    GameSystem.all.each_with_index { |game_system, index| puts "#{index+1}. #{game_system.name}"}
+  # Lists the game systems scraped from http://freeroms.com and saved in Romloader::GameSystem.all (e.g. 1. Amiga, 2. Atari, etc...)
+  def list_systems 
+    RomLoader::GameSystem.all.each_with_index { |game_system, index| puts "#{index+1}. #{game_system.name}"}
     print "\n"
   end
 
-  # Purpose: Retrieves an individual GameSystem object from GameSystem.all
+  # Retrieves an individual Romloader::GameSystem object from Romloader::GameSystem.all
   def select_system(index)
-    # Arguments:
-    # => 1. index (Fixnum): Retrieved from user input
-    #
-    # Return:
-    # => GameSystem object
-    GameSystem.all[index-1]
+    RomLoader::GameSystem.all[index-1]
   end
 
-  # Purpose: List game index for the selected system by letter (e.g. A B C D...)
+  # List game index for the selected system by letter (e.g. A B C D...)
   def list_system_index(selected_system)
-    # Arguments:
-    # => 1. selected_system (String): Retrieved from user input 
-    #
-    # Return:
-    # => nil
     if selected_system.get_rom_indices.empty?
-      selected_system.rom_indices = FreeromsScraper.rom_index_scrape(selected_system.rom_index_url)
+      selected_system.rom_indices = RomLoader::FreeromsScraper.rom_index_scrape(selected_system.rom_index_url)
     end
     
     puts "#{selected_system.name} index:"
@@ -149,64 +119,34 @@ class RomloaderCli
     puts "\n\n"
   end
 
-  # Purpose: Retrieves all the games available for the selected system under the selected index (e.g. NES,"G")
+  # Retrieves all the games available for the selected system under the selected index (e.g. NES,"G")
   def select_game_collection_by_index(system, letter)
-    # Arguments:
-    # => 1. system (GameSystem): Selected by user through CLI
-    # => 2. letter (String): Retrieved from user input
-    #
-    # Return:
-    # => Array<GameRom>
     puts "Loading roms...\n"
     games_list = system.get_roms_by_letter(letter)
-    games_list ||= system.add_roms_to_collection_by_letter(letter,GameRom.create_collection(FreeromsScraper.rom_scrape(system.get_rom_collection_url(letter))))
+    games_list ||= system.add_roms_to_collection_by_letter(letter,RomLoader::GameRom.create_collection(RomLoader::FreeromsScraper.rom_scrape(system.get_rom_collection_url(letter))))
   end
 
-  # Purpose: List all the games available for the selected index (e.g. "S": 1. Super Castlevania, 2. Super Mario World, etc...)
+  # List all the games available for the selected index (e.g. "S": 1. Super Castlevania, 2. Super Mario World, etc...)
   def list_games(games)
-    # Arguments:
-    # => 1. games (Array<GameRom>): Selected by user through CLI
-    #
-    # Return:
-    # => nil
     games.each_with_index {|game,index| puts "#{index+1}. #{game.name}"}
     print "\n"
   end
 
-  # Purpose: Selects an individual game from the provided collection via index
+  # Selects an individual game from the provided collection via index
   def select_game(game_collection,index)
-    # Arguments:
-    # => 1. game_collection (GameRom): Selected by user through CLI
-    # => 2. index (Fixnum): Retrieved from user input
-    #
-    # Return:
-    # => GameRom object
-    game_collection[index-1].set_rom_details(FreeromsScraper.rom_details(game_collection[index-1].rom_detail_url))
+    game_collection[index-1].set_rom_details(RomLoader::FreeromsScraper.rom_details(game_collection[index-1].rom_detail_url))
     game_collection[index-1]
   end
 
-  # Purpose: List the details of the selected game (e.g. Chrono Trigger | 5.38 MB | .zip)
+  # List the details of the selected game (e.g. Chrono Trigger | 5.38 MB | .zip)
   def display_rom_details(game)
-    # Arguments:
-    # => 1. game (GameRom): Selected by user through CLI
-    #
-    # Return:
-    # => nil
     puts "Rom details:"
     puts "#{game.name} | System: #{game.system.name} | File size: #{game.size} | File type: #{game.file_ext}"
-    puts "NOTE: To uncompress 7-Zip (.7z) files, please download a system compatible version at http://www.7-zip.org/download.html" if game.file_ext == ".7z"
     print "\n"
   end
 
-  # Purpose: Prints a custom message, takes user input, asesses whether the input is valid, and returns the input
+  # Prints a custom message, takes user input, asesses whether the input is valid, and returns the input
   def input_prompt(message,accepted_input,control_flow_level=nil)
-    # Arguments:
-    # => 1. message (String): A custom message detailing the expected input
-    # => 2. accepted_input (Regexp or Range): Will be checked against the input to verify validity
-    # => 3. control_flow_level [optional] (Fixnum): Used to assess input in context specific situations (e.g. user providing "back" when CLI is on it's first screen)
-    #
-    # Return:
-    # => Fixnum
     valid = false
     until valid 
       print message + " "
@@ -227,20 +167,20 @@ class RomloaderCli
     input
   end
 
-  # Purpose: Downloads the selected game to the local directory (~/videogame_roms)
+  # Downloads the selected game to the local directory (~/videogame_roms)
   def download_rom(game)
-    # Arguments:
-    # => 1. game (GameRom): Selected by user through CLI
-    #
-    # Return:
-    # => nil
+    file_or_dir_to_open = ""
     puts "Downloading #{game.name} (#{game.size})..."
-    result = Dir.chdir(File.join(Dir.home,"videogame_roms")) do
-      system("curl -Og# \"#{game.download_url}\"")
+    result = Dir.chdir(File.join(Dir.home,"videogame_roms")) { system("curl -Og# \"#{game.download_url}\"") }
+    if result == true 
+      puts "Finished downloading #{game.filename} to #{File.join(Dir.home,"videogame_roms")}.\n"
+      file_or_dir_to_open = RomLoader::ArchiveExtractor.extract(File.join(Dir.home,"videogame_roms",game.filename),game)
+    else
+      puts "An error occured, the rom couldn't be downloaded.\n"
     end
-    result ? puts("Finished downloading to #{File.join(Dir.home,"videogame_roms")}.\n") : puts("An error occured, the rom couldn't be downloaded.\n")
     sleep 3
     puts "\n"
+    file_or_dir_to_open
   end
   
 end

data/lib/romloader/scraping_error/errors.rb

@@ -0,0 +1,8 @@
+
+module RomLoader::ScrapingError
+
+  class NoElementFound < StandardError
+    
+  end
+
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: romloader
 version: !ruby/object:Gem::Version
-  version: 0.0.0
+  version: 1.0.0
 platform: ruby
 authors:
 - Efrain Perez Jr
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-10-01 00:00:00.000000000 Z
+date: 2016-10-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -24,6 +24,48 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.6'
+- !ruby/object:Gem::Dependency
+  name: rubyzip
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.2'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.2'
+- !ruby/object:Gem::Dependency
+  name: seven_zip_ruby
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.2'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.2'
+- !ruby/object:Gem::Dependency
+  name: fileutils
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.7'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.7'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -79,11 +121,12 @@ files:
 - README.md
 - bin/romloader
 - lib/romloader.rb
+- lib/romloader/archive_extractor.rb
 - lib/romloader/freeroms_scraper.rb
 - lib/romloader/game_rom.rb
 - lib/romloader/game_system.rb
 - lib/romloader/romloader_cli.rb
-- lib/romloader/scraping_error/no_element_found.rb
+- lib/romloader/scraping_error/errors.rb
 homepage: https://github.com/jinstrider2000/romloader-cli-gem
 licenses:
 - MIT

data/lib/romloader/scraping_error/no_element_found.rb

@@ -1,7 +0,0 @@
-module ScrapingError
-
-  # A custom error to signal when scraping has returned no results 
-  class NoElementFound < StandardError 
-  end
-
-end