From 703362b33106b424c9525be49e27ec51f2ac092b Mon Sep 17 00:00:00 2001 From: Rajashree K Maiya Date: Tue, 12 Jan 2016 16:00:16 -0500 Subject: [PATCH 1/2] Rubocop-Solution-Raj --- GemCity.rb | 56 ------------------------------------------------ gem_city.rb | 54 ++++++++++++++++++++++++++++++++++++++++++++++ gem_city_spec.rb | 6 +++--- 3 files changed, 57 insertions(+), 59 deletions(-) delete mode 100644 GemCity.rb create mode 100644 gem_city.rb diff --git a/GemCity.rb b/GemCity.rb deleted file mode 100644 index 690dc84..0000000 --- a/GemCity.rb +++ /dev/null @@ -1,56 +0,0 @@ -class GemCity - -=begin -This class represents the town of GemCity -This is a town riddled with crime but we can find out how happy the town is -=end - def initialize - @people, @population = { - :thieves => 5, - :Officers => 1}, - 50 - end - - def thieves thieves_number=@people[:thieves] - return @people[:thieves] = thieves_number - end - - def officers - return @people[:Officers] - end - - def population; return @population; end - - def setOfficers officers - @people[:Officers] = officers - end - - def happiness_of_town - #happiness is random... people don't know what they want! - happinessVals = Array.new - happiness = 0 - for index in (1..@population) - happinessVals.push(rand((100 - successful_crime_rate) .. 100)) - index += 1 - end - happinessVals.each do |value| - happiness += value - end - return happiness / 100 - end - - def successful_crime_rate - thieves = @people[:thieves] - officers = @people[:Officers] - if thieves <= 0 - odds_percent = 0 - elsif officers > thieves - odds_percent = 0 - else - odds = 1 \ - - officers.to_f / thieves.to_f - odds_percent = odds * 100 - end - return odds_percent - end -end \ No newline at end of file diff --git a/gem_city.rb b/gem_city.rb new file mode 100644 index 0000000..13f806b --- /dev/null +++ b/gem_city.rb @@ -0,0 +1,54 @@ +# This class represents the town of Gem_City +class GemCity + # This is a town riddled with crime but we can find out how happy the town is + attr_reader :population + def initialize + @people = + { + thieves: 5, + Officers: 1 + } + @population = 50 + end + + def thieves(thieves_number = @people[:thieves]) + @people[:thieves] = thieves_number + end + + def officers + @people[:Officers] + end + + # def population + # @population + # end + + def _officers(officers) + @people[:Officers] = officers + end + + def happiness_of_town + # happiness is random... people don't know what they want! + happiness_vals = [] + happiness = 0 + (1..@population).each do + happiness_vals.push(rand((100 - successful_crime_rate)..100)) + end + happiness_vals.each do |value| + happiness += value + end + happiness / 100 + end + + def successful_crime_rate + thieves = @people[:thieves] + officers = @people[:Officers] + if thieves <= 0 || officers > thieves + odds_percent = 0 + else + odds = 1 - officers.to_f / thieves.to_f + odds_percent = odds * 100 + end + odds_percent + end +end diff --git a/gem_city_spec.rb b/gem_city_spec.rb index 884b3cb..0231f42 100644 --- a/gem_city_spec.rb +++ b/gem_city_spec.rb @@ -1,5 +1,5 @@ require 'rspec' -require_relative 'GemCity' # This line may need to be changed +require_relative 'gem_city' # This line may need to be changed describe 'Gem City' do let(:city) { GemCity.new } @@ -16,7 +16,7 @@ it 'officers = thieves' do city.thieves 1 - city.setOfficers 1 # This line may need to be changed + city._officers 1 # This line may need to be changed expect(city.successful_crime_rate).to eq(0) end @@ -27,7 +27,7 @@ it 'officers > thieves' do city.thieves 1 - city.setOfficers 2 # This line may need to be changed + city._officers 2 # This line may need to be changed expect(city.successful_crime_rate).to eq(0) end end From 3e464982c9945c7fefd29b5a42d36cc5de53a550 Mon Sep 17 00:00:00 2001 From: Rajashree K Maiya Date: Tue, 12 Jan 2016 19:46:07 -0500 Subject: [PATCH 2/2] All tests passed-Rubocop --- gem_city.rb | 43 +++++++++++++++++++------------------------ gem_city_spec.rb | 46 +++++++++++++++++++++++----------------------- 2 files changed, 42 insertions(+), 47 deletions(-) diff --git a/gem_city.rb b/gem_city.rb index 13f806b..6734c17 100644 --- a/gem_city.rb +++ b/gem_city.rb @@ -1,32 +1,13 @@ # This class represents the town of Gem_City class GemCity # This is a town riddled with crime but we can find out how happy the town is - attr_reader :population + attr_accessor :population, :thieves, :officers def initialize - @people = - { - thieves: 5, - Officers: 1 - } + @thieves = 5 + @officers = 1 @population = 50 end - def thieves(thieves_number = @people[:thieves]) - @people[:thieves] = thieves_number - end - - def officers - @people[:Officers] - end - - # def population - # @population - # end - - def _officers(officers) - @people[:Officers] = officers - end - def happiness_of_town # happiness is random... people don't know what they want! happiness_vals = [] @@ -41,8 +22,6 @@ def happiness_of_town end def successful_crime_rate - thieves = @people[:thieves] - officers = @people[:Officers] if thieves <= 0 || officers > thieves odds_percent = 0 else @@ -51,4 +30,20 @@ def successful_crime_rate end odds_percent end + + def city_demographics + civilians = population - (thieves + officers) + demographics = + { + thieves: (get_percentage(thieves)) + '%', + officers: (get_percentage(officers)) + '%', + civilians: (get_percentage(civilians)) + '%' + } + demographics + end + + def get_percentage(num) + num = ((num.to_f / population.to_f) * 100.0).to_i + num.to_s + end end diff --git a/gem_city_spec.rb b/gem_city_spec.rb index 0231f42..72fed07 100644 --- a/gem_city_spec.rb +++ b/gem_city_spec.rb @@ -15,19 +15,19 @@ end it 'officers = thieves' do - city.thieves 1 - city._officers 1 # This line may need to be changed + city.thieves = 1 + city.officers = 1 # This line may need to be changed expect(city.successful_crime_rate).to eq(0) end it 'thieves = 0' do - city.thieves 0 + city.thieves = 0 expect(city.successful_crime_rate).to eq(0) end it 'officers > thieves' do - city.thieves 1 - city._officers 2 # This line may need to be changed + city.thieves = 1 + city.officers = 2 # This line may need to be changed expect(city.successful_crime_rate).to eq(0) end end @@ -38,26 +38,26 @@ end it 'successful_crime_rate = 0' do - city.thieves 0 + city.thieves = 0 expect(city.happiness_of_town).to eq(50) end end - # context 'city_demographics' do - # it 'initialize' do - # demographics = city.city_demographics - # expect(demographics[:thieves]).to eq('10%') - # expect(demographics[:officers]).to eq('2%') - # expect(demographics[:civilians]).to eq('88%') - # end - - # it 'thieves = 10, officers = 25' do - # city.thieves = 10 - # city.officers = 25 - # demographics = city.city_demographics - # expect(demographics[:thieves]).to eq('20%') - # expect(demographics[:officers]).to eq('50%') - # expect(demographics[:civilians]).to eq('30%') - # end - # end + context 'city_demographics' do + it 'initialize' do + demographics = city.city_demographics + expect(demographics[:thieves]).to eq('10%') + expect(demographics[:officers]).to eq('2%') + expect(demographics[:civilians]).to eq('88%') + end + + it 'thieves = 10, officers = 25' do + city.thieves = 10 + city.officers = 25 + demographics = city.city_demographics + expect(demographics[:thieves]).to eq('20%') + expect(demographics[:officers]).to eq('50%') + expect(demographics[:civilians]).to eq('30%') + end + end end