weatherpup 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ccf304b879c999b5e58da02ad948c15dc7041847c94e362a970ceaaa20abdf7b
-  data.tar.gz: 32c5cc102476833813df34b4d432c02b999ff2e42b8b7cb8dc0163dfee20282b
+  metadata.gz: f1e92311036112a9b35d4eb6b651a067c56a2043b7a978c23de3787649103e17
+  data.tar.gz: b300cb0a6f00afa06ea04fc01775397e4ec2f1ec123cd189e8509ce6e70e95df
 SHA512:
-  metadata.gz: 9430501c9668e79f68369d01150cd615ce05ccee7b954e8e7ef2cb4607d5e2490d07d7ce16ac03ee83426a22fbf2af55c4d976a9fe6e47dfa18e6e6f4715e4cb
-  data.tar.gz: af63828fdc866c270e200d87ef2b935a5c4d3b751696007164b604a5fa8f63648ceeed97b623d7681293208c86629e85cacc36862f1a30c8f466952bac9cecac
+  metadata.gz: 6ff6964a4bdcf7b80e2a2d8617c807b510e5a3a28c14c3e7f2c303404b04d222fef41626e6094f6b1c40da6d2a269d053261be83b2ca2e21f24944149ef935fb
+  data.tar.gz: ad7d512aa35ca94df9d87c3c9beb3d2be15c6ccd8f486cc4d2991ec28955a22ea15bfd73d8f3d299d44230aee16ccf06a9fa6ea43f267b1a10cddbeb32453ae8

data/CHANGELOG.md

@@ -1,3 +1,4 @@
 Version 0.1.0: Initial working version
 Version 0.1.1: Updated gemspec file
-Version 0.1.2: Fixed bug that caused it not to load
+Version 0.1.2: Fixed bug that caused it not to load
+Version 0.1.3: Improved code readability, added additional abstractions

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    weatherpup (0.1.1)
+    weatherpup (0.1.3)
       colorize (~> 0.8)
       httparty (~> 0.17)
 

data/README.md

@@ -1,6 +1,6 @@
 # WeatherPup
 
-Welcome to WeatherPup!  WeatherPup is a CLI written in Ruby that fetches current weather information based on a user entered US Zip Code or a GPS Coordinate Pair (Latitude and Longitude)
+Welcome to WeatherPup!  WeatherPup is a CLI written in Ruby that fetches current weather information based on a user entered US Zip Code or a GPS Coordinate Pair (Latitude and Longitude).  Previously fetched weather conditions can also be viewed.
 
 ## Installation
 
@@ -30,9 +30,9 @@ After running the `weatherpup` executable from the `bin` folder, you'll be prese
 
 Type in `1` to get the current weather conditions by US Zip Code.  After doing this, you'll be asked to type in the 5 digit zip code.  You'll then be shown the current weather for that zip code.  From there you'll have the option to go `back` to the main menu.
 
-Type in `2` to get the current weather conditions by GPS Coordinates (Latitude and Longitude pair).  After doing this, you'll be asked to type in the latitude in decimal format. Example Latitude in decimal format: `40.705204` 
+Type in `2` to get the current weather conditions by GPS Coordinates (Latitude and Longitude pair).  After doing this, you'll be asked to type in the _latitude_ in decimal format. Example Latitude in decimal format: `40.705204` 
 
-You'll then be asked to type in the longitude in decimal format. Example Longitude in decimal format: `-74.013845`
+You'll then be asked to type in the _longitude_ in decimal format. Example Longitude in decimal format: `-74.013845`
 
 Then you'll be shown the current weather for that GPS Coordinate pair.  From there you'll have the option to go `back` to the main menu.
 

data/lib/weatherpup.rb

@@ -1,4 +1,4 @@
-#This is acting as my environment file. 
+#This is acting as my environment file.  Leaving it called "weatherpup.rb" per rubygem standards. 
 require_relative "./weatherpup/version"
 require_relative "./weatherpup/cli"
 require_relative "./weatherpup/weather_conditions"
