[Ruby On Rails]ZIPファイルを一時ファイルに解凍・保存して、順番にカラムに入れたい

Question

コントローラでzipファイルを取得したあと、tempfileに一時的に解凍・保存してproduct_images1 ~ product_images10カラムに自動的に振り分けてレコードを作成する、という機能を作りたいと思っています。

# products_controller.rb
def create
    @product = Product.new(product_params)

    zipfile = product_params[:zip]
    Zip::File.open(zipfile.path) do |zip|
      num = 0
      zip.each do |entry|
        num = num + 1
        return if num > 30
        ext = File.extname(entry.name) #拡張子を取得
        # 隠しファイルやゴミファイルは飛ばす
        next if ext.blank? || File.basename(entry.name).count(".") > 1
        # 一時ファイルを作る
        Tempfile.open(["tempProductImageFile", ext]) do |t|
          entry.extract(t.path) { true }
　　　　　　# カラムに値を入れる
          @product.send("products_image#{num}=", t)
        end
        p "ccc"
      end
      p "ddd"
    end

    p "jjj"
    respond_to do |format|
        if @product.save
          p "kkk"
          format.html { redirect_to [:manage, @product], notice: "#{@product.building_name}を作成しました。" }
          format.json { render :show, status: :created, location: @product }
        else
          format.html { render :new }
          format.json { render json: @product.errors, status: :unprocessable_entity }
        end
      end
    end
  end

##結果

上記のようなコードを書いてこのアクションを起こしたところ、
Missing template manage/products/create, manage/application/create
というエラーが出ました。

ご教示いただけることがあれば、何卒よろしくお願いいたします。

Comments

[mather]

> 以下のように実装すると、zipファイル内のファイルが一つ一つイテレータで回されているのがわかります。
とありますが、本当ですか？ `Tempfile.open` は一時ファイルを新しく作成して開くだけで、中身は空っぽのはずです。再検証してみてください。
#{num} の使い方は「式展開」で調べてみてください。

[s_diff]

ご回答有難うございます。
Tempfile.open(["tempProductImageFile", ext]).each do |t|
　　p "aaaaaaaaaaaaaaaaaaaaaaaaaaaa"
　　p t
　　p "aaaaaaaaaaaaaaaaaaaaaaaaaaaa"
entry.extract(t.path) { true }
@product.send("products_image#{num}=", t)
end

のようにしてサーバーのログを見ると、別のtempfile名になっているということが言いたかったのですが、うまく伝えられず申し訳ありません。修正させていただきます。

式展開の方法はわかるのですが、もしそれぞれのtempfileの順番がわかるメソッドがあればそれをnumに入れれば良いと思いました。

Answer 1

Missing template manage/products/create, manage/application/create

まず、エラーメッセージにある単語で検索してみましょう。「rails Missing template」と検索して出てくる以下のページが参考になりませんか？

https://qiita.com/kazutosato/items/ffceb94b136f69563beb

それぞれのtempfileの順番がわかるメソッドがあればそれをnumに入れれば良いと思いました。

これはどういう意味でしょうか？質問者の意図がわからないです。
tempfileは毎回新規で作るはずなので、順番はないはずです。
zipファイルのエントリー順ということでしょうか。基本的にエントリー順は気にするべきではないと思います。
もし、一意な順番で取得したいなら

zip.entries.sort_by { |entry| entry.name }

など欲しい順序でソートするといいと思います。

Comments

[s_diff]

有難うございます。いろいろと大きな勘違いをしていました。
問題は全く別のところにあったようです。
お騒がせして申し訳ありません ^^;