newegg_scraper_chsbr 0.1.3 → 0.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 71b56640f51ce8b1c269b6a76524aff3d7c57924639c9b8267ab86c51e575c2c
-  data.tar.gz: 825f95f41ede0a578f1035d9f169d175ff268388bcdb836134e8d9abf4f6c4ad
+  metadata.gz: f34ca8fd0dbc560e05743c291654c9bdd092cc8c30c534978898666fdad5eaa3
+  data.tar.gz: 55eec0e26900218bf9ad0ae8ae37c65863add7639365a3df818ece9625e0aaa9
 SHA512:
-  metadata.gz: 7e6effb7aa9bd7b9fbe4a35bf1be6fef37c429bb60ad919e4749e621fef47c3da913ebf849aebd7e45b7b5caa37733ea809d1e22a0fabc7b2b0a47131991a35a
-  data.tar.gz: 88bfeaa3403f4bebebc09bd96d21d370bfabd52e2e65528f6120edd56ea23d5c9bce5dc65a8c54a6ecbe81d991ec6d849f2d9399be9bc9e554ebeca8464ce30e
+  metadata.gz: b50c23b2586cb80492649ed42cfce04f1f1e099baa4d99ddf9539fcfd0b66013f989e47ed56fd2a6f2fc0f9fe8428a338ac6e5d0b16acd993acb6fc8c06f39a9
+  data.tar.gz: 1bf45a023b6508aa5c2bd90996928291667945e4c6e588c7776da5f5dbc6232e684f9f598fbc865b605d4fd9054d97b808fdb08554497c59e895297bb9d45b24

data/lib/environment.rb

@@ -1,6 +1,8 @@
 require 'open-uri'
 require 'nokogiri'
 require "bundler/setup"
+
+
 require_relative "./newegg_scraper_chsbr/version.rb"
 require_relative './newegg_scraper_chsbr/data_grabber.rb'
 require_relative './newegg_scraper_chsbr/scraper.rb'

data/lib/newegg_scraper_chsbr/cli.rb

@@ -1,4 +1,3 @@
-
 class NeweggScraperChsbr::CLI
     attr_reader :user
     def initialize
@@ -6,26 +5,54 @@ class NeweggScraperChsbr::CLI
     end
     def call
         puts "Were you interested in purchasing some CPU's?"
-        puts "Please enter y to do so."
+        puts "Please enter y to do so."  
         @counter = gets.strip
-        while @counter == 'y' do
+        corr_input = control_bool
+        while corr_input
+            puts "Were you interested in purchasing some CPU's?"
+            puts "Please enter y to do so.\nPlease enter n or exit to quit."  
+            @counter = gets.strip
+            corr_input = control_bool
+        end
+        while @counter == 'y'
+            get_price_range
             get_init_data_for_user
-            NeweggScraperChsbr::Cpu.display_cpu(@user)
-            @user.choose_cpu
+            printed_cpus = NeweggScraperChsbr::Cpu.display_cpu(@user)
+
+
+            @user.choose_cpu(printed_cpus)
+    
+            
             NeweggScraperChsbr::Cpu.display_cpu_with_extras(@user)
-            puts "If you would like to continue please enter : #{@counter}"
+            puts "If you would like to restart the program please enter : #{@counter}.\nTo exit type n or exit."
             @counter = gets.strip
+            corr_input = control_bool
+            while corr_input
+                puts "Were you interested in purchasing some CPU's?"
+                puts "Please enter y to do so.\nPlease enter n or exit to quit."  
+                @counter = gets.strip
+                corr_input = control_bool
+            end
+            
         end
         puts "Thanks for using Chesbro's Scraper!"
     end
     def get_init_data_for_user
         
         if @counter == 'y'
-            get_price_range
+            
             puts "Are you searching for Intel or AMD?"
             puts "Please put i for Intel, or a for AMD."
             puts "Please put x for either."
             @cpu_make = gets.strip