@@ -6,9 +6,10 @@ require_relative "./weatherpup/weather_conditions"
 require 'httparty'
 require 'colorize'
 require 'yaml'
-require 'pry'
 
+#Read in a YAML file that contains all the valid US zip codes -- provided by www.aggdata.com
 VALID_US_ZIP_CODES = YAML.load(File.read("./lib/weatherpup/valid_us_zip_codes.yml"))
+
 APPID = "e4ece4d3cbcfdd05d69889c6753d57da"
 
 #These are the ranges that will allow me to figure out the direction indicator is when I get back information from the weather API for the wind direction in degrees

data/lib/weatherpup/cli.rb

@@ -1,8 +1,10 @@
 # CLI Controller
 class WeatherPup::CLI
    def call
+      #Clears the screen to make the interface look nicer / cleaner
       system "clear"
-      #puts "Welcome to WeatherPup!"  #simple intro
+      
+      #Welcome the user
       puts <<~WELCOME
       　　　　　 ▒█░░▒█ █▀▀ █░░ █▀▀ █▀▀█ █▀▄▀█ █▀▀ 　 ▀▀█▀▀ █▀▀█    
       　　　　　 ▒█▒█▒█ █▀▀ █░░ █░░ █░░█ █░▀░█ █▀▀ 　 ░▒█░░ █░░█ 
@@ -18,12 +20,16 @@ class WeatherPup::CLI
            　                ░▄▀░██▀██▀▀▀▀▀██▀▀▄
            　                ░░░░█▄░▀█▄░░░░▀█▄▀▀
       WELCOME
-      main_menu
+      #Run the main menu
+      self.main_menu
    end
 
    def main_menu
-      input = "no input"
+      #Initialize my trigger variable "input" for my until loop
+      input = "no input yet"
       until input.downcase == 'exit'
+         
+         #Show the user their options and provide instructions on how to use.
          puts <<~MAINMENU
 
             #{"How would you like me to fetch the current weather conditions for you today?".colorize(:light_red)}
@@ -34,8 +40,11 @@ class WeatherPup::CLI
 
             Please type #{"1".colorize(:green)}, #{"2".colorize(:light_blue)}, #{"3".colorize(:cyan)} or type #{"exit".colorize(:red)} to quit.
          MAINMENU
+         
+         #get input from the user
          input = gets.chomp.downcase
 
+         #Check that input for a valid input.  If valid, then do what the user selected. Else, tell them the input is not valid and display the options again. 
          case input
          when "1"
             self.fetch_by_zip
@@ -49,20 +58,21 @@ class WeatherPup::CLI
             puts "\nSorry, that isn’t a valid input.".colorize(:red)
          end
       end
+      
+      #Once the user types exit, clear the screen and then display the goodbye message
       system "clear"
       self.goodbye
    end
 
-   #fetch the current conditions via zip code input from the user
+   #fetches the current conditions via zip code input from the user
    def fetch_by_zip
-      #may not need zip_code variable declaration here) 
-      #zip_code = nil
+      #initialize my trigger variable zip_code_valid for my until loop
       zip_code_valid = nil
       
       #Clear my screen first to make things look nicer 
       system "clear"
 
-      #Ask user for the zip code
+      #Ask user for the zip code until we get a valid input. 
       puts "\nOk, Current Weather Conditions by Zip Code!".colorize(:green)
       until zip_code_valid
          puts "\nPlease enter in the 5 Digit Zip Code:".colorize(:green)
@@ -74,25 +84,25 @@ class WeatherPup::CLI
             break
          end
 
-         #Otherwise, check to see if what the user put in was valid
+         #Otherwise, run #zip_code_valid to check to see if what the user put in was valid
          zip_code_valid = zip_code_valid?(zip_code)
 
