전체적인 예외처리 추가

Author: max312444

app/models/reservation.rb

@@ -8,7 +8,6 @@ class Reservation < ApplicationRecord
   # ========================================
   before_validation :set_defaults
   before_validation :generate_code, on: :create
-  before_validation :check_and_handle_reservation_conflicts, on: [:create, :update]
 
   # ========================================
   # Validations
@@ -19,7 +18,7 @@ class Reservation < ApplicationRecord
   validates :priority, presence: true
   validates :start_time, presence: true
   validates :end_time, presence: true
-  validate :start_time_before_end_time
+  validate  :start_time_before_end_time
 
   # ========================================
   # Default scope (Soft delete)
@@ -29,7 +28,7 @@ class Reservation < ApplicationRecord
   # ========================================
   # Soft delete method
   # ========================================
-  def soft_delete
+  def soft_delete(deleted_by: nil)
     update(deleted_at: Time.current)
   end
 
@@ -40,7 +39,7 @@ def self.with_deleted
   private
 
   # ========================================
-  # Default values (NEW)
+  # Default values
   # ========================================
   def set_defaults
     self.priority ||= 0
@@ -54,70 +53,13 @@ def generate_code
   end
 
   # ========================================
-  # Custom validation methods
+  # Time validation
   # ========================================
   def start_time_before_end_time
     return unless start_time && end_time
 
     if start_time >= end_time
-      errors.add(:start_time, "시작 시간은 종료 시간보다 빨라야 합니다.")
+      errors.add(:start_time, "must be earlier than end_time")
     end
   end
-
-  def check_and_handle_reservation_conflicts
-    # 운영 시간 확인
-    unless is_available_during_operating_hours?
-      errors.add(:base, "요청하신 시간은 스터디룸 운영 시간 범위에 포함되지 않거나 휴일입니다.")
-      throw :abort
-    end
-
-    # 중복 예약 확인
-    conflicting_reservations = Reservation.where(room_id: room_id)
-                                          .where.not(id: id)
-                                          .where("start_time < ? AND end_time > ?", end_time, start_time)
-
-    conflicting_reservations.each do |existing_reservation|
-      if priority > existing_reservation.priority
-        existing_reservation.soft_delete
-      else
-        errors.add(:base, "요청하신 시간에 이미 예약이 존재하며, 우선순위가 낮거나 같아 예약할 수 없습니다.")
-        throw :abort
-      end
-    end
-  end
-
-  def is_available_during_operating_hours?
-    reservation_date = start_time.to_date
-    start_time_str = start_time.strftime('%H:%M')
-    end_time_str = end_time.strftime('%H:%M')
-
-    # RoomException 우선 확인
-    room_exception = room.room_exceptions.find_by(holiday_date: reservation_date)
-
-    if room_exception.present?
-      # Case 1: 완전 휴무
-      if room_exception.opening_time.nil? && room_exception.closing_time.nil?
-        return false
-      end
-
-      # Case 2: 특별 운영 시간
-      if room_exception.opening_time.present? && room_exception.closing_time.present?
-        exception_opening_time_str = room_exception.opening_time.strftime('%H:%M')
-        exception_closing_time_str = room_exception.closing_time.strftime('%H:%M')
-        return start_time_str >= exception_opening_time_str && end_time_str <= exception_closing_time_str
-      end
-
-      # Case 3: 이상 데이터
-      return false
-    end
-
-    # 일반 운영 시간 확인
-    operating_hour = room.room_operating_hours.find_by(day_of_week: start_time.wday)
-    return false if operating_hour.nil?
-
-    opening_time_str = operating_hour.opening_time.strftime('%H:%M')
-    closing_time_str = operating_hour.closing_time.strftime('%H:%M')
-
-    start_time_str >= opening_time_str && end_time_str <= closing_time_str
-  end
 end

app/models/room.rb

@@ -1,29 +1,53 @@
 class Room < ApplicationRecord
-  # 스터디룸 상태 정의: 공실(vacant) 또는 입실(occupied)
-  # ERD의 'status' ENUM('공실', '입실')에 해당하며, Rails에서는 integer 타입으로 저장됩니다.
-
-  # 이게 postman 테스트할 때 계속 문제라서 일단 주석처리함
-  # 나중에 지문인식이나 얼굴인식으로 입실확인 하는거 넣으면 같이 추가하기로
-  # enum status: { vacant: 0, occupied: 1 } 
-
-  # 다른 모델과의 연관 관계 정의
-  # RoomOperatingHour 모델과 일대다 관계를 가집니다.
-  has_many :room_operating_hours
-  # RoomException 모델과 일대다 관계를 가집니다。
-  has_many :room_exceptions
-  # Reservation 모델과 일대다 관계를 가집니다.
+  # 연관 관계
+  has_many :room_operating_hours, dependent: :destroy
+  has_many :room_exceptions, dependent: :destroy
   has_many :reservations
 