+            not_found = cpu_make_bool
+            while not_found
+                puts "Are you searching for Intel or AMD?"
+                puts "Please put i for Intel, or a for AMD."
+                puts "Please put x for either."
+                @cpu_make = gets.strip
+                not_found = cpu_make_bool
+            end
             puts "\n\nYou want to search for (an) #{NeweggScraperChsbr::Cpu.cpu_maker(@cpu_make)} CPU."
             if @max_price.to_i != 0 && @min_price.to_i != 0
                 puts "You entered a maximum price of $#{@max_price}, and a minimum price of $#{@min_price}."
@@ -40,28 +67,62 @@ class NeweggScraperChsbr::CLI
         end
     end
     def if_correct_info
-        puts "\nIf you're satisfied with your selection, the computer will print\na list of CPUs that satisfy the criteria. Enter y to continue."
+        puts "\nIf you're satisfied with your selection, the computer will print\na list of CPUs that satisfy the criteria. Enter y to continue. Enter n to choose again."
         @counter = gets.strip
         if @counter == 'y'
             @user = NeweggScraperChsbr::User.new(@cpu_make, @min_price, @max_price)
         else
-            get_init_data_for_user
+            @counter = 'y'
+            call
         end
     end
     def get_price_range
         puts "Welcome. Would you like to set a budget? If no, set your maximum budget to 0"
+        puts "Please enter the integer value of your maximum price."
+        puts "$500.50, please enter as 501, or 500 to stay below budget."        
+        @max_price = gets.strip
+        while @max_price.to_i < 0 || @max_price.to_i > 10000 || @max_price.include?("$") || @max_price.include?(".") || @max_price == nil
+            puts "Welcome. Would you like to set a budget? If no, set your maximum budget to 0"
             puts "Please enter the integer value of your maximum price."
             puts "$500.50, please enter as 501, or 500 to stay below budget."        
             @max_price = gets.strip
-            if @max_price.to_i == 0
-                @min_price = "0"
-            else
+        end
+        if @max_price.to_i != 0
+            puts "Please enter the minimum you would spend on a CPU."
+            puts "Please use the same format as above."
+            @min_price = gets.strip
+            
+            while @min_price.to_i <= 0  || @min_price.to_i >= @max_price.to_i || @min_price.include?("$") || @min_price.include?(".") || @min_price == nil
                 puts "Please enter the minimum you would spend on a CPU."
                 puts "Please use the same format as above."
                 @min_price = gets.strip
             end
-        
+        end
     end
+    def control_bool
+        if @counter.include?("y")
+            false
+        elsif @counter.include?("exit")
+            false
+        elsif @counter.include?("n") 
+            false
+        else
+            true
+        end
+    end
+    def cpu_make_bool
+        if @cpu_make.include?("x")
+            @cpu_make = "x"
+            false
+        elsif @cpu_make.include?("a")
+            @cpu_make = "a"
+            false
+        elsif @cpu_make.include?("i")
+            @cpu_make = "i"
+            false
+        else
+            true
+        end
 
-
+    end
 end

data/lib/newegg_scraper_chsbr/cpu.rb

@@ -6,7 +6,7 @@ class NeweggScraperChsbr::Cpu
         @name = NeweggScraperChsbr::Name.new(name)
         @price = NeweggScraperChsbr::Price.new(price)
         @desc = NeweggScraperChsbr::Descriptors.new(desc)
-        @shipping = NewEggScraperChsbr::Shipping.new(shipping)
+        @shipping = NeweggScraperChsbr::Shipping.new(shipping)
         @@all << self
     end
     def self.all
@@ -23,32 +23,50 @@ class NeweggScraperChsbr::Cpu
     end
     def self.display_cpu(user) 
         counter = 1
+        printed_cpus = []
         @@all.each do | computer |
             if user.max_price.to_i != 0 && user.min_price.to_i == 0
