daily-trending-apps 0.1.7 → 0.1.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2f37b6f4c7eb7b32f7e42ca614fe3e44217c1c0f537b6cd3a219d951fe95c3d2
-  data.tar.gz: 00b5050ff39947692f921cd5c79f2a862218cb9331836a8f297ac73d86558b90
+  metadata.gz: 9d64862a39481fe7803dbc67e92bf17d1430e955308530bc5785c36d42d8b4ef
+  data.tar.gz: 179567ec2b353889b6cac6468a7d68b56d718672f18d064977e947c67a1d468f
 SHA512:
-  metadata.gz: 9a50592eaa10f7cff2e8dd4013ff2b2caa6389f4096ff45bab70b4469f39bd8cef523caec6fbc943cf306765e13fdfa5377ea7c6824df32e00a237592aa57430
-  data.tar.gz: 84513f514a08d4a815fe3af575a718ba023fb11d353a8b1a37e4d1863d9490fbe32dc3ef6cb32ecfbf2578589ece2b7c9b7cd20d6a3b0dc9c525cb5322fce26c
+  metadata.gz: c076c5c33ebd88ffee0ca7ee20811ddf39c8afa45ee46a0ccdd00bd111aa046e523ff255f2a9f3408be119b26aee586b42f3598ef609c4ae206e9e1d90a1ae31
+  data.tar.gz: 2888a2fc353c1d7fef7331bd15069d058854028c259f4a6f787ead460928bed88238e2b54df0de64a499bdeadefcac1f3ee5c076e652d96728106bb68908e67a

data/lib/daily_trending/app.rb

@@ -3,22 +3,13 @@ class DailyTrending::App
   @@all = []
 
 
-
-    # Iterates  #scrape_play_store method to # Makes the apps and assigns
-    #  the attributes available on Google play store
-
+  def self.all
+    @@all
+  end
 
 
-  # def self.make_apps
-  def self.new_from_index(a)
-      app = self.new
-      app.title = a.css('a.title').attribute('title').value
-      app.dev = a.css('a.subtitle').attribute('title').value
-      app.dev_url = ("https://play.google.com" + a.css('a.subtitle').attribute('href').value)
-      app.app_url = ("https://play.google.com" + a.css('a.title').attribute('href').value)
-      app.rating = a.css('div.tiny-star').attribute('aria-label').value.strip.slice(/\d.\S/)<<"/5 Stars"
-      app.price = a.at_css('span.display-price').text
-      app.exist? ? all.freeze : app.save
+  def self.find(input)
+    @@all[input.to_i-1]
   end
 
 
@@ -31,16 +22,7 @@ class DailyTrending::App
     @@all << self
   end
 
-  def self.all
-    @@all
-  end
-
-
-  def make_app(xml_info)
-    page = xml_info
-    self.genre = page.css('a.document-subtitle.category').text
-    self.description = page.css('div.show-more-content.text-body').text
-    self.con_rating = page.css('span.document-subtitle.content-rating-title').text
-    self.rate_cnt = page.css('span.rating-count').text
+  def open_in_browser
+    system("open '#{app_url}'")
   end
 end

data/lib/daily_trending/cli.rb

@@ -3,7 +3,6 @@ class DailyTrending::Cli
   def call
     list_apps
     menu
-    goodbye
   end
 
   def list_apps
@@ -11,9 +10,8 @@ class DailyTrending::Cli
     puts ""
     puts "  #{s}"+"New And Updated Apps!"+s
     puts ""
-
-    @apps = DailyTrending::Scraper.make_apps unless @apps != nil
-    @apps.each.with_index(1) do |app, i|
+    DailyTrending::Scraper.new.scrape_index_page
+    DailyTrending::App.all.each.with_index(1) do |app, i|
       puts <<-DOC
       #{i}. #{app.title.colorize(:blue)}
         #{app.rating}        Cost: #{app.price}
@@ -21,27 +19,31 @@ class DailyTrending::Cli
       DOC
     end
     puts ""
+    puts "Enter the number of the app you'd like more info on, or type exit"
   end
 
-    def menu
-      input = nil
-      until input == 'exit'
-        puts "Enter the number of the app you'd like more info on, or type exit"
-        input = gets.strip.downcase
-
-        if input.to_i > 0
-          app_info(@apps[input.to_i-1])
-        elsif input == 'list'
-          list_apps
-        else
-          puts "#{input} not an option, type list or exit" unless input == 'exit'
-        end
+
+  def menu
+    input = nil
+    until input == 'exit'
+      input = gets.strip.downcase
+
+      if input.to_i > 0 && input.to_i <= DailyTrending::App.all.size
+        app_info(DailyTrending::App.find(input))
+      elsif input == 'list'
+        list_apps
+      elsif input == 'exit'
+        goodbye
+      else
+        puts "  #{input} not an option, type list or exit" unless input == 'exit'
       end
     end
+  end
+
+
 
     def app_info(app)
