sportify 0.2.0 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: df846ec020824028a0fdcd73e19c520966faaff5
-  data.tar.gz: 3b5b34d9da2e045ec7e7907e1102428ae7f335e9
+  metadata.gz: 95b846b5835f54e704d73d55a9f5a1a0983941dd
+  data.tar.gz: 7e1f7429029bed8128774ac2bfb3aab92087f163
 SHA512:
-  metadata.gz: 0c6fe292bb3dcce5cd6e6e5402c2f8c15a818f7427a0413f8af4f0e484ce2c1096ae621cae9d4b821123c6a6a31cf6d19efedcea2339d901d39e33c1f20e0740
-  data.tar.gz: 436a804c080d3e91a0e410d281abcf3f35d6df55e00038bd0d2bd7724199ea7e22aa107d49133382b92c014ceebc9cbf8dd866e5c5de425c3c8c510f994c7054
+  metadata.gz: 0578f797e0b91ce55838b433931d5eff6052c27fa206d7a702d5efa17eff1f5431b66c6026fc0f49df75623cfe1e10e967bbe600e5daed5992b2d06509dcaf26
+  data.tar.gz: 3bacf9cb36ea92bbd5a3b40b82e99373d1b34fbb80593d86d16d15f0acddb6b58d94d9d6393135df060ae33d86c1410ed4504dcd12e50e8d7b70eb2af0530558

data/config/environment.rb

@@ -3,5 +3,6 @@ require "open-uri"
 require "bundler"
 
 require_relative "../lib/cli/cli"
-require_relative "../lib/leagues/mlb_teams"
+require_relative "../lib/teams/mlb_team"
 require_relative "../lib/scrapers/mlb_scraper"
+require_relative "../lib/players/mlb_player"

data/lib/cli/cli.rb

@@ -1,27 +1,27 @@
 class Sportify::CLI
 
   def initialize
-    @mlb = Sportify::MLBteams.new
+    Sportify::MLBscraper.team_builder
     main_menu
-    team_selector
   end
 
   def main_menu
     puts "***** Welcome To Sportify *****"
     puts ""
-    @mlb.teams.each_with_index do |team, index|
-      puts "#{index + 1}) #{team[:team_name]}"
+    Sportify::MLBteam.teams.each_with_index do |team, index|
+      puts "#{index + 1}) #{team.name}"
     end
     puts ""
     puts "Enter the number of the team you want information on or type 'exit' to leave Sportify"
     puts ""
+    team_selector
   end
 
   def team_selector
     input = gets.strip
     puts ""
-    if input.to_i.between?(1, @mlb.teams.length)
-      team_menu(@mlb.teams[input.to_i - 1])
+    if input.to_i.between?(1, Sportify::MLBteam.teams.length)
+      team_menu(Sportify::MLBteam.teams[input.to_i - 1])
     elsif input.downcase == "exit"
       puts "Goodbye"
     else
@@ -31,8 +31,7 @@ class Sportify::CLI
   end
 
   def team_menu(team)
-    puts ""
-    puts "You've selected the #{team[:team_name]}"
+    puts "You've selected the #{team.name}"
     puts""
     puts "Please choose an option"
     puts ""
@@ -48,11 +47,14 @@ class Sportify::CLI
     puts ""
     case input
     when 1
-      puts "***** Active Roster *****"
-      puts ""
-      active_roster_display(team)
+      if team.roster_url == nil
+        Sportify::MLBscraper.roster_url(team)
+        active_roster_display(team)
+      else
+        active_roster_display(team)
+      end
     when 2
-      Sportify::CLI.new
+      main_menu
     when 3
       puts "Goodbye"
     else
@@ -62,8 +64,11 @@ class Sportify::CLI
   end
 
   def active_roster_display(team)
-    @mlb.roster(team).each_with_index do |player, index|
-      puts "#{index + 1}) #{player[:name]}"
+    if team.players.length == 0
+      Sportify::MLBscraper.player_builder(team)
+    end
+    team.players.each_with_index do |player, index|
+      puts "#{index + 1}) #{player.name}"
     end
     player_selector(team)
   end
@@ -74,30 +79,32 @@ class Sportify::CLI
     puts ""
     input = gets.strip
     puts ""