-         #If the zipcode input is valid, clear the screen, instantiate a new instance of WeatherConditions class, go fetch the raw api data, process it into an attributes hash to mass assign, mass assign the attributes of the WeatherConditions intance using #write_attributes, then print the current conditions to screen gps
+         #If the zipcode input is valid, get and display the weather info. 
          if zip_code_valid
+            #clear the screen
             system "clear" 
-            zip_weather_conditions = WeatherPup::WeatherConditions.new
-            zip_weather_conditions.zip_code = zip_code
-            api_raw_data = zip_weather_conditions.zip_api_fetch(zip_code)
-            api_processed_data_hash = zip_weather_conditions.zip_process_api_data_to_attribs_hash(api_raw_data)
-            
-            #This next line takes the zip_weather_conditions variable which is a WeatherConditions Object Instance, taps into it and writes all of the attributes that I collected, then it prints the information located in the object itself.
-            zip_weather_conditions.tap {|weather_conditions_obj| weather_conditions_obj.write_attributes(api_processed_data_hash)}.print_zip_conditions
+ 
+            #Do the actual work of getting the data from the API, processing it, and printing it to screen
+            self.get_and_print_by_zip(zip_code)
             
             #Next I wait for the user to type in "back" to return to the main menu
-            return_to_main_menu_prompt
+            self.return_to_main_menu_prompt
+            
+            #Once the user types back, clear the screen and break out of this until loop
             system "clear"
             break
          else
+            #If given junk data (invalid zip, jibberish input, etc) tell the user its not valid then tell them how to get back to the main menu if they want.
             puts <<~INVALID_ZIP
                \n#{"Invalid zip code detected!".colorize(:light_red)}
                
@@ -101,62 +111,77 @@ class WeatherPup::CLI
          end
       end    
    end
-   
-   def return_to_main_menu_prompt
-      return_to_main = ""
-      until return_to_main.downcase == "back"
-         puts "\nType #{"back".colorize(:red)} to return to the main menu."
-         return_to_main = gets.chomp
-      end
-   end
 
-   
+   #Input validation method - Checks the VALID_US_ZIP_CODES constant to see if the zip code is real
    def zip_code_valid?(zip_to_check)
-      #checks the VALID_US_ZIP_CODES constant to see if the zip code is real
       VALID_US_ZIP_CODES.include?(zip_to_check)
+   end   
+   
+   def get_and_print_by_zip(zip_code)
+      #Instantiate a new instance of WeatherConditions class
+      zip_weather_conditions = WeatherPup::WeatherConditions.new
+ 
+      #set the zip_code attribute to the zip code entered by the user
+      zip_weather_conditions.zip_code = zip_code
+
+      #go fetch the raw api data
+      api_raw_data = zip_weather_conditions.zip_api_fetch(zip_code)
+
+      #process raw data into an attributes hash to use with mass assignment method #write_attributes
+      api_processed_data_hash = zip_weather_conditions.zip_process_api_data_to_attribs_hash(api_raw_data)
+      
+      #This next line takes the zip_weather_conditions variable which is a WeatherConditions Object Instance, taps into it and writes all of the attributes that I collected & processed from the API, then it prints the information located in the object itself to the user.
+      zip_weather_conditions.tap {|weather_conditions_obj| weather_conditions_obj.write_attributes(api_processed_data_hash)}.print_zip_conditions
    end
 
+
    #fetch the current conditions via zip code input from the user
    def fetch_by_gps
-      latitude = nil
-      longitude = nil
+      #initialize my trigger variable valid_coordinates for my until loop
       valid_coordinates = nil
+
+      #Clear my screen first to make things look nicer 
       system "clear"
-      puts "\nOk, Current Weather Conditions by GPS coordinates!".colorize(:blue)
+      
+      #Ask user for the latitude and longitude until we get a valid input. 
+      puts "\nOk, Current Weather Conditions by GPS coordinates!".colorize(:light_blue)
       
       until valid_coordinates
          #Grab the Lat from the user