-  # 유효성 검사 (Validations)
-  # name 컬럼은 필수이며, 최대 100자까지 허용합니다.
+  # -------------------------------
+  # 필수값 검증
+  # -------------------------------
   validates :name, presence: true, length: { maximum: 100 }
-  # minimum_member 컬럼은 필수이며, 0 이상의 정수만 허용합니다.
-  validates :maximum_member, presence: true, numericality: { only_integer: true, greater_than_or_equal_to: 0 }
-  # status 컬럼은 필수입니다.
-  # validates :status, presence: true
+  validates :maximum_member, presence: true,
+                             numericality: { only_integer: true, greater_than_or_equal_to: 1 }
+
+  # room_type이 enum이라면 자동 처리 
+  # enum room_type: { study_room: 0, lecture_room: 1, lab: 2 }, _prefix: :type
+  #
+  # enum 타입이라면 다음 validation은 안 넣어도 되는데
+  # enum이 아니라면 아래처럼 강제해야 함:
+  # validates :room_type, inclusion: { in: ["study", "lecture", "lab"] }
+
+  # -------------------------------
+  # 이름 중복 검증
+  # 1) department_id가 같은 방끼리는 같은 name 금지
+  # 2) department_id가 nil(공용 방)일 경우, 전체에서 name 중복 금지
+  # -------------------------------
+  validate :unique_name_within_department
+
+  def unique_name_within_department
+    if department_id.present?
+      # 같은 학과 내 중복 name 금지
+      if Room.where(department_id: department_id, name: name).where.not(id: id).exists?
+        errors.add(:name, "해당 학과에 동일한 이름의 방이 이미 존재합니다.")
+      end
+    else
+      # 공용 방은 전체에서 name 중복 금지
+      if Room.where(department_id: nil, name: name).where.not(id: id).exists?
+        errors.add(:name, "공용 방 이름은 전체에서 중복될 수 없습니다.")
+      end
+    end
+  end
+
+  # -------------------------------
+  # 삭제된 방 수정 금지
+  # -------------------------------
+  before_update :prevent_update_if_deleted
 
-  # department_id는 외부 서비스(유저 서비스)를 참조하므로,
-  # 여기서는 데이터베이스 레벨의 외래 키 제약 조건은 추가하지 않습니다.
-  # 유효성 검사는 애플리케이션 로직에서 처리하거나, 필요에 따라 추가할 수 있습니다.
-  # validates :department_id, presence: true
+  def prevent_update_if_deleted
+    if deleted_at.present?
+      errors.add(:base, "삭제된 방은 수정할 수 없습니다.")
+      throw :abort
+    end
+  end
 end

app/models/room_exception.rb

@@ -5,14 +5,17 @@ class RoomException < ApplicationRecord
   # Callbacks
   after_save :cancel_conflicting_reservations
 
-  # 유효성 검사 (Validations)
-  # room_id, holiday_date, created_by는 필수입니다.
+  # 기본 유효성 검사
   validates :room_id, presence: true
   validates :holiday_date, presence: true
   validates :created_by, presence: true
 
+  # 추가 유효성 검사
+  validate :validate_time_rule
+  validate :validate_time_order
+  validate :validate_duplicate_date
+
   # 소프트 삭제 (Soft Delete)
-  # deleted_at 컬럼에 값이 있으면 삭제된 것으로 간주합니다.
   default_scope { where(deleted_at: nil) }
 
   def soft_delete
@@ -25,22 +28,62 @@ def self.with_deleted
 
   private
 
