nba-stats 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ff8b26c9529f427f4afdf28357b6935a132b8c76
-  data.tar.gz: bf8c356ba645fd0d9de36cc914165a08771190e2
+  metadata.gz: 77fe915ee660f8d723ff533f725e222c34ebeed4
+  data.tar.gz: 36246b0af9b5d0095cd289a6e3653eea513ac1e4
 SHA512:
-  metadata.gz: 08030467660a1c4adc947b3d61fd16728893ca704f0fc82bd644a580698964c50c21a01d3df0b51729f90c661f389fea0c5d321cd416ddd21f0a561c22644891
-  data.tar.gz: e60c4b156fb650e5d56b17818701078312569da65163cbee572d3a45d141600adb81b7f76f0d6bc44470c4f1fdcb8a4feded5efd01e53c333d1cba4707e29e10
+  metadata.gz: 52be75b9d14722ea257eeddff435dbf01086aa14d56364d4c727577268892dfeb0da5f69887df5c7ac4a9ec1de4c196aef68fcb7a50d57d60fd8be992e2f4663
+  data.tar.gz: 9cefc1ecd4e4db91ce1bbafc28bc175dea2a769600b48bc5149320224413493dee629793173c10ddc6ed359157aefdfc3f3f6b8eb3cd533b2107111daa0ecc54

data/lib/nba_stats/cli.rb

@@ -1,32 +1,50 @@
 class NbaStats::CLI
   
   def call
+    puts "Welcome to the NBA Stats CLI Gem"
     start
   end
 
   def start
-    puts "Welcome to the NBA Stats CLI Gem"
+    puts "Input one of the team names below to load their roster."
+    puts "Input exit to leave this program."
     puts "Here's the list of current teams"
-    make_teams
-    NbaStats::Team.all.each do |team|
-      puts team.name
-    end
 
-    choose_team
-    choose_player
+    make_teams if NbaStats::Team.all.empty?
+
+    rows = [["Eastern Conference", "Western Conference"]]
+    west_teams = NbaStats::Team.western_names
+    east_teams = NbaStats::Team.eastern_names
 
-    puts "Do you want to look up another player on this team? y/n"
-    response = gets.strip
-    while response == "y"
-      choose_player
-      puts "Do you want to look up another player on this team? y/n"
-      response = gets.strip
+    i = 0
+    while i < 15
+      rows << [east_teams[i], west_teams[i]]
+      i += 1
     end
 
-    puts "Do you want to look up another team? y/n"
-    response = gets.strip
-    if response == "y"
-      start
+    team_table = Terminal::Table.new rows: rows
+    puts team_table
+
+    input = ""
+    while input != "exit"
+      puts "Input a team name to see their roster: "
+      input = gets.strip
+      if NbaStats::Team.team_names.include? input
+        display_roster(input)
+        puts "Input a player name to see their individual stats: "
+        input = gets.strip
+        while NbaStats::Player.player_names.include? input
+          display_player_stats(input)
+          puts "Input another player name from this team to see their stats."
+          puts "Or input change teams to see another team's roster."
+          input = gets.strip
+          if input == "change teams" || input == "change team"
+            start
+          elsif input == "exit"
+            break
+          end
+        end
+      end
     end
   end
 
@@ -35,15 +53,9 @@ class NbaStats::CLI
     NbaStats::Team.create_from_collection(teams_array)
   end
 
-  def choose_team
-    puts "Enter a team name to see their roster: "
-    requested_team = gets.strip
-    while !NbaStats::Team.team_names.include? requested_team
-      puts "That team doesn't exist. Try again."
-      requested_team = gets.strip
-    end
+  def display_roster(requested_team)
     team = NbaStats::Team.all.detect {|team| team.name == requested_team}
-    team.add_players
+    team.add_players if team.players.empty?
     puts team.name + " roster:"
     rows = [["Number", "Name", "Position", "Height", "Experience"]]
     team.players.each do |player|
@@ -53,13 +65,7 @@ class NbaStats::CLI
     puts roster_table
   end
 