-         puts "\nPlease enter in the ".colorize(:blue) + "latitude".colorize(:green).underline + " in decimal notation:".colorize(:blue)
+         puts "\nPlease enter in the ".colorize(:light_blue) + "latitude".colorize(:green).underline + " in decimal notation:".colorize(:light_blue)
          latitude = gets.chomp
          
-         #Checking to see if the user wants out of this loop and back to the main menu
+         #Checking to see if the user wants out of this loop and back to the main menu before proceeding
          if latitude.downcase == "back"
             system "clear"
             break
          end
 
-         #Grab the Long from the user
-         puts "\nPlease enter in the ".colorize(:blue) + "longitude".colorize(:magenta).underline + " in decimal notation:".colorize(:blue)
+         #Grab the Longitude from the user
+         puts "\nPlease enter in the ".colorize(:light_blue) + "longitude".colorize(:magenta).underline + " in decimal notation:".colorize(:light_blue)
          longitude = gets.chomp
+         
          #Checking to see if the user wants out of this loop and back to the main menu
          if longitude.downcase == "back"
             system "clear"
             break
          end
 
-         #Checking to see if the coordinates are valid using class method #valid_coordinate_pair?
+         #Checking to see if the coordinates are valid using method #valid_coordinate_pair?
          valid_coordinates = self.valid_coordinate_pair?(latitude, longitude)
          
-         # If the coordinates are valid, then then instantiate a new instance of WeatherConditions Object, hit the weather conditions api, write the object's attributes using mass assignment (#write_attributes), then #print_gps_conditions to screen
+         # If the coordinates are valid, get and display the weather info.
          if valid_coordinates
+            #Clear the screen to make it look nicer 
             system "clear" 
-            gps_weather_conditions = WeatherPup::WeatherConditions.new
-            api_raw_data = gps_weather_conditions.gps_api_fetch(latitude, longitude)
-            api_processed_data_hash = gps_weather_conditions.gps_process_api_data_to_attribs_hash(api_raw_data)
-            #This next line takes the gps_weather_conditions variable which is a WeatherConditions Object Instance, taps into it and writes all of the attributes that I collected, then it prints the information located in the object itself.
-            gps_weather_conditions.tap {|weather_conditions_obj| weather_conditions_obj.write_attributes(api_processed_data_hash)}.print_gps_conditions
+
+            #Do the actual work of getting the data from the API, processing it, and printing it to screen
+            self.get_and_print_by_gps(latitude, longitude)
             
-            return_to_main_menu_prompt
+            #Next I wait for the user to type in "back" to return to the main menu 
+            self.return_to_main_menu_prompt
+
+            #Once the user types back, clear the screen and break out of this until loop
             system "clear"
             break
          else
@@ -181,44 +206,62 @@ class WeatherPup::CLI
       #returns true if the lat and long coordinate pair is valid.
    end
 
+   def get_and_print_by_gps(latitude, longitude)
+      #Instantiate a new instance of WeatherConditions class
+      gps_weather_conditions = WeatherPup::WeatherConditions.new
+
+      #go fetch the raw api data
+      api_raw_data = gps_weather_conditions.gps_api_fetch(latitude, longitude)
+      
+      #process raw data into an attributes hash to use with mass assignment method #write_attributes
+      api_processed_data_hash = gps_weather_conditions.gps_process_api_data_to_attribs_hash(api_raw_data)
+      #This next line takes the gps_weather_conditions variable which is a WeatherConditions Object Instance, taps into it and writes all of the attributes that I collected, then it prints the information located in the object itself to the user.
+      gps_weather_conditions.tap {|weather_conditions_obj| weather_conditions_obj.write_attributes(api_processed_data_hash)}.print_gps_conditions
+   end
+
    def fetch_previous
+      #Clear the screen to make it look nicer. 
       system "clear"
