nyt-bestsellers 0.0.4 → 0.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 364b65e29799c494484045f477d1fe2094d0f172
-  data.tar.gz: 4be1d126014b763428f274def89093b436dc2af1
+  metadata.gz: ce03add5d949656a5a1f634cf4405f043502abc3
+  data.tar.gz: 80a8b71c17efef3eb02188d38f3427f8229ec254
 SHA512:
-  metadata.gz: ad9c40794d6af2f71e94c51d4945b6c263090c965308f776a6e4d1edddea0f09ca9e3bf01146ae4a831cf63e69dfe6de5a36221ec739ce78e88eafafe86879d8
-  data.tar.gz: 951595ba0534a305b7480ff8c9bb67fdee9073e6a13670b71d0e2c6ea7e5f8c4921bfba8651c47f6600e55f5a6fe79e418988894d1fc455cf628821bf414f612
+  metadata.gz: 131a105fbd6c5c88aec6ecb88a34000f7744735fe9380488b4e2f360d79558288887dcc5e7d4d8bd29cdd0da5e8fd44017e884de95b14e3ea2831b2b14c0c846
+  data.tar.gz: c825661ef3a52c018994974b0dba48facb6418d77f72b6b286f83d9d0f026d3c9d66e37a262905002ff9505e9851bf6b4f2e5bfbc54dbd22972351f6086241cf

data/bin/nyt-bestsellers

@@ -1,5 +1,5 @@
 #!/usr/bin/env ruby
 
-require_relative '../lib/nyt_bestsellers.rb'
+require_relative '../lib/nyt_bestsellers'
 
-CLI.new.call
+NYTBestsellers::CLI.new.call

data/lib/nytimes/book.rb

@@ -1,18 +1,18 @@
-class Book
+class NYTBestsellers::Book
 
   attr_accessor :genre, :title, :author, :publisher, :wol, :summary
 
   @@all = []
 
-  def initialize(genre=nil, title=nil, author=nil, publisher=nil, wol=nil, summary=nil)
-    @genre = Genre.find_by_name(genre)
-    @genre.books << self if !wol.empty? || !summary.empty?
-    @title = title
-    @author = author
-    @publisher = publisher
-    @wol = wol
-    @summary = summary
-    @@all << self if !wol.empty? || !summary.empty?
+  def initialize(hash = {})
+    hash.each do |key, value|
+      self.send("#{key}=", value)
+    end
+    @genre = NYTBestsellers::Genre.find_by_name(hash[:genre])
+    if !wol.empty? || !summary.empty?
+      @genre.books << self 
+      @@all << self
+    end
   end
 
   def self.all
@@ -25,6 +25,5 @@ class Book
         book
       end
     end
-  end
-  
+  end 
 end

data/lib/nytimes/cli.rb

@@ -1,8 +1,8 @@
-class CLI
+class NYTBestsellers::CLI
 
   def call 
-    Scraper.make_genres
-    Scraper.make_books
+    NYTBestsellers::Scraper.make_genres
+    NYTBestsellers::Scraper.make_books
     puts ""
     puts "Welcome to the New York Times Bestsellers List".blue.bold
     run
@@ -11,7 +11,7 @@ class CLI
   def run
     puts "-------------------"
     puts ""
-    puts "NYT Bestsellers - Week of #{Scraper.get_date}".bold.blue
+    puts "NYT Bestsellers - Week of #{NYTBestsellers::Scraper.get_date}".bold.blue
     puts "(Lists are published early online.)".bold.blue
     puts ""
     puts "Here are the categories:"
@@ -34,17 +34,17 @@ class CLI
   end
 
   def genre_count
-    Genre.all.count
+    NYTBestsellers::Genre.all.count
   end
 
   def display_genres
-    Genre.all.each_with_index do |genre, index|
+    NYTBestsellers::Genre.all.each_with_index do |genre, index|
       puts "#{index+1} - #{genre.name}" 
     end
   end
 
   def genre_books(response)
-    genre = Genre.find_by_num(response)
+    genre = NYTBestsellers::Genre.find_by_num(response)
     puts ""
     puts "****----#{genre.name.upcase}----****".blue.bold
     puts ""
@@ -81,7 +81,7 @@ class CLI
   end
 
   def display_book_info(response, book_input)
-    genre = Genre.find_by_num(response)
+    genre = NYTBestsellers::Genre.find_by_num(response)
 
     genre.books.each_with_index do |book, index|
       if book_input.to_i == index+1

data/lib/nytimes/genre.rb

@@ -1,12 +1,13 @@
-class Genre
+class NYTBestsellers::Genre
 
   attr_accessor :name, :url
 
   @@all = []
 
-  def initialize(name=nil, url=nil)
-    @name = name
-    @url = url
+  def initialize(hash = {})
+    hash.each do |key, value|
+      self.send("#{key}=", value)
+    end  
     @@all << self
     @books = []
   end

data/lib/nytimes/scraper.rb

@@ -1,26 +1,21 @@
-class Scraper
+class NYTBestsellers::Scraper
 
   def self.get_page
-    agent = Mechanize.new
-    page = agent.get("http://www.nytimes.com/books/best-sellers/")
-  end
-
-  def self.scrape_genres
-    get_page.css("section.subcategory")
+    @@page ||= Mechanize.new.get("http://www.nytimes.com/books/best-sellers/")
   end
 
   def self.make_genres
-    scrape_genres.each do |category|
-      Genre.new(
-        category.css("a.subcategory-heading-link").text.strip,
-        "http://www.nytimes.com#{category.css("a").attr("href").text}"
-        )
+    get_page.css("section.subcategory").each do |category|
+      NYTBestsellers::Genre.new({
+        name: category.css("a.subcategory-heading-link").text.strip, 
+        url: "http://www.nytimes.com#{category.css("a").attr("href").text}"
+      })
     end
   end
 
   def self.get_genre_pages
     agent = Mechanize.new
-    Genre.all.collect do |genre|
+    NYTBestsellers::Genre.all.collect do |genre|
       page = agent.get("#{genre.url}")
     end
   end
@@ -30,14 +25,14 @@ class Scraper
       books = page.css("div.book-body")
 
       books.each do |book|
-        Book.new(
-          page.css("h1").text.split(/\s-\s/)[0].strip, #genre
-          book.css("h2.title").text.split.collect(&:capitalize).join(" "),
-          book.css("p.author").text.split.delete_if{|x| x == "by"}.join(" "),
-          book.css("p.publisher").text,
-          book.css("p.freshness").text,
-          book.css("p.description").text
-          )
+        NYTBestsellers::Book.new({
+          genre: page.css("h1").text.split(/\s-\s/)[0].strip, #genre
+          title: book.css("h2.title").text.split.collect(&:capitalize).join(" "),
+          author: book.css("p.author").text.split.delete_if{|x| x == "by"}.join(" "),
+          publisher: book.css("p.publisher").text,
+          wol: book.css("p.freshness").text,
+          summary: book.css("p.description").text
+        })
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: nyt-bestsellers
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.6
 platform: ruby
 authors:
 - Amy Back