goodreads-books 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 89779d99588337f482f8d2d784e2af6917beeb3f620ec38ea345e1abbac6f621
-  data.tar.gz: d6fb61b1dd36ede436aa991c782cc39b2c091d20749f0025d77d2e5116cfd79c
+  metadata.gz: 9e4b864502b272cfc85b099364e5620d33808a4a6de5da2f2fa961cd25754676
+  data.tar.gz: b12f7f449a2701de75e55c9675a423b7b478a3c6e86318c773ba7fa549ff1865
 SHA512:
-  metadata.gz: 4593aca0913571d19fb263dbdf769d7d3442f471dee736f298c90290a9bc59819701c096f4bd118c06b9c9d487296a7b183f9b9bb604e8125ff6adf8561982e4
-  data.tar.gz: ea0407c0d895aa8e1064f6334956531c3048d164e2a1e859c3de54d5aa7fec12cf7f4e63d19d4a719e043d3fb0182c068cefa667518fe37ac2685a80662f843b
+  metadata.gz: b9352d2861fab111920c9f8d35662412bf4babf16658df60baefcc5b7897380927c8684a4ab27b41c490c1feb351a16ca2a18fdb58b7cdcddc74d132a42aa98d
+  data.tar.gz: 166d78168a329e0fc6aab2a61962c040464b0d74fdb2cfcc1c6f4aa5278680762861cd72bc97482ef3035b2097581f10d0cccaab5b9d6c40ae518273d48f66f1

data/lib/goodreads_books/book.rb

@@ -1,7 +1,5 @@
 class GoodreadsBooks::Book
-  attr_accessor :awards_year, :category, :title, :author, :vote, :description, :cate_url, :url
-
-  BASE_URL = "https://www.goodreads.com"
+  attr_accessor :awards_year, :category, :title, :author, :vote, :description, :category_url, :url
 
   @@all = []
 
@@ -24,54 +22,8 @@ class GoodreadsBooks::Book
     self.class.all << self
   end #-- save --
 
-  def self.all_by_year(awards_year)
-    all.select { |book| book.awards_year == awards_year }
-  end #-- find_by_year --
-
-  def author
-    get_book_details if !@author
-    @author
-  end #-- author --
-
-  def vote
-    get_book_details if !@vote
-    @vote
-  end #-- vote --
-
-  def description
-    get_book_details if !@description
-    @description
-  end #-- description --
-
-  def url
-    get_book_details if !@url
-    @url
-  end #-- url --
-
-  def get_book_details
-    # Next level of scraping (get details of best book within each category_url)
-    book_doc = Nokogiri::HTML(open(self.cate_url))
-
-    self.vote = book_doc.css(".gcaRightContainer .gcaWinnerHeader").text.split(" ")[1]
-    self.author = book_doc.css(".gcaRightContainer h3 .gcaAuthor a.authorName").text
-    self.url = "#{BASE_URL}#{book_doc.css(".gcaRightContainer h3 a.winningTitle").attr("href").text}"
-
-    # goodreads description is encoded, so need to add .encode("ISO-8859-1") to print the special characters eg. â\u0080\u0099s in printable character of '
-    # if self.awards_year < 2017, use the span tag, else there's no span tag so don't check for it
-    descript = book_doc.css(".gcaRightContainer .readable.stacked span")[1]
-    if descript
-      self.description = book_doc.css(".gcaRightContainer .readable.stacked span")[1].text.encode("ISO-8859-1")
-    else
-      self.description = book_doc.css(".gcaRightContainer .readable.stacked").text.encode("ISO-8859-1")
-    end
-    #binding.pry
-  end #-- get_book_details --
-
-  def self.populate_book_details(award_year)
-    all_by_year(awards_year).each do |book|
-      book.get_book_details
-    end
-    #binding.pry
-  end #-- self.populate_book_details --
+  def self.find_all_by_year(awards_year)
+    self.all.select { |book| book.awards_year == awards_year }
+  end #-- self.find_all_by_year --
 
 end

data/lib/goodreads_books/cli.rb

@@ -1,13 +1,8 @@
 class GoodreadsBooks::CLI
 
-  # This application only works for year 2010 to current year - 1.
+  # This application only works for year 2010 to latest awards year (usually, current year - 1).
   # Goodreads Choice Awards Winner 2009 page setup differs from 2010 onwards.
   BASE_YEAR = 2010
-  END_YEAR = Time.now.year - 1
-
-  def initialize
-    @@choice_awards = nil
-  end
 
   def call
     system "clear"
@@ -16,46 +11,51 @@ class GoodreadsBooks::CLI
     puts "           ----------------------------------------"
     puts ""
 