-  def choose_player
-    puts "Enter a player name to see their individual stats: "
-    requested_player = gets.strip
-    while !NbaStats::Player.player_names.include? requested_player
-      puts "That player isn't on this team. Try again."
-      requested_player = gets.strip
-    end
+  def display_player_stats(requested_player)
     player = NbaStats::Player.all.detect {|player| player.name == requested_player}
     stats_hash = NbaStats::Scraper.get_player_stats(player)
     player.add_player_stats(stats_hash)

data/lib/nba_stats/player.rb

@@ -2,6 +2,7 @@ class NbaStats::Player
   attr_accessor :name, :team, :number, :player_url, :position, :height, :experience, :points_pg, :assists_pg, :rebounds_pg, :blocks_pg, :steals_pg, :minutes_pg, :fg_percentage, :three_percentage, :ft_percentage
 
   @@all = []
+  
   def initialize(player_hash)
     player_hash.each do |key, value|
       self.send("#{key}=", value)

data/lib/nba_stats/scraper.rb

@@ -4,18 +4,24 @@ class NbaStats::Scraper
     Nokogiri::HTML(open(url))
   end
 
-  # Returns an array of hashes with current team names and links to 2015-16 team page
   def self.get_teams
-    page = open_page("http://www.basketball-reference.com/teams")
-    teams_array = []
-    teams = page.css("table#active tr.full_table a")
+    page = open_page("http://www.basketball-reference.com/leagues/NBA_2016_standings.html")
+
+    east_teams = page.css("table#E_standings td a")
+    west_teams = page.css("table#W_standings td a")
+
+    assign_teams(west_teams, "West") + assign_teams(east_teams, "East")
+  end
+
+  def self.assign_teams(teams, conference)
+    assigned_teams = []
     teams.each do |team|
       name = team.text
-      team_url = "http://www.basketball-reference.com"+ team["href"] + "2016.html"
-      hash = {name: name, team_url: team_url}
-      teams_array << hash
+      team_url = "http://www.basketball-reference.com"+ team["href"]
+      hash = {name: name, team_url: team_url, conference: conference}
+      assigned_teams << hash unless assigned_teams.include? hash
     end
-    teams_array
+    assigned_teams
   end
 
   def self.get_roster(team)

data/lib/nba_stats/team.rb

@@ -1,6 +1,6 @@
 class NbaStats::Team
   
-  attr_accessor :name, :team_url, :players
+  attr_accessor :name, :team_url, :players, :conference
 
   @@all = []
 
@@ -9,7 +9,7 @@ class NbaStats::Team
       self.send("#{key}=", value)
     end
     @players = []
-    @@all << self
+    @@all << self unless @@all.include? self
   end
 
   def add_players
@@ -19,7 +19,7 @@ class NbaStats::Team
 
   def self.create_from_collection(teams_array)
     teams_array.each do |team|
-      new_team = NbaStats::Team.new(team)
+      new_team = NbaStats::Team.new(team) unless @@all.include? team
     end
   end
 
@@ -31,4 +31,14 @@ class NbaStats::Team
     @@all.collect {|team| team.name }
   end
 
+  def self.western_names
+    west = @@all.select {|team| team.conference == "West"}
+    west.collect {|team| team.name}.sort
+  end
+
+  def self.eastern_names
+    east = @@all.select {|team| team.conference == "East"}
+    east.collect {|team| team.name}.sort
+  end
+
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: nba-stats
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Alex Barron
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-07 00:00:00.000000000 Z
+date: 2016-02-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pry
@@ -108,7 +108,7 @@ dependencies:
     - - '='
       - !ruby/object:Gem::Version
         version: 1.5.2
-description: A command line interface for looking 2015-16 NBA player stats
+description: A command line interface for looking up 2015-16 NBA player stats
 email: alexbarron@gmail.com
 executables:
 - nba-stats
@@ -122,7 +122,7 @@ files:
 - lib/nba_stats/player.rb
 - lib/nba_stats/scraper.rb
 - lib/nba_stats/team.rb
-homepage: http://rubygems.org/gems/nba-stats
+homepage: https://github.com/alexbarron/nba-stats-cli-gem
 licenses:
 - MIT
 metadata: {}
@@ -145,5 +145,5 @@ rubyforge_project:
 rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
-summary: NBA Stats
+summary: Lookup NBA Player Stats
 test_files: []