+
+      #first check to see if there have been any previous checks -- if the array is blank, tell the user that there isn't anything to show.  Otherwise, print out a list of each WeatherConditions instance with details for the user pick from. 
       if WeatherPup::WeatherConditions.all == []
          system "clear"
          puts "\nThere are no previous fetches to display!".colorize(:cyan)
-         return_to_main_menu_prompt
+         self.return_to_main_menu_prompt
          system "clear"
       else
          #puts a blank line to give some headroom when displaying
          puts "\n"
-         #Can possibly abstract this next section into a method called "list_all_previous" to make the #fetch_previous method a bit less cumbersome to read
-         WeatherPup::WeatherConditions.all.each.with_index(1) do |wc_obj, index|
-            case wc_obj.current_conditions_means
-            when "Zip Code"
-               puts "#{index}. Weather by Zip Code: #{wc_obj.zip_code.colorize(:green)} (#{wc_obj.city_name.colorize(:green)}) fetched at #{wc_obj.when_fetched.colorize(:red)}"
-            when "GPS Coordinates"
-               puts "#{index}. Weather by GPS: #{wc_obj.lat.colorize(:light_blue)}, #{wc_obj.long.colorize(:light_blue)} (#{wc_obj.city_name.colorize(:light_blue)}) fetched at #{wc_obj.when_fetched.colorize(:red)}"
-            end
-         end 
+         
+         #ask the WeatherConditions class to print out a list of all its instances
+         WeatherPup::WeatherConditions.list_all_previous
 
+         #set my valid input range between 1 instance and however many instances of WeatherConditions there are in WeatherConditions @@all array.
          valid_input_range = 1..WeatherPup::WeatherConditions.all.length
+         
+         #Initialize my trigger to exit my until loop
          valid_input = nil
          
+         #Until the user gives me a valid input, keep asking for a vaild input & give them the option to go back to the main menu.  If they do give me valid input, then go display that historic WeatherConditions object.
          until valid_input
             puts <<~USER_PROMPT
                \nPlease type in the #{"number".colorize(:cyan)} of the previous fetch you would like to view
                or type #{"back".colorize(:red)} to return to the main menu.
             USER_PROMPT
-            user_selection = gets.chomp
+            
+            #get the input fro mthe user
+            user_selection = gets.chomp.downcase
 
-            if user_selection.downcase == "back"
+            if user_selection == "back"
                system "clear"
                break
             end
 
+            #convert my user input to an integer so that I can check to see if its in the valid range. 
             user_integer = user_selection.to_i
             valid_input = valid_input_range.member?(user_integer)
 
+            #If it is in the valid range, go and print out the selected WeatherConditions objects info again.  Otherwise, keep asking for valid input and tell them how to go back. 
             if valid_input 
                selected_wc_obj = WeatherPup::WeatherConditions.all[user_integer - 1]
                type = selected_wc_obj.current_conditions_means
@@ -227,13 +270,13 @@ class WeatherPup::CLI
                when "Zip Code"
                   system "clear"
                   selected_wc_obj.print_zip_conditions
-                  return_to_main_menu_prompt
+                  self.return_to_main_menu_prompt
                   system "clear"
                   break
                when "GPS Coordinates"
                   system "clear"
                   selected_wc_obj.print_gps_conditions
-                  return_to_main_menu_prompt
+                  self.return_to_main_menu_prompt
                   system "clear"
                   break
                end
@@ -243,7 +286,18 @@ class WeatherPup::CLI
          end
       end
    end
+
    
+
+   #This helper method displays instructions for the user to return to the main menu and then waits for the user to type in "back" before proceeding
+   def return_to_main_menu_prompt
+      return_to_main = ""
+      until return_to_main.downcase == "back"
+         puts "\nType #{"back".colorize(:red)} to return to the main menu."
+         return_to_main = gets.chomp
+      end
+   end
+
    #Says thank you and goodbye to user. 
    def goodbye
       system "clear"

data/lib/weatherpup/version.rb

@@ -1,3 +1,3 @@
 module WeatherPup
