Week 3 pull request - Panda and Tiger Levels

[moserrya]

Spent some time in the Tiger on handling bad inputs. Would love your take on this!

Think I can complete Eagle without fully understanding everything going on, so I am going go spend more time digging in to ActiveRecord first.

Thanks!

[jwo]

You could probably pull this out into two different methods... maybe something like:

def retry_find_date(times)
  retry_attempts = 0
  begin
    retry_attempts += 1
    return find_date
  rescue InvalidDateError
    if retry_attempts < times
      puts "That is not a valid day of the week. Please try again."
      retry
    else
      puts "No valid date entered, giving shows for today (#{user_date})"
      return days_of_the_week[Time.new.wday]
    end
  end
end

class InvalidDateError < StandardError; end

def find_date
  Date::DAYNAMES.fetch(gets.chomp.capitalize) { raise InvalidDateError.new }
end

Then you'd use it in subjectively cleaner way:

date = retry_find_date(3)
Network.all###

[jwo]

Most excellent job on this. the comments I left were dealing with organizing code into smaller methods, not with your logic (which was sound). Let me know if you have questions.