codebreaker_sn 0.1.0 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8b4bc64fa02fe1525021ca41c2f25989ad79ef13c7f52df4e54f272b6526c2d3
-  data.tar.gz: ac89230e4c36e093aa783dace5d1f9c7e12d380107a55f7d450f531eddbe2843
+  metadata.gz: a9b1441a0483d35cfbd56b0ed3c0a6dd3ec3501b4cae97770e5f7a765211de03
+  data.tar.gz: 55ce5e6e368e9cad3c1ea50b21440813ca3c84164c0daaf4b3ab2928a573358e
 SHA512:
-  metadata.gz: 3002e08c2f92d7bbe4813389ebde8fc31a123c15c919b823d33ee719320e9f1455c8f40e8350c9ec7e65567aa999998b9ce33a25a2f7f349274ca62b265a6af6
-  data.tar.gz: 034aa1c0516dc53a557f5f672028bb13a57a706f349dc5f19473bf7d746761629195ff36fbdeddd90694afd02fd1ac7ff0c72e05a4c9030a674d2e42d3875f4d
+  metadata.gz: 5162f080fef1f3280d08e12ec005505e294df53388a1526e7a1794ec1e93a92441344bfa67844a8a6d1bedf3586fd5063327fe0d5832615c01ef50cf8c9c7157
+  data.tar.gz: bdf5faf548352291c6eb15ab222582f1829d5c04c73ab8b30635eff9a85b10ddff2d9d5291951e06952b5337049a85513745ad707dc47ebae62b52406a7c7130

data/.circleci/config.yml

@@ -19,7 +19,7 @@ commands:
           keys:
             - *bundle_cache_full
             - *bundle_cache
-      - run: bundle config set path 'vendor/bundle'
+      - run: bundle install --path vendor/bundle
       - save_cache:
           key: *bundle_cache_full
           paths:

data/.fasterer.yml

@@ -1,4 +1,2 @@
-speedups:
-  each_with_index_vs_while: false
 exclude_paths:
   - "vendor/**/*.rb"

data/.rubocop.yml

@@ -1,6 +1,8 @@
-require: rubocop-rspec
+require:
+  - rubocop-performance
+  - rubocop-rspec
 
 Style/Documentation:
   Enabled: false
-Metrics/MethodLength:
-  Max: 12
+Metrics/BlockLength:
+  ExcludedMethods: ['describe', 'context']

data/Gemfile

@@ -2,5 +2,4 @@
 
 source 'https://rubygems.org'
 
-# Specify your gem's dependencies in codebreaker.gemspec
 gemspec

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    codebreaker (0.1.0)
+    codebreaker_sn (0.1.1)
 
 GEM
   remote: https://rubygems.org/
@@ -44,6 +44,8 @@ GEM
       unicode-display_width (>= 1.4.0, < 2.0)
     rubocop-ast (0.1.0)
       parser (>= 2.7.0.1)
+    rubocop-performance (1.7.1)
+      rubocop (>= 0.82.0)
     rubocop-rspec (1.41.0)
       rubocop (>= 0.68.1)
     ruby-progressbar (1.10.1)
@@ -60,11 +62,12 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
-  codebreaker!
+  codebreaker_sn!
   fasterer
   rake
   rspec
   rubocop
+  rubocop-performance
   rubocop-rspec
   simplecov
 

data/codebreaker.gemspec

@@ -25,6 +25,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rake'
   spec.add_development_dependency 'rspec'
   spec.add_development_dependency 'rubocop'
+  spec.add_development_dependency 'rubocop-performance'
   spec.add_development_dependency 'rubocop-rspec'
   spec.add_development_dependency 'simplecov'
 end

data/lib/autoload.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+require 'yaml'
+require_relative 'codebreaker/version'
+require_relative 'moduls/validations'
+require_relative 'codebreaker/game'
+require_relative 'codebreaker/user'
+require_relative 'codebreaker/codemaker'
+require_relative 'codebreaker/codechecker'
+require_relative 'codebreaker/guess'
+require_relative 'codebreaker/statistic'
+require_relative 'codebreaker/sorter'
+require_relative 'errors/validation_error'
+require_relative 'errors/length_error'
+require_relative 'errors/presence_error'
+require_relative 'errors/out_of_range_error'
+require_relative 'errors/class_type_error'
+require_relative 'errors/inclusion_error'

data/lib/codebreaker.rb

@@ -1,23 +1,6 @@
 # frozen_string_literal: true
 
-RANGE_OF_NUMBERS = (1..6).freeze
-AMOUNT_OF_NUMBERS = 4
-DIFF_ARRAY = %w[hell medium easy].freeze
-RESULTS_FILE = 'lib/files/results_file.csv'
-RULES_FILE = 'lib/files/rules.txt'
+require_relative 'autoload'
 
 module Codebreaker
-  require 'csv'
-  require 'codebreaker/version'
-  require 'moduls/validations'
-  require 'codebreaker/game'
-  require 'codebreaker/user'
-  require 'codebreaker/codemaker'
-  require 'codebreaker/guess'
-  require 'errors/validation_error'
-  require 'errors/length_error'
-  require 'errors/presence_error'
-  require 'errors/out_of_range_error'
-  require 'errors/class_type_error'
-  require 'errors/inclusion_error'
 end

data/lib/codebreaker/codechecker.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+module Codebreaker
+  class Codechecker
+    attr_reader :guess_array, :secret_code_array, :response
+
+    def initialize(guess_array, secret_code_array)
+      @guess_array = guess_array
+      @secret_code_array = secret_code_array
+      @response = { in_plase: 0, out_of_place: 0 }
+    end
+
+    def call
+      in_plase_check
+      out_of_place_check
+    end
+
+    private
+
+    def in_plase_check
+      index = 0
+      while index < guess_array.size
+        if guess_array[index] == secret_code_array[index]
+          response[:in_plase] += 1
+          guess_array[index] = nil
+          secret_code_array[index] = nil
+        end
+        index += 1
+      end
+    end
+
+    def out_of_place_check
+      guess_array.each do |number|
+        if !number.nil? && secret_code_array.include?(number)
+          response[:out_of_place] += 1
+          secret_code_array[secret_code_array.index(number)] = nil
+        end
+      end
+    end
+  end
+end

data/lib/codebreaker/codemaker.rb

@@ -2,37 +2,15 @@
 
 module Codebreaker
   class Codemaker
-    NON_OCCURRING_VALUE_FIRST = 'X'
-    NON_OCCURRING_VALUE_SECOND = 'Y'
-    NON_OCCURRING_VALUE_THIRD = 'Z'
+    RANGE_OF_NUMBERS = (1..6).freeze
+    AMOUNT_OF_NUMBERS = 4
 
-    attr_reader :secret_code, :response
-
-    def initialize
-      @secret_code = ''
-      @response = { in_plase: 0, out_of_place: 0 }
-    end
+    attr_reader :secret_code
 
     def generate_secret_code(amout = AMOUNT_OF_NUMBERS, range = RANGE_OF_NUMBERS)
       array_of_numbers = []
       amout.times { array_of_numbers << rand(range) }
       @secret_code = array_of_numbers.join.to_i
     end
-
-    def check(guess_array, secret_code_array)
-      guess_array.each_with_index do |number, index|
-        next unless number == secret_code_array[index]
-
-        @response[:in_plase] += 1
-        guess_array[index] = NON_OCCURRING_VALUE_FIRST
-        secret_code_array[index] = NON_OCCURRING_VALUE_SECOND
-      end
-      guess_array.each do |number|
-        if secret_code_array.include?(number)
-          @response[:out_of_place] += 1
-          secret_code_array[secret_code_array.index(number)] = NON_OCCURRING_VALUE_THIRD
-        end
-      end
-    end
   end
 end

data/lib/codebreaker/game.rb

@@ -2,15 +2,16 @@
 
 module Codebreaker
   class Game
-    include Validations
-
     DIFFICULTY_HASH = { easy: { attempts: 15, hints: 2 },
                         medium: { attempts: 10, hints: 1 },
                         hell: { attempts: 5, hints: 1 } }.freeze
 
-    validate :difficulty, :inclusion, %i[easy medium hell]
+    include Validations
+
+    validate :difficulty, :inclusion, DIFFICULTY_HASH.keys
 
-    attr_reader :user, :difficulty, :secret_code
+    attr_reader :user, :difficulty, :secret_code, :hints_used
+    attr_accessor :attempts_used
 
     def initialize(user, difficulty, secret_code)
       @user = user
@@ -22,45 +23,36 @@ module Codebreaker
     end
 
     def check(guess)
-      @attempts_used += 1
+      self.attempts_used += 1
 
-      guess_array = guess.to_s.split('').map(&:to_i)
-      secret_code_array = @secret_code.to_s.split('').map(&:to_i)
+      guess_array = guess.digits.reverse
+      secret_code_array = secret_code.digits.reverse
 
-      codemaker = Codebreaker::Codemaker.new
-      codemaker.check(guess_array, secret_code_array)
-      codemaker.response
+      codechecker = Codechecker.new(guess_array, secret_code_array)
+      codechecker.call
+      codechecker.response
     end
 
     def any_hints_left?
-      DIFFICULTY_HASH[@difficulty][:hints] > @hints_used.count
+      DIFFICULTY_HASH[difficulty][:hints] > hints_used.count
     end
 
     def take_hint
-      secret_code_array = @secret_code.to_s.split('').map(&:to_i)
-      @hints_used.each do |hint_used|
+      secret_code_array = secret_code.digits.reverse
+      hints_used.each do |hint_used|
         secret_code_array.delete_at(secret_code_array.index(hint_used))
       end
-      random_element = secret_code_array.sample
-      @hints_used << random_element
-      random_element
+      hint = secret_code_array.sample
+      hints_used << hint
+      hint
     end
 
     def win?(guess)
-      guess == @secret_code
+      guess == secret_code
     end
 
     def lose?
-      @attempts_used >= DIFFICULTY_HASH[@difficulty][:attempts]
-    end
-
-    def save_result
-      data = "#{@user.name},#{@difficulty},"
-      data += "#{DIFFICULTY_HASH[@difficulty][:attempts]},#{@attempts_used},"
-      data += "#{DIFFICULTY_HASH[@difficulty][:hints]},#{@hints_used.count}"
-      File.open(RESULTS_FILE, 'a') do |file|
-        file.puts data
-      end
+      attempts_used >= DIFFICULTY_HASH[difficulty][:attempts]
     end
   end
 end

data/lib/codebreaker/guess.rb

@@ -8,8 +8,8 @@ module Codebreaker
 
     validate :numbers, :presence
     validate :numbers, :type, Integer
-    validate :numbers, :length, AMOUNT_OF_NUMBERS
-    validate :numbers, :range, RANGE_OF_NUMBERS
+    validate :numbers, :length, Codebreaker::Codemaker::AMOUNT_OF_NUMBERS
+    validate :numbers, :range, Codebreaker::Codemaker::RANGE_OF_NUMBERS
 
     def initialize(numbers)
       @numbers = numbers

data/lib/codebreaker/sorter.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+module Codebreaker
+  class Sorter
+    CONDITIONAL_ARRAY = %i[hell medium easy].freeze
+
+    attr_reader :data
+
+    def initialize(data)
+      @data = data
+    end
+
+    def call
+      sorted = sort_by_hints(data)
+      sorted = sort_by_attempts(sorted)
+      sorted = sort_by_difficulties(sorted)
+      sorted
+    end
+
+    private
+
+    def sort_by_hints(data)
+      data.sort_by { |row| row[:hints_used] }
+    end
+
+    def sort_by_attempts(data)
+      data.sort_by { |row| row[:attempts_used] }
+    end
+
+    def sort_by_difficulties(data)
+      data.sort_by { |row| CONDITIONAL_ARRAY.index(row[:difficulty]) }
+    end
+  end
+end

data/lib/codebreaker/statistic.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+module Codebreaker
+  class Statistic
+    attr_reader :file_path
+
+    def initialize(file_path)
+      @file_path = file_path
+    end
+
+    def save(game)
+      data = File.file?(file_path) && !File.zero?(file_path) ? read : []
+      data << new_record(game)
+      file = File.open(file_path, 'w')
+      file.write(data.to_yaml)
+      file.close
+    end
+
+    def read
+      YAML.load_file(file_path)
+    end
+
+    private
+
+    def new_record(game)
+      user = game.user.name
+      attempts_total = Codebreaker::Game::DIFFICULTY_HASH[game.difficulty][:attempts]
+      attempts_used = game.attempts_used
+      hints_total = Codebreaker::Game::DIFFICULTY_HASH[game.difficulty][:hints]
+      hints_used = game.hints_used.size
+      { user: user, difficulty: game.difficulty, attempts_total: attempts_total,
+        attempts_used: attempts_used, hints_total: hints_total, hints_used: hints_used }
+    end
+  end
+end

data/lib/codebreaker/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Codebreaker
-  VERSION = '0.1.0'
+  VERSION = '0.1.2'
 end

data/lib/moduls/validations.rb

@@ -30,9 +30,7 @@ module Validations
     end
 
     def validate_range(value, name, range)
-      return unless value.to_s.split('').map(&:to_i).any? { |number| !range.include?(number) }
-
-      raise OutOfRangeError, "#{name} must be in #{range}"
+      raise OutOfRangeError, "#{name} must be in #{range}" if out_of_range?(value, range)
     end
 
     def validate_type(value, name, type_class)
@@ -54,5 +52,9 @@ module Validations
         length.member?(count)
       end
     end
+
+    def out_of_range?(value, range)
+      value.digits.reverse.any? { |number| !range.include?(number) }
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: codebreaker_sn
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.2
 platform: ruby
 authors:
 - Sergio
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-07-08 00:00:00.000000000 Z
+date: 2020-07-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fasterer
@@ -66,6 +66,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-performance
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
@@ -117,10 +131,15 @@ files:
 - bin/console
 - bin/setup
 - codebreaker.gemspec
+- codebreaker_sn-0.1.0.gem
+- lib/autoload.rb
 - lib/codebreaker.rb
+- lib/codebreaker/codechecker.rb
 - lib/codebreaker/codemaker.rb
 - lib/codebreaker/game.rb
 - lib/codebreaker/guess.rb
+- lib/codebreaker/sorter.rb
+- lib/codebreaker/statistic.rb
 - lib/codebreaker/user.rb
 - lib/codebreaker/version.rb
 - lib/errors/class_type_error.rb
@@ -129,8 +148,6 @@ files:
 - lib/errors/out_of_range_error.rb
 - lib/errors/presence_error.rb
 - lib/errors/validation_error.rb
-- lib/files/results_file.csv
-- lib/files/rules.txt
 - lib/moduls/validations.rb
 homepage: https://github.com/sergeynakul/codebreaker
 licenses:
@@ -153,7 +170,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.0.4
 signing_key: 
 specification_version: 4
 summary: Codebreaker game

data/lib/files/rules.txt

@@ -1,14 +0,0 @@
-Game Rules
-Codebreaker is a logic game in which a code-breaker tries to break a secret code created by a code-maker. The codemaker, which will be played by the application we’re going to write, creates a secret code of four numbers between 1 and 6.
-The codebreaker gets some number of chances to break the code (depends on chosen difficulty). In each turn, the codebreaker makes a guess of 4 numbers. The codemaker then marks the guess with up to 4 signs - + or - or empty spaces.
-A + indicates an exact match: one of the numbers in the guess is the same as one of the numbers in the secret code and in the same position. For example:
-Secret number - 1234
-Input number - 6264
-Number of pluses - 2 (second and fourth position)
-A - indicates a number match: one of the numbers in the guess is the same as one of the numbers in the secret code but in a different position. For example:
-Secret number - 1234
-Input number - 6462
-Number of minuses - 2 (second and fourth position)
-An empty space indicates that there is not a current digit in a secret number.
-If codebreaker inputs the exact number as a secret number - codebreaker wins the game. If all attempts are spent - codebreaker loses.
-Codebreaker also has some number of hints(depends on chosen difficulty). If a user takes a hint - he receives back a separate digit of the secret code.