-  VERSION = "0.1.2"
+  VERSION = "0.1.3"
 end

data/lib/weatherpup/weather_conditions.rb

@@ -3,32 +3,35 @@ class WeatherPup::WeatherConditions
                  :temperature, :pressure, :humidity, :current_weather_description, :pressure, 
                  :wind_speed, :wind_direction_indicator, :wind_direction_indicator_string, :reading_date_and_time, 
                  :city_name, :when_fetched
-   #With this class I should be able to create a WeatherConditions instance by Zip code or GPS coordinates
+   #With this class I should be able to create a WeatherConditions instance by Zip code or GPS coordinates.  Also I should be able to list out all the previous WeatherConditions instances that I have.
 
+   #WeatherConditions instance tracker array class variable
    @@all = []
 
+   #Adds the current WeatherConditions instance to the @@all instance tracker
    def initialize
       @@all << self
    end
    
+   #Class method used to read the all the instances of WeatherConditions in existence.
    def self.all
       @@all
    end
 
+   #Go actually hit the OpenWeatherMap Api and then return all the information 
    def zip_api_fetch(zip_code, country_code = "us")
       #this will actually hit the OpenWeatherMap Api
       #I have defaulted the country code to US for now, but have built the GET request to be flexible to add multi-country postal codes later
       HTTParty.get("https://api.openweathermap.org/data/2.5/weather?zip=#{zip_code},#{country_code}&APPID=#{APPID}&units=imperial")
    end
 
-   def zip_process_api_data_to_attribs_hash(api_data)
-      #May want to separate out this wind direction checker or completely wipe out the wind direction information all together for GPS feature
-      #REFACTOR: take out the line below and rename the arg to api_info
-      api_info = api_data
+   #Process the raw API data that was grabbed by #zip_api_fetch and and output a hash that will be used by #write_attributes to mass assign WeatherConditions instance attributes
+   def zip_process_api_data_to_attribs_hash(api_info)
+      #massage the wind_direction information before writing it to the attributes hash.
       wind_direction_indicator_deg = api_info["wind"]["deg"].round
       wind_direction_indicator_string = wind_direction_indicator_to_text(wind_direction_indicator_deg)
 
-      #creates attributes_hash_for_mass_assignment to be used by the #write_attributes method
+      #creates attributes hash for mass assignment to be used by the #write_attributes method
       {
          :current_weather_description => api_info["weather"][0]["main"],
          #I'm leaving :current_condtions_means in here in case I want to combine my print_gps and print_zip methods, otherwise, it's not needed at the moment
@@ -45,18 +48,16 @@ class WeatherPup::WeatherConditions
       }
    end   
 
+   #Go actually hit the OpenWeatherMap Api and then return all the information 
    def gps_api_fetch(latitude, longitude)
       
-      #I think I want to split this out into two methods -- #gps_api_data_fetch which will just do the fetch based on the Lat, Longs given to it.  Then have another method, #create_api_data_hash that will do the rest of this in another method.  The #create_api_data_hash method will need to be run in the #new_by_gps method and the create_api_data_hash method will call the #gps_apa_data_fetch method and save that into a variable called api_info 
-
-      #this will actually hit the OpenWeatherMap Api and then return all the information 
       HTTParty.get("https://api.openweathermap.org/data/2.5/weather?lat=#{latitude}&lon=#{longitude}&APPID=#{APPID}&units=imperial")
    end
 
-   def gps_process_api_data_to_attribs_hash(api_data)
-      #May want to separate out this wind direction checker or completely wipe out the wind direction information all together for GPS feature
-      
-      api_info = api_data
+   #Process the raw API data that was grabbed by #gps_api_fetch and and output a hash that will be used by #write_attributes to mass assign WeatherConditions instance attributes
+   def gps_process_api_data_to_attribs_hash(api_info)
+      #the data from the API for Wind direction for a GPS check is not as reliable and sometimes is missing.
+      #Check to see if the data is there first, then write it out the variables used in the hash
       wind_direction_indicator_deg = api_info["wind"]["deg"]
       if wind_direction_indicator_deg.nil?
          wind_direction_indicator_deg = "No Data"