-    if input.to_i.between?(1, @mlb.roster(team).length)
-      player_display(@mlb.roster(team)[input.to_i - 1])
-      team_menu(team)
+    if input.to_i.between?(1, team.players.length)
+      player_display(team.players[input.to_i - 1])
     elsif input.downcase == "menu"
       team_menu(team)
     else
-      puts "Invalid input.  Please try again."
-      player_selector(team)
+      active_roster_display(team)
     end
   end
 
   def player_display(player)
-    player = @mlb.player_builder(player)
-    puts "You've selected #{player[:name]}"
+    if player.number == nil
+      Sportify::MLBscraper.add_player_data(player)
+    end
+    puts "You've selected #{player.name}"
+    puts ""
+    puts "Jersey Number: #{player.number}"
     puts ""
-    puts "Jersey Number: #{player[:number]}"
+    puts "Position: #{player.position}"
     puts ""
-    puts "Position: #{player[:position]}"
+    puts "#{player.bats_and_throws}"
     puts ""
-    puts "#{player[:bats_and_throws]}"
+    puts "Height/Weight: #{player.height_weight}"
     puts ""
-    puts "Height/Weight: #{player[:height_weight]}"
+    puts "#{player.age}"
     puts ""
-    puts "#{player[:age]}"
+    team_menu(player.team)
   end
 
 end

data/lib/players/mlb_player.rb

@@ -0,0 +1,18 @@
+class Sportify::MLBplayer
+
+  attr_accessor :name, :player_url, :team, :number, :position, :bats_and_throws, :height_weight, :age
+
+  @@all = []
+
+  def initialize(name = nil, player_url = nil, team = nil)
+    @name = name
+    @player_url = player_url
+    @team = team
+    @@all << self
+  end
+
+  def self.all
+    @@all
+  end
+
+end

data/lib/scrapers/mlb_scraper.rb

@@ -1,81 +1,37 @@
 class Sportify::MLBscraper
 
-  @@root_url = "https://www.mlb.com/team"
-  @@root_html = open(@@root_url)
-  @@root_doc = Nokogiri::HTML(@@root_html)
+  @@team_data = Nokogiri::HTML(open("https://www.mlb.com/team")).css("div.p-featured-content__body")
 
-  def teams
-    teams = []
-    @@root_doc.css("div.u-text-h4.u-text-flow").each do |team|
-      teams << team.text
+  def self.team_builder
+    @@team_data.each do |data|
+      Sportify::MLBteam.new(name = data.css("a").css("div.u-text-h4.u-text-flow").text, team_url = data.css("div.p-wysiwyg").css("a")[0]["href"])
     end
-    teams
   end
 
-  def team_urls
-    team_urls = []
-    @@root_doc.css("div.p-wysiwyg").each do |team|
-      team_urls << team.css("a")[0]["href"]
-    end
-    team_urls
-  end
-
-  def roster_doc(team)
-    team_html = open(team[:team_url])
-    team_doc = Nokogiri::HTML(team_html)
-    if team[:team_url] == "https://www.mlb.com/reds" || team[:team_url] == "https://www.mlb.com/dodgers" || team[:team_url] == "https://www.mlb.com/giants"
-      roster_url = team_doc.css("div.megamenu-navbar").css("a.megamenu-static-navbar__menu-item.megamenu-static-navbar__menu-item--roster").attribute("href").value
+  def self.roster_url(team)
+    team_home_page = Nokogiri::HTML(open(team.team_url))
+    if team.team_url == "https://www.mlb.com/reds" || team.team_url == "https://www.mlb.com/dodgers" || team.team_url == "https://www.mlb.com/giants"
+      roster_url = team_home_page.css("div.megamenu-navbar").css("a.megamenu-static-navbar__menu-item.megamenu-static-navbar__menu-item--roster").attribute("href").value
     else
-      roster_url = team_doc.css("div.megamenu-navbar").css("a.megamenu-static-navbar__menu-item.megamenu-static-navbar__menu-item--team").attribute("href").value
+      roster_url = team_home_page.css("div.megamenu-navbar").css("a.megamenu-static-navbar__menu-item.megamenu-static-navbar__menu-item--team").attribute("href").value
     end
-    roster_html = open(roster_url)
-    Nokogiri::HTML(roster_html)
+    team.roster_url = roster_url
   end
 