-                binding.pry
+                
 
                 user.min_price = "1"
             end
             if user.max_price.to_i == 0 &&
                 user.min_price.to_i == 0 &&
                 (NeweggScraperChsbr::Cpu.cpu_maker(user.cpu_make) == "Intel" || NeweggScraperChsbr::Cpu.cpu_maker(user.cpu_make) == "AMD")
-                    computer.name.name.include?(NeweggScraperChsbr::Cpu.cpu_maker(user.cpu_make)) ? puts("\n  #{counter}:\n     Name: #{computer.name.name}\n     Price: #{computer.price.price}") : nil
-            elsif user.max_price.to_i == 0 &&
-                user.min_price.to_i == 0 &&
-                user.cpu_make == 'x'
+                    if computer.name.name.include?(NeweggScraperChsbr::Cpu.cpu_maker(user.cpu_make))
+                        puts("\n  #{counter}:\n     Name: #{computer.name.name}\n     Price: #{computer.price.price}")
+                        printed_cpus << counter
+                    else
+                        nil
+                    end
+            elsif user.max_price.to_i == 0 && user.min_price.to_i == 0 && user.cpu_make == 'x'
+                    
                     puts("\n   #{counter}:\n     Name: #{computer.name.name}\n     Price: #{computer.price.price}")
+                    printed_cpus << counter
             elsif user.max_price.to_i >= computer.price.convert_price &&
                 user.min_price.to_i <= computer.price.convert_price &&
                 (NeweggScraperChsbr::Cpu.cpu_maker(user.cpu_make) == "Intel" || NeweggScraperChsbr::Cpu.cpu_maker(user.cpu_make) == "AMD")
-                    computer.name.name.include?(NeweggScraperChsbr::Cpu.cpu_maker(user.cpu_make)) ? puts("\n  #{counter}:\n     Name: #{computer.name.name}\n     Price: #{computer.price.price}") : nil
+                    if computer.name.name.include?(NeweggScraperChsbr::Cpu.cpu_maker(user.cpu_make)) 
+                        puts("\n  #{counter}:\n     Name: #{computer.name.name}\n     Price: #{computer.price.price}")
+                        printed_cpus << counter
+                    else
+                        nil
+                    end
             
             elsif user.max_price.to_i >= computer.price.convert_price &&
                 user.min_price.to_i <= computer.price.convert_price &&
                 user.cpu_make == 'x'
                     puts("\n  #{counter}:\n     Name: #{computer.name.name}\n     Price: #{computer.price.price}")
+                    printed_cpus << counter
+
+            elsif printed_cpus.size == 0
+                puts "You did not match any CPU criteria."
             end
+
             counter += 1
+            
         end
+        printed_cpus
 
     end
     def self.display_cpu_with_extras(user)

data/lib/newegg_scraper_chsbr/data_grabber.rb

@@ -1,4 +1,3 @@
-
 class NeweggScraperChsbr::DataGrabber
     attr_reader :cpus
 
@@ -9,7 +8,7 @@ class NeweggScraperChsbr::DataGrabber
         names = getNames
         descHash = getCpuDesc
         @cpus = makeCpus(prices, names, shipping_price, descHash)                     
-    end #end initialize
+    end 
 
     def getCpuDesc
         scraped = NeweggScraperChsbr::Scraper.new
@@ -18,7 +17,10 @@ class NeweggScraperChsbr::DataGrabber
         css_next_link = scraped.xml_obj.css  ".item-title"
         css_next_link.each_with_index do | element, index |
             if index != 0
-                pages << element.attributes["href"].value
+        
+                if element.attributes["href"] != nil
+                    pages << element.attributes["href"].text
+                end
             end
         end
         description = {}
@@ -44,19 +46,19 @@ class NeweggScraperChsbr::DataGrabber
 
 
     end
-    def split_price(html_element)               # This method checks each individual text element, this is to make sure the prices are accurate.
+    def split_price(html_element)              
         counter = 0
         until counter == 100  
