次のコードをリファクタリングしますか？

Question

私は2つのモデルのブログとコメントを持っています。ブログには複数のコメントが含まれ、コメントは1つのブログにしか属しません。 ブログ/ 2 /コメント/ 1

のようなAPIを示すために

私はコメントコントローラ

def show 
    @blog = Blog.find(params[:blog_id]) 
    @comments = @blog.comments 
    @comment = @comments[params[:id].to_i - 1] 
    end 

の内側に次のコードを使用する必要があり、私はこのコードは、特に最後の、かなりakwardと感じone @comments[params[:id].to_i - 1]。パラメータIDを整数に変換してから、0ベースの配列インデックスに変更する必要があります。任意のリファクタリングがここで発生する可能性があります。

Answer 1

「Rails」をもっと見せてください。このような？

def show 
    @blog = Blog.find(params[:blog_id]) 
    @comment = @blog.comments.find params[:id] 
end 

OPのコメントによると、あなたの変換は必要ですです。ただし、コメント全体を読み込まずに、必要なものだけを読み込むことができます。

@comment = @blog.comments.offset(params[:id].to_i - 1).limit(1)