-  def roster(team)
-    roster = []
-    roster_doc(team).css("td.dg-name_display_first_last").css("a").each do |player|
-      roster << player.text
+  def self.player_builder(team)
+    @team =  team
+    Nokogiri::HTML(open(@team.roster_url)).css("td.dg-name_display_first_last").css("a").each do |data|
+      @team.add_player(Sportify::MLBplayer.new(player_name = data.text, player_url = team.team_url + data["href"], team = @team))
     end
-    roster
-  end
-
-  def player_urls(team)
-    player_urls = []
-    team_url = team[:team_url]
-    roster_doc(team).css("td.dg-name_display_first_last").css("a").each do |player|
-      player_urls << team_url + player["href"]
-    end
-    player_urls
-  end
-
-  def player_doc(player)
-    html = open(player[:player_url])
-    Nokogiri::HTML(html)
-  end
-
-  def name(player)
-    player_doc(player).css("span.player-header--vitals-name").text
-  end
-
-  def number(player)
-    player_doc(player).css("span.player-header--vitals-number").text
-  end
-
-  def position(player)
-    player_doc(player).css("div.player-header--vitals").css("ul").css("li")[0].text
-  end
-
-  def bats_and_throws(player)
-    player_doc(player).css("div.player-header--vitals").css("ul").css("li")[1].text
-  end
-
-  def height_weight(player)
-    player_doc(player).css("div.player-header--vitals").css("ul").css("li.player-header--vitals-height").text
   end
 
-  def age(player)
-    player_doc(player).css("div.player-header--vitals").css("ul").css("li.player-header--vitals-age").text
+  def self.add_player_data(player)
+    data = Nokogiri::HTML(open(player.player_url))
+    player.number = data.css("span.player-header--vitals-number").text
+    player.position = data.css("div.player-header--vitals").css("ul").css("li")[0].text
+    player.bats_and_throws = data.css("div.player-header--vitals").css("ul").css("li")[1].text
+    player.height_weight = data.css("div.player-header--vitals").css("ul").css("li.player-header--vitals-height").text
+    player.age = data.css("div.player-header--vitals").css("ul").css("li.player-header--vitals-age").text
   end
 
 end

data/lib/teams/mlb_team.rb

@@ -0,0 +1,27 @@
+class Sportify::MLBteam
+
+  attr_accessor :name, :team_url, :roster_url
+
+  @@teams = []
+
+  def initialize(name = nil, team_url = nil)
+    @name = name
+    @team_url = team_url
+    @@teams << self
+    @players = []
+  end
+
+  def self.teams
+    @@teams.sort_by(&:name)
+  end
+
+  def add_player(player)
+    @players << player
+    player.team = self
+  end
+
+  def players
+    @players
+  end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sportify
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 1.0.0
 platform: ruby
 authors:
 - Bruce Hillsworth
@@ -108,9 +108,10 @@ files:
 - bin/sportify
 - config/environment.rb
 - lib/cli/cli.rb
-- lib/leagues/mlb_teams.rb
+- lib/players/mlb_player.rb
 - lib/scrapers/mlb_scraper.rb
 - lib/sportify.rb
+- lib/teams/mlb_team.rb
 homepage: https://rubygems.org/gems/sportify
 licenses:
 - MIT

data/lib/leagues/mlb_teams.rb

@@ -1,28 +0,0 @@
-require 'pry'
-
-class Sportify::MLBteams
-
-  def initialize
-    @scraper = Sportify::MLBscraper.new
-  end
-
-  def teams
-    @scraper.teams.zip(@scraper.team_urls).map { |team, team_url| {team_name: team, team_url: team_url} }.sort_by { |team| team[:team_name] }
-  end
-
-  def roster(team)
-    @scraper.roster(team).zip(@scraper.player_urls(team)).map { |name, player_url| {name: name, player_url: player_url}}
-  end
-
-  def player_builder(player)
-    {
-      name: @scraper.name(player),
-      number: @scraper.number(player),
-      position: @scraper.position(player),
-      bats_and_throws: @scraper.bats_and_throws(player),
-      height_weight: @scraper.height_weight(player),
-      age: @scraper.age(player)
-    }
-  end
-
-end