-      xml_info = DailyTrending::Scraper.scrape_app(app.app_url)
-      app.make_app(xml_info)
+      DailyTrending::Scraper.new.scrape_app(app)
       puts <<-DOC
               #{app.title.upcase.colorize(:blue)}
                 Developers: #{app.dev}
@@ -58,23 +60,28 @@ class DailyTrending::Cli
       More Apps By #{app.dev}: #{app.dev_url.colorize(:red)}
 
       DOC
-    input = nil
+      # open_browser(app)
+
     puts "  Type 'list' to see apps again or 'exit' to leave"
-    until input == 'list'|| input == 'exit'
-      input = gets.strip.downcase
-      if input == 'list'
-        list_apps
-      elsif input == 'exit'
-        goodbye
-        exit
-      else
-        puts "  #{input} not an option, type list or exit" unless input == 'exit'
-      end
-    end
+      menu
   end
 
 
-  def goodbye
-    puts "See you next time!!"
-  end
+    def goodbye
+      puts "See you next time!!"
+      exit
+    end
+
+
+  # def open_browser(app)
+  #   puts "Would you like to visit this App page?? (y/n)"
+  #   input = gets.strip.downcase
+  #
+  #   if input == 'y'
+  #     app.open_in_browser
+  #   else
+  #     list_apps
+  #   end
+  # end
+
 end

data/lib/daily_trending/scraper.rb

@@ -1,19 +1,33 @@
 class DailyTrending::Scraper
 
-  def self.scrape_play_store
-    doc = Nokogiri::HTML(open("https://play.google.com/store/apps/collection/promotion_3000792_new_releases_apps?clp=SpEBCikKI3Byb21vdGlvbl8zMDAwNzkyX25ld19yZWxlYXNlc19hcHBzEAcYAxpkCl5uZXdfaG9tZV9kZXZpY2VfZmVhdHVyZWRfcmVjczJfdG9waWNfdjFfbGF1bmNoX2FwcHNfVVNfXzE1MTQ0NDgwMDAwMDBfNl9wcm9tb18xNTE0NDkxNzcwMTgwMDAwEAwYAw%3D%3D%3AS%3AANO1ljLrBj0&hl=en"))
-    doc.css('div.card-content.id-track-click.id-track-impression')
+
+  def initialize
+    @base_link = "https://play.google.com"
   end
+  # Why was instance variable @base_link not available in #scrape_app before I made #initialize??
+  # Didn't work in #scrape_app when I assigned it in #scrape_index_page
 
-  def self.make_apps
-    scrape_play_store.each do |a|
-      DailyTrending::App.new_from_index(a)
+  def scrape_index_page
+    index_page = Nokogiri::HTML(open("#{@base_link}/store/apps/collection/promotion_3000792_new_releases_apps?clp=SpEBCikKI3Byb21vdGlvbl8zMDAwNzkyX25ld19yZWxlYXNlc19hcHBzEAcYAxpkCl5uZXdfaG9tZV9kZXZpY2VfZmVhdHVyZWRfcmVjczJfdG9waWNfdjFfbGF1bmNoX2FwcHNfVVNfXzE1MTQ0NDgwMDAwMDBfNl9wcm9tb18xNTE0NDkxNzcwMTgwMDAwEAwYAw%3D%3D%3AS%3AANO1ljLrBj0&hl=en"))
+    index_page.css('div.card-content.id-track-click.id-track-impression').each do |a|
+      app = DailyTrending::App.new
+      app.title = a.css('a.title').attribute('title').value
+      app.app_url = (@base_link + a.css('a.title').attribute('href').value)
+      app.rating = a.css('div.tiny-star').attribute('aria-label').value.strip.slice(/\d.\S/)<<"/5 Stars"
+      app.price = a.at_css('span.display-price').text
+      app.exist? ? DailyTrending::App.all.freeze : app.save
     end
-    DailyTrending::App.all
   end
 
-  def self.scrape_app(app_url)
-    Nokogiri::HTML(open(app_url))
-  end
 
+  def scrape_app(app)
+    page = Nokogiri::HTML(open(app.app_url))
+    app.dev = page.css('a.document-subtitle.primary').text.strip
+    app.dev_url = (@base_link + page.css('a.document-subtitle.primary').attribute('href').value)
+    app.genre = page.css('a.document-subtitle.category').text.strip
+    app.description = page.css('div.show-more-content.text-body').text
+    app.con_rating = page.css('span.document-subtitle.content-rating-title').text
+    app.rate_cnt = page.css('span.rating-count').text
+    app
+  end
 end

data/lib/daily_trending/version.rb

@@ -1,3 +1,3 @@
 module DailyTrending
-      VERSION = "0.1.7"
+      VERSION = "0.1.8"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: daily-trending-apps
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.1.8
 platform: ruby
 authors:
 - "'Chad Montoya'"
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-01-06 00:00:00.000000000 Z
+date: 2018-01-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler