クラスインスタンスメソッド内で定義した変数を、同じコントローラー内の、別のクラスインスタンスメソッドで利用したい

Question

##開発環境
Rails(5.2)
Ruby(2.4)
MySQL

##ファイルアップロードにおいて利用した技術
Active Storage

##背景
Railsで3つのファイルアップロードをさせ、そのファイルをPythonプログラムにコマンドライン引数として渡して、シェル経由で実行させる。この結果をWebに公開するということを目的にやっています。

##疑問点
コントローラーのcreateアクションで取得した各ファイルのフルパスをshowアクションで利用したいと考えています。
その方法についてご教授いただきたいです。

##ソースコード

ContextsController

class ContextsController < ApplicationController
  def new
    @context = Context.new
  end

  def create
    @context = Context.create context_params
    @path_info = []
    tempfile_1 = params[:context][:files][0].path
    tempfile_2 = params[:context][:files][1].path
    tempfile_3 = params[:context][:files][2].path
    @path_info.push(tempfile_1)
    @path_info.push(tempfile_2)
    @path_info.push(tempfile_3)

    redirect_to context_path(@context.id)
  end



  def show
    @context = Context.find(params[:id])
    @file_result = ` python3 /Users/~省略~/test.py "#{ファイルのフルパス }" "#{ファイルのフルパス }" "#{ファイルのフルパス }" `
  end


  private
    def context_params
      params.require(:context).permit(files: [])
    end

    def params_info
      params[:context].permit!
    end

end

##自分で考え、調べているが、答えが見つからないこと
・アクションの遷移に伴って、paramsには情報を含めずに、別の形でクラスインスタンスメソッド間でのやりとりをする方法を探す。
→最初は、paramsに連結させて飛ばそうとしていましたが、連結させるとルーティングエラー が出てしまったので、この方法を考えた。
・クラスインスタンスメソッド(createアクション)が呼ばれた場合、欲しい値を取得するためのメソッドを新たに定義し、クラスインスタンスメソッド(showアクション)でこのメソッドを呼び出して、欲しい情報を取得する。
→「createメソッドが呼ばれたら」を条件に、if文を作成し、そこでparamsに含まれる情報から欲しい情報を取得する。

Answer 1

別のアクションでも使用したい値があるのであればモデルにその値用のカラムを追加して記録させれば良いだけなのですが、この場合は単にファイルパスを記録させてもうまくいきません。なぜなら、フォームでアップロードされたファイルはcreateアクションが終わると削除されてしまうからです。フォーム等の入力によってparams経由で渡されるものは全て一時的なものであり、そのアクションが終われば消えて無くなくなってしまいます。だからこそ、create等ではその値をモデルに設定し、DBに保存しておかなくてはならないと言うことです。

では、どうしたら良いかというと方法はだいたい二つです。

1. Active Storageを使う

Active Storageはファイルをモデルに関連付けて永続的に保存する仕組みを提供します。ファイルの中身全てをモデルに入れる(DB上に保存する)方法もありますが、大きなファイルの保存にDBはあまり適していません。ファイルとして扱う場合はActive Storageの方が良いでしょう。

使い方は Active Storage の概要 - Rails ガイド を参考にしてください。

2. pythonの実行結果をモデルに記録する

そもそも、pythonでの実行結果が常に同じになるのであれば、create内でpythonを実行し、その結果をモデルに記録しておいた方が効率的です。show内で実行するとshowが呼び出される度に実行する事になり、無駄が多くなります。

Comments

[退会済みユーザー]

ご回答ありがとうございます。

>1. Active Storageを使う
→Active Storageはすでに使用しています。今回の場合であれば、Activestorage::blobに入っている情報から、ファイルの絶対パス情報を、最も直近でアップロードされたファイルの上位３件を取得するというやり方が良いのでしょうか？

>2. pythonの実行結果をモデルに記録する
→こちらの方が無難かもしれません。pythonプログラムの結果は、createアクション内で実行しても、showアクション内で実行しても結果に変化はないです。
こちらの方針の場合、モデルに記録するというのは、「新しくモデルを作成し、その中にスキーマを作成して、pythonコマンドの実行結果を記録して、Viewで呼び出す」という認識で良いでしょうか？？

[raccy]

Active Storageを使う場合はURL経由でファイルを処理できるようにPythonを書き換えた方がいいかも知れません。ファイルパス取得の方法ではS3等を使う場合は使用できないできないと思われます。ディスクの場合はファイルパスを取得で可能と思います。

ただ、今回は実行しても結果変わらないなら、モデルにカラムを追加した方が良いでしょう。テキスト一つだけであれば、別モデルまで作るのはすこし大げさのような気がします。