Merge pull request #70 from WWC-Hackathon-2023/5refactor

BE | Refactor `PuzzlesController#index` Endpoint
WWC-Hackathon-2023 · Nov 3, 2023 · 0df9a6b · 0df9a6b
2 parents b385b8c + 126d9ea
commit 0df9a6b
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/app/controllers/api/v1/puzzles_controller.rb b/app/controllers/api/v1/puzzles_controller.rb
@@ -1,16 +1,16 @@
 class Api::V1::PuzzlesController < ApplicationController
   skip_before_action :set_current_user, only: [:index]
+  before_action :find_puzzles, only: [:index]
 
   def index
-    find_puzzles(params[:zip_code])
     raise NoPuzzlesFoundException if @puzzles_in_zip_code.empty?
     render json: PuzzleSerializer.new(@puzzles_in_zip_code)
   end
 
   private
 
-  def find_puzzles(zip_code)
-    @puzzles_in_zip_code = Puzzle.find_by_zip_code(zip_code)
+  def find_puzzles
+    @puzzles_in_zip_code = Puzzle.find_by_zip_code(params[:zip_code])
   end
 end