fantasy-football 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5bbf6659d9889db6ad0064054b7cac10c58876f5ca71762882d76fc189cd37f7
-  data.tar.gz: 98d2cba1d007c6e2698154bc910fc8647031cd8ce996fba8be3529a008332013
+  metadata.gz: a4684cfcaed208aa4b48bb90c51a8aa6e7584de367ba55dea2811fb04146db62
+  data.tar.gz: 75ee2021047cef994803c356b80a6c0ae1ca55cd44692611348a3cfd8d82b3a1
 SHA512:
-  metadata.gz: 14ee8a4219df44ec57d426e49adbcaf5d0be1c37c76dd2c18a50193b923519d510abe35be1e3fd3e960b29deb0607c6b0ecd367c5c68607d20f875e47d930275
-  data.tar.gz: 5a39dbd9ea69a765d80cd0324e44282e6153850d217df7c21b3c843ed4a02bdf689f2b3e9b43d8dc364f5b18db75a8c4e393b468c035cdf2f834e0a30598fa26
+  metadata.gz: f5ee4562b6b71d9aea5d5345bb48a3be5ddf59222757a81e94c7cdb176c05adec1cde8b052ccef09537bf90b792129940174a6bc2cef528cebfd3b3d9e7c25ff
+  data.tar.gz: c2bcf01de92742473279e12c587a26ad190dce1e5bfd1d6e708a9ff4bb7d767ca0dafc91d7013bb333c5022ac4a9311cba9571e7c72d6bc1d8e753b83f308d11

data/lib/fantasy_football/CLI.rb

@@ -2,23 +2,19 @@ class FantasyFootball::CLI
 
   POSITIONS = ["qb", "rb", "te", "wr", "k"]
 
-  attr_accessor :position, :size # Stores current state of position choice
-
   def welcome
     puts " "
-    puts "Welcome to the NFL Fantasy Football Rankings and Players!"
+    puts "Welcome to NFL Fantasy Football Rankings and Players!"
     puts " "
   end
 
-  def choose_list_size
-    puts "How many player rankings would you like see per position?"
-    puts "Please enter a number between 1 and 25:"
-    self.size = gets.strip.to_i
-    if size.between?(1,25)
-      return
-    else
+  def choose_list_size(position)
+    puts "How many player rankings would you like see?"
+    puts "Please enter a number between 1 and #{FantasyFootball::Player.find_by_position(position).size}:"
+    @size = gets.strip.to_i
+    if !@size.between?(1,FantasyFootball::Player.find_by_position(position).size)
       puts "Invalid entry - please enter a valid input:"
-      choose_list_size
+      choose_list_size(@position)
     end
   end
 
@@ -26,10 +22,11 @@ class FantasyFootball::CLI
     # Asks for position and lists top players ranked by Fantasypros
     puts "What position would you like to see rankings for?"
     puts "Please enter QB, RB, TE, WR, or K:"
-    self.position = gets.strip.downcase
-    if POSITIONS.include?(position)
-      FantasyFootball::Scraper.scrape_rankings(position, size) if FantasyFootball::Player.find_by_position(position) == []
-      print_rankings
+    @position = gets.strip.downcase
+    if POSITIONS.include?(@position)
+      FantasyFootball::Scraper.scrape_rankings(@position) if FantasyFootball::Player.find_by_position(@position) == []
+      choose_list_size(@position)
+      print_rankings(@size)
       choose_player
     else
       puts "Invalid entry - please enter a valid input:"
@@ -37,11 +34,11 @@ class FantasyFootball::CLI
     end
   end
 
-  def print_rankings
-    # Iterates through Player.all to print player name and rankings by position
+  def print_rankings(size)
+    # Iterates through Player instances to print player name and rankings by position
     puts " "
-    FantasyFootball::Player.find_by_position(position).sort {|a,b| a.rank.to_i <=> b.rank.to_i}.each_with_index {| p, i|
-      puts "-- Top #{size} #{position.upcase}s for Week #{p.week} of #{Time.new.year} --" if i == 0
+    FantasyFootball::Player.find_by_position(@position)[0..@size - 1].each_with_index {| p, i|
+      puts "-- Top #{size} #{@position.upcase}s for Week #{p.week} of #{Time.new.year} --" if i == 0
       puts "#{p.rank}. #{p.name}"}
     puts " "
   end
@@ -50,7 +47,7 @@ class FantasyFootball::CLI
     # Prompts for player rank #, outputs player details
     puts "If you would like to see details about a player, enter their rank number. If not, enter N:"
     rank = gets.strip
-    if rank.to_i.between?(1,size)
+    if rank.to_i.between?(1,@size)
       print_player(rank)
     elsif rank.downcase == "n"
       return
@@ -63,7 +60,7 @@ class FantasyFootball::CLI
   def print_player(rank)
     # Prints specific player using a custom class finder
     blank = " "
-    p = FantasyFootball::Player.find_by_rank_and_position(rank, position)
+    p = FantasyFootball::Player.find_by_rank_and_position(rank, @position)
     FantasyFootball::Scraper.add_attr(p)
     puts blank
     puts "         Player Stats              "
@@ -86,7 +83,7 @@ class FantasyFootball::CLI
     puts "Please enter 1, 2, or quit."
     input = gets.strip.downcase
     if input == "1"
-      print_rankings
+      print_rankings(@size)
       choose_player
     elsif input == "2"
       choose_position
@@ -101,7 +98,6 @@ class FantasyFootball::CLI
 
   def run
     welcome
-    choose_list_size
     choose_position
     again?
   end

data/lib/fantasy_football/player.rb

@@ -17,7 +17,7 @@ class FantasyFootball::Player
   end
 
   def self.find_by_position(position)
-    all.select {|i| i.position == position.upcase}
+    all.select {|i| i.position == position.upcase}.sort {|a,b| a.rank.to_i <=> b.rank.to_i}
   end
 
   def self.find_by_rank_and_position(rank, position)

data/lib/fantasy_football/scraper.rb

@@ -1,22 +1,26 @@
 class FantasyFootball::Scraper
 
-  def self.scrape_rankings(position, size)
+  def self.scrape_rankings(position)
     doc = Nokogiri::HTML(open("https://www.fantasypros.com/nfl/rankings/#{position}.php"))
     week = doc.css('h1').text.split[5]
     table = doc.css('tbody tr') # Selects the HTML table with player rankings
-    build_players(table, position, week, size)
+    build_players(table, position, week)
   end
 
-  def self.build_players(table, position, week, size)
+  def self.build_players(table, position, week)
     # Input is HTML table of player rankings, instantiates Players, assigns name, rank, and url
+
     table.each_with_index do |t, i|
-      if i < size
-        p = FantasyFootball::Player.new
-        p.name = [t.text.split[1], t.text.split[3]].join(" ")
-        p.rank = t.text.split[0]
-        p.url = "https://www.fantasypros.com" + t.css('a')[0]["href"]
-        p.week = week
-        p.position = position.upcase
+      begin
+        if t.css('td')[0].text.to_i > 0
+          p = FantasyFootball::Player.new
+          p.name = t.css('span.full-name').text
+          p.rank = t.css('td')[0].text
+          p.url = "https://www.fantasypros.com" + t.css('a')[0]["href"]
+          p.week = week
+          p.position = position.upcase
+        end
+      rescue NoMethodError
       end
     end
   end

data/lib/fantasy_football/version.rb

@@ -1,3 +1,3 @@
 module FantasyFootball
-  VERSION = "0.2.0"
+  VERSION = "0.2.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fantasy-football
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - "'Mike Dilley'"