UPDATE: 続きを書きました

swanandp / actsaslist - Github

リスト操作といえば昔からActsAsListが定番ですが、「上に移動」「下に移動」のリクエストをどのようなURLで実行していますか？

#from_README_Example.rb
class TodoList < ActiveRecord::Base
  has_many :todo_items, order: "position"
end

class TodoItem < ActiveRecord::Base
  belongs_to :todo_list
  acts_as_list scope: :todo_list
end

todo_list.first.move_to_bottom
todo_list.last.move_higher

モデルの操作についてはREADMEにある通り、誰も迷わないと思います。

しかし、これらを実行するコントローラ、或いは、RESTメソッド的には、どう作るのが最適なのか。 コレが今日の問題です。

きっとこのサンプルなら、まずは以下の様なルーティングで問題ないでしょう。 “` ruby

routes.rb

resources :todolists, shallow: true do resources :todoitems end ”`

問題は、move_higherやmove_to_bottomを呼び出すコントローラのメソッドどのように割り当てるか、です。

私は以下のように書くのが正解かな、と考えています。 “` ruby

routes.rb

resources :todolists, shallow: true do resources :todoitems do member do put :bottom post :higher end end end ”`

さて、なぜか。少しだけ解説します。

PUT :bottom は、何度実行しても同じ結果となるべき

PUTはREST厨的に言えば「何度実行しても同じ結果になる操作」なわけです。（たしか）

よって、前述のmove_to_bottomを実行するコントローラのメソッドはPUTであるべきだと考えました。

なんでメソッド名はbottomだけ？

REST神曰く。

「メソッド名に動詞を含む事なかれ。動詞は既にPUTというカタチで表現されている。」

今回、todo_itemには順序しか無いのでこれで良しとしています。 ただし、このtodo_itemに「優先順位」などという項目があったときには、もう一捻りする必要がありますね。

bottomだけでは「順序を一番下に持っていく」のか「優先順位を最後にする」のかわからなくなりますから。

move_higher は、実行するたびに結果が変わる故、POST

本当ならリソースの更新なので、ここもPUTにしたいところです。残念なことに、１つずつ順序が変わるmove_higherですから、PUTの原則が適用できません。

なので、仕方がないのでPOSTみたいな感じです。

なんでメソッド名はhigherだけ？

REST神曰く(ry

所感

ぶっちゃけpost :move_to_bottom, post :move_higher というメソッドでも悪く無いとは思います。 原理主義者では無いので。しかし、まあ、規約大事ということで。

ここらへんはいろんな流派があって、賛否が別れるとは思いますが「ここって何か釈然としない」と常々思っておりまして、これで一応の解決を図ることができると思っております。(自分の中で)