@@ -66,9 +67,8 @@ class WeatherPup::WeatherConditions
          wind_direction_indicator_string = wind_direction_indicator_to_text(wind_direction_indicator_deg_rounded)
          wind_direction_indicator_deg = wind_direction_indicator_deg.round.to_s + "°"
       end
-      #NOTE: REFACTOR Needed / Bug -- if you do another city I need to localize the time to that location's time zone - especially for sunrise and sunset
 
-      #creates attributes_hash_for_mass_assignment to be used by the #write_attributes method
+      #creates attributes hash for mass assignment to be used by the #write_attributes method
       {
          :current_weather_description => api_info["weather"][0]["main"],
          #I'm leaving :current_condtions_means in here in case I want to combine my print_gps and print_zip methods, otherwise, it's not needed at the moment
@@ -87,13 +87,14 @@ class WeatherPup::WeatherConditions
       }
    end
 
+   #Writes attributes using mass assignment to the WeatherConditions object it is called on when given a processed api hash as an arg.
    def write_attributes(processed_api_data_hash)
-      #this will take whatever the API hash was and create attributes for the object that it's run on.
       processed_api_data_hash.map do |key, value|
          self.send("#{key}=", value)
       end
    end   
 
+   #Prints Zip Conditions to screen from a WeatherConditions Object. 
    def print_zip_conditions
       puts <<~CONDITIONS
          \nThe weather conditions for #{self.city_name.colorize(:green).underline} (#{self.zip_code.colorize(:green)}):  
@@ -114,6 +115,7 @@ class WeatherPup::WeatherConditions
       CONDITIONS
    end
 
+   #Prints GPS conditions to screen from a WeatherConditions object. 
    def print_gps_conditions
       puts <<~CONDITIONS
          \nThe weather conditions for #{self.lat.colorize(:green)}, #{self.long.colorize(:green)} (#{self.city_name.colorize(:green)}):  
@@ -132,9 +134,9 @@ class WeatherPup::WeatherConditions
          **Weather Data provided by OpenWeatherMap.org**
       CONDITIONS
    end
-      
+   
+   #This method figures out what the direction indicator text should when given a numeric wind direction in degrees
    def wind_direction_indicator_to_text(wind_direction_in_degrees)
-      #This code may be a little smelly.  I may be able to do this by iterating over a hash of the constants and check wind_direction_in_degrees agains the value of each then return the key.  Meh... not sure. 
       indicator = nil
       if NORTH_RANGE_PART1.member?(wind_direction_in_degrees) || NORTH_RANGE_PART2.member?(wind_direction_in_degrees)
          indicator = "N"
@@ -155,4 +157,16 @@ class WeatherPup::WeatherConditions
       end
       indicator
    end 
+
+   #This method prints a list out all the instances of WeatherConditions to screen
+   def self.list_all_previous
+      self.all.each.with_index(1) do |wc_obj, index|
+         case wc_obj.current_conditions_means
+         when "Zip Code"
+            puts "#{index}. Weather by Zip Code: #{wc_obj.zip_code.colorize(:green)} (#{wc_obj.city_name.colorize(:green)}) fetched at #{wc_obj.when_fetched.colorize(:red)}"
+         when "GPS Coordinates"
+            puts "#{index}. Weather by GPS: #{wc_obj.lat.colorize(:light_blue)}, #{wc_obj.long.colorize(:light_blue)} (#{wc_obj.city_name.colorize(:light_blue)}) fetched at #{wc_obj.when_fetched.colorize(:red)}"
+         end
+      end
+   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: weatherpup
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Jeremiah Rodden
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-07-11 00:00:00.000000000 Z
+date: 2019-07-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler