Michelle - Edges - Hotel

.gitignore

@@ -1,7 +1,7 @@
 *.gem
 *.rbc
 /.config
-/coverage/
+coverage
 /InstalledFiles
 /pkg/
 /spec/reports/

Guardfile

@@ -1,5 +1,4 @@
-guard :minitest, bundler: false, rubygems: false do
-  # with Minitest::Spec
+guard :minitest, bundler: false, autorun: false, rubygems: false do  # with Minitest::Spec
   watch(%r{^spec/(.*)_spec\.rb$})
   watch(%r{^lib/(.+)\.rb$})         { |m| "spec/#{m[1]}_spec.rb" }
   watch(%r{^spec/spec_helper\.rb$}) { 'spec' }

lib/booking_system.rb

@@ -0,0 +1,104 @@
+require "date"
+
+require_relative 'calendar'
+require_relative 'reservation'
+require_relative 'room_block'
+
+module Hotel
+  class BookingSystem
+    attr_reader :rooms, :reservations, :room_blocks
+
+    def initialize()
+      @rooms = list_all_rooms() #<-- array of all room numbeers
+      @reservations = []
+      @room_blocks = []
+    end
+
+    def list_all_rooms()
+      return (1..20).to_a
+    end
+
+    def construct_cal_checker(check_in:, check_out:)
+      return Calendar.new(check_in: check_in, check_out: check_out)
+    end
+
+    def generate_res_id()
+      if @reservations.empty?
+        return 1
+      else
+        return @reservations.max_by { |reservation| reservation.id}.id + 1
+      end
+    end
+
+    def generate_block_id()
+      if @room_blocks.empty?
+        return 1
+      else
+        return @room_blocks.max_by { |block| block.id}.id + 1
+      end
+    end
+
+    def list_res_for_date(check_date)
+      matching_res = @reservations.select { |reservation| reservation.has_date?(check_date) }
+
+      return matching_res.empty? ? nil : matching_res
+    end
+
+    def list_avail_rooms_for_range(check_in:, check_out:)
+      date_range = construct_cal_checker(check_in: check_in, check_out: check_out)
+
+        booked_rooms = @reservations.select { |reservation| reservation.overlap?(date_range) }.map { |reservation| reservation.room_num }
+
+        avail_rooms = @rooms - booked_rooms
+
+        held_rooms = @room_blocks.select { |block| block.overlap?(date_range) }.map {|block| block.rooms}
+
+        if !held_rooms.empty?
+          held_rooms = held_rooms[0] # list within a list
+        end
+
+        avail_rooms -= held_rooms
+
+      return avail_rooms.empty? ? nil : avail_rooms
+    end
+
+    def create_reservation(check_in:, check_out:)
+      id = generate_res_id() #<-- create new reservation id
+      avail_rooms = list_avail_rooms_for_range(check_in: check_in, check_out: check_out) #<-- grab first available room
+
+      if avail_rooms.nil?
+        raise StandardError, "No rooms are available for the given date range: #{check_in} - #{check_out}."
+      else
+        avail_room = avail_rooms[0]
+      end
+
+      new_reservation = Hotel::Reservation.new(
+        id: id,
+        room_num: avail_room,
+        check_in: check_in,
+        check_out: check_out)
+
+      @reservations << new_reservation
+
+      return new_reservation
+    end
+
+    def create_room_block(check_in:, check_out:, block_size:)
+      avail_rooms = list_avail_rooms_for_range(check_in: check_in, check_out: check_out)
+
+      if avail_rooms.nil? || avail_rooms.length < block_size
+        raise StandardError, "Not enough rooms are available for the given date range: #{check_in} - #{check_out}."
+      else
+        avail_room = avail_rooms[0]
+      end
+
+      hold_rooms = avail_rooms[0..block_size-1]
+      id = generate_block_id()
+
+      new_room_block = Hotel::RoomBlock.new(id: id, check_in: check_in, check_out: check_out, rooms: hold_rooms)
+
+      @room_blocks << new_room_block
+      return new_room_block
+    end
+  end
+end

lib/calendar.rb

@@ -0,0 +1,38 @@
+require 'date'
+
+module Hotel
+  class Calendar
+    attr_reader :check_in, :check_out
+
+    def initialize(check_in:, check_out:)
+
+      unless /\d{4}-\d{1,2}-\d{1,2}/.match(check_in) && /\d{4}-\d{1,2}-\d{1,2}/.match(check_out)
+        raise StandardError, "Improper date format: date must be entered as YYYY-MM-DD."
+      end
+
+      @check_in = Date.parse(check_in)
+      @check_out = Date.parse(check_out)
+
+      unless @check_in < @check_out
+        raise StandardError, "Invalid date range: end date must occur after start date."
+      end
+    end
+
+  def has_date?(other_date)
+    other_date = Date.parse(other_date)
+    if (other_date >= @check_in) && (other_date <= @check_out - 1)
+      return true
+    else
+      return false
+    end
+  end
+
+  def overlap?(other_dates) # param: Calendar obj
+    if !(other_dates.check_in <= @check_out-1) || !(other_dates.check_out-1 >= @check_in)
+      return false
+    else
+      return true
+    end
+  end
+  end
+end

lib/reservation.rb

@@ -0,0 +1,21 @@
+require_relative 'calendar'
+
+module Hotel
+  class Reservation < Calendar
+    attr_reader :id, :room_num, :daily_rate
+
+    def initialize(check_in:, check_out:, id:, room_num:, daily_rate: 200)
+      super(check_in: check_in, check_out: check_out)
+
+      @id = id.to_i
+      @room_num = room_num.to_i
+      @daily_rate = daily_rate.to_f
+
+    end
+
+    def total_stay_cost()
+      length_in_days = @check_out - @check_in
+      return @daily_rate * length_in_days
+    end
+  end
+end

lib/room_block.rb

@@ -0,0 +1,45 @@
+require_relative 'reservation'
+
+module Hotel
+  class RoomBlock < Reservation
+    attr_reader :block_size, :discount, :rooms, :status
+
+    def initialize(id:, daily_rate: 200, check_in:, check_out:, discount:0, rooms:[], status: :available)
+      super(id: id, daily_rate: daily_rate, check_in: check_in, check_out: check_out, room_num: room_num)
+
+      unless !rooms.empty?
+        raise StandardError, "Room blocks cannot be empty!"
+      end
+
+      @discount = discount/100.to_f
+      @rooms = rooms # array of ints
+      @status = status
+      @block_size = rooms.length
+
+      unless @block_size <= 5 && @block_size > 1
+        raise StandardError, "Room blocks must hold at least two rooms and at most five. You entered #{block_size} rooms."
+      end
+    end
+
+    def show_status()
+      rooms_hash = {}
+      @rooms.each do |room|
+        rooms_hash[room.to_s] = @status
+      end
+      return rooms_hash
+    end
+
+    def discounted_rate()
+      return @daily_rate * (1-@discount)
+    end
+
+    def total_stay_cost_room()
+      length_in_days = @check_out - @check_in
+      return discounted_rate() * length_in_days
+    end
+
+    def total_stay_cost_block()
+      return total_stay_cost_room() * @block_size
+    end
+  end
+end

refactors.txt

@@ -0,0 +1,21 @@
+Calendar
+- add separate eror handling method or class
+- spec: add more edge cases: multi month stay?
+- in #overlap? maybe we don't need the object (so remove BookingSystem#construct_cal_checker and change date_range checks back to check_in, check_out
+
+Reservation
+- add hundredths place rounding for money
+- make @daily_rate a constant?
+
+RoomBlock
+- maybe RoomBlock shouldn't inherit from Reservation b/c I don't use the instance var @room_num in RoomBlock -- but got errors when I didn't include it in super() in initialize()
+- like Calendar, this class also has a lot of error handling going on --> split into methods or a class?
+- add more functionality (didn't finish wave 3), like maybe check whether a given room number exists within the block?
+
+BookingSystem
+- spec: #initialize: maybe here see if new reservations and room blocks can be added to @reservations and @room_blocks??? Not sure if this is the right place to test that.
+- spec: check in #initialize that list_all_rooms() worked
+- spec: more testing for #create_reservation?: reject same start-day overlaps, check to see if res are added to @reservations (here?)
+
+Overall
+- Finish Wave 3