-  # RoomException 변경 시 충돌되는 예약을 자동으로 취소하는 로직
+  # ----------------------------------------
+  # 1) 전체 휴일 vs 부분 휴일 규칙
+  # ----------------------------------------
+  def validate_time_rule
+    # 전체 휴일 (both nil) → OK
+    return if opening_time.nil? && closing_time.nil?
+
+    # 부분 휴일 → 둘 다 있어야 함
+    if opening_time.present? ^ closing_time.present?
+      errors.add(:base, "Both opening_time and closing_time must be present for partial holiday")
+    end
+  end
+
+  # ----------------------------------------
+  # 2) 시간 순서 검증
+  # ----------------------------------------
+  def validate_time_order
+    return if opening_time.blank? || closing_time.blank?
+
+    if opening_time >= closing_time
+      errors.add(:opening_time, "must be earlier than closing_time")
+    end
+  end
+
+  # ----------------------------------------
+  # 3) 동일 날짜 중복 방지 (soft-delete 제외)
+  # ----------------------------------------
+  def validate_duplicate_date
+    return if room_id.blank? || holiday_date.blank?
+
+    duplicate = RoomException.with_deleted
+                             .where(room_id: room_id, holiday_date: holiday_date, deleted_at: nil)
+                             .where.not(id: id)
+                             .exists?
+
+    if duplicate
+      errors.add(:holiday_date, "exception already exists for this date")
+    end
+  end
+
+  # ----------------------------------------
+  # 기존 예약 자동 취소 로직 (그대로 유지)
+  # ----------------------------------------
   def cancel_conflicting_reservations
-    # 변경된 예외 날짜에 해당하는 모든 예약을 조회합니다.
     reservations_on_date = Reservation.where(
       room_id: room_id,
       start_time: holiday_date.all_day
     )
 
-    # Case 1: 완전 휴무일 (운영 시간이 모두 nil)
+    # Case 1: 완전 휴무일
     if opening_time.nil? && closing_time.nil?
-      # 해당 날짜의 모든 예약을 시스템(deleted_by: 0)에 의해 소프트 삭제합니다.
       reservations_on_date.find_each { |reservation| reservation.soft_delete(deleted_by: 0) }
       return
     end
 
-    # Case 2: 특별 운영 시간 지정
+    # Case 2: 부분 휴무
     if opening_time.present? && closing_time.present?
       exception_opening_time_str = opening_time.strftime('%H:%M')
       exception_closing_time_str = closing_time.strftime('%H:%M')
@@ -49,11 +92,11 @@ def cancel_conflicting_reservations
         start_time_str = reservation.start_time.strftime('%H:%M')
         end_time_str = reservation.end_time.strftime('%H:%M')
 
-        # 예약이 새로운 특별 운영 시간의 범위 밖에 있는 경우, 소프트 삭제합니다.
-        unless start_time_str >= exception_opening_time_str && end_time_str <= exception_closing_time_str
+        unless start_time_str >= exception_opening_time_str &&
+               end_time_str <= exception_closing_time_str
           reservation.soft_delete(deleted_by: 0)
         end
       end
     end
   end
-end
+end

app/models/room_operating_hour.rb

@@ -1,16 +1,18 @@
 class RoomOperatingHour < ApplicationRecord
-  # Room 모델과 다대일 관계를 가집니다.
+  # Room 모델과 다대일 관계
   belongs_to :room
 
-  # 유효성 검사 (Validations)
-  # room_id, day_of_week, opening_time, closing_time는 필수입니다.
+  # 기본 유효성 검사
   validates :room_id, presence: true
-  validates :day_of_week, presence: true, numericality: { only_integer: true, in: 0..6 } # 0(일)부터 6(토)까지
+  validates :day_of_week, presence: true, numericality: { only_integer: true, in: 0..6 }
   validates :opening_time, presence: true
   validates :closing_time, presence: true
 
-  # 소프트 삭제 (Soft Delete)
-  # deleted_at 컬럼에 값이 있으면 삭제된 것으로 간주합니다.
+  # 추가 유효성 검사 (예외 처리)
+  validate :validate_time_order
+  validate :validate_day_of_week_duplication
+
+  # 소프트 삭제
   default_scope { where(deleted_at: nil) }
 
   def soft_delete
@@ -20,4 +22,33 @@ def soft_delete
   def self.with_deleted
     unscope(where: :deleted_at)
   end
-end
+
+  private
+
+  # ----------------------------------------
+  # 1. opening_time < closing_time 검증
+  # ----------------------------------------
+  def validate_time_order
+    return if opening_time.blank? || closing_time.blank?
+
+    if opening_time >= closing_time
+      errors.add(:opening_time, "must be earlier than closing_time")
+    end
+  end
+
+  # ----------------------------------------
+  # 2. 동일 room_id + day_of_week 중복 금지
+  # ----------------------------------------
+  def validate_day_of_week_duplication
+    return if room_id.blank? || day_of_week.blank?
+
+    duplicate = RoomOperatingHour.with_deleted
+                                 .where(room_id: room_id, day_of_week: day_of_week, deleted_at: nil)
+                                 .where.not(id: id)
+                                 .exists?
+
+    if duplicate
+      errors.add(:day_of_week, "already exists for this room")
+    end
+  end
+end