-    load_choice_awards
+    @latest_awards_year = GoodreadsBooks::Scraper.scrape_awards_year
+    load_choice_awards_books(@latest_awards_year)
 
     main_menu
   end #-- call --
 
-  def load_choice_awards(awards_year = nil)
-    if awards_year == nil
-      puts "Loading The Latest Winners of Goodreads Choice Awards Books..."
-    else
-      puts "Loading The Winners of #{awards_year} Goodreads Choice Awards Books..."
+  def load_choice_awards_books(awards_year)
+    @awards_year = awards_year
+    puts "Loading The Winners of #{awards_year} Goodreads Choice Awards Books..."
+    if GoodreadsBooks::Book.find_all_by_year(awards_year).empty?
+      books = GoodreadsBooks::Scraper.scrape_books(awards_year)
     end
-
-    @choice_awards = GoodreadsBooks::Scraper.find_or_create_by_year(awards_year)
-
-    @book_count = GoodreadsBooks::Book.all_by_year(@choice_awards.awards_year).count
-  end #-- load_choice_awards --
+  end #-- load_choice_awards_books --
 
   def main_menu
-    system "clear"
-
+    valid_input = true
     input = nil
     while input != "exit"
+      book_count = GoodreadsBooks::Book.find_all_by_year(@awards_year).count
       list_books
 
-      puts ""
-      puts "Enter a number to view details of the book, or select another Choice Awards year (2010 onwards)."
-      puts "Type 'exit' to end the application."
+      if !valid_input
+        puts ""
+        puts "Please enter a number between 1 and #{book_count}, or valid Choice Awards year, or 'exit' to end application.".colorize(:red)
+        valid_input = true
+      else
+        puts ""
+        puts "Enter a number to view details of the book, or select another Choice Awards year (2010 onwards).".colorize(:green)
+        puts "Type 'exit' to end the application.".colorize(:green)
+      end
+
       input = gets.strip
 
       if input.downcase == "exit"
         break
-      elsif input.to_i.between?(1, @book_count)
-        book = GoodreadsBooks::Book.all_by_year(@choice_awards.awards_year)[input.to_i - 1]
+      elsif input.to_i.between?(1, book_count)
+        book = GoodreadsBooks::Book.find_all_by_year(@awards_year)[input.to_i - 1]
+        if !book.author
+          GoodreadsBooks::Scraper.scrape_book_details(book)
+        end
         view_book(book)
-      elsif input.to_i.between?(BASE_YEAR, END_YEAR)
-
-        load_choice_awards(input.to_i)
+      elsif input.to_i.between?(BASE_YEAR, @latest_awards_year)
+        load_choice_awards_books(input.to_i)
       else
-        puts ""
-        puts "Please enter a number between 1 and #{@book_count} or a valid Choice Awards year".colorize(:red)
+        valid_input = false
       end
     end
 
@@ -64,18 +64,20 @@ class GoodreadsBooks::CLI
   end #-- main_menu --
 
   def list_books
+    system "clear"
     puts ""
-    puts "---------- #{@choice_awards.awards_year} Goodreads Choice Awards Books ----------"
+    puts "---------- #{@awards_year} Goodreads Choice Awards Books ----------"
     puts ""
 
-    GoodreadsBooks::Book.all_by_year(@choice_awards.awards_year).each.with_index(1) do |book, index|
+    GoodreadsBooks::Book.find_all_by_year(@awards_year).each.with_index(1) do |book, index|
       puts "#{index}. #{book.category} - #{book.title}"
     end
   end #-- display_books --
 
   def view_book(book)
+    system "clear"
     puts ""
-    puts "---------- #{@choice_awards.awards_year} BEST #{book.category.upcase} Winner ----------"
+    puts "---------- #{@awards_year} BEST #{book.category.upcase} Winner ----------"
     puts ""
     puts "Title:        #{book.title}"
     puts "Author:       #{book.author}"
@@ -85,7 +87,7 @@ class GoodreadsBooks::CLI
     puts "#{book.description}"
 
     puts ""
-    puts "Would you like to visit Goodreads website to view this book? Enter Y or N".colorize(:green)
+    puts "Would you like to open Goodreads website to view this book? Enter Y to open the website.".colorize(:green)
     input = gets.strip.downcase
 
     if input.downcase == "y"

data/lib/goodreads_books/scraper.rb

@@ -1,77 +1,57 @@
 class GoodreadsBooks::Scraper
-  attr_accessor :awards_year, :main_url
-
   BASE_URL = "https://www.goodreads.com"
   PAGE_URL = "/choiceawards"
 
