hanachin · March 18, 2016 15:15 · Mar 18, 2016 · Mar 18, 2016 · Mar 18, 2016 · Mar 18, 2016
diff --git a/post_actions_controller.rb b/post_actions_controller.rb
@@ -1,5 +1,5 @@
 # app/controllers/post_actions_controller.rb
-class PostsActionsController < ApplicationController
+class PostActionsController < ApplicationController
   # POST /posts/:id/publish.json
   def publish
     if current_post.update(state: :publish, published_at: Time.zone.now)

diff --git a/actions_controller.rb → post_actions_controller.rb b/actions_controller.rb → post_actions_controller.rb
@@ -1,4 +1,4 @@
-# app/controllers/posts_actions_controller.rb
+# app/controllers/post_actions_controller.rb
 class PostsActionsController < ApplicationController
   # POST /posts/:id/publish.json
   def publish

diff --git a/routes.rb b/routes.rb
@@ -1,6 +1,6 @@
 Rails.application.routes.draw do
   resources :posts do
-    scope controller: :posts_actions do
+    scope controller: :post_actions do
       member do
         post :publish
         post :hide

diff --git a/action.rb b/action.rb
@@ -1,18 +0,0 @@
-# app/models/posts/action.rb
-class Action
-  include ActiveModel::Model
-
-  attr_reader :post_id, :name
-
-  def save
-    return false if invalid?
-
-    @post = Post.find(post_id)
-    if name == 'publish'
-      @post.update state: :publish, published_at: Time.zone.now
-    elsif name == 'xxx'
-      # do something
-    end
-    # 適当にif分岐にしてるけど、本当はもう少しクラス設計を直して、if文がなくなるようにした方が良い
-  end
-end

diff --git a/actions_controller.rb b/actions_controller.rb
@@ -1,17 +1,26 @@
-# app/controllers/posts/actions_controller.rb
-module Posts
-  class ActionsController < ApplicationController
-    # POST /posts/:post_id/actions.json
-    def create
-      @action = Posts::Action.new(post_id: params[:post_id], name: params[:name])
-
-      respond_to do |format|
-        if @action.save
-          format.json { render json: @action, status: :created }
-        else
-          format.json { render json: @action.errors, status: :unprocessable_entity }
-        else
-      end
+# app/controllers/posts_actions_controller.rb
+class PostsActionsController < ApplicationController
+  # POST /posts/:id/publish.json
+  def publish
+    if current_post.update(state: :publish, published_at: Time.zone.now)
+      〜
+    else
+      〜
     end
   end
+
+  # POST /posts/:id/hide.json
+  def hide
+    if current_post.update(state: :hide, published_at: nil)
+      〜
+    else
+      〜
+    end
+  end
+
+  private
+
+  def current_post
+    Post.find(params[:id])
+  end
 end
diff --git a/routes.rb b/routes.rb
@@ -0,0 +1,10 @@
+Rails.application.routes.draw do
+  resources :posts do
+    scope controller: :posts_actions do
+      member do
+        post :publish
+        post :hide
+      end
+    end
+  end
+end
diff --git a/action.rb b/action.rb
@@ -8,6 +8,11 @@ def save
     return false if invalid?
 
     @post = Post.find(post_id)
-    # name の内容で処理を分岐したり、コールバックを書いたり。
+    if name == 'publish'
+      @post.update state: :publish, published_at: Time.zone.now
+    elsif name == 'xxx'
+      # do something
+    end
+    # 適当にif分岐にしてるけど、本当はもう少しクラス設計を直して、if文がなくなるようにした方が良い
   end
 end
diff --git a/action.rb b/action.rb
@@ -0,0 +1,13 @@
+# app/models/posts/action.rb
+class Action
+  include ActiveModel::Model
+
+  attr_reader :post_id, :name
+
+  def save
+    return false if invalid?
+
+    @post = Post.find(post_id)
+    # name の内容で処理を分岐したり、コールバックを書いたり。
+  end
+end
diff --git a/actions_controller.rb b/actions_controller.rb
@@ -0,0 +1,17 @@
+# app/controllers/posts/actions_controller.rb
+module Posts
+  class ActionsController < ApplicationController
+    # POST /posts/:post_id/actions.json
+    def create
+      @action = Posts::Action.new(post_id: params[:post_id], name: params[:name])
+
+      respond_to do |format|
+        if @action.save
+          format.json { render json: @action, status: :created }
+        else
+          format.json { render json: @action.errors, status: :unprocessable_entity }
+        else
+      end
+    end
+  end
+end
No results found