[ここらへん](https://github.com/RyuPiT/SharePla/blob/32c7445fd468d50788cb7370b7e9828da2425772/app/controllers/documents_controller.rb#L15-L24)のpdf作る責務は`Plan`モデルに移すとよいのでは〜。(`Plan#to_pdf_file`とかそんな感じ?)

そうすると、Controllerの役割がより明確になって、他の箇所でもpdf作る処理を使いまわしたり、pdf作る具体的な実装方法を変更するのも楽になるかな〜と思います。

``` ruby
@plan = Plan.find(params['id'])
pdf_file = @plan.to_pdf_file
send_data File.read(pdf_file.path), filename: @plan.pdf_name, disposition: 'inline'
```