-            if counter < 10                     # If the counter is less than 10, is necessary because the computer doesn't know we are working with money.
-                if html_element.text.include?(".0#{counter.to_s}")  #If the text element includes any dollar ammount below .10 is a neccessary check.
-                                                                    #Otherwise the computer would only check [.0.. .9], and we need it to check [.00.. .09]
-                    price = html_element.text.split ".0#{counter.to_s}" #Once we know it's the correct element we can assign it to the price variable.
-                                                                        #This is only necessary to be able to delete the data we are splitting off.
-                                                                        # (We are splitting anything after the cents amount because it was uneccessary.)
+            if counter < 10                    
+                if html_element.text.include?(".0#{counter.to_s}")  
+                                                                   
+                    price = html_element.text.split ".0#{counter.to_s}" 
+                                                                        
+                                                                        
                     price.delete_at 1
-                    return price, ".0#{counter.to_s}"                   # Return the dollar amount, and the cents amount in an array.
+                    return price, ".0#{counter.to_s}"                   
                 end
-            elsif counter >= 10                                     # This does the same as above, but for any number over 9.
+            elsif counter >= 10                                     
                 if html_element.text.include?(".#{counter.to_s}") 
 
                     price = (html_element.text.split ".#{counter.to_s}")
@@ -67,59 +69,35 @@ class NeweggScraperChsbr::DataGrabber
         counter += 1
         end
     end
-    def isCoolerOrMB?(name) # This methods name means is it a cooler or a motherboard?
-        name.include?("Water") || name.include?("Air") || name.include?("Motherboard")  # this will return true if
-                                                                                        # any of the products include
-                                                                                        # words that describe motherboards and/or 
-                                                                                        # cooling products.
+    def isCoolerOrMB?(name) 
+        name.include?("Water") || name.include?("Air") || name.include?("Motherboard") || name.include?("AIO") || name.include?("FLY")
     end
     def getPrice
         scraped_info = NeweggScraperChsbr::Scraper.new
-        css_price = scraped_info.xml_obj.css ".price-current"    # This will set the variable equal to the
-                                                            # html that contains what I need to find the price.
+        css_price = scraped_info.xml_obj.css ".price-current"    
 
-        prices = []                                         # This will hold all of the prices in an array.
-
-        css_price.each do | piece |                         # Going 1 level deep into the html obj that contains the price text element.
-
-            temp_price = split_price piece                  # split_price will split the price, and return
-                                                            # two variables in an array. The first element of
-                                                            # the array will be the dollar amount.
-                                                            # The second element will be the cent ammount because
-                                                            # I deemed it easier to remove anything after the
-                                                            # floating point, it was the only pattern that was
-                                                            # consistent.
+        prices = []                                        
 
+        css_price.each do | piece |                        
+            temp_price = split_price piece                 
             if temp_price != nil
-                temp_price.flatten!                             # When returned by #split_price the dollar amount is
-                                                                # nested inside of another array. I use flatten!
-                                                                # to return the flattened array to temp_price.
-
-                prices << "#{temp_price[0]}#{temp_price[1]}"    # Here prices is being shoveled the 
-                                                                # two elements, concatenated to create
-                                                                # a string similar to "$00.00"
+                temp_price.flatten!                             
+                prices << "#{temp_price[0]}#{temp_price[1]}"    
             end
 
-        end #end prices.each
+        end 
         prices
     end
     def getNames
         scraped = NeweggScraperChsbr::Scraper.new
-        names = []                                          # Names will hold the names of 
-                                                            # objects found on the page.
+        names = []                                          
 
-        css_name = scraped.xml_obj.css ".item-title"        # css_name will hold the html 
-                                                            # object to be enumerated upon.
+        css_name = scraped.xml_obj.css ".item-title"       
 
-        css_name.each_with_index do | name, index |         # css_name will be enumerated upon
-                                                            # to retrieve the strings.
+        css_name.each_with_index do | name, index |        
 