-  @@all = []
-
-  def initialize(awards_year = nil)
-    @awards_year = awards_year
-  end #-- initialize --
-
-  def self.all
-    @@all
-  end #-- self.all --
-
-  def save
-    self.class.all << self
-  end #-- save --
-
-  def self.find_or_create_by_year(awards_year = nil)
-    if !(choice_awards = find_by_year(awards_year))
-       choice_awards = create(awards_year)
-       choice_awards.scrape_books
-    end
-    find_by_year(choice_awards.awards_year)
-  end #-- self.find_or_create_by_year --
-
-  def self.find_by_year(awards_year = nil)
-    all.detect { |r| r.awards_year == awards_year }
-  end #-- self.find_by_year --
-
-  def self.create(awards_year = nil)
-    #choice_awards = new(awards_year)
-    #choice_awards.save
-    #replaced with one line of code below using .tap method
-    choice_awards = new(awards_year).tap { |s| s.save }
-
+  def self.scrape_awards_year
     # if awards_year is missing from the url,
     # goodreads.com defaults to latest choice awards year
     # /best-books-#{latest awards year}"
-    if awards_year == nil
-      choice_awards.main_url = "#{BASE_URL}#{PAGE_URL}"
-      doc = Nokogiri::HTML(open(choice_awards.main_url))
-      choice_awards.awards_year = doc.css("head title").text.split(" ")[2].to_i
-    else
-      choice_awards.main_url = "#{BASE_URL}#{PAGE_URL}/best-books-#{awards_year}"
-      choice_awards.awards_year = awards_year
-    end
+    main_url = "#{BASE_URL}#{PAGE_URL}"
+    html = open(main_url)
+    html.base_uri.to_s.split("-").last.to_i
+  end #-- self.scrape_awards_year
 
-    choice_awards # return instance of scraper
-  end #-- self.create --
+  def self.scrape_books(awards_year)
+    main_url = "#{BASE_URL}#{PAGE_URL}/best-books-#{awards_year}"
+    doc = Nokogiri::HTML(open(main_url))
 
-  def scrape_books
-    doc = Nokogiri::HTML(open(@main_url))
-
-    # Category winners page
+    # Category winners page: iterate through the best book of each category
+    books = []
     doc.css(".category.clearFix").each do |category|
-      cate_name = category.css("h4").text
-      cate_url = category.css("a").attr("href").text
-      cate_title = category.css("img").attr("alt").text
+      category_name = category.css("h4").text
+      category_url = category.css("a").attr("href").text
+      category_title = category.css("img").attr("alt").text
 
       # for each winner element, assemble the book_details hash
       book_details = {
-        :awards_year => @awards_year,
-        :category => cate_name,
-        :title => cate_title,
-        :cate_url => "#{BASE_URL}#{cate_url}"
+        :awards_year => awards_year,
+        :category => category_name,
+        :title => category_title,
+        :category_url => "#{BASE_URL}#{category_url}"
       }
 
-      GoodreadsBooks::Book.new_from_web_page(book_details)
+      books = GoodreadsBooks::Book.new_from_web_page(book_details)
     end
 
-    #binding.pry
-  end #-- scrape_books --
+    books
+  end #-- self.scrape_books --
+
+  def self.scrape_book_details(book)
+    # Next level of scraping (get details of best book within each category_url)
+    book_doc = Nokogiri::HTML(open(book.category_url))
+
+    book.vote = book_doc.css(".gcaRightContainer .gcaWinnerHeader").text.split(" ")[1]
+    book.author = book_doc.css(".gcaRightContainer h3 .gcaAuthor a.authorName").text
+    book.url = "#{BASE_URL}#{book_doc.css(".gcaRightContainer h3 a.winningTitle").attr("href").text}"
+
+    # goodreads description is encoded, so need to add .encode("ISO-8859-1") to print the special characters eg. â\u0080\u0099s in printable character of '
+    # if self.awards_year < 2017, use the span tag, else there's no span tag so don't check for it
+    description = book_doc.css(".gcaRightContainer .readable.stacked span")[1]
+    if description
+      book.description = book_doc.css(".gcaRightContainer .readable.stacked span")[1].text.encode("ISO-8859-1")
+    else
+      book.description = book_doc.css(".gcaRightContainer .readable.stacked").text.encode("ISO-8859-1")
+    end
+  end #-- self.scrape_book_details --
 
 end

data/lib/goodreads_books/version.rb

@@ -1,3 +1,3 @@
 module GoodreadsBooks
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: goodreads-books
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Ni Chia
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-09-04 00:00:00.000000000 Z
+date: 2018-09-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler