battleroom 0.0.87 → 0.0.871

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ea6eacbcacf28f4bca5940747bf9f4da36ef5d13
-  data.tar.gz: 21b8d9c2b78ab1a42fef3cc600288fe854213c3f
+  metadata.gz: 409f7278a020fb96436c88637a9e4b66fe6a5474
+  data.tar.gz: 734eb52c7e3e8f4ad0b45d7d83d26c06e5136ae9
 SHA512:
-  metadata.gz: 3eb408861d7139e022230ccda9065e86f532a826c6d23b3d7f1b8941ac1e6c407f61b5b5b5b37069c6259706a793d124908c3187166ae80e510aa8f0de53f65f
-  data.tar.gz: 566149a7846b4534eb531a3480c95575a89e294319e2316c67681837d5ed555cc385690adce577519400bb121abac2c3ea1fb195acd69300ff090309cb3672b4
+  metadata.gz: 4544cad6b48559e668db2fff4d1c56875f804090ec706c7cc9f911d4f930278fd6848119e4ee371babe6f884c5a886a8acadfadef9b9c10aa5a2cf188c422d99
+  data.tar.gz: 2bd6d79c5a80621e26b54ea7d61f9a8b64c2a66b87a12db7a1396f50d085fa1e543483f7eedde0c68755cf3fb8f3f4eeeaa0a343cb3eba26f80977a093ce6d78

data/lib/battleroom.rb

@@ -28,11 +28,11 @@ loop do
     5.times do
       q = [ArrayAssignmentQuestion.new(b), HashAssignmentQuestion.new(b)].sample
     end
-  # when '4'
-    # 5.times do
-    #   q = MethodDefinitionQuestion.new(b)
-    #   follow_up_question = MethodInvocationQuestion.new(b, q)
-    # end
+  when '4'
+    5.times do
+      q = MethodDefinitionQuestion.new(b)
+      follow_up_question = MethodInvocationQuestion.new(b, q)
+    end
   when '4'
     5.times { NestedDataStructureAccessQuestion.new(b) }
   else

data/lib/battleroom/battleroom_machinery.rb

@@ -1,96 +1,19 @@
-module BattleroomMachinery
+require_relative 'helpers/printable'
+require_relative 'helpers/exceptionable'
 
-  CONGRATULATIONS = [
-    'Lovely work, my friend!',
-    'Beautiful!',
-    'Lovely!',
-    'Splendid!',
-    'Awesome.',
-    'Stand up job, my friend.',
-    'Nailed it.',
-    'Nice job! Keep it rolling!',
-    'Capital work, battlestar!',
-    'Exemplary work!',
-    'Yeah!',
-    'Roll on.',
-    'Bullseye!',
-    'Woo!',
-    'Let it ride!',
-    'Touchdown!',
-    "You're on your way!",
-    "You're making a prosperous go of this programming thing.",
-  ]
+module BattleroomMachinery
+  include Battleroom::Printable
+  include Battleroom::Exceptionable
 
   def clear_display
     `reset`
   end
 
-  def print_menu_options
-    battleprint ' What would you like to work on?'.blue
-    battleprint '1. Working with Variables'
-    battleprint '2. Accessing Values in Arrays and Hashes'
-    battleprint '3. Adding Values to Arrays and Hashes'
-    # battleprint '4. Working with Methods'
-    battleprint "Q. Quit\r\n\n"
-  end
-
-  def random_congratulation
-    CONGRATULATIONS.sample
-  end
-
-
-  def print_congratulation
-    battleprint "\n#{random_congratulation}\n".green
-  end
-
   def rotate_array(array)
     item = array.shift
     array << item
   end
 
-  def print_colorized_error_prompt(error)
-    method_or_variable = (error.class == NoMethodError) ? "method" : "variable"
-    battleprint "\nYou're referencing a #{method_or_variable} that doesn't exist, probably as the result of a mispelling. This results in a common Ruby error that reads: \n".red
-    battleprint "\tundefined local variable or method \'WHATEVER_YOU_MISTYPED\'\n".green
-    battleprint "Get used to it and try again.\n".red
-  end
-
-  def indent_all_lines_for_stdout(string_to_indent)
-    string_to_indent.gsub(/^.*/) { |match| "\t" + match }
-  end
-
-  def dynamic_word_wrap(string)
-    string.wrap(150)
-  end
-
-  def battleprint(string)
-    puts dynamic_word_wrap(string)
-  end
-
-  def isolate_variable_name_from_name_error(error)
-    /`(.+)'/i.match(error.message)
-  end
-
-  def window_width
-    begin
-      width_string = `tput cols`
-    # for cases where systems don't respond to the tput command
-    rescue StandardError => e
-      width_string = "80"
-    end
-    width_string.strip.to_i
-  end
-
-  def print_unexpected_end_of_input_explanation(error)
-    battleprint "\nNope! You just triggered a common Ruby error that reads:\n".red
-    battleprint "\tsyntax error, unexpected end-of-input\n".green
-    if error.message.include?(']')
-      battleprint "Basically, you used an opening square bracket '[', but didn't pair it with a closing square bracket. Try again.\n".red
-    else
-      battleprint "Basically, you told Ruby you were going to assign a value to a variable, but you neglected to provide a valid value. Try again.\n".red
-    end
-  end
-
   def determine_variable_follow_up_question(eval_scope, question)
     if question.variable_value == true || question.variable_value == false
       VariableReassignmentQuestion.new(eval_scope, question)
@@ -101,30 +24,6 @@ module BattleroomMachinery
     end
   end
 
-  def print_colorized_type_error_prompt(error)
-    battleprint "\nNope! You just triggered a common Ruby error that reads:\n".red
-    battleprint "\tin '[]', #{error.message}".green
-    error.message.match /conversion\sof\s(.+)\sinto\sInteger/i
-    battleprint "\nBasically, you put a #{$1} between square brackets, whereas Ruby ".red +
-         "was expecting an index value, i.e. an integer. This commonly arises ".red +
-         "when programmers think they're dealing with a hash, when in fact ".red +
-         "they're dealing with an array. Try again.\n".red
-  end
-
-  def format_value_for_stdout_and_eval(object)
-    case object.class.to_s
-    when 'String'
-      if object.include?("'")
-        '"' + object + '"'
-      else
-        "'#{object}'"
-      end
-    when 'Symbol' then return ":#{object}"
-    else
-      object.to_s
-    end
-  end
-
   def naughty_input?(user_input)
     if user_input.match(/(require|`|binding)/)
       battleprint "No way no how! Try again.\n".red
@@ -134,9 +33,4 @@ module BattleroomMachinery
     end
   end
 
-  def format_class_for_output(klass)
-    return 'Boolean' if klass.to_s.match /(TrueClass|FalseClass)/
-    klass.to_s
-  end
-
 end

data/lib/battleroom/config/boot.rb

@@ -5,6 +5,7 @@ require 'pry'
 require 'colorize'
 require 'coderay'
 require 'word_wrap'
+require 'rubygems/dependency_installer'
 
 path = File.expand_path("../models/*.rb", File.dirname(__FILE__))
 Dir[path].each { |file| require file }

data/lib/battleroom/config/pry_config.rb

@@ -3,17 +3,14 @@ I18n.config.enforce_available_locales = false
 
 def configure_pry
   Pry.config.default_window_size = 0
-  Pry.config.quiet = true
-  Pry.prompt = [
-    proc { "> ".blue },
-    proc { "* ".blue }
-  ]
-  Pry.config.memory_size = 10
+  Pry.config.quiet               = true
+  Pry.config.memory_size         = 10
+  Pry.prompt                     = [proc { "> ".blue }, proc { "* ".blue }]
 
   # removes pry's exit command, which is run before any of the before_eval hooks, resulting in a pry session users can't escape
   Pry::Commands.delete("exit")
 
-  # # quiets pry error caused by self_termination (below)
+  # quiets pry error caused by self_termination (below)
   Pry.config.exception_handler = proc { |output, exception, _| }
 
   # short circuits pry REPL before eval, saving eval for Question
@@ -21,7 +18,7 @@ def configure_pry
     begin
       # exports user input for availability
       $input = last_input
-      unless last_input.include?("revert_pry_to_defaults")
+      unless last_input.include?("restore_pry_defaults")
         pry_instance.run_command("continue")
       end
     # quiets ArgumentError thrown by pry-byebug as a result of self_termination hook (above)
@@ -30,11 +27,12 @@ def configure_pry
   end
 end
 
+# for use in debugging
 def restore_pry_defaults
   Pry.config.default_window_size = 15
-  Pry.config.quiet = false
-  Pry.prompt = Pry::DEFAULT_PROMPT
-  Pry.config.hooks.delete_hook :before_eval, :self_terminate
+  Pry.config.quiet               = false
+  Pry.prompt                     = Pry::DEFAULT_PROMPT
+  Pry.config.hooks.delete_hook     :before_eval, :self_terminate
 end
 
 configure_pry

data/lib/battleroom/data/array_questions.rb

@@ -1,4 +1,4 @@
-require_relative '../data_generation_machinery'
+require_relative '../helpers/data_generation_machinery'
 include DataGenerationMachinery
 
 $random_names_array = gen_random_names_array

data/lib/battleroom/data/hash_questions.rb

@@ -1,4 +1,4 @@
-require_relative '../data_generation_machinery'
+require_relative '../helpers/data_generation_machinery'
 include DataGenerationMachinery
 require 'active_support/inflector'
 

data/lib/battleroom/data/variable_assignment_questions.rb

@@ -1,4 +1,4 @@
-require_relative '../data_generation_machinery'
+require_relative '../helpers/data_generation_machinery'
 include DataGenerationMachinery
 
 $random_names_array = gen_random_names_array

data/lib/battleroom/helpers/exceptionable.rb

@@ -0,0 +1,36 @@
+module Battleroom
+  module Exceptionable
+
+    def print_colorized_error_prompt(error)
+      method_or_variable = (error.class == NoMethodError) ? "method" : "variable"
+      battleprint "\nYou're referencing a #{method_or_variable} that doesn't exist, probably as the result of a mispelling. This results in a common Ruby error that reads: \n".red
+      battleprint "\tundefined local variable or method \'WHATEVER_YOU_MISTYPED\'\n".green
+      battleprint "Get used to it and try again.\n".red
+    end
+
+    def isolate_variable_name_from_name_error(error)
+      /`(.+)'/i.match(error.message)
+    end
+
+    def print_unexpected_end_of_input_explanation(error)
+      battleprint "\nNope! You just triggered a common Ruby error that reads:\n".red
+      battleprint "\tsyntax error, unexpected end-of-input\n".green
+      if error.message.include?(']')
+        battleprint "Basically, you used an opening square bracket '[', but didn't pair it with a closing square bracket. Try again.\n".red
+      else
+        battleprint "Basically, you told Ruby you were going to assign a value to a variable, but you neglected to provide a valid value. Try again.\n".red
+      end
+    end
+
+    def print_colorized_type_error_prompt(error)
+      battleprint "\nNope! You just triggered a common Ruby error that reads:\n".red
+      battleprint "\tin '[]', #{error.message}".green
+      error.message.match /conversion\sof\s(.+)\sinto\sInteger/i
+      battleprint "\nBasically, you put a #{$1} between square brackets, whereas Ruby ".red +
+           "was expecting an index value, i.e. an integer. This commonly arises ".red +
+           "when programmers think they're dealing with a hash, when in fact ".red +
+           "they're dealing with an array. Try again.\n".red
+    end
+
+  end
+end

data/lib/battleroom/helpers/printable.rb

@@ -0,0 +1,82 @@
+module Battleroom
+  module Printable
+
+    CONGRATULATIONS = [
+      'Lovely work, my friend!',
+      'Beautiful!',
+      'Lovely!',
+      'Splendid!',
+      'Awesome.',
+      'Stand up job, my friend.',
+      'Nailed it.',
+      'Nice job! Keep it rolling!',
+      'Capital work, battlestar!',
+      'Exemplary work!',
+      'Yeah!',
+      'Roll on.',
+      'Bullseye!',
+      'Woo!',
+      'Let it ride!',
+      'Touchdown!',
+      "You're on your way!",
+      "You're making a prosperous go of this programming thing.",
+    ]
+
+    def print_menu_options
+      battleprint ' What would you like to work on?'.blue
+      battleprint '1. Working with Variables'
+      battleprint '2. Accessing Values in Arrays and Hashes'
+      battleprint '3. Adding Values to Arrays and Hashes'
+      battleprint '4. Working with Methods'
+      battleprint "Q. Quit\r\n\n"
+    end
+
+    def random_congratulation
+      CONGRATULATIONS.sample
+    end
+
+    def print_congratulation
+      battleprint "\n#{random_congratulation}\n".green
+    end
+
+    def indent_all_lines_for_stdout(string_to_indent)
+      string_to_indent.gsub(/^.*/) { |match| "\t" + match }
+    end
+
+    def dynamic_word_wrap(string)
+      string.wrap(150)
+    end
+
+    def battleprint(string)
+      puts dynamic_word_wrap(string)
+    end
+
+    def format_value_for_stdout_and_eval(object)
+      case object.class.to_s
+      when 'String'
+        if object.include?("'")
+          '"' + object + '"'
+        else
+          "'#{object}'"
+        end
+      when 'Symbol' then return ":#{object}"
+      else
+        object.to_s
+      end
+    end
+
+    def format_class_for_output(klass)
+      return 'Boolean' if klass.to_s.match /(TrueClass|FalseClass)/
+      klass.to_s
+    end
+
+    def codify(str)
+      str.black.on_light_white
+    end
+
+    def colorized_arithmetic_operator_list
+      codify(" + ") + ", ".blue + codify(" - ") + ", ".blue + codify(" * ") + ", or ".blue + codify(" / ")
+    end
+
+  end
+end

data/lib/battleroom/models/array_access_question.rb

@@ -5,8 +5,8 @@ class ArrayAccessQuestion < DataStructureAccessQuestion
 
   @questions = ARRAY_QUESTIONS.shuffle
 
-  def initialize(scope)
-    super(scope)
+  def post_initialize
+    super
     format_array_for_access
     print_data_structure_access_prompt
     evaluate_data_structure_access_input

data/lib/battleroom/models/array_assignment_question.rb

@@ -5,8 +5,8 @@ class ArrayAssignmentQuestion < DataStructureAssignmentQuestion
 
   @questions = ARRAY_QUESTIONS
 
-  def initialize(scope)
-    super(scope)
+  def post_initialize
+    super
     format_array_for_assignment
     print_data_structure_assignment_prompt
     evaluate_data_structure_assignment_input

data/lib/battleroom/models/data_structure_access_question.rb

@@ -2,29 +2,45 @@ require_relative './data_structure_question'
 
 class DataStructureAccessQuestion < DataStructureQuestion
 
-  def initialize(eval_scope)
-    super(eval_scope)
-  end
-
   def print_data_structure_access_prompt
     answer_value_class = format_class_for_output(answer_value.class)
     answer_value_string = format_value_for_stdout_and_eval(answer_value)
     battleprint "Given the data structure below, access the #{answer_value_class} value ".blue + answer_value_string.yellow + ".\n".blue
     print "#{variable_name} = ".green
-    if data_structure.to_s.length < 40
-      battleprint data_structure.to_s.gsub('{', '{ ').gsub('=>', ' => ').gsub('}', ' }')
+    if data_structure.to_s.length < 45
+      print_data_structure_on_single_line
     else
       ap(data_structure, { indent: -2, index: false, multiline: true, plain: true })
     end
     battleprint ''
   end
 
+  def provide_evaluation_scope_with_variable_assignment_necessary_for_answer_eval
+    evaluation_scope.eval("#{variable_name} = #{data_structure.to_s}")
+  end
+
+  def generate_readable_single_line_hash
+    data_structure.to_s.gsub('{', '{ ').gsub('=>', ' => ').gsub('}', ' }')
+  end
+
+  def print_data_structure_on_single_line
+    single_line_hash_as_string = generate_readable_single_line_hash()
+    battleprint(single_line_hash_as_string)
+  end
+
+  def eval_returns_the_correct_value?(user_input)
+    evaluation_scope.eval(user_input) == answer_value
+  end
+
+  def user_input_uses_variable?(user_input)
+    user_input.include?(variable_name)
+  end
+
   def evaluate_data_structure_access_input
     enter_evaluation_loop do |user_submission|
       begin
-        # provides the evaluation scope with variable assignment necessary for answer eval
-        evaluation_scope.eval("#{variable_name} = #{data_structure.to_s}")
-        if evaluation_scope.eval(user_submission) == answer_value && user_submission.include?(variable_name)
+        provide_evaluation_scope_with_variable_assignment_necessary_for_answer_eval()
+        if eval_returns_the_correct_value?(user_submission) && user_input_uses_variable?(user_submission)
           true
         else
           battleprint "Remember, #{hint} Try again.".red

data/lib/battleroom/models/data_structure_assignment_question.rb

@@ -19,8 +19,8 @@ class DataStructureAssignmentQuestion < DataStructureQuestion
     'Terrific'
   ]
 
-  def initialize(eval_scope)
-    super(eval_scope)
+  def post_initialize
+    super
   end
 
   def print_data_structure

data/lib/battleroom/models/data_structure_question.rb

@@ -3,8 +3,7 @@ require_relative './question'
 class DataStructureQuestion < Question
   attr_accessor :data_structure, :hint, :possible_assignments
 
-  def initialize(eval_scope)
-    super(eval_scope)
+  def post_initialize
     @data_structure = data[:data_structure].clone
     if data_structure.class == Array
       # randomizes and shuffles the items in the arrays, so repeats remain interesting

data/lib/battleroom/models/follow_up_question.rb

@@ -8,6 +8,11 @@ class FollowUpQuestion < Question
     @evaluation_scope = evaluation_scope
     @original_question = original_question
     @input_mechanism = 'readline'
+    post_initialize
+  end
+
+  def post_initialize
+    nil
   end
 
 end

data/lib/battleroom/models/hash_access_question.rb

@@ -5,8 +5,8 @@ class HashAccessQuestion < DataStructureAccessQuestion
 
   @questions = HASH_QUESTIONS.shuffle
 
-  def initialize(scope)
-    super(scope)
+  def post_initialize
+    super
     format_hash
     print_data_structure_access_prompt
     evaluate_data_structure_access_input
@@ -14,12 +14,12 @@ class HashAccessQuestion < DataStructureAccessQuestion
 
   def format_hash
     cull_hash_to_valid_size_for_output
-    remove_multiple_booleans
+    remove_multiple_booleans_to_avoid_possibility_of_ambiguous_prompt
     self.answer_value = data_structure[data_structure.keys.sample]
     self.hint = 'you have to use the EXACT hash key to retrieve the associated value.'
   end
 
-  def remove_multiple_booleans
+  def remove_multiple_booleans_to_avoid_possibility_of_ambiguous_prompt
     boolean_count = find_number_of_boolean_values_in_hash
     while boolean_count > 1
       grouped_by_value = data_structure.group_by { |k, v| v }

data/lib/battleroom/models/hash_assignment_question.rb

@@ -5,8 +5,8 @@ class HashAssignmentQuestion < DataStructureAssignmentQuestion
 
   @questions = HASH_QUESTIONS.shuffle
 
-  def initialize(scope)
-    super(scope)
+  def post_initialize
+    super
     format_hash_for_assignment
     print_data_structure_assignment_prompt
     evaluate_data_structure_assignment_input
@@ -14,11 +14,11 @@ class HashAssignmentQuestion < DataStructureAssignmentQuestion
 
   def format_hash_for_assignment
     cull_hash_to_valid_size_for_output
-    assignment = possible_assignments.sample
-    self.assignment_value = assignment.values[0]
-    self.assignment_value_class = format_class_for_output(assignment_value.class)
+    assignment                      = possible_assignments.sample
+    self.assignment_value           = assignment.values[0]
+    self.assignment_value_class     = format_class_for_output(assignment_value.class)
     self.formatted_assignment_value = format_value_for_stdout_and_eval(assignment_value)
-    self.assignment_key = format_value_for_stdout_and_eval(assignment.keys[0])
+    self.assignment_key             = format_value_for_stdout_and_eval(assignment.keys[0])
   end
 
   def print_data_structure_assignment_prompt

data/lib/battleroom/models/method_definition_question.rb

@@ -1,102 +1,101 @@
 require_relative '../data/method_questions'
 require_relative './question'
 
-# class MethodDefinitionQuestion < Question
+class MethodDefinitionQuestion < Question
 
-#   attr_accessor :method_name, :arg_count, :spec, :eval_string, :eval_answer,
-#                 :return_value, :user_answer_verified
+  attr_accessor :method_name, :arg_count, :spec, :eval_string, :eval_answer,
+                :return_value, :user_answer_verified
 
-#   @questions = METHOD_QUESTONS.shuffle
+  @questions = METHOD_QUESTONS.shuffle
 
-#   def initialize(scope)
-#     super(scope)
-#     @method_name = data[:method_name]
-#     @arg_count = data[:arg_count]
-#     @spec = data[:spec]
-#     @eval_string = data[:eval_string]
-#     @eval_answer = data[:eval_answer]
-#     @input_mechanism = 'pry'
-#     print_prompt
-#     evaluate_method_definition_input
-#   end
+  def post_initialize
+    @method_name = data[:method_name]
+    @arg_count = data[:arg_count]
+    @spec = data[:spec]
+    @eval_string = data[:eval_string]
+    @eval_answer = data[:eval_answer]
+    @input_mechanism = 'pry'
+    print_prompt
+    evaluate_method_definition_input
+  end
 
-#   def print_prompt
-#     battleprint('Define a method called '.blue + method_name.yellow +  ' that takes '.blue + arg_count.to_s.yellow + " argument(s) and #{spec}.\n".blue)
-#   end
+  def print_prompt
+    battleprint('Define a method called '.blue + method_name.yellow +  ' that takes '.blue + arg_count.to_s.yellow + " argument(s) and #{spec}.\n".blue)
+  end
 
-#   def handle_name_error_exceptions(error, user_submission)
-#     if user_submission.include?('def') == false
-#       print_no_method_error_prompt
-#     else
-#       print_colorized_error_prompt(error)
-#     end
-#   end
+  def handle_name_error_exceptions(error, user_submission)
+    if user_submission.include?('def') == false
+      print_no_method_error_prompt
+    else
+      print_colorized_error_prompt(error)
+    end
+  end
 
-#   def handle_incorrect_method_definition(return_value)
-#     battleprint "\nWhen invoking your method via ".red + eval_string + ", your method returned #{return_value || 'nil'}. It should have returned #{eval_answer}. Try again.\n".red
-#   end
+  def handle_incorrect_method_definition(return_value)
+    battleprint "\nWhen invoking your method via ".red + eval_string + ", your method returned #{return_value || 'nil'}. It should have returned #{eval_answer}. Try again.\n".red
+  end
 
-#   def print_puts_explanation
-#     battleprint 'The last line of your method definition uses Ruby\'s "puts" method. The "puts" method is helpful for logging errors and statuses to the console, but its actual '.red + "return".red.underline + " value is always nil, and thus your method returns nil. Try again, this time without using \"puts\".\n".red
-#   end
+  def print_puts_explanation
+    battleprint 'The last line of your method definition uses Ruby\'s "puts" method. The "puts" method is helpful for logging errors and statuses to the console, but its actual '.red + "return".red.underline + " value is always nil, and thus your method returns nil. Try again, this time without using \"puts\".\n".red
+  end
 
-#   def print_no_method_error_prompt
-#     battleprint "\nYou just trigged a common Ruby error that reads: \n".red
-#     battleprint "\tundefined method \'WHATEVER_METHOD_YOU_TRIED_TO_INVOKE\'\n".green
-#     battleprint "Basically, you tried to use a method before you defined it, and Ruby said, \"You haven't told me how to do that yet.\" To let Ruby know that you're defining a method, you'll use the the \"def\" keyword, followed by the desired method name, and in many cases an optional list of arguments between parentheses. Once you've finished writing the method's inner code, you'll let Ruby know that your method definition is finished by using the \"end\" keyword.\n".red
-#   end
+  def print_no_method_error_prompt
+    battleprint "\nYou just trigged a common Ruby error that reads: \n".red
+    battleprint "\tundefined method \'WHATEVER_METHOD_YOU_TRIED_TO_INVOKE\'\n".green
+    battleprint "Basically, you tried to use a method before you defined it, and Ruby said, \"You haven't told me how to do that yet.\" To let Ruby know that you're defining a method, you'll use the the \"def\" keyword, followed by the desired method name, and in many cases an optional list of arguments between parentheses. Once you've finished writing the method's inner code, you'll let Ruby know that your method definition is finished by using the \"end\" keyword.\n".red
+  end
 
-#   def fresh_binding
-#     binding
-#   end
+  def fresh_binding
+    binding
+  end
 
-#   def print_wrong_method_error(error, user_submission)
-#     definition_pattern = Regexp.new("def\s*#{method_name}")
-#     if user_submission.match(definition_pattern)
-#       handle_incorrect_method_definition(user_submission)
-#     else
-#       battleprint "\nYou defined the wrong method, probably as the result of a mispelling. Try again.".red
-#     end
-#   end
+  def print_wrong_method_error(error, user_submission)
+    definition_pattern = Regexp.new("def\s*#{method_name}")
+    if user_submission.match(definition_pattern)
+      handle_incorrect_method_definition(user_submission)
+    else
+      battleprint "\nYou defined the wrong method, probably as the result of a mispelling. Try again.".red
+    end
+  end
 
-#   def print_argument_error_prompt(e)
-#     e.message.match(/wrong number of arguments \((\d) for (\d)\)/)
-#     passed_arg_count = $1.to_i
-#     expected_arg_count = $2.to_i
-#     battleprint "\nLooks like you defined #{method_name} to take #{expected_arg_count} argument(s), when it should take #{arg_count}. Try again.\n".red
-#   end
+  def print_argument_error_prompt(e)
+    e.message.match(/wrong number of arguments \((\d) for (\d)\)/)
+    passed_arg_count = $1.to_i
+    expected_arg_count = $2.to_i
+    battleprint "\nLooks like you defined #{method_name} to take #{expected_arg_count} argument(s), when it should take #{arg_count}. Try again.\n".red
+  end
 
-#   def clean_eval_scope_of_method_definition
-#     if evaluation_scope.eval "respond_to?(:#{method_name}, true)"
-#       evaluation_scope.eval 'Object.class_eval("remove_method :' + method_name + '")'
-#     end
-#   end
+  def clean_eval_scope_of_method_definition
+    if evaluation_scope.eval "respond_to?(:#{method_name}, true)"
+      evaluation_scope.eval 'Object.class_eval("remove_method :' + method_name + '")'
+    end
+  end
 
-#   def evaluate_method_definition_input
-#     enter_evaluation_loop do |user_submission|
-#       begin
-#         clean_eval_scope_of_method_definition
-#         # I want to make sure that the user's method isn't invoked if it uses the puts method
-#         if user_submission.include?('puts')
-#           print_puts_explanation
-#         else
-#           evaluation_scope.eval(user_submission)
-#           return_value = evaluation_scope.eval(eval_string)
-#           if (return_value == eval_answer)
-#             self.user_answer_verified = user_submission
-#             true
-#           else
-#             handle_incorrect_method_definition(return_value)
-#           end
-#         end
-#       rescue ArgumentError => e
-#         print_argument_error_prompt(e)
-#       rescue NoMethodError => e
-#         print_wrong_method_error(e, user_submission)
-#       rescue NameError => e
-#         handle_name_error_exceptions(e, user_submission)
-#       end
-#     end
-#   end
+  def evaluate_method_definition_input
+    enter_evaluation_loop do |user_submission|
+      begin
+        clean_eval_scope_of_method_definition
+        # I want to make sure that the user's method isn't invoked if it uses the puts method
+        if user_submission.include?('puts')
+          print_puts_explanation
+        else
+          evaluation_scope.eval(user_submission)
+          return_value = evaluation_scope.eval(eval_string)
+          if (return_value == eval_answer)
+            self.user_answer_verified = user_submission
+            true
+          else
+            handle_incorrect_method_definition(return_value)
+          end
+        end
+      rescue ArgumentError => e
+        print_argument_error_prompt(e)
+      rescue NoMethodError => e
+        print_wrong_method_error(e, user_submission)
+      rescue NameError => e
+        handle_name_error_exceptions(e, user_submission)
+      end
+    end
+  end
 
-# end
+end

data/lib/battleroom/models/method_invocation_question.rb

@@ -1,90 +1,89 @@
 require_relative './follow_up_question'
 
-# class MethodInvocationQuestion < FollowUpQuestion
+class MethodInvocationQuestion < FollowUpQuestion
 
-#   attr_reader :desired_answer_formatted, :desired_answer_class_formatted
+  attr_reader :desired_answer_formatted, :desired_answer_class_formatted
 
-#   def initialize(evaluation_scope, question_to_follow_up_on)
-#     super(evaluation_scope, question_to_follow_up_on)
-#     @desired_answer_formatted = format_value_for_stdout_and_eval(original_question.eval_answer)
-#     @desired_answer_class_formatted = format_class_for_output(original_question.eval_answer.class)
-#     print_method_invocation_prompt
-#     evaluate_user_input
-#   end
+  def post_initialize
+    @desired_answer_formatted = format_value_for_stdout_and_eval(original_question.eval_answer)
+    @desired_answer_class_formatted = format_class_for_output(original_question.eval_answer.class)
+    print_method_invocation_prompt
+    evaluate_user_input
+  end
 
-#   def format_method_definition_for_stdout
-#     code = CodeRay.scan($input, :ruby)
-#     ansi_prepped_string = code.term
-#     indent_all_lines_for_stdout(ansi_prepped_string)
-#   end
+  def format_method_definition_for_stdout
+    code = CodeRay.scan($input, :ruby)
+    ansi_prepped_string = code.term
+    indent_all_lines_for_stdout(ansi_prepped_string)
+  end
 
-#   def print_method_invocation_prompt
-#     battleprint "You now have the method defined below at your disposal.\n".blue
-#     battleprint format_method_definition_for_stdout
-#     battleprint "\nInvoke the '#{original_question.method_name}' method such that it returns the ".blue + desired_answer_class_formatted.blue + " value ".blue + desired_answer_formatted.yellow + ".\n".blue
-#   end
+  def print_method_invocation_prompt
+    battleprint "You now have the method defined below at your disposal.\n".blue
+    battleprint format_method_definition_for_stdout
+    battleprint "\nInvoke the '#{original_question.method_name}' method such that it returns the ".blue + desired_answer_class_formatted.blue + " value ".blue + desired_answer_formatted.yellow + ".\n".blue
+  end
 
-#   def print_no_method_error_prompt
-#     battleprint "\nYou just trigged a common Ruby error that reads: \n".red
-#     battleprint "\tundefined local variable or method \'WHATEVER_METHOD_YOU_TRIED_TO_INVOKE\'\n".green
-#     battleprint "Basically, you tried to invoke a method that doesn't exist. Try again.\n".red
-#   end
+  def print_no_method_error_prompt
+    battleprint "\nYou just trigged a common Ruby error that reads: \n".red
+    battleprint "\tundefined local variable or method \'WHATEVER_METHOD_YOU_TRIED_TO_INVOKE\'\n".green
+    battleprint "Basically, you tried to invoke a method that doesn't exist. Try again.\n".red
+  end
 
-#   def print_name_error_prompt(error, user_submission)
-#     battleprint "\nYou just triggered a common Ruby error that reads:\n".red
-#     battleprint "\tNameError: #{error.message}\n".green
-#     referenced_variable = isolate_variable_name_from_name_error(error)
-#     parameters = isolate_argument_names_from_method_def
-#     if parameters.find { |parameter| parameter == referenced_variable }
-#       battleprint "Looks like the variable you referenced was one of the argument names in your method #{'definition'.red.underline}. The arguments in method definitions are just placeholders for whatever values end up getting passed in when the method is called. Think on this for a moment before trying again.\n".red
-#     else
-#       battleprint "Basically, you're referencing a variable, \"#{referenced_variable}\", that hasn't been assigned a value.\n".red
-#     end
-#   end
+  def print_name_error_prompt(error, user_submission)
+    battleprint "\nYou just triggered a common Ruby error that reads:\n".red
+    battleprint "\tNameError: #{error.message}\n".green
+    referenced_variable = isolate_variable_name_from_name_error(error)
+    parameters = isolate_argument_names_from_method_def
+    if parameters.find { |parameter| parameter == referenced_variable }
+      battleprint "Looks like the variable you referenced was one of the argument names in your method #{'definition'.red.underline}. The arguments in method definitions are just placeholders for whatever values end up getting passed in when the method is called. Think on this for a moment before trying again.\n".red
+    else
+      battleprint "Basically, you're referencing a variable, \"#{referenced_variable}\", that hasn't been assigned a value.\n".red
+    end
+  end
 
-#   def isolate_variable_name_from_name_error(error)
-#     error.name.to_s
-#   end
+  def isolate_variable_name_from_name_error(error)
+    error.name.to_s
+  end
 
-#   def print_argument_error_prompt(e)
-#     e.message.match(/wrong number of arguments \((\d) for (\d)\)/)
-#     passed_arg_count = $1.to_i
-#     expected_arg_count = $2.to_i
-#     battleprint "You just triggered a common Ruby error that reads:\n".red
-#     battleprint "\tArgumentError: #{e.message}\n".green
-#     battleprint "Basically, you defined the '#{original_question.method_name}' method to expect #{expected_arg_count} argument(s), and you're only passing it #{passed_arg_count}. Try again.\n".red
-#   end
+  def print_argument_error_prompt(e)
+    e.message.match(/wrong number of arguments \((\d) for (\d)\)/)
+    passed_arg_count = $1.to_i
+    expected_arg_count = $2.to_i
+    battleprint "You just triggered a common Ruby error that reads:\n".red
+    battleprint "\tArgumentError: #{e.message}\n".green
+    battleprint "Basically, you defined the '#{original_question.method_name}' method to expect #{expected_arg_count} argument(s), and you're only passing it #{passed_arg_count}. Try again.\n".red
+  end
 
-#   def isolate_argument_names_from_method_def
-#     isolate_parameter_list_string_from_method_def.scan(/([^,\s]+)+/x).flatten
-#   end
+  def isolate_argument_names_from_method_def
+    isolate_parameter_list_string_from_method_def.scan(/([^,\s]+)+/x).flatten
+  end
 
-#   def isolate_parameter_list_string_from_method_def
-#     original_question.user_answer_verified.match /\((.+)\)/
-#     $1
-#   end
+  def isolate_parameter_list_string_from_method_def
+    original_question.user_answer_verified.match /\((.+)\)/
+    $1
+  end
 
-#   def print_incorrect_input_prompt(return_from_eval)
-#     battleprint "Your code returned the #{format_class_for_output(return_from_eval.class)} value #{return_from_eval.to_s} when it should have returned the #{desired_answer_class_formatted} value #{desired_answer_formatted}. Try again.".red
-#   end
+  def print_incorrect_input_prompt(return_from_eval)
+    battleprint "Your code returned the #{format_class_for_output(return_from_eval.class)} value #{return_from_eval.to_s} when it should have returned the #{desired_answer_class_formatted} value #{desired_answer_formatted}. Try again.".red
+  end
 
-#   def evaluate_user_input
-#     enter_evaluation_loop do |user_submission|
-#       begin
-#         return_from_eval = original_question.evaluation_scope.eval(user_submission)
-#         if return_from_eval == original_question.eval_answer
-#           true
-#         else
-#           print_incorrect_input_prompt(return_from_eval)
-#         end
-#       rescue NoMethodError => e
-#         print_no_method_error_prompt
-#       rescue NameError => e
-#         print_name_error_prompt(e, user_submission)
-#       rescue ArgumentError => e
-#         print_argument_error_prompt(e)
-#       end
-#     end
-#   end
+  def evaluate_user_input
+    enter_evaluation_loop do |user_submission|
+      begin
+        return_from_eval = original_question.evaluation_scope.eval(user_submission)
+        if return_from_eval == original_question.eval_answer
+          true
+        else
+          print_incorrect_input_prompt(return_from_eval)
+        end
+      rescue NoMethodError => e
+        print_no_method_error_prompt
+      rescue NameError => e
+        print_name_error_prompt(e, user_submission)
+      rescue ArgumentError => e
+        print_argument_error_prompt(e)
+      end
+    end
+  end
 
-# end
+end

data/lib/battleroom/models/nested_data_structure_access_question.rb

@@ -6,8 +6,7 @@ class NestedDataStructureAccessQuestion < DataStructureAccessQuestion
   # class instance variable used within the initialize method for Question
   @questions = NESTED_DATA_STRUCTURE_ACCESS_QUESTIONS.shuffle
 
-  def initialize(eval_scope)
-    super(eval_scope)
+  def post_initialize
     if data_structure.class == Array
       self.inner_hash = data_structure.sample
       self.answer_value = inner_hash[inner_hash.keys.sample]

data/lib/battleroom/models/question.rb

@@ -9,10 +9,14 @@ class Question
     @data = self.class.generate_question
     @variable_name = rotate_array(data[:possible_variable_names] || []).first
     @input_mechanism = 'readline'
+    post_initialize
   end
 
-  # retrieves question from front of the array and rotates it to the back
-  # to avoid immediate re-sampling of questions
+  def post_initialize
+    nil
+  end
+
+  # avoids immediate re-sampling of questions
   def self.generate_question
     # calls upon *class instance* variable assigned in the subclasses
     question = @questions.shift

data/lib/battleroom/models/variable_assignment_question.rb

@@ -6,8 +6,7 @@ class VariableAssignmentQuestion < Question
   attr_accessor :formatted_value, :formatted_class
   @questions = VARIABLE_QUESTIONS.shuffle
 
-  def initialize(eval_scope)
-    super(eval_scope)
+  def post_initialize
     @variable_value = rotate_array(data[:possible_variable_values]).first
     @formatted_value = format_value_for_stdout_and_eval(variable_value)
     print_variable_assignment_prompt

data/lib/battleroom/models/variable_reassignment_question.rb

@@ -4,8 +4,7 @@ class VariableReassignmentQuestion < FollowUpQuestion
 
   attr_accessor :toggled_boolean
 
-  def initialize(evaluation_scope, original_question)
-    super(evaluation_scope, original_question)
+  def post_initialize
     @toggled_boolean = !(original_question.variable_value)
     print_variable_reassignment_prompt
     evaluate_variable_reassignment_input
@@ -18,11 +17,15 @@ class VariableReassignmentQuestion < FollowUpQuestion
   def evaluate_variable_reassignment_input
     enter_evaluation_loop do |user_submission|
       input = user_submission
-      evaluation_scope.eval(input)
-      if (evaluation_scope.eval(original_question.variable_name) == toggled_boolean)
-        true
-      else
-        battleprint "Nope. Try again.".red
+      begin
+        evaluation_scope.eval(input)
+        if (evaluation_scope.eval(original_question.variable_name) == toggled_boolean)
+          true
+        else
+          battleprint "Nope. Try again.".red
+        end
+      rescue NameError, NoMethodError => e
+        print_colorized_error_prompt(e)
       end
     end
   end

data/lib/battleroom/models/variable_reference_question.rb

@@ -2,54 +2,62 @@ require_relative './follow_up_question'
 
 class VariableReferenceQuestion < FollowUpQuestion
 
-  attr_accessor :required_return_value
+  attr_accessor :required_return_value, :required_class
+  attr_reader   :variable_name, :variable_value, :formatted_variable_value,
+                :formatted_variable_class
 
-  def initialize(scope, original_question)
-    super(scope, original_question)
+  def post_initialize
+    @variable_name            = original_question.variable_name
+    @variable_value           = original_question.variable_value
+    @formatted_variable_value = original_question.formatted_value
+    @formatted_variable_class = original_question.formatted_class
     print_variable_reference_prompt
     evaluate_variable_reference_input
   end
 
   def print_variable_reference_prompt
     battleprint "You now have the variable assignment below at your disposal.\n".blue
-    battleprint "\t#{original_question.variable_name} = #{original_question.formatted_value}\n\n"
-    prompt = develop_prompt
+    battleprint "\t#{variable_name} = #{formatted_variable_value}\n\n"
+    prompt = develop_prompt()
     puts prompt
   end
 
   def a_value_within_12_more_or_12_less_than_the_original_assigned_value
     range = (-12..12).to_a
     range.delete(0)
-    original_question.variable_value + range.sample
+    variable_value + range.sample
   end
 
   def a_value_that_doesnt_result_from_imprecise_float_arithmetic
-    if original_question.variable_value < 15
-      new_target_value = original_question.variable_value + rand(1..15)
+    if variable_value < 15
+      new_target_value = variable_value + rand(1..15)
     else
       new_target_value = a_value_within_12_more_or_12_less_than_the_original_assigned_value
     end
     count = 1
     while new_target_value.to_s.split(".").last.length > 4
-      new_target_value = original_question.variable_value + count
+      new_target_value = variable_value + count
       count += 1
     end
     new_target_value
   end
 
+  def a_value_that_requires_low_arithmetic_complexity_to_arrive_at_via_multiplication_or_division
+    if variable_value < 12
+      variable_value * [3,4,5,6].sample
+    elsif variable_value < 50
+      variable_value * 2
+    else
+      variable_value / 2
+    end
+  end
+
   def generate_appropriate_value
-    value = original_question.variable_value
-    if value.class == Fixnum
-      if value.even? # && [1,2,3,4,5].sample.even?
-        if value < 12
-          value * [3,4,5,6].sample
-        elsif value < 50
-          value * 2
-        else
-          value / 2
-        end
+    if variable_value.class == Fixnum
+      if variable_value.even?
+        a_value_that_requires_low_arithmetic_complexity_to_arrive_at_via_multiplication_or_division()
       else
-        a_value_within_12_more_or_12_less_than_the_original_assigned_value
+        a_value_within_12_more_or_12_less_than_the_original_assigned_value()
       end
     else
       a_value_that_doesnt_result_from_imprecise_float_arithmetic()
@@ -57,17 +65,22 @@ class VariableReferenceQuestion < FollowUpQuestion
   end
 
   def develop_prompt
-    case original_question.variable_value.class.to_s
-    when 'Fixnum', 'Float'
-     self.required_return_value = generate_appropriate_value()
-     prompt = "Use ".blue + original_question.variable_name.green + " in combination with an arithmetic operator like ".blue +  " + ".black.on_light_white + ", ".blue + " - ".black.on_light_white + ", ".blue + " * ".black.on_light_white + ", or ".blue + " / ".black.on_light_white + " to return the #{original_question.formatted_class} value ".blue + required_return_value.to_s.green + ".\n".blue
-    else
-      battleprint "Something's gone wrong. This code should never run.".red
-    end
+    self.required_return_value = generate_appropriate_value()
+    required_return_value_colorized = required_return_value.to_s.green
+    self.required_class = original_question.formatted_class
+    "Use ".blue + variable_name.green + " in combination with an arithmetic operator like ".blue + colorized_arithmetic_operator_list + " to return the #{required_class} value ".blue + required_return_value_colorized + ".\n".blue
   end
 
   def provide_evaluation_scope_with_original_variable_assignment
-    evaluation_scope.eval("#{original_question.variable_name} = #{original_question.formatted_value}")
+    evaluation_scope.eval("#{variable_name} = #{formatted_variable_value}")
+  end
+
+  def user_doesnt_use_variable?(user_input)
+    !user_input.include?(variable_name)
+  end
+
+  def coach_user_and_reprompt(return_value_of_user_input)
+    puts "Your code returned the #{format_class_for_output(return_value_of_user_input.class)} value #{return_value_of_user_input.to_s} when it should have returned the #{formatted_variable_class} value #{format_value_for_stdout_and_eval(required_return_value)}. Try again.".red
   end
 
   def evaluate_variable_reference_input
@@ -75,14 +88,14 @@ class VariableReferenceQuestion < FollowUpQuestion
       begin
         provide_evaluation_scope_with_original_variable_assignment
         returned_value = evaluation_scope.eval(user_submission)
-        if !user_submission.include?(original_question.variable_name)
-          battleprint "You didn't make use of the '#{original_question.variable_name}' variable, which is the entire purpose of this exercise. Try again.".red
+        if user_doesnt_use_variable?(user_submission)
+          battleprint "You didn't make use of the '#{variable_name}' variable, which is the entire purpose of this exercise. Try again.".red
         elsif user_submission.include?("=")
-          battleprint "Looks like you simply assigned (or reassigned) a value to a variable, rather than making use of the value stored in '#{original_question.variable_name}'. Reread the directions and try again!".red
+          battleprint "Looks like you simply assigned (or reassigned) a value to a variable, rather than making use of the value stored in '#{variable_name}'. Reread the directions and try again!".red
         elsif (returned_value == required_return_value)
           true
         else
-          puts "Your code returned the #{format_class_for_output(returned_value.class)} value #{returned_value.to_s} when it should have returned the #{original_question.formatted_class} value #{format_value_for_stdout_and_eval(required_return_value)}. Try again.".red
+          coach_user_and_reprompt(returned_value)
         end
       rescue NameError, NoMethodError => e
         print_colorized_error_prompt(e)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: battleroom
 version: !ruby/object:Gem::Version
-  version: 0.0.87
+  version: 0.0.871
 platform: ruby
 authors:
 - Travis Vander Hoop
@@ -80,6 +80,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.10'
+- !ruby/object:Gem::Dependency
+  name: pry-byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.3.3
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.3.3
 - !ruby/object:Gem::Dependency
   name: coderay
   requirement: !ruby/object:Gem::Requirement
@@ -128,9 +142,9 @@ files:
 - lib/battleroom/data/nested_data_structure_access_questions.rb
 - lib/battleroom/data/scratch.rb
 - lib/battleroom/data/variable_assignment_questions.rb
-- lib/battleroom/data_generation_machinery.rb
-- lib/battleroom/helpers/array_formatter.rb
-- lib/battleroom/helpers/hash_formatter.rb
+- lib/battleroom/helpers/data_generation_machinery.rb
+- lib/battleroom/helpers/exceptionable.rb
+- lib/battleroom/helpers/printable.rb
 - lib/battleroom/models/array_access_question.rb
 - lib/battleroom/models/array_assignment_question.rb
 - lib/battleroom/models/data_structure_access_question.rb
@@ -158,7 +172,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.1'
+      version: '2.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="

data/lib/battleroom/helpers/array_formatter.rb

@@ -1,3 +0,0 @@
-module ArrayFormatter
-
-end

data/lib/battleroom/helpers/hash_formatter.rb

@@ -1,3 +0,0 @@
-module HashFormatter
-
-end