-            index != 0 ? names << name.text : nil           # I have to skip the first name
-                                                            # because it does not have a price.
-                                                            # After I skip the first name
-                                                            # I will shovel the text from the 
-                                                            # html object into the names array.
-        end #end names.each_with_index
+            index != 0 ? names << name.text : nil           
+        end 
         names
     end
     def getShipping
@@ -136,23 +114,13 @@ class NeweggScraperChsbr::DataGrabber
                                               
                                                             
         cpus = []                                                    
-        prices.each_with_index do | price, index |          # I will then enumerate on
-                                                            # each of the prices with their
-                                                            # index so that the enumation can
-                                                            # access the names array at the
-                                                            # same time.
+        prices.each_with_index do | price, index |         
 
-            if !isCoolerOrMB?(names[index])                 # Here the program check if
-                                                            # the name is a cooler
-                                                            # or motherboard because
-                                                            # there is a price associated
-                                                            # with the coolers on the
-                                                            # website.
-                
+            if !isCoolerOrMB?(names[index])                
                 cpus << NeweggScraperChsbr::Cpu.new(names[index], price, shipping[index], desc_hash[index])      
                                                             
-            end #end if
-        end #end prices.each_with_index
+            end 
+        end 
         cpus
     end
 end

data/lib/newegg_scraper_chsbr/name.rb

@@ -2,7 +2,7 @@
 
 class NeweggScraperChsbr::Name
     attr_accessor :name
-    # @@all = []
+ 
     def initialize(name)
         @name = name
     end

data/lib/newegg_scraper_chsbr/scraper.rb

@@ -4,7 +4,7 @@ class NeweggScraperChsbr::Scraper
     
     URL = "https://www.newegg.com/p/pl?d=cpu"
     def initialize(url = URL)
-        @xml_obj = Nokogiri::HTML(open(url))  # This opens the URL with open-uri, then parses it as an XML Object
+        @xml_obj = Nokogiri::HTML(URI.open(url)) 
     end
     
 

data/lib/newegg_scraper_chsbr/user.rb

@@ -7,14 +7,27 @@ class NeweggScraperChsbr::User
         @max_price = max_price
         @chosen_cpu = []
     end
-    def choose_cpu
-        puts "Please enter the number of the CPU you'd like to choose."
-        chosen = gets.strip.to_i
-        @chosen_cpu << chosen
-        puts "Would you like to add another CPU?\nEnter y to do so."
-        if gets.strip == 'y'
-            choose_cpu
+    def choose_cpu(printed_cpus)
+        puts "Please enter the number of the CPU you'd like to see more about."
+        chosen = gets.strip
+        while !is_numeric?(chosen)
+            puts "Please enter the number of the CPU you'd like to see more about."
+            chosen = gets.strip
+        end
+        if !printed_cpus.include?(chosen.to_i)
+            puts "You chose a CPU outside of your budget please choose another."
+            choose_cpu(printed_cpus)
+        end
+        @chosen_cpu << chosen.to_i
+        puts "Would you like to see more about another CPU?\nEnter y to do so.\nEnter n to see details about the chosen CPU(s)"
+            input = gets.strip
+            
+        if input == 'y'
+            choose_cpu(printed_cpus)
         end
         
     end
+    def is_numeric?(obj) 
+        obj.match(/\A[^+-]?\d+?(\^.\d+)?\Z/) == nil ? false : true
+     end
 end

data/lib/newegg_scraper_chsbr/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module NeweggScraperChsbr
-  VERSION = "0.1.3"
+  VERSION = "0.2.3"
 end

metadata

@@ -1,43 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: newegg_scraper_chsbr
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.2.3
 platform: ruby
 authors:
 - Lrd134
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-04-16 00:00:00.000000000 Z
+date: 2021-04-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.11.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.11.3
 - !ruby/object:Gem::Dependency
   name: open-uri
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.1'
 description: To scrape CPU's off of Newegg and display them nicely.
 email:
 - larryc3200@gmail.com