カラムに入っている[]を外す方法を教えてください

Question

前提・実現したいこと

Rails初心者です。
複数画像選択の実装をしていて、保存には成功したのですが、カラムに[""]が入ってしまいます。
createメソッドが実行する前にこのカッコを外したいのですが、やり方がわからず質問させていただきました。

どなたかわかる方がいらっしゃいましたら、ご回答いただけると非常に助かります。

発生している問題・エラーメッセージ

複数画像選択で、カラム保存の時["couple-260899_1920.jpg"]などとはいってしまうので余計なものを外したい。

products_controller

省略

def new
    @product = Product.new
    @product.product_images.build
  end

  def create
    @product = current_user.products.build(product_params)
    unless @product.valid?
      render "/products/new"
    else
      @product.save
      redirect_to root_path
    end
  end

省略

def product_params
    params.require(:product).permit(
      :name,
      :description,
      :price,
      :quality,
      :delivery_origin,
      :delivery_status,
      :delivery_cost,
      :prep_days,
      :category_id,
      product_images_attributes: [:id, :image_url[]]
    )
end

products/new.html.haml

= form_for @product do |f|
  .sell-product__container__form
    .preview-content#sell-content
      .preview-content__main
        .preview-content__main__box
          %label
            商品画像
            %span.require 必須
          %p
            最大3枚までアップロードできます(ドラッグ&ドロップまたはクリックしてファイルをアップロード)
          .preview-content__main__box__container
            .preview-content__main__box__dropbox
              %pre
              %ul
                - 3.times do
                  %li
                    .view_box
                      = f.fields_for :product_images do |i|
                        = i.file_field :image_url, class: 'file',multiple: true

model/products.rb

省略

 has_many :product_images, dependent: :destroy
  accepts_nested_attributes_for :product_images, allow_destroy: true

省略

model/product_images.rb

class ProductImage < ApplicationRecord
  # Association
  belongs_to :product

  mount_uploaders :image_url, ImageUploader
end

補足情報（FW/ツールのバージョンなど）

Ruby version 2.5.1
Rails version 5.2.3

Answer 1

文字列の配列なので["str"]となっているだけのように見えますが、
本当に[" "]を取り除いてもいいのですか？
例では要素が1つなのでいいのかもしれませんが、複数の場合の動作はどうなるでしょうか？
もっと別のアプローチが必要なのでは？例えば以下のような...

attribute :image_url, :string, array: true

@product = current_user.products.build(product_params) do |product|
  product.product_images.build(*image_params)
end

def product_params
  params.require(:product).permit(%i[name description price quality delivery_origin delivery_status delivery_cost prep_days category_id])
end

def image_params
  params.require(:product).permit(product_images_attributes: [:id, :image_url[]])
end

雰囲気で書いたんですけどこんな感じですかね...？

Comments

[rui-watanabe]

ご回答ありがとうございます。
ご指摘通り、モデルに対して、attrubute :image_url, :string, array: trueの記述をしました。
しかし今度はコントローラのnewアクションのところの @product.product_images.build　に、unknown keyword: arrayと出て、配列型ではないというエラーが出ます。
理解不足で大変申し訳ないのですが、ここでの配列型の指定についてご教授いただけると非常に助かります。

[rhiroe]

typo してました。
attrubute じゃなくて attribute です。

[rhiroe]

あと書いてなかったですが、array: true を使えるのはDBがPostgreSQLの時だけだったかもしれません。

[rui-watanabe]

あ、attributeで指定して以上のようなエラーが出てしまいました。
https://gyazo.com/2122a633193256eb3a54a776b7da352b

[rui-watanabe]

そうなのですね、DBはMySqlでやってます。
もし他の指定でお分かりになることがあったら、ご教授いただけると非常に助かります。

[rhiroe]

であれば取り出すときに整形するとかですかね...。
お勧めではないですが eval('["str"]') とかやると ["str"] の配列が返ってきたりします。
どちらにせよ複数の画像選択ということであれば image_url は配列で複数あると思うので
テーブルを分けて子レコードとして複数持つか、
取り出すときに整形するかのどちらかのアプローチがいいと思います。

[rui-watanabe]

迅速なご回答誠にありがとうございます。
カラムに入れたものを整形する時は、例えばproductsのindexページで、
= image_tag("#{@product.product_images.last.eval('["image_url"]')}", class:"main__image__size")とするとまた別のエラーが出てしまいました。
https://gyazo.com/1eeedab8365e9059fe63682c72d10274
度々で申し訳ございませんが。上記のエラーの対応などをご教示いただけると非常に助かります。
指定の仕方などが間違っているのでしょうか。

[rhiroe]

eval の使い方が誤っています。あとeval を使うのはあまりお勧めできないです。
モデルに serialize を指定する方法もあるんですが、これもお勧めできないです。
複数のレコードに分割させるのが一番いいと思います。

[rui-watanabe]

記載通り記述したら解決しました。
ご丁寧に教えてくださりありがとうございます！
非常に助かりました。